jekyll-minibundle 2.1.2 → 2.2.0

data/test/integration/static_files_as_asset_sources_test.rb

@@ -13,7 +13,7 @@ module Jekyll::Minibundle::Test
           destination_path(CSS_BUNDLE_DESTINATION_FINGERPRINT_PATH),
           destination_path(JS_BUNDLE_DESTINATION_FINGERPRINT_PATH)
         ].sort
-        assert_equal expected, actual
+        assert_equal(expected, actual)
       end
     end
 
@@ -28,9 +28,9 @@ module Jekyll::Minibundle::Test
 
           asset_files = Dir[destination_path('assets') + '/screen*.css']
 
-          assert_equal 1, asset_files.size
-          assert_equal contents, File.read(destination_path('assets/shared.css'))
-          assert_equal contents, File.read(asset_files.first)
+          assert_equal(1, asset_files.size)
+          assert_equal(contents, File.read(destination_path('assets/shared.css')))
+          assert_equal(contents, File.read(asset_files.first))
         end
       end
     end
@@ -45,8 +45,8 @@ module Jekyll::Minibundle::Test
 
         generate_site(:development)
 
-        assert_equal dep_contents, File.read(destination_path('assets/dependency.js'))
-        assert_equal app_contents, File.read(destination_path('assets/app.js'))
+        assert_equal(dep_contents, File.read(destination_path('assets/dependency.js')))
+        assert_equal(app_contents, File.read(destination_path('assets/app.js')))
       end
     end
 
@@ -63,10 +63,10 @@ module Jekyll::Minibundle::Test
 
         asset_files = Dir[destination_path('assets/site-*.js')]
 
-        assert_equal 1, asset_files.size
-        assert_equal dep_contents, File.read(destination_path('assets/dependency.js'))
-        assert_equal app_contents, File.read(destination_path('assets/app.js'))
-        assert_equal bundled_contents, File.read(asset_files.first)
+        assert_equal(1, asset_files.size)
+        assert_equal(dep_contents, File.read(destination_path('assets/dependency.js')))
+        assert_equal(app_contents, File.read(destination_path('assets/app.js')))
+        assert_equal(bundled_contents, File.read(asset_files.first))
       end
     end
   end

data/test/support/assertions.rb

@@ -17,7 +17,7 @@ module Jekyll::Minibundle::Test
       end
 
       assert(remaining.empty?, lambda do
-        "Expected #{mu_pp(collection)} to include only #{mu_pp(expected_elements)}"
+        "Expected\n\n#{mu_pp(collection)}\n\nto include only\n\n#{mu_pp(expected_elements)}"
       end)
     end
   end

data/test/support/{static_file_api_config.rb → static_file_config.rb}

@@ -1,17 +1,20 @@
 module Jekyll::Minibundle::Test
-  module StaticFileAPIConfig
-    STATIC_FILE_API_PROPERTIES = [
+  module StaticFileConfig
+    STATIC_FILE_PROPERTIES = [
+      :basename,
+      :data,
       :defaults,
       :destination_rel_dir,
-      :name,
       :extname,
       :modified_time,
       :mtime,
+      :name,
       :path,
       :placeholders,
       :relative_path,
       :to_liquid,
       :type,
+      :url,
       :write?
     ].freeze
   end

data/test/support/test_case.rb

@@ -84,10 +84,10 @@ module Jekyll::Minibundle::Test
       OpenStruct.new(source: dir, static_files: [])
     end
 
-    def make_real_site
+    def make_real_site(dir = Dir.pwd)
       config = nil
       capture_io do
-        config = Jekyll.configuration('source' => Dir.pwd, 'destination' => '_site')
+        config = Jekyll.configuration('source' => dir, 'destination' => '_site')
       end
       Jekyll::Site.new(config)
     end
@@ -125,7 +125,7 @@ module Jekyll::Minibundle::Test
     end
 
     def get_minifier_cmd_count(count_file = 'minifier_cmd_count')
-      if File.exist?(count_file)
+      if File.file?(count_file)
         File.read(count_file).to_i
       else
         0
@@ -162,7 +162,10 @@ module Jekyll::Minibundle::Test
     end
 
     def _generate_site(test_options)
-      AssetFileRegistry.clear_all if test_options.fetch(:clear_cache, true)
+      if test_options.fetch(:clear_cache, true)
+        AssetFileRegistry.clear_all
+        VariableTemplateRegistry.clear
+      end
       site = make_real_site
       capture_io { site.process }
     end

data/test/unit/asset_bundle_test.rb

@@ -6,7 +6,7 @@ module Jekyll::Minibundle::Test
     def test_raise_exception_if_bundle_command_fails
       capture_io do
         err = assert_raises(RuntimeError) { make_bundle('read _ignore ; false') }
-        assert_equal "Bundling assets/site.js failed with exit status 1, command: 'read _ignore ; false'", err.to_s
+        assert_equal('Bundling "assets/site.js" failed with exit status 1, command: "read _ignore ; false"', err.to_s)
       end
     end
 
@@ -16,27 +16,27 @@ module Jekyll::Minibundle::Test
         assert_raises(RuntimeError) { make_bundle(cmd) }
       end
       expected_stderr = <<-END
-Minibundle: Bundling assets/site.js failed with exit status 1, command: '#{cmd}', last 16 bytes of minifier output:
+Minibundle: Bundling "assets/site.js" failed with exit status 1, command: #{cmd.inspect}, last 16 bytes of minifier output:
 Minibundle: line 1\\x07
 Minibundle: line 2\\xa4
       END
-      assert_equal expected_stderr, actual_stderr.gsub(/\e\[\d+m/, '').gsub(/^ +/, '')
+      assert_equal(expected_stderr, actual_stderr.gsub(/\e\[\d+m/, '').gsub(/^ +/, ''))
     end
 
     def test_raise_exception_if_bundle_command_not_found
       err = assert_raises(RuntimeError) { make_bundle('no-such-jekyll-minibundle-cmd') }
-      assert_equal 'Bundling assets/site.js failed: No such file or directory - no-such-jekyll-minibundle-cmd', err.to_s
+      assert_equal('Bundling "assets/site.js" failed: No such file or directory - no-such-jekyll-minibundle-cmd', err.to_s)
     end
 
     def test_raise_exception_if_bundle_command_not_configured
       err = assert_raises(RuntimeError) { make_bundle(nil) }
       expected_errmsg = <<-END
-Missing minification command for bundling assets/site.js. Specify it in
+Missing minification command for bundling "assets/site.js". Specify it in
 1) minibundle.minifier_commands.js setting in _config.yml,
 2) $JEKYLL_MINIBUNDLE_CMD_JS environment variable, or
 3) minifier_cmd setting inside minibundle block.
       END
-      assert_equal expected_errmsg, err.to_s
+      assert_equal(expected_errmsg, err.to_s)
     end
 
     private

data/test/unit/asset_file_drop_test.rb

@@ -0,0 +1,65 @@
+require 'json'
+require 'support/test_case'
+require 'support/fixture_config'
+require 'jekyll/minibundle/asset_file_drop'
+require 'jekyll/minibundle/stamp_file'
+
+module Jekyll::Minibundle::Test
+  class AssetFileDropTest < TestCase
+    include FixtureConfig
+
+    def test_method_access_via_subscript_method
+      with_drop do |drop|
+        assert_equal("screen-#{STAMP_FINGERPRINT}.css", drop['name'])
+      end
+    end
+
+    def test_method_access_via_forwarding
+      with_drop do |drop|
+        assert_equal("screen-#{STAMP_FINGERPRINT}.css", drop['name'])
+      end
+    end
+
+    def test_key?
+      with_drop do |drop|
+        assert(drop.key?('name'))
+      end
+    end
+
+    def test_to_h
+      with_drop do |drop|
+        hash = drop.to_h
+        assert_equal(AssetFileDrop::KEYS.sort, hash.keys.sort)
+        assert_equal("screen-#{STAMP_FINGERPRINT}.css", hash['name'])
+        assert_equal('.css', hash['extname'])
+        assert_equal("screen-#{STAMP_FINGERPRINT}", hash['basename'])
+        assert_instance_of(Time, hash['modified_time'])
+        assert_equal("/#{STAMP_SOURCE_PATH}", hash['path'])
+        assert_nil(hash['collection'])
+      end
+    end
+
+    def test_to_hash
+      with_drop do |drop|
+        assert_equal(drop.to_h, drop.to_hash)
+      end
+    end
+
+    def test_inspect
+      with_drop do |drop|
+        assert_equal(drop.to_h.keys, JSON.parse(drop.inspect).keys)
+      end
+    end
+
+    private
+
+    def with_drop(&block)
+      with_fake_site do |site|
+        file = StampFile.new(site, STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH)
+        file.destination_path_for_markup
+        drop = AssetFileDrop.new(file)
+        block.call(drop)
+      end
+    end
+  end
+end

data/test/unit/asset_file_registry_test.rb

@@ -1,59 +1,77 @@
 require 'support/test_case'
+require 'support/fixture_config'
 require 'jekyll/minibundle/asset_file_registry'
 
 module Jekyll::Minibundle::Test
   class AssetFileRegistryTest < TestCase
+    include FixtureConfig
+
     def setup
       AssetFileRegistry.clear_all
-      @site = make_site
     end
 
     def test_register_returns_same_bundle_file_for_same_bundle_config
-      first = AssetFileRegistry.register_bundle_file(@site, bundle_config)
-      second = AssetFileRegistry.register_bundle_file(@site, bundle_config)
-      assert_same first, second
-      assert_equal 1, asset_file_registry_size
-      assert_contains_only @site.static_files, [first]
+      with_fake_site do |site|
+        first = AssetFileRegistry.register_bundle_file(site, bundle_config)
+        second = AssetFileRegistry.register_bundle_file(site, bundle_config)
+        assert_same(first, second)
+        assert_equal(1, asset_file_registry_size)
+        assert_contains_only(site.static_files, [first])
+      end
     end
 
     def test_register_returns_same_development_file_collection_for_same_bundle_config
-      first = AssetFileRegistry.register_development_file_collection(@site, bundle_config)
-      second = AssetFileRegistry.register_development_file_collection(@site, bundle_config)
-      assert_same first, second
-      assert_equal 1, asset_file_registry_size
-      assert_contains_only @site.static_files, first.files
+      with_fake_site do |site|
+        first = AssetFileRegistry.register_development_file_collection(site, bundle_config)
+        second = AssetFileRegistry.register_development_file_collection(site, bundle_config)
+        assert_same(first, second)
+        assert_equal(1, asset_file_registry_size)
+        assert_contains_only(site.static_files, first.files)
+      end
     end
 
     def test_register_returns_different_bundle_files_for_bundle_configs_with_different_destination_paths
-      first = AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('destination_path' => 'assets/dest1'))
-      second = AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('destination_path' => 'assets/dest2'))
-      refute_same first, second
-      assert_equal 2, asset_file_registry_size
-      assert_contains_only @site.static_files, [first, second]
+      with_fake_site do |site|
+        first = AssetFileRegistry.register_bundle_file(site, bundle_config.merge('destination_path' => 'assets/dest1'))
+        second = AssetFileRegistry.register_bundle_file(site, bundle_config.merge('destination_path' => 'assets/dest2'))
+        refute_same first, second
+        assert_equal(2, asset_file_registry_size)
+        assert_contains_only(site.static_files, [first, second])
+      end
     end
 
     def test_register_returns_different_development_file_collections_for_bundle_configs_with_different_destination_paths
-      first = AssetFileRegistry.register_development_file_collection(@site, bundle_config.merge('destination_path' => 'assets/dest1'))
-      second = AssetFileRegistry.register_development_file_collection(@site, bundle_config.merge('destination_path' => 'assets/dest2'))
-      refute_same first, second
-      assert_equal 2, asset_file_registry_size
-      assert_contains_only @site.static_files, (first.files + second.files)
+      with_fake_site do |site|
+        first = AssetFileRegistry.register_development_file_collection(site, bundle_config.merge('destination_path' => 'assets/dest1'))
+        second = AssetFileRegistry.register_development_file_collection(site, bundle_config.merge('destination_path' => 'assets/dest2'))
+        refute_same first, second
+        assert_equal(2, asset_file_registry_size)
+        assert_contains_only(site.static_files, (first.files + second.files))
+      end
     end
 
     def test_register_returns_different_bundle_files_for_bundle_configs_with_different_types
-      first = AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('type' => :css))
-      second = AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('type' => :js))
-      refute_same first, second
-      assert_equal 2, asset_file_registry_size
-      assert_contains_only @site.static_files, [first, second]
+      with_fake_site do |site|
+        FileUtils.touch(File.join(JS_BUNDLE_SOURCE_DIR, 'dependency.css'))
+        FileUtils.touch(File.join(JS_BUNDLE_SOURCE_DIR, 'app.css'))
+        first = AssetFileRegistry.register_bundle_file(site, bundle_config.merge('type' => :css))
+        second = AssetFileRegistry.register_bundle_file(site, bundle_config.merge('type' => :js))
+        refute_same first, second
+        assert_equal(2, asset_file_registry_size)
+        assert_contains_only(site.static_files, [first, second])
+      end
     end
 
     def test_register_returns_different_development_file_collections_for_bundle_configs_with_different_types
-      first = AssetFileRegistry.register_development_file_collection(@site, bundle_config.merge('type' => :css))
-      second = AssetFileRegistry.register_development_file_collection(@site, bundle_config.merge('type' => :js))
-      refute_same first, second
-      assert_equal 2, asset_file_registry_size
-      assert_contains_only @site.static_files, (first.files + second.files)
+      with_fake_site do |site|
+        FileUtils.touch(File.join(JS_BUNDLE_SOURCE_DIR, 'dependency.css'))
+        FileUtils.touch(File.join(JS_BUNDLE_SOURCE_DIR, 'app.css'))
+        first = AssetFileRegistry.register_development_file_collection(site, bundle_config.merge('type' => :css))
+        second = AssetFileRegistry.register_development_file_collection(site, bundle_config.merge('type' => :js))
+        refute_same first, second
+        assert_equal(2, asset_file_registry_size)
+        assert_contains_only(site.static_files, (first.files + second.files))
+      end
     end
 
     [
@@ -61,17 +79,19 @@ module Jekyll::Minibundle::Test
       {description: 'source_directory', config_diff: {'source_dir' => '_assets/src2'}}
     ].each do |spec|
       define_method :"test_raise_exception_if_registering_bundle_file_with_same_destination_path_but_with_different_#{spec.fetch(:description)}" do
-        first_config = bundle_config
-        first_file = AssetFileRegistry.register_bundle_file(@site, first_config)
-        second_config = bundle_config.merge(spec[:config_diff])
-        err = assert_raises(RuntimeError) do
-          AssetFileRegistry.register_bundle_file(@site, second_config)
+        with_fake_site do |site|
+          first_config = bundle_config
+          first_file = AssetFileRegistry.register_bundle_file(site, first_config)
+          second_config = bundle_config.merge(spec[:config_diff])
+          err = assert_raises(RuntimeError) do
+            AssetFileRegistry.register_bundle_file(site, second_config)
+          end
+          assert_equal(<<-END, err.to_s)
+Two or more minibundle blocks with the same destination path "assets/site.js", but having different asset configuration: #{second_config.inspect} vs. #{first_config.inspect}
+          END
+          assert_equal(1, asset_file_registry_size)
+          assert_contains_only(site.static_files, [first_file])
         end
-        assert_equal <<-END, err.to_s
-Two or more minibundle blocks with the same destination path 'assets/dest.css', but having different asset configuration: #{second_config.inspect} vs. #{first_config.inspect}
-        END
-        assert_equal 1, asset_file_registry_size
-        assert_contains_only @site.static_files, [first_file]
       end
     end
 
@@ -80,17 +100,19 @@ Two or more minibundle blocks with the same destination path 'assets/dest.css',
       {description: 'source_directory', config_diff: {'source_dir' => '_assets/src2'}}
     ].each do |spec|
       define_method :"test_raise_exception_if_registering_development_file_collection_with_same_destination_path_but_with_different_#{spec.fetch(:description)}" do
-        first_config = bundle_config
-        first_file = AssetFileRegistry.register_development_file_collection(@site, first_config)
-        second_config = bundle_config.merge(spec[:config_diff])
-        err = assert_raises(RuntimeError) do
-          AssetFileRegistry.register_development_file_collection(@site, second_config)
+        with_fake_site do |site|
+          first_config = bundle_config
+          first_file = AssetFileRegistry.register_development_file_collection(site, first_config)
+          second_config = bundle_config.merge(spec[:config_diff])
+          err = assert_raises(RuntimeError) do
+            AssetFileRegistry.register_development_file_collection(site, second_config)
+          end
+          assert_equal(<<-END, err.to_s)
+Two or more minibundle blocks with the same destination path "assets/site.js", but having different asset configuration: #{second_config.inspect} vs. #{first_config.inspect}
+          END
+          assert_equal(1, asset_file_registry_size)
+          assert_contains_only(site.static_files, first_file.files)
         end
-        assert_equal <<-END, err.to_s
-Two or more minibundle blocks with the same destination path 'assets/dest.css', but having different asset configuration: #{second_config.inspect} vs. #{first_config.inspect}
-        END
-        assert_equal 1, asset_file_registry_size
-        assert_contains_only @site.static_files, first_file.files
       end
     end
 
@@ -99,52 +121,68 @@ Two or more minibundle blocks with the same destination path 'assets/dest.css',
       {description: 'development_file', method: :register_development_file}
     ].each do |spec|
       define_method :"test_register_returns_same_#{spec.fetch(:description)}_for_same_source_and_destination_paths" do
-        first = AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src1.css', 'assets/dest1.css')
-        second = AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src1.css', 'assets/dest1.css')
-        assert_same first, second
-        assert_equal 1, asset_file_registry_size
-        assert_contains_only @site.static_files, [first]
+        with_fake_site do |site|
+          first = AssetFileRegistry.send(spec.fetch(:method), site, STAMP_SOURCE_PATH, 'assets/dest1.css')
+          second = AssetFileRegistry.send(spec.fetch(:method), site, STAMP_SOURCE_PATH, 'assets/dest1.css')
+          assert_same(first, second)
+          assert_equal(1, asset_file_registry_size)
+          assert_contains_only(site.static_files, [first])
+        end
       end
 
       define_method :"test_register_returns_different_#{spec.fetch(:description)}s_for_different_source_and_destination_paths" do
-        first = AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src1.css', 'assets/dest1.css')
-        second = AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src2.css', 'assets/dest2.css')
-        refute_same first, second
-        assert_equal 2, asset_file_registry_size
-        assert_contains_only @site.static_files, [first, second]
+        with_fake_site do |site|
+          first = AssetFileRegistry.send(spec.fetch(:method), site, STAMP_SOURCE_PATH, 'assets/dest1.css')
+          second = AssetFileRegistry.send(spec.fetch(:method), site, STAMP_SOURCE_PATH, 'assets/dest2.css')
+          refute_same first, second
+          assert_equal(2, asset_file_registry_size)
+          assert_contains_only(site.static_files, [first, second])
+        end
       end
 
       define_method :"test_raise_exception_if_registering_#{spec.fetch(:description)}s_with_different_source_and_same_destination_paths" do
-        first = AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src1.css', 'assets/dest1.css')
-        err = assert_raises(RuntimeError) do
-          AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src2.css', 'assets/dest1.css')
+        with_fake_site do |site|
+          source_paths = %w{src1.css src2.css}.map do |file|
+            File.join(CSS_BUNDLE_SOURCE_DIR, file)
+          end
+          source_paths.each { |path| FileUtils.touch(path) }
+          first = AssetFileRegistry.send(spec.fetch(:method), site, source_paths[0], 'assets/dest1.css')
+          err = assert_raises(RuntimeError) do
+            AssetFileRegistry.send(spec.fetch(:method), site, source_paths[1], 'assets/dest1.css')
+          end
+          assert_equal(<<-END, err.to_s)
+Two or more ministamp tags with the same destination path "assets/dest1.css", but different asset source paths: "#{source_paths[1]}" vs. "#{source_paths[0]}"
+          END
+          assert_equal(1, asset_file_registry_size)
+          assert_contains_only(site.static_files, [first])
         end
-        assert_equal <<-END, err.to_s
-Two or more ministamp tags with the same destination path 'assets/dest1.css', but different asset source paths: '_assets/src2.css' vs. '_assets/src1.css'
-        END
-        assert_equal 1, asset_file_registry_size
-        assert_contains_only @site.static_files, [first]
       end
     end
 
     def test_raise_exception_if_registering_stamp_file_with_same_destination_path_as_existing_bundle_file
-      file = AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('type' => :css, 'destination_path' => 'assets/dest'))
-      err = assert_raises(RuntimeError) do
-        AssetFileRegistry.register_stamp_file(@site, '_assets/src1.css', 'assets/dest.css')
+      with_fake_site do |site|
+        FileUtils.touch('_assets/src.js')
+        file = AssetFileRegistry.register_bundle_file(site, bundle_config.merge('destination_path' => 'assets/dest'))
+        err = assert_raises(RuntimeError) do
+          AssetFileRegistry.register_stamp_file(site, '_assets/src.js', 'assets/dest.js')
+        end
+        assert_equal('ministamp tag has the same destination path as a minibundle block: assets/dest.js', err.to_s)
+        assert_equal(1, asset_file_registry_size)
+        assert_contains_only(site.static_files, [file])
       end
-      assert_equal "ministamp tag has the same destination path as a minibundle block: 'assets/dest.css'", err.to_s
-      assert_equal 1, asset_file_registry_size
-      assert_contains_only @site.static_files, [file]
     end
 
     def test_raise_exception_if_registering_bundle_file_with_same_destination_path_as_existing_stamp_file
-      file = AssetFileRegistry.register_stamp_file(@site, '_assets/src1.css', 'assets/dest.css')
-      err = assert_raises(RuntimeError) do
-        AssetFileRegistry.register_bundle_file(@site, bundle_config.merge('type' => :css, 'destination_path' => 'assets/dest'))
+      with_fake_site do |site|
+        FileUtils.touch('_assets/src.js')
+        file = AssetFileRegistry.register_stamp_file(site, '_assets/src.js', 'assets/dest.js')
+        err = assert_raises(RuntimeError) do
+          AssetFileRegistry.register_bundle_file(site, bundle_config.merge('destination_path' => 'assets/dest'))
+        end
+        assert_equal('minibundle block has the same destination path as a ministamp tag: assets/dest.js', err.to_s)
+        assert_equal(1, asset_file_registry_size)
+        assert_contains_only(site.static_files, [file])
       end
-      assert_equal "minibundle block has the same destination path as a ministamp tag: 'assets/dest.css'", err.to_s
-      assert_equal 1, asset_file_registry_size
-      assert_contains_only @site.static_files, [file]
     end
 
     [
@@ -152,14 +190,16 @@ Two or more ministamp tags with the same destination path 'assets/dest1.css', bu
       {description: 'development_file_collection', method: :register_development_file_collection}
     ].each do |spec|
       define_method :"test_remove_unused_#{spec.fetch(:description)}" do
-        AssetFileRegistry.send(spec.fetch(:method), @site, bundle_config)
-        AssetFileRegistry.clear_unused
+        with_fake_site do |site|
+          AssetFileRegistry.send(spec.fetch(:method), site, bundle_config)
+          AssetFileRegistry.clear_unused
 
-        assert_equal 1, asset_file_registry_size
+          assert_equal(1, asset_file_registry_size)
 
-        AssetFileRegistry.clear_unused
+          AssetFileRegistry.clear_unused
 
-        assert_equal 0, asset_file_registry_size
+          assert_equal(0, asset_file_registry_size)
+        end
       end
     end
 
@@ -168,14 +208,16 @@ Two or more ministamp tags with the same destination path 'assets/dest1.css', bu
       {description: 'development_file', method: :register_development_file}
     ].each do |spec|
       define_method :"test_remove_unused_#{spec.fetch(:description)}" do
-        AssetFileRegistry.send(spec.fetch(:method), @site, '_assets/src.css', 'assets/dest.css')
-        AssetFileRegistry.clear_unused
+        with_fake_site do |site|
+          AssetFileRegistry.send(spec.fetch(:method), site, STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH)
+          AssetFileRegistry.clear_unused
 
-        assert_equal 1, asset_file_registry_size
+          assert_equal(1, asset_file_registry_size)
 
-        AssetFileRegistry.clear_unused
+          AssetFileRegistry.clear_unused
 
-        assert_equal 0, asset_file_registry_size
+          assert_equal(0, asset_file_registry_size)
+        end
       end
     end
 
@@ -183,10 +225,10 @@ Two or more ministamp tags with the same destination path 'assets/dest1.css', bu
 
     def bundle_config
       {
-        'type'             => :css,
-        'source_dir'       => '_assets/src',
+        'type'             => :js,
+        'source_dir'       => JS_BUNDLE_SOURCE_DIR,
         'assets'           => %w{dependency app},
-        'destination_path' => 'assets/dest',
+        'destination_path' => JS_BUNDLE_DESTINATION_PATH,
         'minifier_cmd'     => 'unused_minifier_cmd'
       }
     end
@@ -194,9 +236,5 @@ Two or more ministamp tags with the same destination path 'assets/dest1.css', bu
     def asset_file_registry_size
       AssetFileRegistry.instance_variable_get(:@_files).size
     end
-
-    def make_site
-      make_fake_site('.')
-    end
   end
 end