rubocop 0.28.0 → 0.29.0

data/spec/rubocop/cop/rails/scope_args_spec.rb

@@ -7,19 +7,19 @@ describe RuboCop::Cop::Rails::ScopeArgs do
 
   it 'registers an offense a scope with a method arg' do
     inspect_source(cop,
-                   ['scope :active, where(active: true)'])
+                   'scope :active, where(active: true)')
     expect(cop.offenses.size).to eq(1)
   end
 
   it 'accepts a lambda arg' do
     inspect_source(cop,
-                   ['scope :active, -> { where(active: true) }'])
+                   'scope :active, -> { where(active: true) }')
     expect(cop.offenses).to be_empty
   end
 
   it 'accepts a proc arg' do
     inspect_source(cop,
-                   ['scope :active, proc { where(active: true) }'])
+                   'scope :active, proc { where(active: true) }')
     expect(cop.offenses).to be_empty
   end
 end

data/spec/rubocop/cop/rails/validation_spec.rb

@@ -8,13 +8,13 @@ describe RuboCop::Cop::Rails::Validation do
   described_class::BLACKLIST.each_with_index do |validation, number|
     it "registers an offense for #{validation}" do
       inspect_source(cop,
-                     ["#{validation} :name"])
+                     "#{validation} :name")
       expect(cop.offenses.size).to eq(1)
     end
 
     it "outputs the correct message for #{validation}" do
       inspect_source(cop,
-                     ["#{validation} :name"])
+                     "#{validation} :name")
       expect(cop.offenses.first.message)
         .to include(described_class::WHITELIST[number])
     end
@@ -22,7 +22,7 @@ describe RuboCop::Cop::Rails::Validation do
 
   it 'accepts new style validations' do
     inspect_source(cop,
-                   ['validates :name'])
+                   'validates :name')
     expect(cop.offenses).to be_empty
   end
 end

data/spec/rubocop/cop/style/alias_spec.rb

@@ -7,7 +7,7 @@ describe RuboCop::Cop::Style::Alias do
 
   it 'registers an offense for alias with symbol args' do
     inspect_source(cop,
-                   ['alias :ala :bala'])
+                   'alias :ala :bala')
     expect(cop.offenses.size).to eq(1)
     expect(cop.messages)
       .to eq(['Use `alias_method` instead of `alias`.'])
@@ -20,7 +20,7 @@ describe RuboCop::Cop::Style::Alias do
 
   it 'registers an offense for alias with bareword args' do
     inspect_source(cop,
-                   ['alias ala bala'])
+                   'alias ala bala')
     expect(cop.offenses.size).to eq(1)
     expect(cop.messages)
       .to eq(['Use `alias_method` instead of `alias`.'])
@@ -33,19 +33,19 @@ describe RuboCop::Cop::Style::Alias do
 
   it 'does not register an offense for alias_method' do
     inspect_source(cop,
-                   ['alias_method :ala, :bala'])
+                   'alias_method :ala, :bala')
     expect(cop.offenses).to be_empty
   end
 
   it 'does not register an offense for :alias' do
     inspect_source(cop,
-                   ['[:alias, :ala, :bala]'])
+                   '[:alias, :ala, :bala]')
     expect(cop.offenses).to be_empty
   end
 
   it 'does not register an offense for alias with gvars' do
     inspect_source(cop,
-                   ['alias $ala $bala'])
+                   'alias $ala $bala')
     expect(cop.offenses).to be_empty
   end
 

data/spec/rubocop/cop/style/align_hash_spec.rb

@@ -296,7 +296,7 @@ describe RuboCop::Cop::Style::AlignHash, :config do
     end
 
     it 'accepts a single method argument entry with colon' do
-      inspect_source(cop, ['merge(parent: nil)'])
+      inspect_source(cop, 'merge(parent: nil)')
       expect(cop.offenses).to be_empty
     end
   end

data/spec/rubocop/cop/style/align_parameters_spec.rb

@@ -47,7 +47,7 @@ describe RuboCop::Cop::Style::AlignParameters do
     end
 
     it 'accepts calls that only span one line' do
-      inspect_source(cop, ['find(path, s, @special[sexp[0]])'])
+      inspect_source(cop, 'find(path, s, @special[sexp[0]])')
       expect(cop.offenses).to be_empty
     end
 
@@ -100,12 +100,12 @@ describe RuboCop::Cop::Style::AlignParameters do
     end
 
     it "doesn't get confused by symbols with embedded expressions" do
-      inspect_source(cop, ['send(:"#{name}_comments_path")'])
+      inspect_source(cop, 'send(:"#{name}_comments_path")')
       expect(cop.offenses).to be_empty
     end
 
     it "doesn't get confused by regexen with embedded expressions" do
-      inspect_source(cop, ['a(/#{name}/)'])
+      inspect_source(cop, 'a(/#{name}/)')
       expect(cop.offenses).to be_empty
     end
 
@@ -142,7 +142,7 @@ describe RuboCop::Cop::Style::AlignParameters do
     end
 
     it 'can handle a call embedded in a string' do
-      inspect_source(cop, ['model("#{index(name)}", child)'])
+      inspect_source(cop, 'model("#{index(name)}", child)')
       expect(cop.offenses).to be_empty
     end
 
@@ -163,12 +163,12 @@ describe RuboCop::Cop::Style::AlignParameters do
     end
 
     it 'can handle a ternary condition with a block reference' do
-      inspect_source(cop, ['cond ? a : func(&b)'])
+      inspect_source(cop, 'cond ? a : func(&b)')
       expect(cop.offenses).to be_empty
     end
 
     it 'can handle parentheses used with no parameters' do
-      inspect_source(cop, ['func()'])
+      inspect_source(cop, 'func()')
       expect(cop.offenses).to be_empty
     end
 
@@ -180,12 +180,12 @@ describe RuboCop::Cop::Style::AlignParameters do
     end
 
     it 'can handle method calls without parentheses' do
-      inspect_source(cop, ['a(b c, d)'])
+      inspect_source(cop, 'a(b c, d)')
       expect(cop.offenses).to be_empty
     end
 
     it 'can handle other method calls without parentheses' do
-      src = ['chars(Unicode.apply_mapping @wrapped_string, :uppercase)']
+      src = 'chars(Unicode.apply_mapping @wrapped_string, :uppercase)'
       inspect_source(cop, src)
       expect(cop.offenses).to be_empty
     end

data/spec/rubocop/cop/style/and_or_spec.rb

@@ -13,8 +13,7 @@ describe RuboCop::Cop::Style::AndOr, :config do
 
     %w(and or).each do |operator|
       it "accepts \"#{operator}\" outside of conditional" do
-        inspect_source(cop,
-                       ["x = a + b #{operator} return x"])
+        inspect_source(cop, "x = a + b #{operator} return x")
         expect(cop.offenses).to be_empty
       end
 
@@ -51,14 +50,12 @@ describe RuboCop::Cop::Style::AndOr, :config do
 
     %w(&& ||).each do |operator|
       it "accepts #{operator} inside of conditional" do
-        inspect_source(cop,
-                       ["test if a #{operator} b"])
+        inspect_source(cop, "test if a #{operator} b")
         expect(cop.offenses).to be_empty
       end
 
       it "accepts #{operator} outside of conditional" do
-        inspect_source(cop,
-                       ["x = a #{operator} b"])
+        inspect_source(cop, "x = a #{operator} b")
         expect(cop.offenses).to be_empty
       end
     end
@@ -73,28 +70,24 @@ describe RuboCop::Cop::Style::AndOr, :config do
     let(:cop_config) { cop_config }
 
     it 'registers an offense for "or"' do
-      inspect_source(cop,
-                     ['test if a or b'])
+      inspect_source(cop, 'test if a or b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end
 
     it 'registers an offense for "and"' do
-      inspect_source(cop,
-                     ['test if a and b'])
+      inspect_source(cop, 'test if a and b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `&&` instead of `and`.'])
     end
 
     it 'accepts ||' do
-      inspect_source(cop,
-                     ['test if a || b'])
+      inspect_source(cop, 'test if a || b')
       expect(cop.offenses).to be_empty
     end
 
     it 'accepts &&' do
-      inspect_source(cop,
-                     ['test if a && b'])
+      inspect_source(cop, 'test if a && b')
       expect(cop.offenses).to be_empty
     end
 
@@ -132,57 +125,49 @@ describe RuboCop::Cop::Style::AndOr, :config do
     end
 
     it 'warns on short-circuit (and)' do
-      inspect_source(cop,
-                     ['x = a + b and return x'])
+      inspect_source(cop, 'x = a + b and return x')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `&&` instead of `and`.'])
     end
 
     it 'also warns on non short-circuit (and)' do
-      inspect_source(cop,
-                     ['x = a + b if a and b'])
+      inspect_source(cop, 'x = a + b if a and b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `&&` instead of `and`.'])
     end
 
     it 'also warns on non short-circuit (and) (unless)' do
-      inspect_source(cop,
-                     ['x = a + b unless a and b'])
+      inspect_source(cop, 'x = a + b unless a and b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `&&` instead of `and`.'])
     end
 
     it 'warns on short-circuit (or)' do
-      inspect_source(cop,
-                     ['x = a + b or return x'])
+      inspect_source(cop, 'x = a + b or return x')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end
 
     it 'also warns on non short-circuit (or)' do
-      inspect_source(cop,
-                     ['x = a + b if a or b'])
+      inspect_source(cop, 'x = a + b if a or b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end
 
     it 'also warns on non short-circuit (or) (unless)' do
-      inspect_source(cop,
-                     ['x = a + b unless a or b'])
+      inspect_source(cop, 'x = a + b unless a or b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end
 
     it 'also warns on while (or)' do
-      inspect_source(cop,
-                     ['x = a + b while a or b'])
+      inspect_source(cop, 'x = a + b while a or b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end
 
     it 'also warns on until (or)' do
-      inspect_source(cop,
-                     ['x = a + b until a or b'])
+      inspect_source(cop, 'x = a + b until a or b')
       expect(cop.offenses.size).to eq(1)
       expect(cop.messages).to eq(['Use `||` instead of `or`.'])
     end

data/spec/rubocop/cop/style/array_join_spec.rb

@@ -7,23 +7,23 @@ describe RuboCop::Cop::Style::ArrayJoin do
 
   it 'registers an offense for an array followed by string' do
     inspect_source(cop,
-                   ['%w(one two three) * ", "'])
+                   '%w(one two three) * ", "')
     expect(cop.offenses.size).to eq(1)
   end
 
   it 'does not register an offense for numbers' do
     inspect_source(cop,
-                   ['%w(one two three) * 4'])
+                   '%w(one two three) * 4')
     expect(cop.offenses).to be_empty
   end
 
   it 'does not register an offense for ambiguous cases' do
     inspect_source(cop,
-                   ['test * ", "'])
+                   'test * ", "')
     expect(cop.offenses).to be_empty
 
     inspect_source(cop,
-                   ['%w(one two three) * test'])
+                   '%w(one two three) * test')
     expect(cop.offenses).to be_empty
   end
 end

data/spec/rubocop/cop/style/ascii_comments_spec.rb

@@ -15,8 +15,7 @@ describe RuboCop::Cop::Style::AsciiComments do
   end
 
   it 'accepts comments with only ascii chars' do
-    inspect_source(cop,
-                   ['# AZaz1@$%~,;*_`|'])
+    inspect_source(cop, '# AZaz1@$%~,;*_`|')
     expect(cop.offenses).to be_empty
   end
 end

data/spec/rubocop/cop/style/ascii_identifiers_spec.rb

@@ -16,7 +16,7 @@ describe RuboCop::Cop::Style::AsciiIdentifiers do
 
   it 'accepts identifiers with only ascii chars' do
     inspect_source(cop,
-                   ['x.empty?'])
+                   'x.empty?')
     expect(cop.offenses).to be_empty
   end
 
@@ -30,7 +30,7 @@ describe RuboCop::Cop::Style::AsciiIdentifiers do
 
   it 'does not get confused by an empty file' do
     inspect_source(cop,
-                   [''])
+                   '')
     expect(cop.offenses).to be_empty
   end
 end

data/spec/rubocop/cop/style/begin_block_spec.rb

@@ -6,7 +6,7 @@ describe RuboCop::Cop::Style::BeginBlock do
   subject(:cop) { described_class.new }
 
   it 'reports an offense for a BEGIN block' do
-    src = ['BEGIN { test }']
+    src = 'BEGIN { test }'
     inspect_source(cop, src)
     expect(cop.offenses.size).to eq(1)
   end

data/spec/rubocop/cop/style/block_comments_spec.rb

@@ -15,7 +15,7 @@ describe RuboCop::Cop::Style::BlockComments do
 
   it 'accepts regular comments' do
     inspect_source(cop,
-                   ['# comment'])
+                   '# comment')
     expect(cop.offenses).to be_empty
   end
 

data/spec/rubocop/cop/style/block_end_newline_spec.rb

@@ -6,7 +6,7 @@ describe RuboCop::Cop::Style::BlockEndNewline do
   subject(:cop) { described_class.new }
 
   it 'does not register an offense for a one-liner' do
-    inspect_source(cop, ['test do foo end'])
+    inspect_source(cop, 'test do foo end')
     expect(cop.messages).to be_empty
   end
 

data/spec/rubocop/cop/style/blocks_spec.rb

@@ -12,13 +12,13 @@ describe RuboCop::Cop::Style::Blocks do
   end
 
   it 'registers an offense for a single line block with do-end' do
-    inspect_source(cop, ['each do |x| end'])
+    inspect_source(cop, 'each do |x| end')
     expect(cop.messages)
       .to eq(['Prefer {...} over do...end for single-line blocks.'])
   end
 
   it 'accepts a single line block with braces' do
-    inspect_source(cop, ['each { |x| }'])
+    inspect_source(cop, 'each { |x| }')
     expect(cop.offenses).to be_empty
   end
 

data/spec/rubocop/cop/style/braces_around_hash_parameters_spec.rb

@@ -9,15 +9,15 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
     after(:each) { expect(cop.offenses).to be_empty }
 
     it 'accepts one non-hash parameter' do
-      inspect_source(cop, ['where(2)'])
+      inspect_source(cop, 'where(2)')
     end
 
     it 'accepts multiple non-hash parameters' do
-      inspect_source(cop, ['where(1, "2")'])
+      inspect_source(cop, 'where(1, "2")')
     end
 
     it 'accepts one empty hash parameter' do
-      inspect_source(cop, ['where({})'])
+      inspect_source(cop, 'where({})')
     end
 
     it 'accepts one empty hash parameter with whitespace' do
@@ -30,23 +30,23 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
     after(:each) { expect(cop.offenses).to be_empty }
 
     it 'accepts one hash parameter without braces' do
-      inspect_source(cop, ['where(x: "y")'])
+      inspect_source(cop, 'where(x: "y")')
     end
 
     it 'accepts one hash parameter without braces and with multiple keys' do
-      inspect_source(cop, ['where(x: "y", foo: "bar")'])
+      inspect_source(cop, 'where(x: "y", foo: "bar")')
     end
 
     it 'accepts one hash parameter without braces and with one hash value' do
-      inspect_source(cop, ['where(x: { "y" => "z" })'])
+      inspect_source(cop, 'where(x: { "y" => "z" })')
     end
 
     it 'accepts property assignment with braces' do
-      inspect_source(cop, ['x.z = { y: "z" }'])
+      inspect_source(cop, 'x.z = { y: "z" }')
     end
 
     it 'accepts operator with a hash parameter with braces' do
-      inspect_source(cop, ['x.z - { y: "z" }'])
+      inspect_source(cop, 'x.z - { y: "z" }')
     end
   end
 
@@ -55,34 +55,34 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
 
     it 'registers an offense for one non-hash parameter followed by a hash ' \
        'parameter with braces' do
-      inspect_source(cop, ['where(1, { y: 2 })'])
+      inspect_source(cop, 'where(1, { y: 2 })')
       expect(cop.messages).to eq([msg])
       expect(cop.highlights).to eq(['{ y: 2 }'])
     end
 
     it 'registers an offense for one object method hash parameter with ' \
        'braces' do
-      inspect_source(cop, ['x.func({ y: "z" })'])
+      inspect_source(cop, 'x.func({ y: "z" })')
       expect(cop.messages).to eq([msg])
       expect(cop.highlights).to eq(['{ y: "z" }'])
     end
 
     it 'registers an offense for one hash parameter with braces' do
-      inspect_source(cop, ['where({ x: 1 })'])
+      inspect_source(cop, 'where({ x: 1 })')
       expect(cop.messages).to eq([msg])
       expect(cop.highlights).to eq(['{ x: 1 }'])
     end
 
     it 'registers an offense for one hash parameter with braces and ' \
        'whitespace' do
-      inspect_source(cop, ["where(  \n { x: 1 }   )"])
+      inspect_source(cop, "where(  \n { x: 1 }   )")
       expect(cop.messages).to eq([msg])
       expect(cop.highlights).to eq(['{ x: 1 }'])
     end
 
     it 'registers an offense for one hash parameter with braces and multiple ' \
        'keys' do
-      inspect_source(cop, ['where({ x: 1, foo: "bar" })'])
+      inspect_source(cop, 'where({ x: 1, foo: "bar" })')
       expect(cop.messages).to eq([msg])
       expect(cop.highlights).to eq(['{ x: 1, foo: "bar" }'])
     end
@@ -92,51 +92,56 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
     it 'corrects one non-hash parameter followed by a hash parameter with ' \
        'braces' do
       corrected = autocorrect_source(cop, ['where(1, { y: 2 })'])
-      expect(corrected).to eq('where(1,  y: 2 )')
+      expect(corrected).to eq('where(1, y: 2)')
     end
 
     it 'corrects one object method hash parameter with braces' do
       corrected = autocorrect_source(cop, ['x.func({ y: "z" })'])
-      expect(corrected).to eq('x.func( y: "z" )')
+      expect(corrected).to eq('x.func(y: "z")')
     end
 
     it 'corrects one hash parameter with braces' do
       corrected = autocorrect_source(cop, ['where({ x: 1 })'])
-      expect(corrected).to eq('where( x: 1 )')
+      expect(corrected).to eq('where(x: 1)')
     end
 
     it 'corrects one hash parameter with braces and whitespace' do
       corrected = autocorrect_source(cop, ['where(  ',
                                            ' { x: 1 }   )'])
       expect(corrected).to eq(['where(  ',
-                               '  x: 1    )'].join("\n"))
+                               ' x: 1   )'].join("\n"))
     end
 
     it 'corrects one hash parameter with braces and multiple keys' do
       corrected = autocorrect_source(cop, ['where({ x: 1, foo: "bar" })'])
-      expect(corrected).to eq('where( x: 1, foo: "bar" )')
+      expect(corrected).to eq('where(x: 1, foo: "bar")')
     end
 
     it 'corrects one hash parameter with braces and extra leading whitespace' do
       corrected = autocorrect_source(cop, ['where({   x: 1, y: 2 })'])
-      expect(corrected).to eq('where(   x: 1, y: 2 )')
+      expect(corrected).to eq('where(x: 1, y: 2)')
     end
 
     it 'corrects one hash parameter with braces and extra trailing ' \
        'whitespace' do
       corrected = autocorrect_source(cop, ['where({ x: 1, y: 2   })'])
-      expect(corrected).to eq('where( x: 1, y: 2   )')
+      expect(corrected).to eq('where(x: 1, y: 2)')
     end
 
     it 'corrects one hash parameter with braces and a trailing comma' do
       corrected = autocorrect_source(cop, ['where({ x: 1, y: 2, })'])
-      expect(corrected).to eq('where( x: 1, y: 2, )')
+      expect(corrected).to eq('where(x: 1, y: 2,)')
     end
 
     it 'corrects one hash parameter with braces and trailing comma and ' \
        'whitespace' do
       corrected = autocorrect_source(cop, ['where({ x: 1, y: 2,   })'])
-      expect(corrected).to eq('where( x: 1, y: 2,   )')
+      expect(corrected).to eq('where(x: 1, y: 2,)')
+    end
+
+    it 'corrects one hash parameter with braces without adding extra space' do
+      corrected = autocorrect_source(cop, 'get :i, { q: { x: 1 } }')
+      expect(corrected).to eq('get :i, q: { x: 1 }')
     end
   end
 
@@ -157,7 +162,7 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
       end
 
       it 'registers an offense for two hash parameters with braces' do
-        inspect_source(cop, ['where({ x: 1 }, { y: 2 })'])
+        inspect_source(cop, 'where({ x: 1 }, { y: 2 })')
         expect(cop.highlights).to eq(['{ y: 2 }'])
       end
     end
@@ -167,12 +172,12 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
 
       it 'corrects one hash parameter with braces' do
         corrected = autocorrect_source(cop, ['where(1, { x: 1 })'])
-        expect(corrected).to eq('where(1,  x: 1 )')
+        expect(corrected).to eq('where(1, x: 1)')
       end
 
       it 'corrects two hash parameters with braces' do
         corrected = autocorrect_source(cop, ['where(1, { x: 1 }, { y: 2 })'])
-        expect(corrected).to eq('where(1, { x: 1 },  y: 2 )')
+        expect(corrected).to eq('where(1, { x: 1 }, y: 2)')
       end
     end
   end
@@ -185,7 +190,7 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
       include_examples 'no_braces and context_dependent non-offenses'
 
       it 'accepts two hash parameters with braces' do
-        inspect_source(cop, ['where({ x: 1 }, { y: 2 })'])
+        inspect_source(cop, 'where({ x: 1 }, { y: 2 })')
         expect(cop.offenses).to be_empty
       end
     end
@@ -195,7 +200,7 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
 
       it 'registers an offense for one hash parameter with braces and one ' \
          'without' do
-        inspect_source(cop, ['where({ x: 1 }, y: 2)'])
+        inspect_source(cop, 'where({ x: 1 }, y: 2)')
         expect(cop.messages)
           .to eq(['Missing curly braces around a hash parameter.'])
         expect(cop.highlights).to eq(['y: 2'])
@@ -212,7 +217,7 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
 
       it 'corrects one hash parameter with braces' do
         corrected = autocorrect_source(cop, ['where(1, { x: 1 })'])
-        expect(corrected).to eq('where(1,  x: 1 )')
+        expect(corrected).to eq('where(1, x: 1)')
       end
     end
   end
@@ -226,11 +231,11 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
       after(:each) { expect(cop.offenses).to be_empty }
 
       it 'accepts one hash parameter with braces' do
-        inspect_source(cop, ['where({ x: 1 })'])
+        inspect_source(cop, 'where({ x: 1 })')
       end
 
       it 'accepts multiple hash parameters with braces' do
-        inspect_source(cop, ['where({ x: 1 }, { y: 2 })'])
+        inspect_source(cop, 'where({ x: 1 }, { y: 2 })')
       end
 
       it 'accepts one hash parameter with braces and whitespace' do
@@ -246,19 +251,19 @@ describe RuboCop::Cop::Style::BracesAroundHashParameters, :config do
       end
 
       it 'registers an offense for one hash parameter without braces' do
-        inspect_source(cop, ['where(x: "y")'])
+        inspect_source(cop, 'where(x: "y")')
         expect(cop.highlights).to eq(['x: "y"'])
       end
 
       it 'registers an offense for one hash parameter with multiple keys and ' \
          'without braces' do
-        inspect_source(cop, ['where(x: "y", foo: "bar")'])
+        inspect_source(cop, 'where(x: "y", foo: "bar")')
         expect(cop.highlights).to eq(['x: "y", foo: "bar"'])
       end
 
       it 'registers an offense for one hash parameter without braces with ' \
          'one hash value' do
-        inspect_source(cop, ['where(x: { "y" => "z" })'])
+        inspect_source(cop, 'where(x: { "y" => "z" })')
         expect(cop.highlights).to eq(['x: { "y" => "z" }'])
       end
     end