diff options
author | Joffrey BION <joffrey.bion@gmail.com> | 2017-07-08 19:19:05 +0200 |
---|---|---|
committer | Joffrey BION <joffrey.bion@gmail.com> | 2017-07-08 19:19:05 +0200 |
commit | b8a00699849026980913e65d0b42c6ef599f7c21 (patch) | |
tree | df69fa6658bf93cba5dfee8793055075a979297c /backend/src/test | |
parent | Frontend flow-typed regenerated (diff) | |
download | seven-wonders-b8a00699849026980913e65d0b42c6ef599f7c21.tar.gz seven-wonders-b8a00699849026980913e65d0b42c6ef599f7c21.tar.bz2 seven-wonders-b8a00699849026980913e65d0b42c6ef599f7c21.zip |
Remove comments that are now unnecessary
We now use a TestPrincipal class instead of a lambda in order to be
clear enough to be self-explanatory.
Diffstat (limited to 'backend/src/test')
-rw-r--r-- | backend/src/test/java/org/luxons/sevenwonders/controllers/LobbyControllerTest.java | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/backend/src/test/java/org/luxons/sevenwonders/controllers/LobbyControllerTest.java b/backend/src/test/java/org/luxons/sevenwonders/controllers/LobbyControllerTest.java index bee3110b..6958ad8b 100644 --- a/backend/src/test/java/org/luxons/sevenwonders/controllers/LobbyControllerTest.java +++ b/backend/src/test/java/org/luxons/sevenwonders/controllers/LobbyControllerTest.java @@ -48,7 +48,6 @@ public class LobbyControllerTest { @Test(expected = PlayerNotFoundException.class) public void leave_failsWhenPlayerDoesNotExist() { - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.leave(principal); } @@ -56,7 +55,6 @@ public class LobbyControllerTest { @Test(expected = PlayerNotInLobbyException.class) public void leave_failsWhenNotInLobby() { playerRepository.createOrUpdate("testuser", "Test User"); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.leave(principal); } @@ -69,7 +67,6 @@ public class LobbyControllerTest { assertTrue(lobby.getPlayers().contains(player)); assertSame(lobby, player.getLobby()); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.leave(principal); @@ -86,7 +83,6 @@ public class LobbyControllerTest { assertTrue(lobby.getPlayers().contains(player2)); assertSame(lobby, player2.getLobby()); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser2"); lobbyController.leave(principal); @@ -111,7 +107,6 @@ public class LobbyControllerTest { List<String> playerNames = reorderedPlayers.stream().map(Player::getUsername).collect(Collectors.toList()); reorderPlayersAction.setOrderedPlayers(playerNames); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.reorderPlayers(reorderPlayersAction, principal); @@ -131,7 +126,6 @@ public class LobbyControllerTest { List<String> playerNames = reorderedPlayers.stream().map(Player::getUsername).collect(Collectors.toList()); reorderPlayersAction.setOrderedPlayers(playerNames); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser2"); lobbyController.reorderPlayers(reorderPlayersAction, principal); } @@ -159,7 +153,6 @@ public class LobbyControllerTest { newSettings.setWonPointsPerVictoryPerAge(new HashMap<>()); updateSettingsAction.setSettings(newSettings); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.updateSettings(updateSettingsAction, principal); @@ -177,7 +170,6 @@ public class LobbyControllerTest { UpdateSettingsAction updateSettingsAction = new UpdateSettingsAction(); updateSettingsAction.setSettings(new CustomizableSettings()); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser2"); lobbyController.updateSettings(updateSettingsAction, principal); } @@ -191,7 +183,6 @@ public class LobbyControllerTest { addPlayer(lobby, "testuser3"); addPlayer(lobby, "testuser4"); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser"); lobbyController.startGame(principal); @@ -206,7 +197,6 @@ public class LobbyControllerTest { addPlayer(lobby, "testuser2"); addPlayer(lobby, "testuser3"); - // the Principal interface just contains a getName() method Principal principal = new TestPrincipal("testuser2"); lobbyController.startGame(principal); } |