forked from pierront/rock-paper-scissors
dick #1
@@ -8,9 +8,9 @@ public enum Move {
|
||||
|
||||
public boolean beats(Move other) {
|
||||
return switch (this) {
|
||||
case ROCK -> other == SCISSORS || other == WELL;
|
||||
case PAPER -> other == ROCK;
|
||||
case SCISSORS -> other == PAPER || other == WELL;
|
||||
case ROCK -> other == SCISSORS;
|
||||
case PAPER -> other == ROCK || other == WELL;
|
||||
case SCISSORS -> other == PAPER;
|
||||
case WELL -> other == PAPER;
|
||||
};
|
||||
}
|
||||
|
||||
@@ -14,7 +14,7 @@ class MoveTest {
|
||||
|
||||
@Test
|
||||
void should_check_move_size(){
|
||||
Assertions.assertEquals(3, Move.values().length);
|
||||
Assertions.assertEquals(4, Move.values().length);
|
||||
}
|
||||
|
||||
@ParameterizedTest(name = "{0} should beat {1}")
|
||||
@@ -33,7 +33,10 @@ class MoveTest {
|
||||
return Stream.of(
|
||||
Arguments.of(Move.ROCK,Move.SCISSORS),
|
||||
Arguments.of(Move.PAPER,Move.ROCK),
|
||||
Arguments.of(Move.SCISSORS,Move.PAPER)
|
||||
Arguments.of(Move.SCISSORS,Move.PAPER),
|
||||
Arguments.of(Move.WELL,Move.ROCK),
|
||||
Arguments.of(Move.WELL,Move.SCISSORS),
|
||||
Arguments.of(Move.WELL,Move.PAPER)
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -20,7 +20,7 @@ class RandomCpuPickerTest {
|
||||
obtainedMoves.add(picker.pick());
|
||||
}
|
||||
|
||||
assertEquals(3, obtainedMoves.size());
|
||||
assertEquals(4, obtainedMoves.size());
|
||||
assertTrue(obtainedMoves.contains(Move.ROCK));
|
||||
assertTrue(obtainedMoves.contains(Move.PAPER));
|
||||
assertTrue(obtainedMoves.contains(Move.SCISSORS));
|
||||
|
||||
Reference in New Issue
Block a user