From e09b544f26b5c9318d408af274d7f31f6213002e Mon Sep 17 00:00:00 2001
From: Raphael <raphael@joindiaspora.com>
Date: Fri, 26 Nov 2010 19:28:41 -0500
Subject: [PATCH] Add specs to log overrider

---
 spec/controllers/home_controller_spec.rb | 45 +++++++++++++++++++++---
 1 file changed, 40 insertions(+), 5 deletions(-)

diff --git a/spec/controllers/home_controller_spec.rb b/spec/controllers/home_controller_spec.rb
index 8cbf2c472..4fe8f5893 100644
--- a/spec/controllers/home_controller_spec.rb
+++ b/spec/controllers/home_controller_spec.rb
@@ -31,14 +31,49 @@ describe HomeController do
     before do
       logger = FakeLogger.new
       Rails.stub(:logger).and_return(FakeLogger.new)
-      get :show
     end
 
-    it 'logs the routing of a request' do
-      Rails.logger.infos.first.include?('event=request_routed').should be_true
+    context 'routing' do
+      before do
+        get :show, :lasers => 'green'
+        @line = Rails.logger.infos.first
+      end
+      it 'logs the routing of a request' do
+        @line.include?('event=request_routed').should be_true
+      end
+      it 'logs the controller' do
+        @line.include?('controller=HomeController').should be_true
+      end
+      it 'logs the action' do
+        @line.include?('action=show').should be_true
+      end
+      it 'logs params' do
+        @line.include?("params='{\"lasers\"=>\"green\"}'").should be_true
+      end
     end
-    it 'logs the completion of a request' do
-      Rails.logger.infos.last.include?('event=request_completed').should be_true
+    context 'completion' do
+      context 'ok' do
+        before do
+          get :show, :lasers => 'green'
+          @line = Rails.logger.infos.last
+        end
+        it 'logs the completion of a request' do
+          @line.include?('event=request_completed').should be_true
+        end
+        it 'logs an ok' do
+          @line.include?('status=200').should be_true
+        end
+      end
+      context 'redirected' do
+        before do
+          sign_in @user
+          get :show, :lasers => 'green'
+          @line = Rails.logger.infos.last
+        end
+        it 'logs a redirect' do
+          @line.include?('status=302').should be_true
+        end
+      end
     end
   end
   class FakeLogger