Merge pull request #66 from testomatio/cucumber-parametrized-support

Implemented cucumber parametrized tests support

Author: YevheniiVlasenko

java-reporter-core/src/main/java/io/testomat/core/client/NativeApiClient.java

@@ -1,8 +1,5 @@
 package io.testomat.core.client;
 
-import static io.testomat.core.constants.CommonConstants.REPORTER_VERSION;
-import static io.testomat.core.constants.CommonConstants.RESPONSE_UID_KEY;
-
 import io.testomat.core.client.http.CustomHttpClient;
 import io.testomat.core.client.request.NativeRequestBodyBuilder;
 import io.testomat.core.client.request.RequestBodyBuilder;
@@ -18,6 +15,9 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import static io.testomat.core.constants.CommonConstants.REPORTER_VERSION;
+import static io.testomat.core.constants.CommonConstants.RESPONSE_UID_KEY;
+
 /**
  * HTTP client for Testomat.io API operations.
  * Handles test run lifecycle and result reporting with proper error handling.

java-reporter-cucumber/java-reporter-core-0.1.0.jar

@@ -6,7 +6,7 @@
 
     <groupId>io.testomat</groupId>
     <artifactId>java-reporter-cucumber</artifactId>
-    <version>0.6.8</version>
+    <version>0.7.2</version>
     <packaging>jar</packaging>
 
     <name>Testomat.io Java Reporter Cucumber</name>
@@ -50,7 +50,7 @@
         <dependency>
             <groupId>io.testomat</groupId>
             <artifactId>java-reporter-core</artifactId>
-            <version>0.6.462</version>
+            <version>0.7.2</version>
         </dependency>
         <dependency>
             <groupId>io.cucumber</groupId>
@@ -73,6 +73,12 @@
             <version>${junit.version}</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-params</artifactId>
+            <version>${junit.version}</version>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>org.mockito</groupId>
             <artifactId>mockito-core</artifactId>

java-reporter-cucumber/pom.xml

@@ -1,66 +1,55 @@
 package io.testomat.cucumber.constructor;
 
+import io.cucumber.plugin.event.TestCaseFinished;
 import io.testomat.core.model.ExceptionDetails;
 import io.testomat.core.model.TestResult;
-import java.io.PrintWriter;
-import java.io.StringWriter;
-import java.util.Optional;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import io.testomat.cucumber.extractor.TestDataExtractor;
 
 /**
  * Constructs test case results from Cucumber test case finished events.
  * Extracts exception details from Cucumber result errors when available.
  */
 public class CucumberTestResultConstructor {
-    private static final Logger log = LoggerFactory.getLogger(CucumberTestResultConstructor.class);
 
-    public TestResult constructTestRunResult(TestResultWrapper holder) {
-        validateHolder(holder);
+    private final TestDataExtractor testDataExtractor;
 
-        log.debug("Creating Cucumber test result with exception details for: {}",
-                holder.getTestMetadata().getTitle());
-
-        ExceptionDetails exceptionDetails = extractExceptionDetails(holder);
-
-        return TestResult.builder()
-                .withTitle(holder.getTestMetadata().getTitle())
-                .withTestId(holder.getTestMetadata().getTestId())
-                .withSuiteTitle(holder.getTestMetadata().getSuiteTitle())
-                .withFile(holder.getTestMetadata().getFile())
-                .withStatus(holder.getStatus())
-                .withMessage(exceptionDetails.getMessage())
-                .withStack(exceptionDetails.getStack())
-                .build();
+    /**
+     * Creates a new instance with default test data extractor.
+     */
+    public CucumberTestResultConstructor() {
+        this.testDataExtractor = new TestDataExtractor();
     }
 
-    private void validateHolder(TestResultWrapper holder) {
-        if (holder == null) {
-            throw new IllegalArgumentException("TestRunResultWrapper cannot be null");
-        }
-        if (holder.getTestMetadata() == null) {
-            throw new IllegalArgumentException("TestMetadata cannot be null");
-        }
+    /**
+     * Creates a new instance with the specified test data extractor.
+     *
+     * @param testDataExtractor the test data extractor to use
+     */
+    public CucumberTestResultConstructor(TestDataExtractor testDataExtractor) {
+        this.testDataExtractor = testDataExtractor;
     }
 
-    private ExceptionDetails extractExceptionDetails(TestResultWrapper holder) {
-        return Optional.ofNullable(holder.getCucumberTestCaseFinished())
-                .map(event -> event.getResult().getError())
-                .map(this::createExceptionDetails)
-                .orElse(ExceptionDetails.empty());
-    }
+    /**
+     * Constructs a test result from a Cucumber test case finished event.
+     * Extracts test metadata, status, and error details from the event.
+     *
+     * @param event the Cucumber test case finished event
+     * @return the constructed test result
+     */
+    public TestResult constructTestRunResult(TestCaseFinished event) {
 
-    private ExceptionDetails createExceptionDetails(Throwable throwable) {
-        String message = throwable.getMessage();
-        String stack = getStackTrace(throwable);
-        log.debug("Including error details for failed test");
-        return new ExceptionDetails(message, stack);
-    }
+        ExceptionDetails exceptionDetails = testDataExtractor.extractExceptionDetails(event);
 
-    private String getStackTrace(Throwable t) {
-        StringWriter sw = new StringWriter();
-        PrintWriter pw = new PrintWriter(sw);
-        t.printStackTrace(pw);
-        return sw.toString();
+        return TestResult.builder()
+                .withStatus(testDataExtractor.getNormalizedStatus(event))
+                .withSuiteTitle(event.getTestCase().getUri().toString())
+                .withExample(testDataExtractor.createExample(event))
+                .withTestId(testDataExtractor.extractTestId(event))
+                .withFile(testDataExtractor.extractFileName(event))
+                .withTitle(testDataExtractor.extractTitle(event))
+                .withRid(event.getTestCase().getId().toString())
+                .withMessage(exceptionDetails.getMessage())
+                .withStack(exceptionDetails.getStack())
+                .build();
     }
 }

java-reporter-cucumber/src/main/java/io/testomat/cucumber/constructor/CucumberTestResultConstructor.java

@@ -0,0 +1,7 @@
+package io.testomat.cucumber.exception;
+
+public class CucumberListenerException extends RuntimeException {
+    public CucumberListenerException(String message) {
+        super(message);
+    }
+}

java-reporter-cucumber/src/main/java/io/testomat/cucumber/constructor/TestResultWrapper.java

@@ -0,0 +1,7 @@
+package io.testomat.cucumber.exception;
+
+public class StatusNormalizerException extends RuntimeException {
+    public StatusNormalizerException(String message) {
+        super(message);
+    }
+}