diff --git a/lib/shoulda/matchers/active_record/have_db_column_matcher.rb b/lib/shoulda/matchers/active_record/have_db_column_matcher.rb index c6ab4e412..5fc8911b8 100644 --- a/lib/shoulda/matchers/active_record/have_db_column_matcher.rb +++ b/lib/shoulda/matchers/active_record/have_db_column_matcher.rb @@ -84,7 +84,7 @@ def have_db_column(column) # @private class HaveDbColumnMatcher - OPTIONS = %i(precision limit default null scale primary) + OPTIONS = %i(precision limit default null scale primary).freeze def initialize(column) @column = column @@ -147,7 +147,10 @@ def description def validate_options(opts) invalid_options = opts.keys.map(&:to_sym) - OPTIONS if invalid_options.any? - raise ArgumentError, "Unknown option(s): #{invalid_options.map(&:inspect).join(", ")}" + raise( + ArgumentError, + "Unknown option(s): #{invalid_options.map(&:inspect).join(', ')}", + ) end end diff --git a/spec/unit/shoulda/matchers/active_record/have_db_column_matcher_spec.rb b/spec/unit/shoulda/matchers/active_record/have_db_column_matcher_spec.rb index ab165f39f..8d56315f6 100644 --- a/spec/unit/shoulda/matchers/active_record/have_db_column_matcher_spec.rb +++ b/spec/unit/shoulda/matchers/active_record/have_db_column_matcher_spec.rb @@ -102,7 +102,7 @@ it 'raises an error with the unknown options' do expect { have_db_column(:salary).with_options(preccision: 5, primaryy: true) - }.to raise_error("Unknown option(s): :preccision, :primaryy") + }.to raise_error('Unknown option(s): :preccision, :primaryy') end end