rack-pagespeed 1.0.0 → 1.0.1

data/README.md

@@ -1,14 +1,6 @@
 # rack-pagespeed
 
-This middleware _will_ (as in this is a work in progress) replicate possibly every feature found in Google's [modpagespeed](http://www.modpagespeed.com/).
-
-# To do
-
-Everything. I'm just reserving the name for now.
-
-# Where's rack-bundle?
-
-I kept it in a branch called `bundle`. Though I strongly recommend you _not_ to use it. You can get the same effect (better actually, considering rack-pagespeed doesn't do some dumb stuff rack-bundle used to) by activating only the equivalent features in rack-pagespeed.
+Please refer to [the instructions manual](http://rack-pagespeed.heroku.com) for details. GitHub ain't stylish enough.
 
 # License
 

data/VERSION

@@ -1 +1 @@
-1.0.0
+1.0.1

data/lib/rack/pagespeed/config.rb

@@ -31,8 +31,8 @@ class Rack::PageSpeed::Config
     end
   end
 
-  def method_missing filter
-    raise NoSuchFilter, "No such filter \"#{filter}\". Available filters: #{(Rack::PageSpeed::Filters::Base.available_filters).join(', ')}"
+  def method_missing filter, *args
+    raise NoSuchFilter, "No such filter \"#{filter}\". Available filters: #{(Rack::PageSpeed::Filter.available_filters).join(', ')}"
   end
 
   private
@@ -61,7 +61,7 @@ class Rack::PageSpeed::Config
   end
 
   def filters_to_methods
-    Rack::PageSpeed::Filters::Base.available_filters.each do |klass|
+    Rack::PageSpeed::Filter.available_filters.each do |klass|
       (class << self; self; end).send :define_method, klass.name do |*options|
         default_options = {:public => @options[:public], :store => @store}
         instance = klass.new(options.any? ? default_options.merge(*options) : default_options)

data/rack-pagespeed.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{rack-pagespeed}
-  s.version = "1.0.0"
+  s.version = "1.0.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Julio Cesar Ody"]
-  s.date = %q{2011-01-14}
+  s.date = %q{2011-01-20}
   s.description = %q{Web page speed optimizations at the Rack level}
   s.email = %q{julio@awesomebydesign.com}
   s.extra_rdoc_files = [

data/spec/config_spec.rb

@@ -42,24 +42,22 @@ describe 'rack-pagespeed configuration' do
   end
 
   context 'enabling filters, options hash based' do
-    context 'options[:filters]' do
-      before { File.stub(:directory?).and_return(true) }
+    before { File.stub(:directory?).and_return(true) }
 
-      it "if it's an array, it enables filters listed in it with their default options" do
-        config = Rack::PageSpeed::Config.new :filters => [:makes_it_look_good]
-        config.filters.first.should be_a MakesItLookGood
-      end
+    it "if it's an array, it enables filters listed in it with their default options" do
+      config = Rack::PageSpeed::Config.new :filters => [:makes_it_look_good]
+      config.filters.first.should be_a MakesItLookGood
+    end
 
-      it "if it's a hash, it let's you pass options to the filters logically" do
-        config = Rack::PageSpeed::Config.new :filters => {:makes_it_look_good => {:test => 6000}}
-        filter = config.filters.first
-        filter.should be_a MakesItLookGood
-        filter.options[:test].should == 6000 # yeah, 2 assertions, bad, bad
-      end
+    it "if it's a hash, it let's you pass options to the filters logically" do
+      config = Rack::PageSpeed::Config.new :filters => {:makes_it_look_good => {:test => 6000}}
+      filter = config.filters.first
+      filter.should be_a MakesItLookGood
+      filter.options[:test].should == 6000 # yeah, 2 assertions, bad, bad
     end
 
-    it 'raises a NoSuchFilterError when a non-existing filter is passed to :filters' do
-      expect { Rack::PageSpeed.new page, :filters => [:whoops!] }.to raise_error
+    it 'raises a NoSuchFilter error when a non-existing filter is passed to :filters' do
+      expect { Rack::PageSpeed::Config.new :filters => [:whoops!] }.to raise_error(Rack::PageSpeed::Config::NoSuchFilter)
     end
   end
 
@@ -92,6 +90,10 @@ describe 'rack-pagespeed configuration' do
       config = Rack::PageSpeed::Config.new do needs_store end
       config.filters.should be_empty
     end
+    
+    it 'raises a NoSuchFilter error when a non-existing filter is called ' do
+      expect { Rack::PageSpeed::Config.new do whateva :foo => 'bar' end }.to raise_error(Rack::PageSpeed::Config::NoSuchFilter)
+    end
   end
 
   context 'setting a storage mechanism' do

data/spec/pagespeed_spec.rb

@@ -76,7 +76,7 @@ describe 'rack-pagespeed' do
       end
 
       it "responds with the contents in store that match the asset unique id" do
-        @response.to_s.should == 'Little poney'
+        @response.first.to_s.should == 'Little poney'
       end
     
       it "responds with the appropriate MIME type for the asset stored" do

metadata

@@ -1,13 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: rack-pagespeed
 version: !ruby/object:Gem::Version 
-  hash: 23
   prerelease: false
   segments: 
   - 1
   - 0
-  - 0
-  version: 1.0.0
+  - 1
+  version: 1.0.1
 platform: ruby
 authors: 
 - Julio Cesar Ody
@@ -15,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-14 00:00:00 +11:00
+date: 2011-01-20 00:00:00 +11:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -26,7 +25,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 15
         segments: 
         - 1
         - 4
@@ -42,7 +40,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 29
         segments: 
         - 1
         - 2
@@ -58,7 +55,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 19
         segments: 
         - 1
         - 0
@@ -74,7 +70,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 21
         segments: 
         - 1
         - 0
@@ -90,7 +85,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 15
         segments: 
         - 1
         - 4
@@ -106,7 +100,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 29
         segments: 
         - 1
         - 2
@@ -122,7 +115,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 19
         segments: 
         - 1
         - 0
@@ -138,7 +130,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 47
         segments: 
         - 1
         - 16
@@ -153,7 +144,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 21
         segments: 
         - 1
         - 0
@@ -169,7 +159,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 11
         segments: 
         - 2
         - 1
@@ -185,7 +174,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 23
         segments: 
         - 1
         - 0
@@ -201,7 +189,6 @@ dependencies:
     requirements: 
     - - "="
       - !ruby/object:Gem::Version 
-        hash: 15
         segments: 
         - 0
         - 4
@@ -298,7 +285,6 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 3
       segments: 
       - 0
       version: "0"
@@ -307,7 +293,6 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 3
       segments: 
       - 0
       version: "0"