jekyll-minibundle 0.1.0 → 0.2.0

data/History.md

@@ -1,3 +1,9 @@
+### 0.2.0 / 2012-12-15
+
+* Escape the values of custom attributes given in `minibundle` block
+* Add semicolons between each JavaScript asset in bundling
+* Show error in page output if asset bundling command failed
+
 ### 0.1.0 / 2012-12-07
 
 * Add `ministamp` tag and `minibundle` block for Jekyll

data/License.txt

@@ -0,0 +1,19 @@
+Copyright (c) 2012 Tuomas Kareinen
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

data/Rakefile

@@ -2,9 +2,19 @@ require 'rake/clean'
 require 'shellwords'
 
 require_relative 'lib/jekyll/minibundle/version'
-gem_name = 'jekyll-minibundle'
+
+def get_minibundle_env(overrides = {})
+  bundle_cmd = File.expand_path(File.join(File.dirname(__FILE__), 'test/fixture/site/_bin/remove_comments'))
+  {
+    'JEKYLL_MINIBUNDLE_CMD_JS'  => bundle_cmd,
+    'JEKYLL_MINIBUNDLE_CMD_CSS' => bundle_cmd,
+    'RUBYLIB'                   => File.expand_path(File.join(File.dirname(__FILE__), 'lib'))
+  }.merge(overrides)
+end
 
 namespace :gem do
+  gem_name = 'jekyll-minibundle'
+
   CLEAN.include "#{gem_name}-*.gem"
 
   desc 'Package the software as a gem'
@@ -25,19 +35,17 @@ end
 
 desc 'Run tests'
 task :test do
-  test_dir = 'test'
-  test_glob = 'integration/**/*_test.rb'
-  includes = ['lib', test_dir].join(':')
-  tests = Dir["#{test_dir}/#{test_glob}"].
+  tests = Dir['test/**/*_test.rb'].
     map { |file| %r{^test/(.+)\.rb$}.match(file)[1] }.
     shelljoin
-  test_cmd = %{bundle exec ruby -I#{includes} -e 'ARGV.each { |f| require f }' #{tests}}
-  bundle_cmd = File.expand_path(File.join(File.dirname(__FILE__), test_dir, 'fixture/site/_bin/remove_comments'))
-  env = {
-    'JEKYLL_MINIBUNDLE_CMD_JS' => bundle_cmd,
-    'JEKYLL_MINIBUNDLE_CMD_CSS' => bundle_cmd
-  }
-  sh(env, test_cmd)
+  test_cmd = %{bundle exec ruby -e 'ARGV.each { |f| require f }' #{tests}}
+  sh(get_minibundle_env('RUBYLIB' => 'lib:test'), test_cmd)
+end
+
+desc 'Generate fixture site for debugging'
+task :debug do
+  Dir.chdir 'test/fixture/site'
+  sh(get_minibundle_env, 'jekyll')
 end
 
 CLEAN.include 'test/fixture/site/_site'

data/Readme.md

@@ -4,12 +4,14 @@ A minimalistic plugin for bundling assets to
 [Jekyll](https://github.com/mojombo/jekyll)'s site generation
 directory.
 
-In addition to the plugin, you need your asset bundling tool of choice
-only. The plugin needs no other configuration than setting an
-environment variable.
+In addition to the plugin itself, you only need your asset bundling
+tool of choice. The plugin needs no other configuration than setting
+an environment variable. There are no gem dependencies.
 
 Tested with Ruby MRI 1.9.3. Ruby 1.8 is *not* supported.
 
+[![Build Status](https://secure.travis-ci.org/tkareine/jekyll-minibundle.png)](http://travis-ci.org/tkareine/jekyll-minibundle)
+
 # Features
 
 There are two features: asset stamping with MD5 digest over the
@@ -94,7 +96,7 @@ You can pass custom attributes to the generated markup with
 
 Output:
 
-    <script type="text/javascript" src="assets/site-8e764372a0dbd296033cb2a416f064b5.js" id="my-scripts"></script>
+    <script src="assets/site-8e764372a0dbd296033cb2a416f064b5.js" type="text/javascript" id="my-scripts"></script>
 
 For bundling CSS assets, you use `css` as the argument to `minibundle` block:
 
@@ -117,22 +119,4 @@ See the contents of `test/fixture/site` directory.
 
 # License
 
-Copyright (c) 2012 Tuomas Kareinen
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
+MIT. See `License.txt`.

data/lib/jekyll/minibundle/asset_bundle.rb

@@ -18,6 +18,7 @@ module Jekyll::Minibundle
         @assets.each do |asset|
           puts "  #{asset}"
           IO.foreach(asset) { |line| wr.write line }
+          wr.puts ';' if @type == :js
         end
       end
       self
@@ -40,7 +41,8 @@ module Jekyll::Minibundle
       end
       yield wr
       wr.close
-      Process.waitpid2 pid
+      _, status = Process.waitpid2 pid
+      raise "Bundling #{@type} assets failed with exit status #{status.exitstatus}, command: #{cmd}" if status.exitstatus != 0
     ensure
       wr.close unless wr.closed?
     end

data/lib/jekyll/minibundle/asset_file_support.rb

@@ -7,8 +7,8 @@ module Jekyll::Minibundle
       site.static_files << self unless static_file_exists
     end
 
-    def write_destination(gensite_dir)
-      destination_path = destination gensite_dir
+    def write_destination(site_destination_dir)
+      destination_path = destination site_destination_dir
       FileUtils.mkdir_p File.dirname(destination_path)
       FileUtils.cp path, destination_path
     end

data/lib/jekyll/minibundle/bundle_file.rb

@@ -10,10 +10,10 @@ module Jekyll::Minibundle
 
     def initialize(config)
       @type = config['type']
-      @site_dir = config['site_dir']
-      source_dir = File.join @site_dir, config['source_dir']
-      @destination_path = config['destination_path']
-      @assets = config['assets'].map { |asset_path| File.join source_dir, "#{asset_path}.#{@type}" }
+      @site_source_dir = config['site_dir']
+      asset_source_dir = File.join @site_source_dir, config['source_dir']
+      @assets = config['assets'].map { |asset_path| File.join asset_source_dir, "#{asset_path}.#{@type}" }
+      @asset_destination_path = config['destination_path']
       @attributes = config['attributes']
       update_mtime
     end
@@ -23,11 +23,11 @@ module Jekyll::Minibundle
     end
 
     def asset_path
-      "#{@destination_path}-#{asset_stamp}.#{@type}"
+      "#{@asset_destination_path}-#{asset_stamp}.#{@type}"
     end
 
-    def destination(destination_base_dir)
-      File.join destination_base_dir, asset_path
+    def destination(site_destination_dir)
+      File.join site_destination_dir, asset_path
     end
 
     def mtime
@@ -38,14 +38,14 @@ module Jekyll::Minibundle
       @@mtimes[path] != mtime
     end
 
-    def write(gensite_dir)
+    def write(site_destination_dir)
       rebundle_assets if modified?
-      destination_path = destination gensite_dir
+      destination_path = destination site_destination_dir
 
       return false if File.exist?(destination_path) and !modified?
 
       update_mtime
-      write_destination gensite_dir
+      write_destination site_destination_dir
 
       true
     end
@@ -61,7 +61,7 @@ module Jekyll::Minibundle
     end
 
     def asset_bundle
-      @asset_bundle ||= AssetBundle.new(@type, @assets, @site_dir).make_bundle
+      @asset_bundle ||= AssetBundle.new(@type, @assets, @site_source_dir).make_bundle
     end
 
     def rebundle_assets

data/lib/jekyll/minibundle/bundle_markup.rb

@@ -1,3 +1,5 @@
+require 'cgi'
+
 module Jekyll::Minibundle
   module BundleMarkup
     def self.make_markup(type, path, attributes)
@@ -12,7 +14,11 @@ module Jekyll::Minibundle
     end
 
     def self.make_attributes(attributes)
-      attributes.map { |name, value| %{ #{name}="#{value}"} }.join('')
+      attributes.map { |name, value| make_attribute(name, value) }.join('')
+    end
+
+    def self.make_attribute(name, value)
+      %{ #{name}="#{CGI.escape_html(value)}"}
     end
   end
 end

data/lib/jekyll/minibundle/mini_stamp_tag.rb

@@ -4,12 +4,12 @@ module Jekyll::Minibundle
   class MiniStampTag < Liquid::Tag
     def initialize(tag_name, text, _tokens)
       super
-      @source, @destination = text.split(/\s+/, 3)[0, 2]
+      @asset_source, @asset_destination = text.split(/\s+/, 3)[0, 2]
     end
 
     def render(context)
       site = context.registers[:site]
-      file = StampFile.new File.join(site.source, @source), @destination
+      file = StampFile.new File.join(site.source, @asset_source), @asset_destination
       file.static_file! site
       file.asset_path
     end

data/lib/jekyll/minibundle/stamp_file.rb

@@ -7,25 +7,24 @@ module Jekyll::Minibundle
 
     @@mtimes = Hash.new
 
-    def initialize(source_path, destination_path)
-      @source_path = source_path
-      @destination_dir = File.dirname destination_path
-      @destination_extension = File.extname destination_path
-      base = File.basename destination_path
-      @destination_base_prefix = base[0 .. -(@destination_extension.size + 1)]
+    def initialize(asset_source_path, asset_destination_path)
+      @asset_source_path = asset_source_path
+      @asset_destination_dir = File.dirname asset_destination_path
+      @asset_destination_extension = File.extname asset_destination_path
+      @asset_destination_base_prefix = File.basename(asset_destination_path)[0 .. -(@asset_destination_extension.size + 1)]
       update_mtime
     end
 
     def path
-      @source_path
+      @asset_source_path
     end
 
     def asset_path
-      File.join @destination_dir, destination_basename
+      File.join @asset_destination_dir, asset_destination_basename
     end
 
-    def destination(gensite_dir)
-      File.join gensite_dir, @destination_dir, destination_basename
+    def destination(site_destination_dir)
+      File.join site_destination_dir, @asset_destination_dir, asset_destination_basename
     end
 
     def mtime
@@ -36,22 +35,22 @@ module Jekyll::Minibundle
       @@mtimes[path] != mtime
     end
 
-    def write(gensite_dir)
+    def write(site_destination_dir)
       clear_asset_stamp if modified?
-      destination_path = destination gensite_dir
+      destination_path = destination site_destination_dir
 
       return false if File.exist?(destination_path) and !modified?
 
       update_mtime
-      write_destination gensite_dir
+      write_destination site_destination_dir
 
       true
     end
 
     private
 
-    def destination_basename
-      "#{@destination_base_prefix}-#{asset_stamp}#{@destination_extension}"
+    def asset_destination_basename
+      "#{@asset_destination_base_prefix}-#{asset_stamp}#{@asset_destination_extension}"
     end
 
     def asset_stamp

data/lib/jekyll/minibundle/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module Minibundle
-    VERSION = '0.1.0'
+    VERSION = '0.2.0'
   end
 end

data/test/fixture/site/_assets/scripts/app.js

@@ -2,4 +2,4 @@
 (function(root, dependency) {
   if (!dependency) throw new Error("missing dependency");
   root.app = {};
-})(window, window.dependency);
+})(window, window.dependency)

data/test/fixture/site/_assets/scripts/dependency.js

@@ -1,4 +1,4 @@
 /* comment to be removed in bundling */
 (function(root) {
   root.dependency = {};
-})(window);
+})(window)

data/test/integration/minibundle_test.rb

@@ -3,7 +3,7 @@ require 'support/test_case'
 module Jekyll::Minibundle::Test
   class MiniBundleTest < TestCase
     EXPECTED_CSS_ASSET_PATH = 'assets/site-b2e0ecc1c100effc2c7353caad20c327.css'
-    EXPECTED_JS_ASSET_PATH = 'assets/site-f78e0c4497343c33e9282df5d684540e.js'
+    EXPECTED_JS_ASSET_PATH = 'assets/site-4782a1f67803038d4f8351051e67deb8.js'
 
     def test_css_asset_bundle_has_configured_attributes
       element = find_html_element_from_index(%{head link[href="#{EXPECTED_CSS_ASSET_PATH}"]}).first
@@ -27,44 +27,49 @@ module Jekyll::Minibundle::Test
     end
 
     def test_css_asset_bundle_is_copied_to_destination_dir
-      assert File.exists?(gensite_path(EXPECTED_CSS_ASSET_PATH))
+      assert File.exists?(destination_path(EXPECTED_CSS_ASSET_PATH))
     end
 
     def test_js_asset_bundle_is_copied_to_destination_dir
-      assert File.exists?(gensite_path(EXPECTED_JS_ASSET_PATH))
+      assert File.exists?(destination_path(EXPECTED_JS_ASSET_PATH))
     end
 
     def test_css_asset_bundle_is_concatenated_in_configured_order
-      bundle = File.read(gensite_path(EXPECTED_CSS_ASSET_PATH))
+      bundle = File.read(destination_path(EXPECTED_CSS_ASSET_PATH))
       assert bundle.index('html { margin: 0; }') < bundle.index('p { margin: 0; }')
     end
 
     def test_js_asset_bundle_is_concatenated_in_configured_order
-      bundle = File.read(gensite_path(EXPECTED_JS_ASSET_PATH))
+      bundle = File.read(destination_path(EXPECTED_JS_ASSET_PATH))
       assert bundle.index('root.dependency = {};') < bundle.index('root.app = {};')
     end
 
+    def test_js_asset_bundle_has_inserted_semicolons_between_assets
+      bundle = File.read(destination_path(EXPECTED_JS_ASSET_PATH))
+      assert_match(%r|}\)\(window\)\n;\n\(function|, bundle)
+    end
+
     def test_css_asset_bundle_is_minified
       source_contents_size = source_assets_size('_assets/styles', %w{reset common}, 'css')
-      destination_contents_size = File.read(gensite_path(EXPECTED_CSS_ASSET_PATH)).size
+      destination_contents_size = File.read(destination_path(EXPECTED_CSS_ASSET_PATH)).size
       assert destination_contents_size < source_contents_size
     end
 
     def test_js_asset_bundle_is_minified
       source_contents_size = source_assets_size('_assets/scripts', %w{dependency app}, 'js')
-      destination_contents_size = File.read(gensite_path(EXPECTED_JS_ASSET_PATH)).size
+      destination_contents_size = File.read(destination_path(EXPECTED_JS_ASSET_PATH)).size
       assert destination_contents_size < source_contents_size
     end
 
     private
 
     def find_html_element_from_index(css)
-      find_html_element(read_from_gensite('index.html'), css)
+      find_html_element(read_from_destination('index.html'), css)
     end
 
-    def source_assets_size(fixture_subdir, assets, type)
+    def source_assets_size(source_subdir, assets, type)
       assets.
-        map { |f| File.read fixture_path(fixture_subdir, "#{f}.#{type}") }.
+        map { |f| File.read source_path(source_subdir, "#{f}.#{type}") }.
         join('').
         size
     end

data/test/integration/ministamp_test.rb

@@ -5,17 +5,17 @@ module Jekyll::Minibundle::Test
     EXPECTED_ASSET_PATH = 'assets/screen-390be921ee0eff063817bb5ef2954300.css'
 
     def test_asset_path_has_stamp
-      actual = find_html_element(read_from_gensite('index.html'), 'head link').first['href']
+      actual = find_html_element(read_from_destination('index.html'), 'head link').first['href']
       assert_equal EXPECTED_ASSET_PATH, actual
     end
 
     def test_asset_file_is_copied_to_destination_dir
-      assert File.exists?(gensite_path(EXPECTED_ASSET_PATH))
+      assert File.exists?(destination_path(EXPECTED_ASSET_PATH))
     end
 
     def test_asset_file_is_equal_to_source_file
-      source_contents = File.read fixture_path('_tmp/site.css')
-      destination_contents = File.read gensite_path(EXPECTED_ASSET_PATH)
+      source_contents = File.read source_path('_tmp/site.css')
+      destination_contents = File.read destination_path(EXPECTED_ASSET_PATH)
       assert_equal source_contents, destination_contents
     end
   end

data/test/support/test_case.rb

@@ -7,45 +7,59 @@ require 'jekyll/minibundle'
 
 module Jekyll::Minibundle::Test
   class TestCase < ::MiniTest::Unit::TestCase
-    FIXTURE_DIR = File.expand_path(File.join(File.dirname(__FILE__), '../fixture/site'))
+    include ::Jekyll::Minibundle
 
-    def fixture_path(*args)
-      File.join(FIXTURE_DIR, *args)
+    FIXTURE_DIR = File.expand_path(File.join(File.dirname(__FILE__), '../fixture'))
+    SOURCE_DIR = File.join(FIXTURE_DIR, 'site')
+
+    def source_path(*args)
+      File.join(SOURCE_DIR, *args)
     end
 
-    def gensite_path(*args)
-      File.join(_gensite_dir, *args)
+    def destination_path(*args)
+      File.join(_destination_dir, *args)
     end
 
-    def read_from_gensite(*args)
-      File.read gensite_path(*args)
+    def read_from_destination(*args)
+      File.read destination_path(*args)
     end
 
     def find_html_element(file, css)
       Nokogiri::HTML(file).css(css)
     end
 
+    def with_env(env)
+      org_env = {}
+      env.each do |k, v|
+        org_env[k] = ENV[k]
+        ENV[k] = v
+      end
+      yield
+    ensure
+      org_env.each { |k, v| ENV[k] = v }
+    end
+
     private
 
-    def _gensite_dir(&block)
-      @@_gensite_dir ||= begin
+    def _destination_dir(&block)
+      @@_destination_dir ||= begin
         dir = Dir.mktmpdir('jekyll-minibundle-test-')
         at_exit do
           FileUtils.rm_rf dir
-          puts "Cleaned generated site for tests: #{dir}"
+          puts "\nCleaned generated site for tests: #{dir}"
         end
         Dir.chdir(dir) { _generate_site dir }
-        puts "Generated site for tests: #{dir}"
+        puts "\nGenerated site for tests: #{dir}"
         dir
       end
     end
 
     def _generate_site(destination)
       options = {
-        'source'      => fixture_path,
+        'source'      => source_path,
         'destination' => destination
       }
-      Jekyll::Site.new(Jekyll.configuration(options)).process
+      capture_io { Jekyll::Site.new(Jekyll.configuration(options)).process }
     end
   end
 end

data/test/unit/asset_bundle_test.rb

@@ -0,0 +1,35 @@
+require 'support/test_case'
+require 'jekyll/minibundle/asset_bundle'
+
+module Jekyll::Minibundle::Test
+  class AssetBundleTest < TestCase
+    def test_raise_exception_if_bundle_command_fails
+      capture_io do
+        with_env('JEKYLL_MINIBUNDLE_CMD_JS' => 'false') do
+          err = assert_raises(RuntimeError) { make_bundle }
+          assert_equal 'Bundling js assets failed with exit status 1, command: false', err.to_s
+        end
+      end
+    end
+
+    def test_raise_exception_if_bundle_command_not_found
+      with_env('JEKYLL_MINIBUNDLE_CMD_JS' => 'no-such-jekyll-minibundle-cmd') do
+        assert_raises(Errno::ENOENT) { make_bundle }
+      end
+    end
+
+    def test_raise_exception_if_bundle_command_not_configured
+      with_env('JEKYLL_MINIBUNDLE_CMD_JS' => nil) do
+        err = assert_raises(RuntimeError) { make_bundle }
+        assert_equal 'You need to set bundling command in $JEKYLL_MINIBUNDLE_CMD_JS', err.to_s
+      end
+    end
+
+    private
+
+    def make_bundle
+      bundle = AssetBundle.new :js, [source_path('_assets/scripts/dependency.js')], source_path
+      bundle.make_bundle
+    end
+  end
+end

data/test/unit/bundle_markup_test.rb

@@ -0,0 +1,13 @@
+require 'support/test_case'
+require 'jekyll/minibundle/bundle_markup'
+
+module Jekyll::Minibundle::Test
+  class BundleMarkupTest < TestCase
+    def test_escape_attribute_value
+      attributes = { media: 'screen, projection', extra: '">attack<br' }
+      actual = BundleMarkup.make_markup :css, 'http://localhost', attributes
+      expected = %{<link rel="stylesheet" href="http://localhost" media="screen, projection" extra="&quot;&gt;attack&lt;br">}
+      assert_equal expected, actual
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-minibundle
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-07 00:00:00.000000000 Z
+date: 2012-12-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -59,12 +59,27 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.5.5
-description: ! 'Provides ''minibundle'' block for bundling multiple assets, ''ministamp''
-
-  tag for stamping a single asset.
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 10.0.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 10.0.2
+description: ! 'A minimalistic asset bundling plugin for Jekyll, requiring only your
 
+  bundling tool of choice (no other dependencies, not even other gems).
 
-  No other dependencies than the asset bundler of your choice.
+  Provides asset bundling and filename stamping with MD5 digest.
 
 '
 email: tkareine@gmail.com
@@ -73,6 +88,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - History.md
+- License.txt
 - Rakefile
 - Readme.md
 - lib/jekyll/minibundle.rb
@@ -95,6 +111,8 @@ files:
 - test/fixture/site/index.html
 - test/integration/minibundle_test.rb
 - test/integration/ministamp_test.rb
+- test/unit/asset_bundle_test.rb
+- test/unit/bundle_markup_test.rb
 - test/support/test_case.rb
 homepage: https://github.com/tkareine/jekyll-minibundle
 licenses:
@@ -125,7 +143,7 @@ rubyforge_project:
 rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
-summary: Straightforward asset bundling plugin for Jekyll
+summary: A minimalistic asset bundling plugin for Jekyll
 test_files:
 - test/fixture/site/_assets/scripts/app.js
 - test/fixture/site/_assets/scripts/dependency.js
@@ -137,5 +155,7 @@ test_files:
 - test/fixture/site/index.html
 - test/integration/minibundle_test.rb
 - test/integration/ministamp_test.rb
+- test/unit/asset_bundle_test.rb
+- test/unit/bundle_markup_test.rb
 - test/support/test_case.rb
 has_rdoc: