- * The method adds a new entry containing {@code test_id} and {@code rid} - * only if an entry with the same values does not already exist. - * The check and insertion are performed atomically using synchronization - * to prevent duplicates in concurrent environments. + * The method extracts {@code test_id} and {@code rid} from the provided body + * and stores them as a new entry if an identical pair does not already exist. + * If {@code rid} is {@code null}, the entry is ignored and nothing is stored. + *
+ * The existence check and insertion are performed inside a synchronized block
+ * to guarantee atomicity and prevent duplicate entries in concurrent environments.
*
- * @param body a map containing test execution data; expected to include
- * {@code "test_id"} and {@code "rid"} keys
+ * @param body a map containing test execution data. Expected to contain
+ * {@code "rid"} and optionally {@code "test_id"}.
*/
public static void store(Map
- * The identifier is deterministically derived from the scenario location
- * (feature file path and scenario line number). This ensures that the same
- * scenario always produces the same rId across multiple test runs.
- *
- * The rId is generated using a name-based UUID calculated from the string:
- *
- * <feature-relative-path>:<scenario-line-number>
- *
+ * Generates a report identifier (rId) for a Karate scenario.
+ * The identifier is based on the feature path, scenario line, and example index
+ * (if the scenario is a Scenario Outline).
*
- * @param sr the {@link ScenarioRuntime} representing the executed Karate test scenario
- * @return a deterministic UUID string uniquely identifying the scenario execution
+ * @param sr scenario runtime
+ * @return stable UUID identifying this scenario execution
*/
public String getRid(ScenarioRuntime sr) {
- String raw = String.format("%s:%s",
- sr.scenario.getFeature().getResource().getRelativePath(),
- sr.scenario.getLine()
- );
+ StringBuilder raw = new StringBuilder()
+ .append(sr.scenario.getFeature().getResource().getRelativePath())
+ .append(":")
+ .append(sr.scenario.getLine());
+
+ int exampleIndex = sr.scenario.getExampleIndex();
+ if (exampleIndex >= 0) {
+ raw.append(":").append(exampleIndex);
+ }
- return UUID.nameUUIDFromBytes(raw.getBytes()).toString();
+ return UUID.nameUUIDFromBytes(raw.toString().getBytes(StandardCharsets.UTF_8)).toString();
}
private ExceptionDetails createExceptionDetails(Throwable throwable) {
diff --git a/java-reporter-karate/src/test/java/io/testomat/karate/extractor/TestDataExtractorTest.java b/java-reporter-karate/src/test/java/io/testomat/karate/extractor/TestDataExtractorTest.java
index 8e7330a..b54c596 100644
--- a/java-reporter-karate/src/test/java/io/testomat/karate/extractor/TestDataExtractorTest.java
+++ b/java-reporter-karate/src/test/java/io/testomat/karate/extractor/TestDataExtractorTest.java
@@ -9,9 +9,11 @@
import com.intuit.karate.core.Tags;
import com.intuit.karate.resource.Resource;
import io.testomat.core.model.ExceptionDetails;
+import java.nio.charset.StandardCharsets;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
+import java.util.UUID;
import java.util.stream.Collectors;
import org.junit.jupiter.api.Test;
@@ -143,13 +145,107 @@ void getRidGeneratesStableUuid() {
ScenarioRuntime sr = mockScenarioRuntime();
when(sr.scenario.getLine()).thenReturn(4);
+ when(sr.scenario.getExampleIndex()).thenReturn(-1);
String rid1 = extractor.getRid(sr);
String rid2 = extractor.getRid(sr);
+ assertThat(rid1)
+ .isNotNull()
+ .isEqualTo(rid2);
+ }
+
+ @Test
+ void getRidSameWithoutExampleIndex() {
+ ScenarioRuntime sr1 = mockScenarioRuntime();
+ ScenarioRuntime sr2 = mockScenarioRuntime();
+
+ when(sr1.scenario.getLine()).thenReturn(5);
+ when(sr2.scenario.getLine()).thenReturn(5);
+
+ when(sr1.scenario.getExampleIndex()).thenReturn(-1);
+ when(sr2.scenario.getExampleIndex()).thenReturn(-1);
+
+ String rid1 = extractor.getRid(sr1);
+ String rid2 = extractor.getRid(sr2);
+
assertThat(rid1).isEqualTo(rid2);
}
+ @Test
+ void getRidDifferentLinesProduceDifferentRid() {
+ ScenarioRuntime sr1 = mockScenarioRuntime();
+ ScenarioRuntime sr2 = mockScenarioRuntime();
+
+ when(sr1.scenario.getLine()).thenReturn(1);
+ when(sr2.scenario.getLine()).thenReturn(2);
+
+ when(sr1.scenario.getExampleIndex()).thenReturn(-1);
+ when(sr2.scenario.getExampleIndex()).thenReturn(-1);
+
+ String rid1 = extractor.getRid(sr1);
+ String rid2 = extractor.getRid(sr2);
+
+ assertThat(rid1).isNotEqualTo(rid2);
+ }
+
+ @Test
+ void getRidIncludesExampleIndex() {
+ ScenarioRuntime sr1 = mockScenarioRuntime();
+ ScenarioRuntime sr2 = mockScenarioRuntime();
+
+ when(sr1.scenario.getLine()).thenReturn(10);
+ when(sr2.scenario.getLine()).thenReturn(10);
+
+ when(sr1.scenario.getExampleIndex()).thenReturn(0);
+ when(sr2.scenario.getExampleIndex()).thenReturn(1);
+
+ String rid1 = extractor.getRid(sr1);
+ String rid2 = extractor.getRid(sr2);
+
+ assertThat(rid1).isNotEqualTo(rid2);
+ }
+
+ @Test
+ void getRidDifferentFeatureFilesProduceDifferentRid() {
+ ScenarioRuntime sr1 = mockScenarioRuntime();
+ ScenarioRuntime sr2 = mockScenarioRuntime();
+
+ when(sr1.scenario.getLine()).thenReturn(1);
+ when(sr2.scenario.getLine()).thenReturn(1);
+
+ when(sr1.scenario.getExampleIndex()).thenReturn(-1);
+ when(sr2.scenario.getExampleIndex()).thenReturn(-1);
+
+ when(sr2.scenario.getFeature()
+ .getResource()
+ .getRelativePath())
+ .thenReturn("features/Other.feature");
+
+ String rid1 = extractor.getRid(sr1);
+ String rid2 = extractor.getRid(sr2);
+
+ assertThat(rid1).isNotEqualTo(rid2);
+ }
+
+ @Test
+ void getRidMatchesExpectedUuid() {
+ ScenarioRuntime sr = mockScenarioRuntime();
+
+ when(sr.scenario.getLine()).thenReturn(4);
+ when(sr.scenario.getExampleIndex()).thenReturn(-1);
+
+ String expected =
+ UUID.nameUUIDFromBytes(
+ "features/KarateTest.feature:4"
+ .getBytes(StandardCharsets.UTF_8)
+ ).toString();
+
+ String rid = extractor.getRid(sr);
+
+ assertThat(rid).isEqualTo(expected);
+ }
+
private ScenarioRuntime mockScenarioRuntime() {
ScenarioRuntime sr = mock(ScenarioRuntime.class);
diff --git a/java-reporter-testng/pom.xml b/java-reporter-testng/pom.xml
index 6221211..f18217e 100644
--- a/java-reporter-testng/pom.xml
+++ b/java-reporter-testng/pom.xml
@@ -47,7 +47,7 @@