lackie 0.1.7 → 0.1.8

data/lib/lackie/rack/logger.rb

@@ -0,0 +1,14 @@
+module Lackie
+  module Rack
+    class NilLogger
+      def log(message)
+      end
+    end
+    
+    class SimpleLogger
+      def log(message)
+        Kernel.puts message
+      end
+    end
+  end
+end

data/lib/lackie.rb

@@ -2,7 +2,7 @@ lib = File.dirname(__FILE__)
 $:.unshift(lib) unless $:.include?(lib) || $:.include?(File.expand_path(lib))
 
 module Lackie
-  VERSION = '0.1.7'
+  VERSION = '0.1.8'
 end
 
 require 'lackie/remote_control'

data/spec/lackie/rack/logging_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+require 'lackie/rack'
+
+module Lackie
+  module Rack
+    describe Middleware, "with a logger" do
+      before do
+        Lackie::JavaScript::Surrender.stub!(:new).and_return(mock("surrender", :script => "yowzer"))
+        @app = mock("app")
+        @env = mock("env")
+        @logger = mock("logger", :log => true)
+        @middleware = Middleware.new(@app, @logger)
+      end
+      
+      def request(options)
+        @request = mock("Request: #{options.inspect}", options)
+        ::Rack::Request.stub!(:new).with(@env).and_return(@request)
+        @middleware.call(@env)
+      end
+      
+      def get(path, options)
+        request(options.merge({ :path => path, :get? => true }))
+      end
+      
+      def post(path, body)
+        request(:path => path, :get? => false, :body => mock("io", :read => body))
+      end
+      
+      it "logs the surrendered user agent" do
+        @logger.should_receive(:log).with("surrendered to some-browser")
+        get("/lackie/surrender", :user_agent => "some-browser")
+      end
+      
+      it "logs commands with ids" do
+        @logger.should_receive(:log).with("eval {\"command\":\"foo\",\"id\":1}")
+        post("/lackie/eval", "foo")
+      end
+      
+      it "logs results with ids" do
+        post("/lackie/eval", "bar")
+        @logger.should_receive(:log).with("result {\"id\":1,\"value\":\"bar\"}")
+        post("/lackie/result", "{\"id\":1,\"value\":\"bar\"}")
+      end
+    end
+    
+    describe SimpleLogger do
+      it "logs using Kernel.puts" do
+        Kernel.should_receive(:puts).with("yo").once
+        SimpleLogger.new.log("yo")
+      end
+    end
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: lackie
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 11
   prerelease: 
   segments: 
   - 0
   - 1
-  - 7
-  version: 0.1.7
+  - 8
+  version: 0.1.8
 platform: ruby
 authors: 
 - Josh Chisholm
@@ -157,10 +157,12 @@ files:
 - lib/lackie/javascript/surrender.rb
 - lib/lackie/poller.rb
 - lib/lackie/rack.rb
+- lib/lackie/rack/logger.rb
 - lib/lackie/rack/middleware.rb
 - lib/lackie/remote_control.rb
 - spec/lackie/javascript/surrender_spec.rb
 - spec/lackie/poller_spec.rb
+- spec/lackie/rack/logging_spec.rb
 - spec/lackie/rack/middleware_spec.rb
 - spec/lackie/remote_control_spec.rb
 - spec/spec_helper.rb
@@ -197,7 +199,7 @@ rubyforge_project:
 rubygems_version: 1.4.1
 signing_key: 
 specification_version: 3
-summary: lackie-0.1.7
+summary: lackie-0.1.8
 test_files: 
 - features/remote_control.feature
 - features/step_definitions/lackie_steps.rb
@@ -207,6 +209,7 @@ test_files:
 - features/support/example_app/app.html
 - spec/lackie/javascript/surrender_spec.rb
 - spec/lackie/poller_spec.rb
+- spec/lackie/rack/logging_spec.rb
 - spec/lackie/rack/middleware_spec.rb
 - spec/lackie/remote_control_spec.rb
 - spec/spec_helper.rb