BAEL-9376 use givenWhenThen test names

Author: Sam Gardner

testing-modules/junit-5-advanced-3/src/test/java/com/baeldung/enginetestkit/DisplayTest.java

@@ -16,23 +16,23 @@ public class DisplayTest {
     private final Display display = new Display(Platform.DESKTOP, 1000);
 
     @Test
-    void succeeds() {
+    void whenCorrect_thenSucceeds() {
         assertEquals(1000, display.getHeight());
     }
 
     @Test
-    void fails() {
+    void whenIncorrect_thenFails() {
         assertEquals(500, display.getHeight());
     }
 
     @Test
     @Disabled("Flakey test needs investigating")
-    void skips() {
+    void whenDisabled_thenSkips() {
         assertEquals(999, display.getHeight());
     }
 
     @Test
-    void aborts() {
+    void whenAssumptionsFail_thenAborts() {
         assumeTrue(display.getPlatform() == Platform.MOBILE, "test only runs for mobile");
     }
 

testing-modules/junit-5-advanced-3/src/test/java/com/baeldung/enginetestkit/EngineTestKitDiscoveryUnitTest.java

@@ -22,7 +22,7 @@
 public class EngineTestKitDiscoveryUnitTest {
 
     @Test
-    void verifyTestEngineDiscovery() {
+    void givenJunitJupiterEngine_whenRunningTestSuite_thenTestsAreDiscovered() {
         EngineDiscoveryResults results = EngineTestKit.engine("junit-jupiter")
             .selectors(selectClass(DisplayTest.class))
             .discover();
@@ -32,7 +32,7 @@ void verifyTestEngineDiscovery() {
     }
 
     @Test
-    void verifyVintageDiscovery() {
+    void givenJunitVintageEngine_whenRunningTestSuite_thenTestsAreDiscovered() {
         EngineDiscoveryResults results = EngineTestKit.engine("junit-vintage")
             .selectors(selectClass(DisplayTest.class))
             .discover();
@@ -41,7 +41,7 @@ void verifyVintageDiscovery() {
     }
 
     @Test
-    void verifyHighLevelTestStats() {
+    void givenTestSuite_whenRunningAllTests_thenCollectHighLevelStats() {
         EngineTestKit
             .engine("junit-jupiter")
             .selectors(selectClass(DisplayTest.class))
@@ -52,29 +52,29 @@ void verifyHighLevelTestStats() {
     }
 
     @Test
-    void verifyTestAbortion() {
+    void givenTestSuite_whenRunningTestThatAborts_thenCollectDetailedStats() {
         Events testEvents = EngineTestKit
             .engine("junit-jupiter")
-            .selectors(selectMethod(DisplayTest.class, "aborts"))
+            .selectors(selectMethod(DisplayTest.class, "whenAssumptionsFail_thenAborts"))
             .execute()
             .testEvents();
 
         testEvents.assertThatEvents()
-            .haveExactly(1, event(test("aborts"),
+            .haveExactly(1, event(test("whenAssumptionsFail_thenAborts"),
                     abortedWithReason(instanceOf(TestAbortedException.class),
                         message(message -> message.contains("test only runs for mobile")))));
     }
 
     @Test
-    void verifyTestFailure() {
+    void givenTestSuite_whenRunningTestThatFails_thenCollectDetailedStats() {
         Events testEvents = EngineTestKit
             .engine("junit-jupiter")
-            .selectors(selectMethod(DisplayTest.class, "fails"))
+            .selectors(selectMethod(DisplayTest.class, "whenIncorrect_thenFails"))
             .execute()
             .testEvents();
 
         testEvents.assertThatEvents()
-            .haveExactly(1, event(test("fails"),
+            .haveExactly(1, event(test("whenIncorrect_thenFails"),
                 finishedWithFailure(instanceOf(AssertionFailedError.class))));
     }