Possible fix for issue displaying durations in parameterized tests

serenity-bdd · Jan 30, 2025 · 1ef0248 · 1ef0248
1 parent 35de199
commit 1ef0248
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/serenity-core/src/main/java/net/thucydides/core/steps/StepEventBus.java b/serenity-core/src/main/java/net/thucydides/core/steps/StepEventBus.java
@@ -493,7 +493,7 @@ public void finishTestRun() {
     }
 
     public void testFinished() {
-        testFinished(false);
+        testFinished(this.isCurrentTestDataDriven());
     }
 
     private TestOutcome checkForEmptyScenarioIn(TestOutcome outcome) {

diff --git a/serenity-junit5/src/main/java/net/serenitybdd/junit5/SerenityTestExecutionListener.java b/serenity-junit5/src/main/java/net/serenitybdd/junit5/SerenityTestExecutionListener.java
@@ -298,11 +298,12 @@ public synchronized void executionFinished(TestIdentifier testIdentifier, TestEx
             if (isMethodSource(testIdentifier)) {
                 MethodSource methodSource = ((MethodSource) testIdentifier.getSource().get());
                 String sourceMethod = methodSource.getClassName() + "." + methodSource.getMethodName();
-                testFinished(testIdentifier, methodSource, testExecutionResult);
+//                testFinished(testIdentifier, methodSource, testExecutionResult);
                 DataTable dataTable = dataTables.get(sourceMethod);
                 if (dataTable != null) {
                     eventBusFor(testIdentifier).exampleFinished();
                 }
+                testFinished(testIdentifier, methodSource, testExecutionResult);
             }
         }
         recordSummaryData(testIdentifier, testExecutionResult);