From b70da571249abb383da7e118eb101c0bb3c7efb8 Mon Sep 17 00:00:00 2001
From: Lisa Hamm <lisalhamm@gmail.com>
Date: Thu, 19 Mar 2015 16:27:28 -0500
Subject: [PATCH] Refactored and added TicTacToeController tests

---
 spec/tic_tac_toe_controller_spec.rb | 44 ++++++++++++++++++++++-------
 1 file changed, 34 insertions(+), 10 deletions(-)

diff --git a/spec/tic_tac_toe_controller_spec.rb b/spec/tic_tac_toe_controller_spec.rb
index 83ca188..5fbb8d6 100644
--- a/spec/tic_tac_toe_controller_spec.rb
+++ b/spec/tic_tac_toe_controller_spec.rb
@@ -41,7 +41,7 @@ def app
   end
 
   describe "GET /game" do
-    it "loads the game" do
+    it "loads a game without any moves" do
       get '/game', {}, {'rack.session' => {"computer_opponent" => "O",
                                        "player1_mark" => "X",
                                        "player2_mark" => "O",
@@ -49,28 +49,52 @@ def app
                                        "moves" => [nil, nil, nil, nil, nil, nil, nil, nil, nil]}}
       expect(last_response).to be_ok
       expect(last_response.status).to eq 200
+      expect(last_response.body).to include('<form action="/make_move" method="POST">')
+    end
+
+
+    it "loads the game with a move" do
+      get '/game', {}, {'rack.session' => {"computer_opponent" => "O",
+                                       "player1_mark" => "X",
+                                       "player2_mark" => "O",
+                                       "current_player_mark" => "X",
+                                       "moves" => ["X", nil, nil, nil, nil, nil, nil, nil, nil]}}
+      expect(last_response).to be_ok
+      expect(last_response.status).to eq 200
+      expect(last_response.body).to include('name="0" value=" X " readonly')
     end
   end
 
+
   describe "POST /make_move" do
-    before :each do
+
+    it "adds a human player's mark and a computer player's mark to the board" do
+      @mock_game = instance_double("TicTacToe::Game", :board_to_array => [])
+
+      expect(TicTacToe::Game).to receive(:new).and_return(@mock_game)
+      expect(@mock_game).to receive(:take_turn).with(0)
+      expect(@mock_game).to receive(:over?).and_return(false, false)
+      expect(@mock_game).to receive(:current_player_mark).and_return("X", "O")
+
       post '/make_move', {:move => '0'}, {'rack.session' => {
                                      "computer_opponent" => nil,
                                      "player1_mark" => "X",
                                      "player2_mark" => "O",
                                      "current_player_mark" => "X",
                                      "moves" => [nil, nil, nil, nil, nil, nil, nil, nil, nil]}}
-
+      expect(last_response.redirect?).to eq true
+      expect(last_response.headers["Location"]).to eq ("http://example.org/game")
     end
 
-    it "adds the player's mark to the board" do
-      rack_mock_session.cookie_jar[:moves] == ["X", nil, nil, nil, nil, nil, nil, nil, nil]
-    end
+    it "completes a game" do
+      @mock_game = instance_double("TicTacToe::Game", :board_to_array => [])
+      expect(TicTacToe::Game).to receive(:new).and_return(@mock_game)
+      expect(@mock_game).to receive(:take_turn).with(0)
+      expect(@mock_game).to receive(:over?).and_return true
 
-    it "redirects to /game" do
+      post '/make_move', {:move => '0'}
       expect(last_response.redirect?).to eq true
-      follow_redirect!
-      expect(last_request.path).to eq '/game'
+      expect(last_response.headers["Location"]).to eq ("http://example.org/game_over")
     end
   end
 
@@ -90,4 +114,4 @@ def app
       expect(last_response.body).to include "Result: Player X wins"
     end
   end
-end
\ No newline at end of file
+end