Skip to content

Commit 1e9e525

Browse files
committed
fix: Rubocop offenses
1 parent fd730a6 commit 1e9e525

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

app/models/runtime/feature_flag.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ def self.override_default_flags(feature_flag_overrides)
9090
invalid_keys = feature_flag_overrides.keys.to_set - FeatureFlag::DEFAULT_FLAGS.keys.to_set
9191
raise "Invalid feature flag name(s): #{invalid_keys.to_a}" if invalid_keys.any?
9292

93-
invalid_values = feature_flag_overrides.select { |k,v| !v.in? [true,false] }
93+
invalid_values = feature_flag_overrides.reject { |_, v| v.is_a?(TrueClass) || v.is_a?(FalseClass) }
9494
raise "Invalid feature flag value(s): #{invalid_values}" if invalid_values.any?
9595

9696
feature_flag_overrides.each do |flag_name, flag_value|

spec/unit/models/runtime/feature_flag_spec.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -294,28 +294,28 @@ module VCAP::CloudController
294294
feature_flag_overrides = { an_invalid_name: true }
295295
expect do
296296
FeatureFlag.override_default_flags(feature_flag_overrides)
297-
end.to raise_error("Invalid feature flag name(s): [:an_invalid_name]")
297+
end.to raise_error('Invalid feature flag name(s): [:an_invalid_name]')
298298
end
299299

300300
it 'raises an error for a mix of valid and invalid names' do
301301
feature_flag_overrides = { diego_docker: true, an_invalid_name: true }
302302
expect do
303303
FeatureFlag.override_default_flags(feature_flag_overrides)
304-
end.to raise_error("Invalid feature flag name(s): [:an_invalid_name]")
304+
end.to raise_error('Invalid feature flag name(s): [:an_invalid_name]')
305305
end
306306

307307
it 'raises an error for all invalid names' do
308308
feature_flag_overrides = { invalid_name1: true, invalid_name2: false }
309309
expect do
310310
FeatureFlag.override_default_flags(feature_flag_overrides)
311-
end.to raise_error("Invalid feature flag name(s): [:invalid_name1, :invalid_name2]")
311+
end.to raise_error('Invalid feature flag name(s): [:invalid_name1, :invalid_name2]')
312312
end
313313

314314
it 'raises an error for invalid values' do
315315
feature_flag_overrides = { diego_docker: 'an invalid value', user_org_creation: false }
316316
expect do
317317
FeatureFlag.override_default_flags(feature_flag_overrides)
318-
end.to raise_error("Invalid feature flag value(s): {:diego_docker=>\"an invalid value\"}")
318+
end.to raise_error('Invalid feature flag value(s): {:diego_docker=>"an invalid value"}')
319319
end
320320
end
321321

0 commit comments

Comments
 (0)