Skip to content

Commit fedf135

Browse files
committed
Fix SpotBugs issues and format TowerOfHanoi tests
1 parent 619c462 commit fedf135

File tree

1 file changed

+4
-16
lines changed

1 file changed

+4
-16
lines changed

src/test/java/com/thealgorithms/recursion/TowerOfHanoiTest.java

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.thealgorithms.recursion;
22

33
import static org.junit.jupiter.api.Assertions.assertEquals;
4+
import static org.junit.jupiter.api.Assertions.assertFalse;
45
import static org.junit.jupiter.api.Assertions.assertThrows;
56

67
import java.util.List;
@@ -19,14 +20,7 @@ void testMoveCount(int disks, long expectedMoves) {
1920
}
2021

2122
private static Stream<Arguments> diskCountAndMoveCount() {
22-
return Stream.of(
23-
Arguments.of(1, 1L),
24-
Arguments.of(2, 3L),
25-
Arguments.of(3, 7L),
26-
Arguments.of(4, 15L),
27-
Arguments.of(5, 31L),
28-
Arguments.of(10, 1023L)
29-
);
23+
return Stream.of(Arguments.of(1, 1L), Arguments.of(2, 3L), Arguments.of(3, 7L), Arguments.of(4, 15L), Arguments.of(5, 31L), Arguments.of(10, 1023L));
3024
}
3125

3226
@ParameterizedTest
@@ -37,13 +31,7 @@ void testSolveReturnsCorrectNumberOfMoves(int disks, long expectedMoveCount) {
3731
}
3832

3933
private static Stream<Arguments> diskCountAndExpectedMoves() {
40-
return Stream.of(
41-
Arguments.of(1, 1L),
42-
Arguments.of(2, 3L),
43-
Arguments.of(3, 7L),
44-
Arguments.of(4, 15L),
45-
Arguments.of(5, 31L)
46-
);
34+
return Stream.of(Arguments.of(1, 1L), Arguments.of(2, 3L), Arguments.of(3, 7L), Arguments.of(4, 15L), Arguments.of(5, 31L));
4735
}
4836

4937
@Test
@@ -112,6 +100,6 @@ void testSolveMoveCountMatchesGetMoveCount(int disks, long expectedMoveCount) {
112100
@Test
113101
void testSolveIsNotEmpty() {
114102
List<String> moves = TowerOfHanoi.solve(1, 'A', 'C', 'B');
115-
assertEquals(false, moves.isEmpty());
103+
assertFalse(moves.isEmpty());
116104
}
117105
}

0 commit comments

Comments
 (0)