Skip to content

Commit 2c84483

Browse files
author
Pierre-Yves Fourmond
committed
Renommer la classe utilitaire pour les tests
1 parent bcdd3be commit 2c84483

4 files changed

Lines changed: 11 additions & 12 deletions

File tree

README.md

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -132,8 +132,7 @@ Il ne nous reste plus qu'à :
132132

133133
💡 Avec la JEP 458, il est également possible d'implémenter des tests, que l'on lancera en ligne de commande comme c'est déjà le cas pour l'application.
134134

135-
La classe [LanguageTest](src/LanguageTest.java) qui utilise la classe [TestRunner](src/TestRunner.java) et les
136-
assertions Java montre comment procéder :
135+
La classe [LanguageTest](src/LanguageTest.java) qui utilise la classe [TestHelper](src/TestHelper.java) et les assertions Java montre comment procéder :
137136

138137
```bash
139138
$ make test
@@ -143,16 +142,16 @@ $ make test
143142
./src/LanguageTest.java
144143
✅ Test fromStringTest is successful
145144
❌ Test failingTest has failed
146-
java.lang.AssertionError: Test was expected to fail !
147-
at LanguageTest.failingTest(LanguageTest.java:13)
145+
java.lang.AssertionError: Test was expected to fail.
146+
at LanguageTest.failingTest(LanguageTest.java:14)
148147
at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
149148
at java.base/java.lang.reflect.Method.invoke(Method.java:565)
150-
at TestRunner.runTest(TestRunner.java:21)
151-
at TestRunner.lambda$runTests$0(TestRunner.java:12)
149+
at TestHelper.runTest(TestHelper.java:40)
150+
at TestHelper.lambda$runTests$0(TestHelper.java:31)
152151
at java.base/java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:1024)
153152
at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:803)
154-
at TestRunner.runTests(TestRunner.java:12)
155-
at TestRunner.runTests(TestRunner.java:6)
153+
at TestHelper.runTests(TestHelper.java:31)
154+
at TestHelper.runTests(TestHelper.java:9)
156155
at LanguageTest.main(LanguageTest.java:4)
157156
at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
158157
at java.base/java.lang.reflect.Method.invoke(Method.java:565)

src/ApplicationTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
//usr/bin/env java -enableassertions --class-path ${APP_DIR:-.}/lib/'*' "$0" "$@"; exit $?
22

33
void main() {
4-
TestRunner.runTests(getClass());
4+
TestHelper.runTests(getClass());
55
}
66

77
void sayHelloFrenchTest() {
@@ -20,5 +20,5 @@ void helpFlagTest() {
2020
}
2121

2222
String runApplication(String... args) {
23-
return TestRunner.runApplication("./src/Application.java", args);
23+
return TestHelper.runApplication("./src/Application.java", args);
2424
}

src/LanguageTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
//usr/bin/env java -enableassertions --class-path ${APP_DIR:-.}/lib/'*' "$0" "$@"; exit $?
22

33
void main() {
4-
TestRunner.runTests(getClass());
4+
TestHelper.runTests(getClass());
55
}
66

77
void fromStringTest() {
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import static java.util.stream.Collectors.joining;
44

5-
class TestRunner {
5+
class TestHelper {
66

77
static void runTests(Class<?>... classes) {
88
for (Class<?> clazz : classes) {

0 commit comments

Comments
 (0)