justinko · June 23, 2011 23:45 · Jun 23, 2011 · Jun 21, 2011 · Jun 21, 2011 · Jun 21, 2011
diff --git a/let_with_contexts.rb → do_action.rb b/let_with_contexts.rb → do_action.rb
@@ -2,26 +2,26 @@
   describe "PUT #update" do
     let(:post) { Factory(:post) }
 
-    before(:each) do
-      sign_in user # user is defined as a memoized helper method using let { } below
-
-      put :update, :id => post.id, :body => "new content"
-    end
-
     context "when logged in as post's author" do
-      let(:user) { post.author }
+      before { sign_in post.author }
 
       it "allows the post to be updated" do
+        do_action
         response.should be_success
       end
     end
 
     context "when logged in as another user" do
-      let(:user) { Factory(:user) }
+      before { sign_in Factory(:user) }
 
       it "does not allow the post to be updated" do
+        do_action
         response.should be_forbidden
       end
     end
+
+    def do_action
+      put :update, :id => post.id, :body => "new content"
+    end
   end
 end
diff --git a/let_with_contexts.rb b/let_with_contexts.rb
@@ -3,7 +3,7 @@
     let(:post) { Factory(:post) }
 
     before(:each) do
-      login_as user # user is defined as a memoized helper method using let { } below
+      sign_in user # user is defined as a memoized helper method using let { } below
 
       put :update, :id => post.id, :body => "new content"
     end

diff --git a/let_with_contexts.rb b/let_with_contexts.rb
@@ -3,7 +3,7 @@
     let(:post) { Factory(:post) }
 
     before(:each) do
-      login_as user
+      login_as user # user is defined as a memoized helper method using let { } below
 
       put :update, :id => post.id, :body => "new content"
     end

diff --git a/let_with_contexts.rb b/let_with_contexts.rb
@@ -2,22 +2,26 @@
   describe "PUT #update" do
     let(:post) { Factory(:post) }
 
-    before do
+    before(:each) do
       login_as user
 
       put :update, :id => post.id, :body => "new content"
     end
 
-    context "logged in as post's author" do
+    context "when logged in as post's author" do
       let(:user) { post.author }
 
-      its(:response) { should be_success }
+      it "allows the post to be updated" do
+        response.should be_success
+      end
     end
 
-    context "logged in as another user" do
+    context "when logged in as another user" do
       let(:user) { Factory(:user) }
 
-      its(:response) { should be_forbidden }
+      it "does not allow the post to be updated" do
+        response.should be_forbidden
+      end
     end
   end
 end
diff --git a/let_with_contexts.rb b/let_with_contexts.rb
@@ -0,0 +1,23 @@
+describe PostsController do
+  describe "PUT #update" do
+    let(:post) { Factory(:post) }
+
+    before do
+      login_as user
+
+      put :update, :id => post.id, :body => "new content"
+    end
+
+    context "logged in as post's author" do
+      let(:user) { post.author }
+
+      its(:response) { should be_success }
+    end
+
+    context "logged in as another user" do
+      let(:user) { Factory(:user) }
+
+      its(:response) { should be_forbidden }
+    end
+  end
+end