rubocop: fix code style

.rubocop.yml

@@ -10,7 +10,6 @@ AllCops:
   - lib/jekyll/document.rb
   - lib/jekyll/regenerator.rb
   - lib/jekyll/renderer.rb
-  - lib/jekyll/static_file.rb
   - lib/jekyll/utils.rb
   - bin/**/*
   - benchmark/**/*

lib/jekyll/static_file.rb

@@ -1,8 +1,5 @@
 module Jekyll
   class StaticFile
-    # The cache of last modification times [path] -> mtime.
-    @@mtimes = {}
-
     attr_reader :relative_path, :extname
 
     # Initialize a new StaticFile.
@@ -11,6 +8,7 @@ module Jekyll
     # base - The String path to the <source>.
     # dir  - The String path between <source> and the file.
     # name - The String filename of the file.
+    # rubocop: disable ParameterLists
     def initialize(site, base, dir, name, collection = nil)
       @site = site
       @base = base
@@ -19,7 +17,10 @@ module Jekyll
       @collection = collection
       @relative_path = File.join(*[@dir, @name].compact)
       @extname = File.extname(@name)
+      # The cache of last modification times [path] -> mtime.
+      @mtimes = {}
     end
+    # rubocop: enable ParameterLists
 
     # Returns source file path.
     def path
@@ -56,7 +57,7 @@ module Jekyll
     #
     # Returns true if modified since last write.
     def modified?
-      @@mtimes[path] != mtime
+      @mtimes[path] != mtime
     end
 
     # Whether to write the file to the filesystem
@@ -64,7 +65,7 @@ module Jekyll
     # Returns true unless the defaults for the destination path from
     # _config.yml contain `published: false`.
     def write?
-      defaults.fetch('published', true)
+      defaults.fetch("published", true)
     end
 
     # Write the static file to the destination directory (if modified).
@@ -76,28 +77,16 @@ module Jekyll
       dest_path = destination(dest)
 
       return false if File.exist?(dest_path) && !modified?
-      @@mtimes[path] = mtime
+      @mtimes[path] = mtime
 
       FileUtils.mkdir_p(File.dirname(dest_path))
       FileUtils.rm(dest_path) if File.exist?(dest_path)
-      if @site.safe || Jekyll.env == "production"
+      copy_file(dest_path)
-        FileUtils.cp(path, dest_path)
+      File.utime(@mtimes[path], @mtimes[path], dest_path)
-      else
-        FileUtils.copy_entry(path, dest_path)
-      end
-      File.utime(@@mtimes[path], @@mtimes[path], dest_path)
 
       true
     end
 
-    # Reset the mtimes cache (for testing purposes).
-    #
-    # Returns nothing.
-    def self.reset_cache
-      @@mtimes = {}
-      nil
-    end
-
     def to_liquid
       {
         "extname"       => extname,
@@ -111,9 +100,9 @@ module Jekyll
         :collection => @collection.label,
         :path       => relative_path[
           @collection.relative_directory.size..relative_path.size],
-        :output_ext => '',
+        :output_ext => "",
-        :name => '',
+        :name       => "",
-        :title => ''
+        :title      => ""
       }
     end
 
@@ -128,7 +117,7 @@ module Jekyll
                    :template     => @collection.url_template,
                    :placeholders => placeholders
                  })
-               end.to_s.gsub(/\/$/, '')
+               end.to_s.gsub(%r!/$!, "")
     end
 
     # Returns the type of the collection if present, nil otherwise.
@@ -141,5 +130,14 @@ module Jekyll
     def defaults
       @defaults ||= @site.frontmatter_defaults.all url, type
     end
+
+    private
+    def copy_file(dest_path)
+      if @site.safe || Jekyll.env == "production"
+        FileUtils.cp(path, dest_path)
+      else
+        FileUtils.copy_entry(path, dest_path)
+      end
+    end
   end
 end

test/test_site.rb

@@ -100,7 +100,6 @@ class TestSite < JekyllUnitTest
 
     should "write only modified static files" do
       clear_dest
-      StaticFile.reset_cache
       @site.regenerator.clear
 
       @site.process
@@ -130,7 +129,6 @@ class TestSite < JekyllUnitTest
 
     should "write static files if not modified but missing in destination" do
       clear_dest
-      StaticFile.reset_cache
       @site.regenerator.clear
 
       @site.process