cypher-rack-dickbarblocker 0.2.0 → 0.2.1

data/README.markdown

@@ -18,7 +18,7 @@ Hat-tip to [rue](http://github.com/rue) for suggesting the name.
 
         require 'rack/contrib/dick_bar_blocker'
         
-        use Rack::Contrib::DickBarBlocker
+        use Rack::Contrib::DickBarBlocker, 'name or initials here'
 
 * Or, if you're on Rails, add this to your environment.rb:
 
@@ -26,23 +26,23 @@ Hat-tip to [rue](http://github.com/rue) for suggesting the name.
         
         Rails::Initializer.run do |config|
           config.gem 'cypher-rack-dickbarblocker', :lib => 'rack/contrib/dick_bar_blocker', :source => 'http://gems.github.com'
-          config.middleware.use 'Rack::Contrib::DickBarBlocker'
+          config.middleware.use 'Rack::Contrib::DickBarBlocker', 'name or initials here'
           
           # rest of your config
         end
 
 By default, it shows the following text (sans markup):
 
-> Dear Digg,
+> Dear Digg,  
 > Framing sites is bullshit.
 >
-> Your pal,
-> —J.G.
+> Your pal,  
+> —(here are the supplied initals or name)
 >
-> p.s. Firefox users may enjoy the<br>
+> p.s. Firefox users may enjoy the  
 > DiggBar Killer script for Greasemonkey.
 >
-> p.p.s. Digg users can disable the DiggBar under
+> p.p.s. Digg users can disable the DiggBar under  
 > My Profile → Settings → Viewing Preferences.
 
 You can override this by supplying the middleware with a block that returns whatever you want it to display:

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :major: 0
 :minor: 2
-:patch: 0
+:patch: 1

data/lib/rack/contrib/dick_bar_blocker.rb

@@ -1,8 +1,11 @@
 module Rack
   module Contrib
     class DickBarBlocker
-      def initialize(app, &block)
+      def initialize(app, name = nil,  &block)
+        raise ArgumentError, "You need to supply a name or a block" if name.nil? and block.nil?
+
         @app = app
+        @name = name
         @block = block
       end
 
@@ -41,7 +44,7 @@ module Rack
             Framing sites is bullshit.<br>
             <br>
             Your pal,<br>
-            —J.G.
+            —#{@name}
         </p>
         <p>
             p.s. Firefox users may enjoy the<br>

data/test/spec_rack_dick_bar_blocker.rb

@@ -7,7 +7,7 @@ context "Rack::Contrib::DickBarBlocker" do
 
   context "non-digg-toolbar referrer" do
     specify "returns unmodified response" do
-      status, headers, body = Rack::Contrib::DickBarBlocker.new(app).call({'HTTP_REFERER' => 'http://daringfireball.net'})
+      status, headers, body = Rack::Contrib::DickBarBlocker.new(app, 'J.G.').call({'HTTP_REFERER' => 'http://daringfireball.net'})
 
       status.should.equal 200
       headers['Content-Type'].should.equal 'text/plain'
@@ -17,12 +17,36 @@ context "Rack::Contrib::DickBarBlocker" do
 
   context "via digg bar" do
     specify "return anti-digg-toolbar site" do
-      status, headers, body = Rack::Contrib::DickBarBlocker.new(app).call({'HTTP_REFERER' => 'http://digg.com/d1oNOZ'})
+      status, headers, body = Rack::Contrib::DickBarBlocker.new(app, 'J.G.').call({'HTTP_REFERER' => 'http://digg.com/d1oNOZ'})
 
       status.should.equal 200
       headers['Content-Type'].should.equal 'text/html'
       body.should =~ %r{<title>Don't be a dick, say no to the DiggBar</title>}i
-      body.should =~ %r{Dear Digg,.*Framing sites is bullshit\.}mi
+      body.should =~ %r{Dear Digg,<br>\s+Framing sites is bullshit\.<br>\s+<br>\s+Your pal,<br>\s+—J.G.}mi
+    end
+
+    specify "uses the given name" do
+      name = "test name"
+      status, headers, body = Rack::Contrib::DickBarBlocker.new(app, name).call({'HTTP_REFERER' => 'http://digg.com/d1oNOZ'})
+
+      status.should.equal 200
+      headers['Content-Type'].should.equal 'text/html'
+      body.should =~ %r{Your pal,<br>\s+—#{name}}i
+    end
+
+    specify "raises an error when no name or block is supplied" do
+      lambda {
+        Rack::Contrib::DickBarBlocker.new(app).call({'HTTP_REFERER' => 'http://digg.com/d1oNOZ'})
+      }.should.raise ArgumentError
+    end
+
+    specify "returns custom content when supplied with a block" do
+      block = lambda { "Here be kittens." }
+      status, headers, body = Rack::Contrib::DickBarBlocker.new(app, &block).call({'HTTP_REFERER' => 'http://digg.com/d1oNOZ'})
+
+      status.should.equal 200
+      headers['Content-Type'].should.equal 'text/html'
+      body.should.equal "Here be kittens."
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: cypher-rack-dickbarblocker
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors: 
 - Markus Prinz