Commit 4b56e4cf authored by Alper Akgun's avatar Alper Akgun

Merge branch 'remove-render-dashboard-gold-trial-feature-flag' into 'master'

Remove render_dashboard_gold_trial feature flag

See merge request gitlab-org/gitlab!43129
parents 455e84ca 4f0b8f00
...@@ -55,7 +55,6 @@ module EE ...@@ -55,7 +55,6 @@ module EE
def render_dashboard_gold_trial(user) def render_dashboard_gold_trial(user)
return unless show_gold_trial?(user, GOLD_TRIAL) && return unless show_gold_trial?(user, GOLD_TRIAL) &&
user_default_dashboard?(user) && user_default_dashboard?(user) &&
::Feature.enabled?(:render_dashboard_gold_trial, default_enabled: true) &&
!user.owns_paid_namespace? && !user.owns_paid_namespace? &&
user.any_namespace_without_trial? user.any_namespace_without_trial?
......
---
name: render_dashboard_gold_trial
introduced_by_url:
rollout_issue_url:
group:
type: development
default_enabled: true
...@@ -217,22 +217,6 @@ RSpec.describe EE::UserCalloutsHelper do ...@@ -217,22 +217,6 @@ RSpec.describe EE::UserCalloutsHelper do
helper.render_dashboard_gold_trial(user) helper.render_dashboard_gold_trial(user)
end end
end end
context 'when render_dashboard_gold_trial feature is disabled' do
before do
stub_feature_flags(render_dashboard_gold_trial: false)
allow(helper).to receive(:show_gold_trial?).and_return(true)
allow(helper).to receive(:user_default_dashboard?).and_return(true)
allow(helper).to receive(:has_some_namespaces_with_no_trials?).and_return(true)
end
it 'does not render' do
expect(helper).not_to receive(:render)
helper.render_dashboard_gold_trial(user)
end
end
end end
describe '#render_billings_gold_trial' do describe '#render_billings_gold_trial' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment