Merge pull request #36 from sandorw/feature/tests

HumanPlayer tests should use PlayerFactory
sandorw · Oct 19, 2015 · aca1cee · aca1cee
2 parents adc38e8 + d533928
commit aca1cee
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/test/java/com/github/sandorw/mocabogaso/players/HumanPlayerTest.java b/src/test/java/com/github/sandorw/mocabogaso/players/HumanPlayerTest.java
@@ -18,7 +18,7 @@
  * @author sandorw
  */
 public final class HumanPlayerTest {
-    private HumanPlayer<DefaultGameMove> player;
+    private Player<DefaultGameMove> player;
     private SimpleTestGameState gameState;
 
     @Before
@@ -30,7 +30,7 @@ public void before() {
     public void chooseNextMoveValidMoveTest() {
         InputStream inputStream = new ByteArrayInputStream("1".getBytes(Charset.forName("UTF-8")));
         System.setIn(inputStream);
-        player = new HumanPlayer<>();
+        player = PlayerFactory.getNewHumanPlayer();
         DefaultGameMove move = player.chooseNextMove(gameState);
         assertEquals(move.getLocation(), 1);
         System.setIn(System.in);
@@ -40,7 +40,7 @@ public void chooseNextMoveValidMoveTest() {
     public void chooseNextMoveInvalidMoveTest() {
         InputStream inputStream = new ByteArrayInputStream("4 4 1".getBytes(Charset.forName("UTF-8")));
         System.setIn(inputStream);
-        player = new HumanPlayer<>();
+        player = PlayerFactory.getNewHumanPlayer();
         DefaultGameMove move = player.chooseNextMove(gameState);
         assertEquals(move.getLocation(), 1);
         System.setIn(System.in);

diff --git a/...cabogaso/players/AIPlayerFactoryTest.java → ...mocabogaso/players/PlayerFactoryTest.java b/...cabogaso/players/AIPlayerFactoryTest.java → ...mocabogaso/players/PlayerFactoryTest.java
@@ -10,11 +10,11 @@
 import com.github.sandorw.mocabogaso.games.test.SimpleTestGameState;
 
 /**
- * Test cases for AIPlayerFactory
+ * Test cases for PlayerFactory
  * 
  * @author sandorw
  */
-public final class AIPlayerFactoryTest {
+public final class PlayerFactoryTest {
     SimpleTestGameState gameState;
     Game<DefaultGameMove, SimpleTestGameState> game;