jekyll-minibundle 3.0.0 → 4.0.0

data/test/unit/bundle_file_properties_test.rb

@@ -1,108 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require_relative '../support/fixture_config'
-require_relative '../support/static_file_config'
-require 'jekyll/minibundle/bundle_file'
-
-module Jekyll::Minibundle::Test
-  class BundleFilePropertiesTest < TestCase
-    include FixtureConfig
-    include StaticFileConfig
-
-    def setup
-      @@results ||= with_bundle_file do |file|
-        get_send_results(file, STATIC_FILE_PROPERTIES)
-      end
-    end
-
-    def test_basename
-      assert_equal("site-#{JS_BUNDLE_FINGERPRINT}", @@results.fetch(:basename))
-    end
-
-    def test_data
-      assert_equal({}, @@results.fetch(:data))
-    end
-
-    def test_defaults
-      assert_equal({}, @@results.fetch(:defaults))
-    end
-
-    def test_destination_rel_dir
-      assert_equal('/assets', @@results.fetch(:destination_rel_dir))
-    end
-
-    def test_extname
-      assert_equal('.js', @@results.fetch(:extname))
-    end
-
-    def test_modified_time
-      assert_instance_of(Time, @@results.fetch(:modified_time))
-    end
-
-    def test_mtime
-      mtime = @@results.fetch(:modified_time)
-      assert_equal(mtime.to_i, @@results.fetch(:mtime))
-    end
-
-    def test_name
-      assert_equal("site-#{JS_BUNDLE_FINGERPRINT}.js", @@results.fetch(:name))
-    end
-
-    def test_path
-      assert_match(%r{\A/.+/jekyll-minibundle-.+\.js\z}, @@results.fetch(:path))
-    end
-
-    def test_placeholders
-      assert_equal({}, @@results.fetch(:placeholders))
-    end
-
-    def test_relative_path
-      assert_match(%r{/jekyll-minibundle-.+\.js\z}, @@results.fetch(:relative_path))
-    end
-
-    def test_to_liquid
-      with_bundle_file do |file|
-        drop = file.to_liquid
-        assert_equal("site-#{JS_BUNDLE_FINGERPRINT}.js", drop.name)
-        assert_equal('.js', drop.extname)
-        assert_equal("site-#{JS_BUNDLE_FINGERPRINT}", drop.basename)
-        assert_instance_of(Time, drop.modified_time)
-        assert_match(%r{/jekyll-minibundle-.+\.js\z}, drop.path)
-        assert_nil(drop.collection)
-      end
-    end
-
-    def test_type
-      assert_nil(@@results.fetch(:type))
-    end
-
-    def test_url
-      assert_equal(JS_BUNDLE_DESTINATION_FINGERPRINT_PATH, @@results.fetch(:url))
-    end
-
-    def test_write?
-      assert(@@results.fetch(:write?))
-    end
-
-    private
-
-    def with_bundle_file(&block)
-      with_fake_site do |site|
-        file = BundleFile.new(site, bundle_config(minifier_cmd_to_remove_comments))
-        capture_io { file.destination_path_for_markup }
-        block.call(file)
-      end
-    end
-
-    def bundle_config(minifier_cmd)
-      {
-        'type'             => :js,
-        'source_dir'       => JS_BUNDLE_SOURCE_DIR,
-        'assets'           => %w[dependency app],
-        'destination_path' => JS_BUNDLE_DESTINATION_PATH,
-        'minifier_cmd'     => minifier_cmd
-      }
-    end
-  end
-end

data/test/unit/bundle_file_writing_test.rb

@@ -1,155 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require_relative '../support/fixture_config'
-require 'jekyll/minibundle/bundle_file'
-
-module Jekyll::Minibundle::Test
-  class BundleFileWritingTest < TestCase
-    include FixtureConfig
-
-    def test_raise_error_if_source_directory_does_not_exist
-      err = assert_raises(ArgumentError) do
-        with_fake_site do |site|
-          make_bundle_file(site, 'source_dir' => STAMP_SOURCE_PATH)
-        end
-      end
-      assert_match(%r{\ABundle source directory does not exist: .+/#{Regexp.escape(STAMP_SOURCE_PATH)}\z}, err.to_s)
-    end
-
-    def test_raise_error_if_asset_source_file_does_not_exist
-      err = assert_raises(ArgumentError) do
-        with_fake_site do |site|
-          make_bundle_file(site, 'assets' => %w[no-such])
-        end
-      end
-      assert_match(%r{\ABundle asset source file does not exist: .+/#{Regexp.escape(JS_BUNDLE_SOURCE_DIR)}/no-such.js\z}, err.to_s)
-    end
-
-    def test_calling_destination_path_for_markup_determines_fingerprint_and_destination_write
-      with_fake_site do |site|
-        bundle_file = make_bundle_file(site)
-        source = source_path(JS_BUNDLE_SOURCE_DIR, 'app.js')
-        old_destination = destination_path(JS_BUNDLE_DESTINATION_FINGERPRINT_PATH)
-        org_markup_path, last_markup_path = nil
-
-        # the call to destination_path_for_markup determines the
-        # fingerprint
-
-        capture_io { org_markup_path = bundle_file.destination_path_for_markup }
-
-        assert(write_file(bundle_file))
-
-        org_mtime = file_mtime_of(old_destination)
-
-        assert_equal(1, get_minifier_cmd_count)
-
-        last_markup_path = bundle_file.destination_path_for_markup
-
-        assert_equal(org_markup_path, last_markup_path)
-        assert_equal(1, get_minifier_cmd_count)
-
-        # change content, but don't call destination_path_for_markup yet
-
-        ensure_file_mtime_changes { File.write(source, '(function() {})()') }
-
-        # preserve content's fingerprint
-
-        refute(write_file(bundle_file))
-
-        assert_equal(org_mtime, file_mtime_of(old_destination))
-        assert_equal(1, get_minifier_cmd_count)
-
-        # see content's fingerprint to update after calling
-        # destination_path_for_markup
-
-        capture_io { last_markup_path = bundle_file.destination_path_for_markup }
-
-        refute_equal org_markup_path, last_markup_path
-
-        assert(write_file(bundle_file))
-
-        new_destination = destination_path('assets/site-375a0b430b0c5555d0edd2205d26c04d.js')
-
-        assert_operator(file_mtime_of(new_destination), :>, org_mtime)
-        assert_equal(2, get_minifier_cmd_count)
-      end
-    end
-
-    def test_many_consecutive_destination_path_for_markup_calls_trigger_one_destination_write
-      with_fake_site do |site|
-        bundle_file = make_bundle_file(site)
-        source = source_path(JS_BUNDLE_SOURCE_DIR, 'app.js')
-        destination = destination_path(JS_BUNDLE_DESTINATION_FINGERPRINT_PATH)
-        org_markup_path, last_markup_path = nil
-        capture_io { org_markup_path = bundle_file.destination_path_for_markup }
-        bundle_file.destination_path_for_markup
-
-        assert(write_file(bundle_file))
-
-        org_mtime = file_mtime_of(destination)
-
-        assert_equal(1, get_minifier_cmd_count)
-
-        ensure_file_mtime_changes { FileUtils.touch(source) }
-        capture_io { last_markup_path = bundle_file.destination_path_for_markup }
-        bundle_file.destination_path_for_markup
-
-        assert(write_file(bundle_file))
-        assert_equal(org_markup_path, last_markup_path)
-        assert_operator(file_mtime_of(destination), :>, org_mtime)
-        assert_equal(2, get_minifier_cmd_count)
-      end
-    end
-
-    def test_calling_write_before_destination_path_for_markup_has_no_effect
-      with_fake_site do |site|
-        bundle_file = make_bundle_file(site)
-
-        refute(write_file(bundle_file))
-        assert_empty(Dir[destination_path('assets/*.js')])
-        assert_equal(0, get_minifier_cmd_count)
-
-        capture_io { bundle_file.destination_path_for_markup }
-
-        assert(write_file(bundle_file))
-        assert(File.file?(destination_path(JS_BUNDLE_DESTINATION_FINGERPRINT_PATH)))
-        assert_equal(1, get_minifier_cmd_count)
-      end
-    end
-
-    def test_modified_property_determines_if_write_would_succeed
-      with_fake_site do |site|
-        bundle_file = make_bundle_file(site)
-
-        refute(bundle_file.modified?)
-        refute(write_file(bundle_file))
-
-        capture_io { bundle_file.destination_path_for_markup }
-
-        assert(bundle_file.modified?)
-        assert(write_file(bundle_file))
-
-        refute(bundle_file.modified?)
-        refute(write_file(bundle_file))
-      end
-    end
-
-    private
-
-    def make_bundle_file(site, config = {})
-      bundle_config = {
-        'type'             => :js,
-        'source_dir'       => JS_BUNDLE_SOURCE_DIR,
-        'assets'           => %w[dependency app],
-        'destination_path' => JS_BUNDLE_DESTINATION_PATH,
-        'minifier_cmd'     => minifier_cmd_to_remove_comments_and_count
-      }
-      BundleFile.new(site, bundle_config.merge(config))
-    end
-
-    def write_file(file)
-      file.write(File.join(Dir.pwd, '_site'))
-    end
-  end
-end

data/test/unit/development_file_properties_test.rb

@@ -1,97 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require_relative '../support/fixture_config'
-require_relative '../support/static_file_config'
-require 'jekyll/minibundle/development_file'
-
-module Jekyll::Minibundle::Test
-  class DevelopmentFilePropertiesTest < TestCase
-    include FixtureConfig
-    include StaticFileConfig
-
-    def setup
-      @@results ||= with_development_file do |file|
-        get_send_results(file, STATIC_FILE_PROPERTIES)
-      end
-    end
-
-    def test_basename
-      assert_equal('screen', @@results.fetch(:basename))
-    end
-
-    def test_data
-      assert_equal({}, @@results.fetch(:data))
-    end
-
-    def test_defaults
-      assert_equal({}, @@results.fetch(:defaults))
-    end
-
-    def test_destination_rel_dir
-      assert_equal('/assets', @@results.fetch(:destination_rel_dir))
-    end
-
-    def test_extname
-      assert_equal('.css', @@results.fetch(:extname))
-    end
-
-    def test_modified_time
-      assert_instance_of(Time, @@results.fetch(:modified_time))
-    end
-
-    def test_mtime
-      mtime = @@results.fetch(:modified_time)
-      assert_equal(mtime.to_i, @@results.fetch(:mtime))
-    end
-
-    def test_name
-      assert_equal('screen.css', @@results.fetch(:name))
-    end
-
-    def test_path
-      assert_match(%r{\A/.+/#{STAMP_SOURCE_PATH}\z}, @@results.fetch(:path))
-    end
-
-    def test_placeholders
-      assert_equal({}, @@results.fetch(:placeholders))
-    end
-
-    def test_relative_path
-      assert_equal("/#{STAMP_SOURCE_PATH}", @@results.fetch(:relative_path))
-    end
-
-    def test_to_liquid
-      with_development_file do |file|
-        drop = file.to_liquid
-        assert_equal('screen.css', drop.name)
-        assert_equal('.css', drop.extname)
-        assert_equal('screen', drop.basename)
-        assert_instance_of(Time, drop.modified_time)
-        assert_equal("/#{STAMP_SOURCE_PATH}", drop.path)
-        assert_nil(drop.collection)
-      end
-    end
-
-    def test_type
-      assert_nil(@@results.fetch(:type))
-    end
-
-    def test_url
-      assert_equal(STAMP_DESTINATION_PATH, @@results.fetch(:url))
-    end
-
-    def test_write?
-      assert(@@results.fetch(:write?))
-    end
-
-    private
-
-    def with_development_file(&block)
-      with_fake_site do |site|
-        file = DevelopmentFile.new(site, STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH)
-        block.call(file)
-      end
-    end
-  end
-end

data/test/unit/development_file_writing_test.rb

@@ -1,50 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require_relative '../support/fixture_config'
-require 'jekyll/minibundle/development_file'
-
-module Jekyll::Minibundle::Test
-  class DevelopmentFileWritingTest < TestCase
-    include FixtureConfig
-
-    def test_raise_error_if_source_file_does_not_exist
-      err = assert_raises(ArgumentError) do
-        with_fake_site do |site|
-          DevelopmentFile.new(site, '_tmp', STAMP_DESTINATION_PATH)
-        end
-      end
-      assert_match(%r{\ADevelopment source file does not exist: .+/_tmp\z}, err.to_s)
-    end
-
-    def test_modify_property_determines_if_write_would_succeed
-      with_fake_site do |site|
-        dev_file = make_development_file(site)
-        source = source_path(STAMP_SOURCE_PATH)
-
-        assert(dev_file.modified?)
-        assert(write_file(dev_file))
-
-        ensure_file_mtime_changes do
-          FileUtils.touch(source)
-        end
-
-        assert(dev_file.modified?)
-        assert(write_file(dev_file))
-
-        refute(dev_file.modified?)
-        refute(write_file(dev_file))
-      end
-    end
-
-    private
-
-    def make_development_file(site)
-      DevelopmentFile.new(site, STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH)
-    end
-
-    def write_file(file)
-      file.write(File.join(Dir.pwd, '_site'))
-    end
-  end
-end

data/test/unit/environment_test.rb

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require 'jekyll/minibundle/environment'
-
-module Jekyll::Minibundle::Test
-  class EnvironmentTest < TestCase
-    def test_find_site_config_returns_value_when_found
-      assert_equal(1, Environment.find_site_config(make_site(top: {leaf: 1}), %i[top leaf], Integer))
-    end
-
-    def test_find_site_config_returns_nil_when_not_found
-      assert_nil(Environment.find_site_config(make_site, %i[top leaf], Integer))
-    end
-
-    def test_find_site_config_raises_exception_if_found_value_is_of_unexpected_type
-      err = assert_raises(RuntimeError) do
-        Environment.find_site_config(make_site(top: {leaf: '1'}), %i[top leaf], Integer)
-      end
-      assert_equal('Invalid site configuration for key top.leaf; expecting type Integer', err.to_s)
-    end
-
-    private
-
-    def make_site(config = {})
-      OpenStruct.new(config: config)
-    end
-  end
-end

data/test/unit/files_test.rb

@@ -1,61 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require 'jekyll/minibundle/files'
-
-module Jekyll::Minibundle::Test
-  class FilesTest < TestCase
-    def test_copy_p
-      with_tmp_dir do
-        File.write('foo', 'content')
-        Files.copy_p('foo', 'bar/zap')
-        copied = File.read('bar/zap')
-        assert_equal('content', copied)
-      end
-    end
-
-    def test_read_last_with_non_existing_file_raises_exception
-      assert_raises(Errno::ENOENT) { Files.read_last('no-such-file', 4) }
-    end
-
-    def test_read_last_with_empty_file_returns_empty
-      Tempfile.open('test') do |file|
-        file.close
-
-        assert_equal('', Files.read_last(file.path, 4))
-      end
-    end
-
-    def test_read_last_with_file_bigger_than_max_size_returns_last_bytes
-      Tempfile.open('test') do |file|
-        file.write("1\n2\n3\n4")
-        file.close
-
-        assert_equal("\n3\n4", Files.read_last(file.path, 4))
-      end
-    end
-
-    def test_read_last_with_file_smaller_than_max_size_returns_all_contents
-      Tempfile.open('test') do |file|
-        file.write("1\n2")
-        file.close
-
-        assert_equal("1\n2", Files.read_last(file.path, 100))
-      end
-    end
-
-    def test_read_last_with_max_size_zero_or_negative_returns_empty
-      Tempfile.open('test') do |file|
-        file.write("1\n2")
-        file.close
-
-        assert_equal('', Files.read_last(file.path, 0))
-        assert_equal('', Files.read_last(file.path, -1))
-      end
-    end
-
-    def test_strip_dot_slash_from_path_start
-      assert_equal('path', Files.strip_dot_slash_from_path_start('./path'))
-    end
-  end
-end

data/test/unit/hashes_test.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-require_relative '../support/test_case'
-require 'jekyll/minibundle/hashes'
-
-module Jekyll::Minibundle::Test
-  class HashesTest < TestCase
-    def test_dig_returns_value_when_found
-      assert_equal(1, Hashes.dig({top: {middle: {leaf: 1}}}, :top, :middle, :leaf))
-      assert_equal(1, Hashes.dig({top: [{}, {leaf: 1}]}, :top, 1, :leaf))
-      assert_equal(1, Hashes.dig([{}, {middle: [{}, {leaf: 1}]}], 1, :middle, 1, :leaf))
-    end
-
-    def test_dig_returns_same_found_object
-      leaf_obj = {leaf: 1}
-      assert_same(leaf_obj, Hashes.dig({top: {middle: leaf_obj}}, :top, :middle))
-    end
-
-    def test_dig_returns_nil_when_not_found
-      assert_nil(Hashes.dig({}, :no_such))
-      assert_nil(Hashes.dig([], 0))
-      assert_nil(Hashes.dig({top: {}}, :top, :no_such_leaf))
-      assert_nil(Hashes.dig({top: {leaf: 1}}, :top, :no_such_leaf))
-      assert_nil(Hashes.dig({top: []}, :top, 0))
-      assert_nil(Hashes.dig([{leaf: 1}], 0, :no_such))
-    end
-
-    def test_dig_returns_nil_for_nil
-      assert_nil(Hashes.dig(nil, :no_such_key))
-      assert_nil(Hashes.dig(nil, 0))
-    end
-
-    def test_pick_returns_hash_with_specified_keys
-      assert_equal({}, Hashes.pick(a: 1, 'b' => 2, 'c' => 3))
-      assert_equal({}, Hashes.pick({}))
-      assert_equal({a: 1, 'c' => 3}, Hashes.pick({a: 1, 'b' => 2, 'c' => 3}, :a, 'c'))
-    end
-
-    def test_pick_raises_exception_if_key_does_not_exist
-      assert_raises(KeyError) { Hashes.pick({}, :no_such) }
-    end
-  end
-end