Stop skipping "test exception" test cases, they are not synthetic (#11427)

fix: stop tagging "test exception" testcases as synthetic skip

"test exception" is not a synthetic placeholder emitted by any test
framework — JUnit 4, JUnit Platform, Gradle's adapters, Spock and Kotest
all reserve "initializationError"/"executionError" for that purpose, not
generic English names. In dd-trace-java the entries observed in CI came
from a real Spock feature method (`def "test exception"()` in
HttpServerTest.groovy), not a framework synthetic. Tagging that name
unconditionally silently masks real pass/fail outcomes for any
Spock/JUnit 5 @DisplayName/Kotest test that happens to use it.

Drop "test exception" from JUnitReport.tagSyntheticFailures and document
the criteria for future synthetic entries (must be framework-emitted,
must link to the source pinned to a release tag). Annotate the two
remaining entries — initializationError (JUnit 4 + Gradle) and
executionError (Gradle) — with the canonical source locations.

Update JUnitReport.java

Co-authored-by: Sarah Chen <sarah.chen@datadoghq.com>

Merge branch 'master' into bdu/revert-test-exception-synthetic-skip

Co-authored-by: brice.dutheil <brice.dutheil@datadoghq.com>

Author: bric3

.gitlab/collect-result/JUnitReport.java

@@ -66,16 +66,43 @@ boolean normalizeStableTestNames() {
     return changed;
   }
 
+  /// Tags framework-emitted synthetic testcases so Test Optimization does not treat them as
+  /// real failures.
+  ///
+  /// **Criteria for new entries:**
+  /// - Must be a name the framework/runner emits itself — never use a name that a user-authored test
+  ///   could legitimately have.
+  /// - Must link to the source that emits the literal, pinned to a release tag (not a commit
+  ///   hash).
+  ///
+  /// **Frameworks/libraries to audit before adding a name:** JUnit 4, JUnit Platform engines
+  /// (Vintage / Jupiter), Gradle's JUnit and JUnit Platform adapters, TestNG, Spock, Kotest,
+  /// Maven Surefire/Failsafe. JUnit 5's main sources do not define these literals — Vintage
+  /// forwards `"initializationError"` from JUnit 4 as-is, and `"executionError"` is
+  /// Gradle-specific.
+  ///
+  /// **Do not add** generic English names such as `"test exception"`. Spock feature methods
+  /// (`def "..."()`), JUnit 5 `@DisplayName`, and Kotest specs can all produce them, so the
+  /// tagger would silently mask real pass/fail outcomes.
   void tagSyntheticFailures() {
     Map<String, List<Element>> initializationErrorsByClassname = new LinkedHashMap<>();
     for (var testcase : testcases()) {
-      var name = testcase.getAttribute("name");
-      if ("initializationError".equals(name)) {
-        initializationErrorsByClassname
-            .computeIfAbsent(testcase.getAttribute("classname"), ignored -> new ArrayList<>())
-            .add(testcase);
-      } else if ("executionError".equals(name) || "test exception".equals(name)) {
-        addFinalStatusProperty(testcase, "skip", MissingPropertiesPlacement.APPEND_TO_TESTCASE);
+      switch (testcase.getAttribute("name")) {
+        // JUnit 4 ErrorReportingRunner — initializationError
+        // https://github.com/junit-team/junit4/blob/r4.13.2/src/main/java/org/junit/internal/runners/ErrorReportingRunner.java#L83
+        // Gradle JUnit Platform listener — reuses the same name for synthetic container failures
+        // https://github.com/gradle/gradle/blob/v8.14.5/platforms/jvm/testing-junit-platform/src/main/java/org/gradle/api/internal/tasks/testing/junitplatform/JUnitPlatformTestExecutionListener.java#L248
+        // https://github.com/gradle/gradle/blob/v9.5.0/platforms/jvm/testing-jvm-infrastructure/src/main/java/org/gradle/api/internal/tasks/testing/junitplatform/JUnitPlatformTestExecutionListener.java#L426
+        case "initializationError" ->
+            initializationErrorsByClassname
+                .computeIfAbsent(testcase.getAttribute("classname"), ignored -> new ArrayList<>())
+                .add(testcase);
+        // Gradle JUnit Platform listener — executionError, used when descendants already started
+        // https://github.com/gradle/gradle/blob/v8.14.5/platforms/jvm/testing-junit-platform/src/main/java/org/gradle/api/internal/tasks/testing/junitplatform/JUnitPlatformTestExecutionListener.java#L248
+        // https://github.com/gradle/gradle/blob/v9.5.0/platforms/jvm/testing-jvm-infrastructure/src/main/java/org/gradle/api/internal/tasks/testing/junitplatform/JUnitPlatformTestExecutionListener.java#L426
+        case "executionError" ->
+            addFinalStatusProperty(testcase, "skip", MissingPropertiesPlacement.APPEND_TO_TESTCASE);
+        default -> {}
       }
     }