Rubocop: test/test_regenerator.rb

test/test_regenerator.rb

@@ -1,4 +1,4 @@
-require 'helper'
+require "helper"
 
 class TestRegenerator < JekyllUnitTest
   context "The site regenerator" do
@@ -39,12 +39,11 @@ class TestRegenerator < JekyllUnitTest
       # we need to create the destinations for these files,
       # because regenerate? checks if the destination exists
       [@page, @post, @document, @asset_file].each do |item|
-        if item.respond_to?(:destination)
+        next unless item.respond_to?(:destination)
         dest = item.destination(@site.dest)
         FileUtils.mkdir_p(File.dirname(dest))
         FileUtils.touch(dest)
       end
-      end
       @regenerator.write_metadata
       @regenerator = Regenerator.new(@site)
 
@@ -69,7 +68,7 @@ class TestRegenerator < JekyllUnitTest
       [@page, @post, @document, @asset_file].each do |item|
         if item.respond_to?(:destination)
           dest = item.destination(@site.dest)
-          File.unlink(dest) unless !File.exist?(dest)
+          File.unlink(dest) if File.exist?(dest)
         end
       end
 
@@ -171,7 +170,7 @@ class TestRegenerator < JekyllUnitTest
       metadata_file = source_dir(".jekyll-metadata")
       @regenerator = Regenerator.new(@site)
 
-      File.open(metadata_file, 'w') do |f|
+      File.open(metadata_file, "w") do |f|
         f.write(@regenerator.metadata.to_yaml)
       end
 
@@ -182,7 +181,7 @@ class TestRegenerator < JekyllUnitTest
     should "not crash when reading corrupted marshal file" do
       metadata_file = source_dir(".jekyll-metadata")
       File.open(metadata_file, "w") do |file|
-        file.puts Marshal.dump({ foo: 'bar' })[0,5]
+        file.puts Marshal.dump({ :foo => "bar" })[0, 5]
       end
 
       @regenerator = Regenerator.new(@site)
@@ -282,7 +281,7 @@ class TestRegenerator < JekyllUnitTest
     end
 
     should "not regenerate again if multiple dependencies" do
-      multi_deps = @regenerator.metadata.select {|k,v| v['deps'].length > 2}
+      multi_deps = @regenerator.metadata.select { |_k, v| v["deps"].length > 2 }
       multi_dep_path = multi_deps.keys.first
 
       assert @regenerator.metadata[multi_dep_path]["deps"].length > 2