From 624782758ad2e4caeabc8ec29004c17b7112f5e8 Mon Sep 17 00:00:00 2001
From: ghay <graham.hay@7digital.com>
Date: Wed, 19 Oct 2011 16:13:21 +0100
Subject: [PATCH] Return who played

---
 scripts/controller.js             | 16 +++++++++++-----
 unit-tests/spec/ControllerSpec.js | 25 ++++++++++++++++---------
 2 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/scripts/controller.js b/scripts/controller.js
index f27411f..17f4a05 100644
--- a/scripts/controller.js
+++ b/scripts/controller.js
@@ -9,8 +9,8 @@ TICTACTOE.controller = function(umpire){
 			board = [[],[],[]];
 			turnsPlayed = 0;
 			gameOver = false;
-			currentPlayer  = "0";
-			return "Player 0's turn";
+			currentPlayer = "0";
+			return { message : "Player 0's turn" };
 		},
 	
 		play : function(x, y) {	
@@ -33,19 +33,25 @@ TICTACTOE.controller = function(umpire){
 
 			board[x][y] = currentPlayer;
 			
+            var result = { playedBy : currentPlayer };
+            
 			var hasAWinner = umpire.checkForWinner(board); 
 			if (hasAWinner) {				
 				gameOver = true;
-				return "Player " + currentPlayer + " rules!";
+                result.message = "Player " + currentPlayer + " rules!";
+                return result;
 			}
 			
 			if (turnsPlayed >= 9){
 				gameOver = true;
-				return "Game is a draw";
+				result.message = "Game is a draw";
+                return result;
 			}
 						
 			currentPlayer = currentPlayer === "0" ? "X" : "0";
-			return "Player " + currentPlayer + "'s turn";
+			
+            result.message = "Player " + currentPlayer + "'s turn";
+            return result;
 		}
 	};
 };
\ No newline at end of file
diff --git a/unit-tests/spec/ControllerSpec.js b/unit-tests/spec/ControllerSpec.js
index 73bb5e4..3473a52 100644
--- a/unit-tests/spec/ControllerSpec.js
+++ b/unit-tests/spec/ControllerSpec.js
@@ -21,17 +21,24 @@
 		
 			it("should return 'Player X's turn' if it's player X's turn next", function() {
 				var result = controller.play(1, 1);	
-				expect(result).toEqual("Player X's turn");
+				expect(result.message).toEqual("Player X's turn");
 			});
 			
 			it("should return 'Player 0's turn' if it's player 0's turn next", function() {
 				var result = controller.play(1, 1);	
 				result = controller.play(1, 2);	
-				expect(result).toEqual("Player 0's turn");
+				expect(result.message).toEqual("Player 0's turn");
 			});
 
 		});
 		
+        describe("returns played by", function () {		
+			it("should return who played, innit", function() {
+				var result = controller.play(1, 1);	
+				expect(result.playedBy).toEqual("0");
+			});
+		});
+        
 		describe("draw", function(){
 			it("should return expected message if game is drawn", function(){
 				var result;
@@ -40,7 +47,7 @@
 						result = controller.play(i, j);
 					}
 				}
-				expect(result).toEqual("Game is a draw");
+				expect(result.message).toEqual("Game is a draw");
 			});	
 		});
 		
@@ -57,10 +64,10 @@
 			
 			it("should set current player to 0 when game is reset", function(){
 				controller.play(1,2);
-				var currentPlayer =                   controller.reset();
-				expect(currentPlayer).toEqual("Player 0's turn");
-				currentPlayer = controller.play(2,2);
-				expect(currentPlayer).toEqual("Player X's turn");
+				var result = controller.reset();
+				expect(result.message).toEqual("Player 0's turn");
+				result = controller.play(2,2);
+				expect(result.message).toEqual("Player X's turn");
 			});	
 		});
 			
@@ -125,7 +132,7 @@
                     return true;
                 }
                 var result = controller.play(0, 0);
-                expect(result).toEqual("Player 0 rules!");
+                expect(result.message).toEqual("Player 0 rules!");
             });
             
             it("asks the umpire if there's a winner after the 2nd play is recorded", function(){      
@@ -134,7 +141,7 @@
                     return true;
                 }
                 var result = controller.play(1, 1);
-                expect(result).toEqual("Player X rules!");
+                expect(result.message).toEqual("Player X rules!");
             });
         });