Skip to content

Commit 717511e

Browse files
author
Gonzalo Diaz
committed
[REFACTOR] [Hacker Rank] Tests messages rewritten.
1 parent afa0282 commit 717511e

File tree

6 files changed

+62
-23
lines changed

6 files changed

+62
-23
lines changed

algorithm-exercises-java/src/test/java/ae/hackerrank/CompareTripletsTest.java

Lines changed: 18 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,13 @@
44

55
import java.util.Arrays;
66
import java.util.List;
7+
import org.junit.jupiter.api.BeforeAll;
78
import org.junit.jupiter.api.Test;
9+
import org.junit.jupiter.api.TestInstance;
10+
import org.junit.jupiter.api.TestInstance.Lifecycle;
811

912

13+
@TestInstance(Lifecycle.PER_CLASS)
1014
class CompareTripletsTest {
1115

1216
public class CompareTripletsTestCase {
@@ -21,24 +25,28 @@ public CompareTripletsTestCase(List<Integer> _la, List<Integer> _lb, List<Intege
2125
}
2226
}
2327

24-
@Test void testCompareTriplets() {
28+
public List<CompareTripletsTestCase> testCases;
2529

26-
CompareTripletsTestCase[] testCases = {
30+
@BeforeAll
31+
public void setup() {
32+
this.testCases = Arrays.asList(
2733
new CompareTripletsTestCase(
2834
Arrays.asList(5, 6, 7),
2935
Arrays.asList(3, 6, 10),
30-
Arrays.asList(1, 1)
31-
)
32-
};
36+
Arrays.asList(1, 1))
37+
);
38+
}
3339

34-
for (CompareTripletsTestCase testCase : testCases) {
40+
@Test void testCompareTriplets() {
41+
for (CompareTripletsTestCase testCase : this.testCases) {
3542
List<Integer> solutionFound = CompareTriplets.compareTriplets(testCase.la, testCase.lb);
3643

3744
assertEquals(testCase.expected, solutionFound,
38-
String.format(
39-
"CompareTriplets.compareTriplets() answer must be: %s",
40-
testCase.expected.toString()
41-
)
45+
String.format("%s(%s, %s) answer must be: %s",
46+
"CompareTriplets.compareTriplets",
47+
testCase.la.toString(),
48+
testCase.lb.toString(),
49+
testCase.expected.toString())
4250
);
4351
}
4452
}

algorithm-exercises-java/src/test/java/ae/hackerrank/DiagonalDifferenceTest.java

Lines changed: 33 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,49 @@
44

55
import java.util.Arrays;
66
import java.util.List;
7+
import org.junit.jupiter.api.BeforeAll;
78
import org.junit.jupiter.api.Test;
9+
import org.junit.jupiter.api.TestInstance;
10+
import org.junit.jupiter.api.TestInstance.Lifecycle;
811

912

13+
@TestInstance(Lifecycle.PER_CLASS)
1014
class DiagonalDifferenceTest {
1115

12-
@Test void testDiagonalDifference() {
16+
public class DiagonalDifferenceTestCase {
17+
public List<List<Integer>> matrix;
18+
public Integer expected;
19+
20+
public DiagonalDifferenceTestCase(List<List<Integer>> _input, int _expected) {
21+
this.matrix = _input;
22+
this.expected = _expected;
23+
}
24+
}
1325

14-
List<List<Integer>> matrix = Arrays.asList(
26+
public List<DiagonalDifferenceTestCase> testCases;
27+
28+
@BeforeAll
29+
public void setup() {
30+
this.testCases = Arrays.asList(
31+
new DiagonalDifferenceTestCase(Arrays.asList(
1532
Arrays.asList(11, 2, 4),
1633
Arrays.asList(4, 5, 6),
1734
Arrays.asList(10, 8, -12)
35+
), 15)
1836
);
19-
Integer expected = 15;
20-
Integer resultFound = DiagonalDifference.diagonalDifference(matrix);
37+
}
2138

22-
assertEquals(expected, resultFound,
23-
String.format("DiagonalDifference.diagonalDifference answer must be: %d", expected)
24-
);
39+
@Test void testDiagonalDifference() {
40+
41+
for (DiagonalDifferenceTestCase testCase : this.testCases) {
42+
Integer resultFound = DiagonalDifference.diagonalDifference(testCase.matrix);
43+
44+
assertEquals(testCase.expected, resultFound,
45+
String.format("%s(%s) answer must be: %s",
46+
"MiniMaxSum.miniMaxSum",
47+
testCase.matrix.toString(),
48+
testCase.expected.toString())
49+
);
50+
}
2551
}
2652
}

algorithm-exercises-java/src/test/java/ae/hackerrank/MiniMaxSumTest.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,11 @@ void testMiniMaxSum() {
4141
String solutionFound = MiniMaxSum.miniMaxSum(testCase.input);
4242

4343
assertEquals(testCase.expected, solutionFound,
44-
String.format(
45-
"CompareTriplets.compareTriplets() answer must be: %s",
46-
testCase.expected.toString()));
44+
String.format("%s(%s) answer must be: %s",
45+
"MiniMaxSum.miniMaxSum",
46+
testCase.input.toString(),
47+
testCase.expected.toString())
48+
);
4749
}
4850
}
4951

algorithm-exercises-java/src/test/java/ae/hackerrank/SimpleArraySumTest.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@ class SimpleArraySumTest {
1717
Integer solutionFound = SimpleArraySum.simpleArraySum(ar);
1818

1919
assertEquals(answer, solutionFound,
20-
String.format("Problem 0000 answer must be: %d", answer)
20+
String.format("%s(%s) answer must be: %d",
21+
"SimpleArraySum.simpleArraySum",
22+
ar.toString(),
23+
answer)
2124
);
2225
}
2326
}

algorithm-exercises-java/src/test/java/ae/hackerrank/SolveMeFirstTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class SolveMeFirstTest {
1616
Integer solutionFound = SolveMeFirst.solveMeFirst(a, b);
1717

1818
assertEquals(answer, solutionFound,
19-
String.format("Problem 0000 answer must be: %d", answer)
19+
String.format("%s(%d, %d) answer must be: %d", "SolveMeFirst.solveMeFirst", a, b, answer)
2020
);
2121
}
2222
}

algorithm-exercises-java/src/test/java/ae/hackerrank/VeryBigSumTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ class VeryBigSumTest {
1717
Long solutionFound = VeryBigSum.aVeryBigSum(arr);
1818

1919
assertEquals(answer, solutionFound,
20-
String.format("Problem 0000 answer must be: %d", answer)
20+
String.format("%s(%s) answer must be: %d", "VeryBigSum.aVeryBigSum", arr.toString(), answer)
2121
);
2222
}
2323
}

0 commit comments

Comments
 (0)