diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2019-02-13 12:41:56 +0300 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2019-02-13 12:41:56 +0300 |
commit | 559169f613ee1ca596cdc530929cc07b03288bc2 (patch) | |
tree | 6f001cdcbac302863a915d86d1b9e593e426f3ba | |
parent | 032e800b93dfe36604d896f2128ea05ad1330ece (diff) | |
parent | 198f6965cd1b48e61243d49ba28dce593ea7c9bb (diff) |
Merge branch 'fix-feature_flags_environment_scope-feature-flag-ce' into 'master'
CE Backport: Fix unleash server side cannot return feature flags
See merge request gitlab-org/gitlab-ce!25185
-rw-r--r-- | spec/support/helpers/stub_feature_flags.rb | 27 |
1 files changed, 23 insertions, 4 deletions
diff --git a/spec/support/helpers/stub_feature_flags.rb b/spec/support/helpers/stub_feature_flags.rb index 4061a8d1bc9..48258692304 100644 --- a/spec/support/helpers/stub_feature_flags.rb +++ b/spec/support/helpers/stub_feature_flags.rb @@ -1,11 +1,30 @@ module StubFeatureFlags # Stub Feature flags with `flag_name: true/false` # - # @param [Hash] features where key is feature name and value is boolean whether enabled or not + # @param [Hash] features where key is feature name and value is boolean whether enabled or not. + # Alternatively, you can specify Hash to enable the flag on a specific thing. + # + # Examples + # - `stub_feature_flags(ci_live_trace: false)` ... Disable `ci_live_trace` + # feature flag globally. + # - `stub_feature_flags(ci_live_trace: { enabled: false, thing: project })` ... + # Disable `ci_live_trace` feature flag on the specified project. def stub_feature_flags(features) - features.each do |feature_name, enabled| - allow(Feature).to receive(:enabled?).with(feature_name, any_args) { enabled } - allow(Feature).to receive(:enabled?).with(feature_name.to_s, any_args) { enabled } + features.each do |feature_name, option| + if option.is_a?(Hash) + enabled, thing = option.values_at(:enabled, :thing) + else + enabled = option + thing = nil + end + + if thing + allow(Feature).to receive(:enabled?).with(feature_name, thing, any_args) { enabled } + allow(Feature).to receive(:enabled?).with(feature_name.to_s, thing, any_args) { enabled } + else + allow(Feature).to receive(:enabled?).with(feature_name, any_args) { enabled } + allow(Feature).to receive(:enabled?).with(feature_name.to_s, any_args) { enabled } + end end end end |