From 5847e2485251996c76f5e2f0c9f2497a25da8d21 Mon Sep 17 00:00:00 2001
From: Liviu Carausu <liviu.carausu@gmail.com>
Date: Sat, 16 Dec 2023 23:35:34 +0100
Subject: [PATCH] take back  "isInDataDrivenTest" flag for stepFailed (#3360)

---
 .../net/thucydides/core/steps/BaseStepListener.java    | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/serenity-core/src/main/java/net/thucydides/core/steps/BaseStepListener.java b/serenity-core/src/main/java/net/thucydides/core/steps/BaseStepListener.java
index 2dc3b2f23..7dd9e0c9a 100644
--- a/serenity-core/src/main/java/net/thucydides/core/steps/BaseStepListener.java
+++ b/serenity-core/src/main/java/net/thucydides/core/steps/BaseStepListener.java
@@ -924,15 +924,9 @@ public void stepFailed(StepFailure failure, List<ScreenshotAndHtmlSource> screen
             getCurrentTestOutcome().appendTestFailure(failureCause);
 
             recordFailureDetails(failure);
-            // Step marked as done with the appropriate result before
-            if (isInDataDrivenTest) {
-                currentStepDone(failureAnalysis.resultFor(failure));
-            }
-        }
-        if (!isInDataDrivenTest) {
-            // Step marked as done with the appropriate result before
-            currentStepDone(failureAnalysis.resultFor(failure));
         }
+        // Step marked as done with the appropriate result before
+        currentStepDone(failureAnalysis.resultFor(failure));
     }