jekyll-minibundle 2.2.0 → 3.0.0

data/test/unit/files_test.rb

@@ -1,4 +1,6 @@
-require 'support/test_case'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
 require 'jekyll/minibundle/files'
 
 module Jekyll::Minibundle::Test

data/test/unit/hashes_test.rb

@@ -1,4 +1,6 @@
-require 'support/test_case'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
 require 'jekyll/minibundle/hashes'
 
 module Jekyll::Minibundle::Test

data/test/unit/jekyll_static_file_api_test.rb

@@ -1,22 +1,26 @@
-require 'support/test_case'
-require 'support/static_file_config'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
+require_relative '../support/static_file_config'
 require 'jekyll/minibundle/development_file'
 require 'jekyll/minibundle/bundle_file'
 require 'jekyll/minibundle/stamp_file'
 
 module Jekyll::Minibundle::Test
   class JekyllStaticFileAPITest < TestCase
+    IGNORED_STATIC_FILE_METHODS = %i[cleaned_relative_path].freeze
+
     include StaticFileConfig
 
-    def test_development_file_has_all_static_file_methods
+    def test_development_file_has_most_static_file_methods
       assert_empty(missing_static_file_methods(DevelopmentFile))
     end
 
-    def test_bundle_file_has_all_static_file_methods
+    def test_bundle_file_has_most_static_file_methods
       assert_empty(missing_static_file_methods(BundleFile))
     end
 
-    def test_stamp_file_has_all_static_file_methods
+    def test_stamp_file_has_most_static_file_methods
       assert_empty(missing_static_file_methods(StampFile))
     end
 
@@ -42,7 +46,7 @@ module Jekyll::Minibundle::Test
 
         return_type_diff = diff_non_nil_response_types_of_static_file_properties(
           make_static_file(site, 'static.txt'),
-          make_bundle_file(site, %w{dependency app})
+          make_bundle_file(site, %w[dependency app])
         )
 
         assert_empty(return_type_diff)
@@ -82,7 +86,7 @@ module Jekyll::Minibundle::Test
         FileUtils.touch('app.js')
 
         expected_keys = make_static_file(site, 'static.txt').to_liquid.keys.sort
-        actual_keys = make_bundle_file(site, %w{dependency app}).to_liquid.keys.sort
+        actual_keys = make_bundle_file(site, %w[dependency app]).to_liquid.keys.sort
 
         assert_equal(expected_keys, actual_keys)
       end
@@ -103,7 +107,7 @@ module Jekyll::Minibundle::Test
     private
 
     def missing_static_file_methods(clazz)
-      Jekyll::StaticFile.public_instance_methods - clazz.public_instance_methods
+      Jekyll::StaticFile.public_instance_methods - IGNORED_STATIC_FILE_METHODS - clazz.public_instance_methods
     end
 
     def response_types_of_methods(file, methods)
@@ -112,7 +116,7 @@ module Jekyll::Minibundle::Test
           begin
             value = file.send(method_name)
             {returned_type: value.class}
-          rescue => e
+          rescue StandardError => e
             {raised: e}
           end
       end

data/test/unit/mini_bundle_block_test.rb

@@ -1,5 +1,7 @@
-require 'support/test_case'
-require 'support/fixture_config'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
+require_relative '../support/fixture_config'
 require 'jekyll/minibundle/mini_bundle_block'
 
 module Jekyll::Minibundle::Test
@@ -18,34 +20,39 @@ module Jekyll::Minibundle::Test
     end
 
     def test_raise_exception_if_missing_block_contents
-      rendered = render_template(<<-END)
-{% minibundle css %}
+      err = assert_raises(ArgumentError) do
+        render_template(<<~LIQUID)
+          {% minibundle css %}
 
-{% endminibundle %}
-        END
-      expected = "Liquid error: Missing configuration for minibundle block; pass configuration in YAML syntax\n"
-      assert_equal(expected, rendered)
+          {% endminibundle %}
+        LIQUID
+      end
+      assert_equal('Missing configuration for minibundle block; pass configuration in YAML syntax', err.to_s)
     end
 
     def test_raise_exception_if_invalid_block_contents_syntax
-      rendered = render_template(<<-END)
-{% minibundle css %}
-}
-{% endminibundle %}
-        END
+      err = assert_raises(ArgumentError) do
+        render_template(<<~LIQUID)
+          {% minibundle css %}
+          }
+          {% endminibundle %}
+        LIQUID
+      end
       expected =
-        'Liquid error: Failed parsing minibundle block contents syntax as YAML: "}". ' \
-        "Cause: (<unknown>): did not find expected node content while parsing a block node at line 2 column 1\n"
-      assert_equal(expected, rendered)
+        'Failed parsing minibundle block contents syntax as YAML: "}". ' \
+        'Cause: (<unknown>): did not find expected node content while parsing a block node at line 2 column 1'
+      assert_equal(expected, err.to_s)
     end
 
     def test_raise_exception_if_unsupported_block_contents_type
-      rendered = render_template(<<-END)
-{% minibundle css %}
-str
-{% endminibundle %}
-        END
-      assert_equal("Liquid error: Unsupported minibundle block contents type (String), only Hash is supported: \nstr\n\n", rendered)
+      err = assert_raises(ArgumentError) do
+        render_template(<<~LIQUID)
+          {% minibundle css %}
+          str
+          {% endminibundle %}
+        LIQUID
+      end
+      assert_equal("Unsupported minibundle block contents type (String), only Hash is supported: \nstr\n", err.to_s)
     end
 
     [
@@ -86,19 +93,19 @@ str
       }
     ].each do |spec|
       define_method :"test_normalizing_baseurl_with_#{spec.fetch(:description)}" do
-        actual_output = render_template(<<-END)
-{% minibundle css %}
-source_dir: _assets/styles
-destination_path: assets/site
-#{spec.fetch(:baseurl_config)}
-assets:
-  - reset
-  - common
-minifier_cmd: #{minifier_cmd_to_remove_comments}
-{% endminibundle %}
-        END
+        actual_output = render_template(<<~LIQUID)
+          {% minibundle css %}
+          source_dir: _assets/styles
+          destination_path: assets/site
+          #{spec.fetch(:baseurl_config)}
+          assets:
+            - reset
+            - common
+          minifier_cmd: #{minifier_cmd_to_remove_comments}
+          {% endminibundle %}
+        LIQUID
 
-        expected_output = %{<link rel="stylesheet" href="#{spec.fetch(:expected_asset_url)}">\n}
+        expected_output = %(<link rel="stylesheet" href="#{spec.fetch(:expected_asset_url)}">\n)
 
         assert_equal(expected_output, actual_output)
       end
@@ -110,7 +117,7 @@ minifier_cmd: #{minifier_cmd_to_remove_comments}
       compiled = Liquid::Template.parse(template)
       rendered = nil
       capture_io do
-        rendered = compiled.render({}, registers: {site: make_fake_site(site_fixture_path)})
+        rendered = compiled.render!({}, registers: {site: make_fake_site(site_fixture_path)})
       end
       rendered
     end

data/test/unit/mini_stamp_tag_test.rb

@@ -1,5 +1,7 @@
-require 'support/test_case'
-require 'support/fixture_config'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
+require_relative '../support/fixture_config'
 require 'jekyll/minibundle/mini_stamp_tag'
 
 module Jekyll::Minibundle::Test

data/test/unit/stamp_file_properties_test.rb

@@ -1,6 +1,8 @@
-require 'support/test_case'
-require 'support/fixture_config'
-require 'support/static_file_config'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
+require_relative '../support/fixture_config'
+require_relative '../support/static_file_config'
 require 'jekyll/minibundle/stamp_file'
 
 module Jekyll::Minibundle::Test

data/test/unit/stamp_file_writing_test.rb

@@ -1,5 +1,7 @@
-require 'support/test_case'
-require 'support/fixture_config'
+# frozen_string_literal: true
+
+require_relative '../support/test_case'
+require_relative '../support/fixture_config'
 require 'jekyll/minibundle/stamp_file'
 
 module Jekyll::Minibundle::Test

data/test/unit/variable_template_test.rb

@@ -1,6 +1,6 @@
-# coding: utf-8
+# frozen_string_literal: true
 
-require 'support/test_case'
+require_relative '../support/test_case'
 require 'jekyll/minibundle/variable_template'
 
 module Jekyll::Minibundle::Test
@@ -49,10 +49,10 @@ module Jekyll::Minibundle::Test
 
     def test_raise_error_if_unclosed_open_tag
       err = assert_raises(VariableTemplate::SyntaxError) { VariableTemplate.compile('beg{{var}end') }
-      expected = <<-END
-Missing closing tag ("}}") for variable opening tag ("{{") at position 5 in template (position highlighted with "@"):
-beg{{@var}end
-      END
+      expected = <<~MESSAGE
+        Missing closing tag ("}}") for variable opening tag ("{{") at position 5 in template (position highlighted with "@"):
+        beg{{@var}end
+      MESSAGE
       assert_equal(expected, err.to_s)
     end
 
@@ -85,7 +85,7 @@ beg{{@var}end
     end
 
     def test_generator_escapes_text_token
-      assert_equal('#{}', render('#{}', {}))
+      assert_equal('#{}', render('#{}', {}))  # rubocop:disable Lint/InterpolationCheck
       assert_equal('"hey"', render('"hey"', {}))
       assert_equal("'hey'", render("'hey'", {}))
     end
@@ -94,7 +94,7 @@ beg{{@var}end
       assert_equal('42', render('{{ \\ }}', '\\' => 42))
       assert_equal('42', render("{{ ' }}", "'" => 42))
       assert_equal('42', render('{{ " }}', '"' => 42))
-      assert_equal('42', render('{{ #{} }}', '#{}' => 42))
+      assert_equal('42', render('{{ #{} }}', '#{}' => 42))  # rubocop:disable Lint/InterpolationCheck
       assert_equal("beg'42'end", render("beg'{{ 'var' }}'end", "'var'" => 42))
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-minibundle
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Tuomas Kareinen
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2017-03-10 00:00:00.000000000 Z
+date: 2020-09-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -28,14 +28,14 @@ dependencies:
   name: jekyll
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
@@ -86,28 +86,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.47.0
+        version: 0.90.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.47.0
+        version: 0.90.0
 description: |
   A straightforward asset bundling plugin for Jekyll, utilizing external
   minification tool of your choice. It provides asset concatenation for
@@ -187,7 +187,7 @@ homepage: https://github.com/tkareine/jekyll-minibundle
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options:
 - "--line-numbers"
 - "--title"
@@ -200,16 +200,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.4.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.10
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: A minimalistic asset bundling plugin for Jekyll
 test_files: