padrino-assets 0.2.2 → 0.3.0

data/spec/spec.rb

@@ -0,0 +1,37 @@
+PADRINO_ENV = 'test'
+
+require 'rspec'
+require 'rspec-html-matchers'
+require 'padrino-assets'
+
+module TestHelpers
+  def app
+    @app ||= Sinatra.new(Padrino::Application) do
+      register Padrino::Assets
+      set :manifest_file, File.join(settings.root, 'fixtures', 'compiled_assets', 'manifest.json')
+      set :logging, false
+    end
+  end
+
+  def settings
+    app.settings
+  end
+
+  def request
+    @request ||= Sinatra::Request.new(nil.to_s)
+  end
+
+  def environment
+    Padrino::Assets.environment
+  end
+
+  def manifest
+    Padrino::Assets.manifest
+  end
+end
+
+RSpec.configure do |configuration|
+  configuration.include TestHelpers
+end
+
+Padrino::Assets.load_paths << File.dirname(__FILE__) + '/fixtures/assets'

data/spec/tasks_spec.rb

@@ -0,0 +1,114 @@
+require_relative 'spec'
+require 'fileutils'
+require 'tmpdir'
+require 'rake'
+
+describe 'Rake Tasks' do
+  let :temp_directory do
+   File.join(Dir.tmpdir, 'sprockets')
+  end
+
+  let(:image)      { File.join(temp_directory, environment['pony.jpg'].digest_path) }
+  let(:stylesheet) { File.join(temp_directory, environment['application.css'].digest_path) }
+  let(:javascript) { File.join(temp_directory, environment['application.js'].digest_path) }
+
+  let :rake do
+    Rake.application
+  end
+
+  before do
+    app.set :manifest_file, File.join(temp_directory, 'manifest.json')
+    app.set :app_obj, app
+
+    Padrino.after_load.each(&:call)
+    Padrino.clear!
+
+    Padrino.insert_mounted_app(app)
+
+    Rake.application = Rake::Application.new
+    Dir.glob(File.expand_path('../../lib/tasks/*.rake', __FILE__)).each do |task|
+      load(task)
+    end
+    rake.load_imports
+  end
+
+  after do
+    FileUtils.rm_rf(temp_directory)
+  end
+
+  context 'assets:precompile' do
+    it 'should allow you to use a regexp to filter assets' do
+      app.set :precompile_assets, [/^application\.(css|js)$/]
+      rake['assets:precompile'].invoke
+
+      File.should     exist(stylesheet)
+      File.should     exist(javascript)
+      File.should_not exist(image)
+    end
+
+    it 'should allow you to use a proc to filter assets' do
+      app.set :precompile_assets, [ ->(asset) { !File.extname(asset).in?(['.css', '.js']) } ]
+      rake['assets:precompile'].invoke
+
+      File.should     exist(image)
+      File.should_not exist(stylesheet)
+      File.should_not exist(javascript)
+    end
+
+    it 'should allow you to use a string to filter assets' do
+      app.set :precompile_assets, ['pony.jpg', 'application.css']
+      rake['assets:precompile'].invoke
+
+      File.should     exist(stylesheet)
+      File.should     exist(image)
+      File.should_not exist(javascript)
+    end
+
+    it 'should automatically compress assets when enabled' do
+      app.set :precompile_assets, ['application.css']
+      app.set :compress_assets, true
+
+      rake['assets:precompile'].invoke
+
+      File.should exist(stylesheet)
+      File.should exist(stylesheet + '.gz')
+    end
+  end
+
+  context 'assets:clobber' do
+    it 'should delete the directory' do
+      rake['assets:precompile'].invoke
+      File.should exist(app.manifest_file)
+
+      rake['assets:clobber'].invoke
+      File.should_not exist(manifest.dir)
+    end
+  end
+
+  context 'assets:compress' do
+    it 'should compress assets' do
+      app.set :precompile_assets, [/^application\.(css|js)$/]
+      app.set :compress_assets, false
+
+      rake['assets:precompile'].invoke
+      rake['assets:compress'].invoke
+
+      File.should exist(stylesheet)
+      File.should exist(stylesheet + '.gz')
+      File.should exist(javascript)
+      File.should exist(javascript + '.gz')
+    end
+
+    it 'should only compress text based assets' do
+      app.set :precompile_assets, [/.+/]
+      app.set :compress_assets, false
+
+      rake['assets:precompile'].invoke
+      rake['assets:compress'].invoke
+
+      File.should     exist(stylesheet + '.gz')
+      File.should     exist(javascript + '.gz')
+      File.should_not exist(image + '.gz')
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: padrino-assets
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-23 00:00:00.000000000Z
+date: 2012-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sprockets
-  requirement: &6755340 !ruby/object:Gem::Requirement
+  requirement: &15358392 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,10 @@ dependencies:
         version: 2.3.0
   type: :runtime
   prerelease: false
-  version_requirements: *6755340
+  version_requirements: *15358392
 - !ruby/object:Gem::Dependency
   name: padrino-core
-  requirement: &6755090 !ruby/object:Gem::Requirement
+  requirement: &15357600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *6755090
+  version_requirements: *15357600
 - !ruby/object:Gem::Dependency
   name: padrino-helpers
-  requirement: &6754830 !ruby/object:Gem::Requirement
+  requirement: &15356832 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *6754830
+  version_requirements: *15356832
 - !ruby/object:Gem::Dependency
-  name: minitest
-  requirement: &6754570 !ruby/object:Gem::Requirement
+  name: rake
+  requirement: &15356448 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,21 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6754570
+  version_requirements: *15356448
 - !ruby/object:Gem::Dependency
-  name: webrat
-  requirement: &6754330 !ruby/object:Gem::Requirement
+  name: rspec
+  requirement: &15356040 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 2.0.0
+  type: :development
+  prerelease: false
+  version_requirements: *15356040
+- !ruby/object:Gem::Dependency
+  name: rspec-html-matchers
+  requirement: &15355728 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,7 +76,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *6754330
+  version_requirements: *15355728
 description: A plugin for the Padrino web framework which uses Sprockets to manage
   and compile assets
 email:
@@ -75,12 +86,14 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rspec
+- .travis.yml
+- .yardopts
 - CHANGELOG.md
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
-- examples/nginx.md
 - lib/padrino-assets.rb
 - lib/padrino-assets/helpers.rb
 - lib/padrino-assets/version.rb
@@ -89,8 +102,15 @@ files:
 - lib/tasks/compress.rake
 - lib/tasks/precompile.rake
 - padrino-assets.gemspec
-- test/test.rb
-- test/test_helpers.rb
+- spec/assets_spec.rb
+- spec/fixtures/assets/application.css
+- spec/fixtures/assets/application.js
+- spec/fixtures/assets/pony.jpg
+- spec/fixtures/compiled_assets/application-b8588c6975a4539fbf2c11471870bdca.css
+- spec/fixtures/compiled_assets/manifest.json
+- spec/helpers_spec.rb
+- spec/spec.rb
+- spec/tasks_spec.rb
 homepage: https://github.com/Cirex/padrino-assets
 licenses: []
 post_install_message: 
@@ -111,12 +131,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.15
+rubygems_version: 1.8.17
 signing_key: 
 specification_version: 3
 summary: A plugin for the Padrino web framework which uses Sprockets to manage and
   compile assets
-test_files:
-- test/test.rb
-- test/test_helpers.rb
+test_files: []
 has_rdoc: 

data/examples/nginx.md

@@ -1,11 +0,0 @@
-Nginx Configuration Example
-=============================
-
-``` nginx
-location ~ ^/(assets)/  {
-  add_header Cache-Control public;
-  root /path/to/public;
-  gzip_static on;
-  expires max;
-}
-```

data/test/test.rb

@@ -1,52 +0,0 @@
-require 'webrat'
-require 'padrino-assets'
-require 'minitest/pride'
-require 'minitest/autorun'
-
-Padrino::Assets.load_paths << File.dirname(__FILE__) + '/fixtures/assets/**'
-
-module MiniTest
-  class Spec
-    class << self
-      alias_method :context, :describe
-    end
-
-    def app
-      @app ||= Sinatra.new(Padrino::Application) do
-        register Padrino::Assets
-        set :environment, :test
-      end
-    end
-
-    def settings
-      app.settings
-    end
-
-    def request
-      nil
-    end
-  end
-
-  module Assertions
-    include Webrat::Matchers
-
-    def assert_has_tag(tag, html, attributes)
-      selector = HaveSelector.new(tag, attributes)
-      assert selector.matches?(html), selector.failure_message
-    end
-
-    def refute_has_tag(tag, html, attributes)
-      selector = HaveSelector.new(tag, attributes)
-      refute selector.matches?(html), selector.failure_message
-    end
-  end
-
-  module Expectations
-    infect_an_assertion :assert_has_tag, :must_have_tag
-    infect_an_assertion :refute_has_tag, :wont_have_tag
-  end
-end
-
-class Object
-  include MiniTest::Expectations
-end

data/test/test_helpers.rb

@@ -1,339 +0,0 @@
-require_relative 'test'
-
-describe 'Helpers' do
-  include Padrino::Helpers::OutputHelpers
-  include Padrino::Helpers::TagHelpers
-  include Padrino::Assets::Helpers
-
-  describe 'assets_path' do
-    it 'should prepend :assets_host when present' do
-      app.set :assets_host, 'http://www.test.com'
-      asset = asset_path('application.css')
-      asset.must_equal 'http://www.test.com/assets/application.css'
-    end
-
-    it 'should allow :assets_host to use a Proc' do
-      app.set :assets_host, ->(asset, request) { 'http://test.com' }
-      asset = asset_path('application.css')
-      asset.must_equal 'http://test.com/assets/application.css'
-    end
-
-    it 'should pass the current source to :assets_host when a Proc is used' do
-      app.set :assets_host, ->(source, request) do
-        if source =~ /application.css/
-          'http://true.com'
-        else
-          'http://false.com'
-        end
-      end
-      asset = asset_path('application.css')
-      asset.must_equal 'http://true.com/assets/application.css'
-    end
-
-    it 'should append extension when provided' do
-      asset = asset_path(:application, :css)
-      asset.must_equal '/assets/application.css'
-    end
-
-    it 'should not append extension when one is present' do
-      asset = asset_path('application.css', :css)
-      asset.wont_equal '/assets/application.css.css'
-    end
-
-    it 'should prepend :assets_prefix when present' do
-      app.set :assets_prefix, '/test'
-      asset = asset_path('application.css')
-      asset.must_equal '/test/application.css'
-    end
-
-    it 'should prepend a forward slash to :assets_prefix when missing' do
-      app.set :assets_prefix, 'test'
-      asset = asset_path('application.css')
-      asset.must_equal '/test/application.css'
-    end
-
-    it 'should not interfere with a reference URI' do
-      asset = asset_path('/application.css')
-      asset.must_equal '/application.css'
-
-      asset = asset_path('//test.com/application.css')
-      asset.must_equal '//test.com/application.css'
-
-      asset = asset_path('http://test.com/application.css')
-      asset.must_equal 'http://test.com/application.css'
-    end
-  end
-
-  describe 'is_uri?' do
-    it 'should return true when given an absolute URI' do
-      is_uri?('https://example.com/application.css').must_equal true
-      is_uri?('http://example.com/application.css').must_equal true
-      is_uri?('ftp://example.com/application.css').must_equal true
-    end
-
-    it 'should return true when given a reference URI' do
-      is_uri?('//example.com/assets/application.css').must_equal true
-      is_uri?('/assets/application.css').must_equal true
-    end
-
-    it 'should return false when given a host without a protocol' do
-      is_uri?('example.com/assets/application.css').must_equal false
-    end
-
-    it 'should return false when given an asset' do
-      is_uri?('application.css').must_equal false
-    end
-  end
-
-  describe 'include_stylesheet' do
-    it 'should accept multiple sources' do
-      stylesheets = include_stylesheets :application, :theme
-      stylesheets.must_have_tag :link, href: '/assets/application.css'
-      stylesheets.must_have_tag :link, href: '/assets/theme.css'
-    end
-
-    it 'should accept multiple sources with attributes' do
-      stylesheets = include_stylesheets :application, :theme, media: 'handheld'
-      stylesheets.must_have_tag :link, href: '/assets/application.css', media: 'handheld'
-      stylesheets.must_have_tag :link, href: '/assets/theme.css', media: 'handheld'
-    end
-
-    it 'should accept a single source' do
-      stylesheet = include_stylesheet :application
-      stylesheet.must_have_tag :link, href: '/assets/application.css'
-    end
-
-    it 'should accept a single source with attributes' do
-      stylesheet = include_stylesheet :application, media: 'handheld'
-      stylesheet.must_have_tag :link, href: '/assets/application.css', media: 'handheld'
-    end
-
-    it 'should allow a source with an extension' do
-      stylesheet = include_stylesheet 'application.css'
-      stylesheet.wont_have_tag :link, hreg: '/assets/application.css.css'
-    end
-
-    it 'should be aliased for compatibility' do
-      respond_to?(:stylesheet_link_tag).must_equal true
-    end
-  end
-
-  describe 'include_javascript' do
-    it 'should accept multiple sources' do
-      javascripts = include_javascripts :application, :jquery
-      javascripts.must_have_tag :script, src: '/assets/application.js'
-      javascripts.must_have_tag :script, src: '/assets/jquery.js'
-    end
-
-    it 'should accept a single source' do
-      javascripts = include_javascript :application
-      javascripts.must_have_tag :script, src: '/assets/application.js'
-    end
-
-    it 'should be aliased for compatibility' do
-      respond_to?(:javascript_include_tag).must_equal true
-    end
-  end
-
-  describe 'image' do
-    it 'should accept multiple sources' do
-      images = images 'application.jpg', 'application.png'
-      images.must_have_tag :img, src: '/assets/application.jpg'
-      images.must_have_tag :img, src: '/assets/application.png'
-    end
-
-    it 'should accept multiple sources with attributes' do
-      images = images 'application.jpg', 'application.png', width: '40', height: '40'
-      images.must_have_tag :img, src: '/assets/application.jpg', width: '40', height: '40'
-      images.must_have_tag :img, src: '/assets/application.png', width: '40', height: '40'
-    end
-
-    it 'should accept a single source' do
-      image = image 'application.jpg'
-      image.must_have_tag :img, src: '/assets/application.jpg'
-    end
-
-    it 'should accept a single source with attributes' do
-      image = image 'application.jpg', width: '40', height: '40'
-      image.must_have_tag :img, width: '40', height: '40'
-    end
-
-    it 'should allow you to set the :width and :height with size' do
-      image = image 'application.jpg', size: '40x40'
-      image.must_have_tag :img, width: '40', height: '40'
-    end
-
-    it 'should allow you to set the :width' do
-      image = image 'application.jpg', width: '40'
-      image.must_have_tag :img, width: '40'
-    end
-
-    it 'should allow you to set the :height' do
-      image = image 'application.jpg', height: '40'
-      image.must_have_tag :img, height: '40'
-    end
-
-    it 'should allow you to set the alternate text' do
-      image = image 'application.jpg', alt: 'My Little Pony'
-      image.must_have_tag :img, alt: 'My Little Pony'
-    end
-
-    it 'should automatically set an alternate text when none present' do
-      image = image 'application.jpg'
-      image.must_have_tag :img, alt: 'Application'
-    end
-
-    it 'should allow you to set the :class' do
-      image = image 'application.jpg', class: 'image'
-      image.must_have_tag :img, class: 'image'
-    end
-
-    it 'should allow you to set the :id' do
-      image = image 'application.jpg', id: 'logo'
-      image.must_have_tag :img, id: 'logo'
-    end
-
-    it 'should be aliased for compatibility' do
-      respond_to?(:image_tag).must_equal true
-    end
-  end
-
-  describe 'video' do
-    it 'should accept multiple sources' do
-      videos = videos 'test.webm', 'test.mov'
-      videos.must_have_tag :source, src: '/assets/test.webm'
-      videos.must_have_tag :source, src: '/assets/test.mov'
-    end
-
-    it 'should accept multiple sources with attributes' do
-      videos = videos 'test.webm', 'test.mov', width: '40', height: '40'
-      videos.must_have_tag :video, width: '40', height: '40'
-      videos.must_have_tag :source, src: '/assets/test.webm'
-      videos.must_have_tag :source, src: '/assets/test.mov'
-    end
-
-    it 'should accept a single source' do
-      video = video 'test.webm'
-      video.must_have_tag :video, src: '/assets/test.webm'
-    end
-
-    it 'should accept a single source with attributes' do
-      video = video 'test.webm', width: '40', height: '40'
-      video.must_have_tag :video, src: '/assets/test.webm', width: '40', height: '40'
-    end
-
-    it 'should allow you to set the :class' do
-      video = video 'test.webm', class: 'video'
-      video.must_have_tag :video, class: 'video'
-    end
-
-    it 'should allow you to set the :id' do
-      video = video 'test.webm', id: 'video'
-      video.must_have_tag :video, id: 'video'
-    end
-
-    it 'should allow you to set the :width and :height with :size' do
-      video = video 'test.webm', size: '40x40'
-      video.must_have_tag :video, width: '40', height: '40'
-    end
-
-    it 'should allow you to set the :width' do
-      video = video 'test.webm', width: '40'
-      video.must_have_tag :video, width: '40'
-    end
-
-    it 'should allow you to set the :height' do
-      video = video 'test.webm', height: '40'
-      video.must_have_tag :video, height: '40'
-    end
-
-    it 'should allow :poster to use an absolute URI' do
-      video = video 'test.webm', poster: 'http://test.com/poster.jpg'
-      video.must_have_tag :video, poster: 'http://test.com/poster.jpg'
-    end
-
-    it 'should use asset_path when :poster is present' do
-      video = video 'test.webm', poster: 'poster.jpg'
-      video.must_have_tag :video, poster: '/assets/poster.jpg'
-    end
-
-    it 'should allow you to set the :preload method' do
-      video = video 'test.webm', preload: 'auto'
-      video.must_have_tag :video, preload: 'auto'
-    end
-
-    it 'should allow you to set :muted to true' do
-      video = video 'test.webm', muted: true
-      video.must_have_tag :video, muted: 'muted'
-    end
-
-    it 'should allow you to set :autoplay to true' do
-      video = video 'test.webm', autoplay: true
-      video.must_have_tag :video, autoplay: 'autoplay'
-    end
-
-    it 'should allow you to set :loop to true' do
-      video = video 'test.webm', loop: true
-      video.must_have_tag :video, loop: 'loop'
-    end
-
-    it 'should allow you to set :controls to true' do
-      video = video 'test.webm', controls: true
-      video.must_have_tag :video, controls: 'controls'
-    end
-
-    it 'should be aliased for compatibility' do
-      respond_to?(:video_tag).must_equal true
-    end
-  end
-
-  describe 'audio' do
-    it 'should accept multiple sources' do
-      audios = audios 'test.ogg', 'test.aac'
-      audios.must_have_tag :source, src: '/assets/test.ogg'
-      audios.must_have_tag :source, src: '/assets/test.aac'
-    end
-
-    it 'should accept multiple sources with attributes' do
-      audios = audios 'test.ogg', 'test.aac', id: 'audio'
-      audios.must_have_tag :audio, id: 'audio'
-      audios.must_have_tag :source, src: '/assets/test.ogg'
-      audios.must_have_tag :source, src: '/assets/test.aac'
-    end
-
-    it 'should accept a single source' do
-      audio = audio 'test.ogg'
-      audio.must_have_tag :audio, src: '/assets/test.ogg'
-    end
-
-    it 'should accept a single source with attributes' do
-      audio = audio 'test.ogg', id: 'audio'
-      audio.must_have_tag :audio, src: '/assets/test.ogg', id: 'audio'
-    end
-
-    it 'should allow you to set the :class' do
-      audio = audio 'test.ogg', class: 'audio'
-      audio.must_have_tag :audio, class: 'audio'
-    end
-
-    it 'should allow you to set the :id' do
-      audio = audio 'test.ogg', id: 'audio'
-      audio.must_have_tag :audio, id: 'audio'
-    end
-
-    it 'should allow you to set :autoplay to true' do
-      audio = audio 'test.ogg', autoplay: true
-      audio.must_have_tag :audio, autoplay: 'autoplay'
-    end
-
-    it 'should allow you to set :loop to true' do
-      audio = audio 'test.ogg', loop: true
-      audio.must_have_tag :audio, loop: 'loop'
-    end
-
-    it 'should allow you to set :controls to true' do
-      audio = audio 'test.ogg', controls: true
-      audio.must_have_tag :audio, controls: 'controls'
-    end
-  end
-end