diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2022-10-20 12:40:42 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2022-10-20 12:40:42 +0300 |
commit | ee664acb356f8123f4f6b00b73c1e1cf0866c7fb (patch) | |
tree | f8479f94a28f66654c6a4f6fb99bad6b4e86a40e /spec/models/operations | |
parent | 62f7d5c5b69180e82ae8196b7b429eeffc8e7b4f (diff) |
Add latest changes from gitlab-org/gitlab@15-5-stable-eev15.5.0-rc42
Diffstat (limited to 'spec/models/operations')
-rw-r--r-- | spec/models/operations/feature_flags/strategy_spec.rb | 26 |
1 files changed, 12 insertions, 14 deletions
diff --git a/spec/models/operations/feature_flags/strategy_spec.rb b/spec/models/operations/feature_flags/strategy_spec.rb index de1b9d2c855..949f92b3b2a 100644 --- a/spec/models/operations/feature_flags/strategy_spec.rb +++ b/spec/models/operations/feature_flags/strategy_spec.rb @@ -130,14 +130,15 @@ RSpec.describe Operations::FeatureFlags::Strategy do context 'when the strategy name is flexibleRollout' do valid_parameters = { rollout: '40', groupId: 'mygroup', stickiness: 'default' } - where(invalid_parameters: [ - nil, - {}, - *valid_parameters.to_a.combination(1).to_a.map { |p| p.to_h }, - *valid_parameters.to_a.combination(2).to_a.map { |p| p.to_h }, - { **valid_parameters, userIds: '4' }, - { **valid_parameters, extra: nil } - ]) + where( + invalid_parameters: [ + nil, + {}, + *valid_parameters.to_a.combination(1).to_a.map { |p| p.to_h }, + *valid_parameters.to_a.combination(2).to_a.map { |p| p.to_h }, + { **valid_parameters, userIds: '4' }, + { **valid_parameters, extra: nil } + ]) with_them do it 'must have valid parameters for the strategy' do strategy = build(:operations_strategy, @@ -180,9 +181,7 @@ RSpec.describe Operations::FeatureFlags::Strategy do expect(strategy).to be_invalid - expect(strategy.errors[:parameters]).to eq([ - 'rollout must be a string between 0 and 100 inclusive' - ]) + expect(strategy.errors[:parameters]).to eq(['rollout must be a string between 0 and 100 inclusive']) end end @@ -243,9 +242,8 @@ RSpec.describe Operations::FeatureFlags::Strategy do expect(strategy).to be_invalid - expect(strategy.errors[:parameters]).to eq([ - 'stickiness parameter must be default, userId, sessionId, or random' - ]) + expect(strategy.errors[:parameters]).to eq( + ['stickiness parameter must be default, userId, sessionId, or random']) end end |