#185: Fixed missed scenarios for BDD

pom.xml

@@ -5,7 +5,7 @@
 
     <groupId>com.github.invictum</groupId>
     <artifactId>serenity-reportportal-integration</artifactId>
-    <version>1.6.0-SNAPSHOT</version>
+    <version>1.6.2-SNAPSHOT</version>
 
     <packaging>jar</packaging>
 
@@ -26,6 +26,11 @@
             <email>zim182@gmail.com</email>
             <organizationUrl>https://github.com/Invictum</organizationUrl>
         </developer>
+        <developer>
+            <name>Daria Yershova</name>
+            <email>dariayershova@icloud.com</email>
+            <organizationUrl>https://github.com/grey-rain</organizationUrl>
+        </developer>
     </developers>
 
     <scm>

src/main/java/com/github/invictum/reportportal/ReportPortalListener.java

@@ -1,6 +1,7 @@
 package com.github.invictum.reportportal;
 
 import com.github.invictum.reportportal.injector.IntegrationInjector;
+import com.github.invictum.reportportal.model.TestType;
 import com.github.invictum.reportportal.recorder.TestRecorder;
 import com.google.inject.Inject;
 import net.thucydides.core.webdriver.ThucydidesWebDriverSupport;
@@ -129,7 +130,10 @@ public void stepFinished(List<ScreenshotAndHtmlSource> list, ZonedDateTime zoned
 
     @Override
     public void testFailed(TestOutcome testOutcome, Throwable throwable) {
-        // Not used by listener
+        TestType testType = TestType.byTestSource(testOutcome.getTestSource());
+        if (testType == TestType.BDD) {
+            testFinished(testOutcome);
+        }
     }
 
     @Override

src/main/java/com/github/invictum/reportportal/model/TestType.java

@@ -0,0 +1,24 @@
+package com.github.invictum.reportportal.model;
+
+import org.apache.commons.lang3.NotImplementedException;
+
+import java.util.Arrays;
+
+public enum TestType {
+    BDD("(cucumber|jbehave)"),
+    JUNIT5("JUnit5");
+
+    private final String regexMatcher;
+
+    TestType(String regexMatcher) {
+        this.regexMatcher = regexMatcher;
+    }
+
+    public static TestType byTestSource(String testSource) {
+        return Arrays.stream(values())
+                .filter(type -> testSource.toLowerCase().matches(type.regexMatcher))
+                .findFirst()
+                .orElseThrow(() -> new NotImplementedException(
+                        String.format("Test source %s is not supported", testSource)));
+    }
+}

src/main/java/com/github/invictum/reportportal/recorder/TestRecorder.java

@@ -5,6 +5,7 @@
 import com.epam.ta.reportportal.ws.model.StartTestItemRQ;
 import com.github.invictum.reportportal.*;
 import com.github.invictum.reportportal.injector.IntegrationInjector;
+import com.github.invictum.reportportal.model.TestType;
 import com.google.inject.Injector;
 import io.reactivex.Maybe;
 import net.thucydides.model.domain.TestOutcome;
@@ -36,7 +37,8 @@ public TestRecorder(SuiteStorage suiteStorage, Launch launch, LogUnitsHolder hol
      */
     public static TestRecorder forTest(TestOutcome testOutcome) {
         Injector injector = IntegrationInjector.getInjector();
-        if (testOutcome.isDataDriven() && testOutcome.getTestSource().toLowerCase().matches("(cucumber|jbehave)")) {
+        boolean isBdd = TestType.byTestSource(testOutcome.getTestSource()) == TestType.BDD;
+        if (testOutcome.isDataDriven() && isBdd) {
             return injector.getInstance(BddDataDriven.class);
         }
         return injector.getInstance(Regular.class);