jekyll-minibundle 1.4.2 → 1.4.3

data/test/unit/development_file_collection_test.rb

@@ -0,0 +1,44 @@
+require 'support/test_case'
+require 'support/fixture_config'
+require 'jekyll/minibundle/development_file'
+
+module Jekyll::Minibundle::Test
+  class DevelopmentFileCollectionTest < TestCase
+    include FixtureConfig
+
+    def test_calling_write_before_markup_writes_destination
+      with_site do
+        dev_files = DevelopmentFileCollection.new(bundle_config)
+
+        assert first_file_of(dev_files).write('_site')
+
+        destination_file = destination_path(JS_BUNDLE_DESTINATION_PATH, 'dependency.js')
+
+        assert File.exists?(destination_file)
+
+        org_mtime = mtime_of(destination_file)
+        dev_files.markup
+
+        refute first_file_of(dev_files).write('_site')
+        assert_equal org_mtime, mtime_of(destination_file)
+      end
+    end
+
+    private
+
+    def bundle_config
+      {
+       'type'             => :js,
+       'site_dir'         => '.',
+       'source_dir'       => JS_BUNDLE_SOURCE_DIR,
+       'assets'           => %w{dependency app},
+       'destination_path' => JS_BUNDLE_DESTINATION_PATH,
+       'attributes'       => {}
+      }
+    end
+
+    def first_file_of(dev_files)
+      dev_files.instance_variable_get(:@files).first
+    end
+  end
+end

data/test/unit/stamp_file_test.rb

@@ -9,27 +9,27 @@ module Jekyll::Minibundle::Test
     def test_calling_markup_determines_fingerprint_and_destination_write
       with_site do
         stamp_file = StampFile.new(STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH, &stamp_basenamer)
-
-        source = source_path STAMP_SOURCE_PATH
-        old_destination = destination_path STAMP_DESTINATION_FINGERPRINT_PATH
-
+        source = source_path(STAMP_SOURCE_PATH)
+        old_destination = destination_path(STAMP_DESTINATION_FINGERPRINT_PATH)
         org_markup = stamp_file.markup
-        stamp_file.write '_site'
-        org_mtime = mtime_of old_destination
 
+        assert stamp_file.write('_site')
+
+        org_mtime = mtime_of(old_destination)
         last_markup = stamp_file.markup
-        ensure_file_mtime_changes { File.write source, 'h1 {}' }
-        stamp_file.write '_site'
+        ensure_file_mtime_changes { File.write(source, 'h1 {}') }
 
         # preserve fingerprint and content seen in last markup phase
+        refute stamp_file.write('_site')
         assert_equal org_markup, last_markup
         assert_equal org_mtime, mtime_of(old_destination)
         assert_equal File.read(site_fixture_path(STAMP_SOURCE_PATH)), File.read(old_destination)
 
         last_markup = stamp_file.markup
-        stamp_file.write '_site'
 
-        new_destination = destination_path 'assets/screen-0f5dbd1e527a2bee267e85007b08d2a5.css'
+        assert stamp_file.write('_site')
+
+        new_destination = destination_path('assets/screen-0f5dbd1e527a2bee267e85007b08d2a5.css')
 
         # see updated fingerprint in the next round
         refute_equal org_markup, last_markup
@@ -41,27 +41,39 @@ module Jekyll::Minibundle::Test
     def test_many_consecutive_markup_calls_trigger_one_destination_write
       with_site do
         stamp_file = StampFile.new(STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH, &stamp_basenamer)
-
-        source = source_path STAMP_SOURCE_PATH
-        destination = destination_path STAMP_DESTINATION_FINGERPRINT_PATH
-
+        source = source_path(STAMP_SOURCE_PATH)
+        destination = destination_path(STAMP_DESTINATION_FINGERPRINT_PATH)
         org_markup = stamp_file.markup
         stamp_file.markup
-        stamp_file.write '_site'
-        org_mtime = mtime_of destination
 
-        ensure_file_mtime_changes { FileUtils.touch source }
+        assert stamp_file.write('_site')
 
+        org_mtime = mtime_of(destination)
+        ensure_file_mtime_changes { FileUtils.touch(source) }
         last_markup = stamp_file.markup
         stamp_file.markup
-        stamp_file.write '_site'
 
+        assert stamp_file.write('_site')
         assert_equal org_markup, last_markup
         assert_operator mtime_of(destination), :>, org_mtime
         assert_equal File.read(source), File.read(destination)
       end
     end
 
+    def test_calling_write_before_markup_has_no_effect
+      with_site do
+        stamp_file = StampFile.new(STAMP_SOURCE_PATH, STAMP_DESTINATION_PATH, &stamp_basenamer)
+
+        refute stamp_file.write('_site')
+        assert_empty Dir[destination_path('assets/*.css')]
+
+        stamp_file.markup
+
+        assert stamp_file.write('_site')
+        assert File.exists?(destination_path(STAMP_DESTINATION_FINGERPRINT_PATH))
+      end
+    end
+
     private
 
     def stamp_basenamer

metadata

@@ -1,71 +1,71 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-minibundle
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.4.3
 platform: ruby
 authors:
 - Tuomas Kareinen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-28 00:00:00.000000000 Z
+date: 2014-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: '1.4'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.4.2
+        version: '1.4'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: '5.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.0
+        version: '5.2'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: '1.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.1
+        version: '1.6'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 10.1.1
+        version: '10.1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 10.1.1
+        version: '10.1'
 description: |
   A straightforward asset bundling plugin for Jekyll, utilizing external
   minification tool of your choice. It provides asset concatenation for
@@ -120,6 +120,7 @@ files:
 - test/unit/asset_file_registry_test.rb
 - test/unit/asset_tag_markup_test.rb
 - test/unit/bundle_file_test.rb
+- test/unit/development_file_collection_test.rb
 - test/unit/stamp_file_test.rb
 homepage: https://github.com/tkareine/jekyll-minibundle
 licenses:
@@ -127,26 +128,26 @@ licenses:
 metadata: {}
 post_install_message: 
 rdoc_options:
-- --line-numbers
-- --title
+- "--line-numbers"
+- "--title"
 - jekyll-minibundle
-- --exclude
+- "--exclude"
 - test
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.1.11
+rubygems_version: 2.2.1
 signing_key: 
 specification_version: 4
 summary: A minimalistic asset bundling plugin for Jekyll
@@ -174,4 +175,5 @@ test_files:
 - test/unit/asset_file_registry_test.rb
 - test/unit/asset_tag_markup_test.rb
 - test/unit/bundle_file_test.rb
+- test/unit/development_file_collection_test.rb
 - test/unit/stamp_file_test.rb