(CONT-811) Correct Style/RedundantArgument

Author: GSPatton

lib/puppet-lint/checks.rb

@@ -129,6 +129,6 @@ def enabled_checks
   #
   # Returns the manifest as a String.
   def manifest
-    PuppetLint::Data.tokens.map(&:to_manifest).join('')
+    PuppetLint::Data.tokens.map(&:to_manifest).join
   end
 end

spec/unit/puppet-lint/lexer_spec.rb

@@ -1289,7 +1289,7 @@
     it 'does not enclose variable with a chained function call' do
       manifest = '"This is ${a.test}"'
       tokens = lexer.tokenise(manifest)
-      expect(tokens.map(&:to_manifest).join('')).to eq(manifest)
+      expect(tokens.map(&:to_manifest).join).to eq(manifest)
     end
   end
 
@@ -1688,7 +1688,7 @@
       END
 
       tokens = lexer.tokenise(manifest)
-      expect(tokens.map(&:to_manifest).join('')).to eq(manifest)
+      expect(tokens.map(&:to_manifest).join).to eq(manifest)
 
       expect(tokens[0].type).to eq(:VARIABLE)
       expect(tokens[0].value).to eq('str')
@@ -1755,7 +1755,7 @@
       expect(tokens[7].raw).to eq('$myvar')
       expect(tokens[7].to_manifest).to eq('$myvar')
 
-      expect(tokens.map(&:to_manifest).join('')).to eq(manifest)
+      expect(tokens.map(&:to_manifest).join).to eq(manifest)
     end
   end