Lower logging level for RepoIndex error (#9697)

Author: daniel-mohedano

dd-java-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/source/index/RepoIndex.java

@@ -70,7 +70,7 @@ private String getFallbackSourcePath(@Nonnull Class<?> c) throws SourceResolutio
     try {
       String fileName = Utils.getFileName(c);
       if (fileName == null) {
-        log.error("Could not retrieve file name for class {}", c.getName());
+        log.debug("Could not retrieve file name for class {}", c.getName());
         return null;
       }
 

dd-java-agent/instrumentation/junit/junit-4.10/src/test/groovy/JUnit4Test.groovy

@@ -77,15 +77,15 @@ class JUnit4Test extends CiVisibilityInstrumentationTest {
 
     where:
     testcaseName                             | success | tests                          | retriedTests
-    //    "test-failed"                            | false   | [TestFailed]                   | []
+    "test-failed"                            | false   | [TestFailed]                   | []
     "test-retry-failed"                      | false   | [TestFailed]                   | [new TestFQN("org.example.TestFailed", "test_failed")]
-    //    "test-failed-then-succeed"               | true    | [TestFailedThenSucceed]        | [new TestFQN("org.example.TestFailedThenSucceed", "test_failed_then_succeed")]
-    //    "test-assumption-is-not-retried"         | true    | [TestAssumption]               | [new TestFQN("org.example.TestAssumption", "test_fail_assumption")]
-    //    "test-skipped-is-not-retried"            | true    | [TestSkipped]                  | [new TestFQN("org.example.TestSkipped", "test_skipped")]
-    //    "test-retry-parameterized"               | false   | [TestFailedParameterized]      | [
-    //      new TestFQN("org.example.TestFailedParameterized", "test_failed_parameterized") /* backend cannot provide parameters for flaky parameterized tests yet */
-    //    ]
-    //    "test-expected-exception-is-not-retried" | true    | [TestSucceedExpectedException] | [new TestFQN("org.example.TestSucceedExpectedException", "test_succeed")]
+    "test-failed-then-succeed"               | true    | [TestFailedThenSucceed]        | [new TestFQN("org.example.TestFailedThenSucceed", "test_failed_then_succeed")]
+    "test-assumption-is-not-retried"         | true    | [TestAssumption]               | [new TestFQN("org.example.TestAssumption", "test_fail_assumption")]
+    "test-skipped-is-not-retried"            | true    | [TestSkipped]                  | [new TestFQN("org.example.TestSkipped", "test_skipped")]
+    "test-retry-parameterized"               | false   | [TestFailedParameterized]      | [
+      new TestFQN("org.example.TestFailedParameterized", "test_failed_parameterized") /* backend cannot provide parameters for flaky parameterized tests yet */
+    ]
+    "test-expected-exception-is-not-retried" | true    | [TestSucceedExpectedException] | [new TestFQN("org.example.TestSucceedExpectedException", "test_succeed")]
   }
 
   def "test early flakiness detection #testcaseName"() {

dd-java-agent/instrumentation/junit/junit-4.10/src/test/resources/test-failed-then-succeed/events.ftl

@@ -16,7 +16,7 @@
       "test.command" : "junit-4.10",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
-      "test.status" : "fail",
+      "test.status" : "pass",
       "test.type" : "test",
       "test_session.name" : "session-name"
     },
@@ -48,7 +48,7 @@
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.module" : "junit-4.10",
-      "test.status" : "fail",
+      "test.status" : "pass",
       "test.type" : "test",
       "test_session.name" : "session-name"
     },
@@ -80,7 +80,7 @@
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.module" : "junit-4.10",
       "test.source.file" : "dummy_source_path",
-      "test.status" : "fail",
+      "test.status" : "pass",
       "test.suite" : "org.example.TestFailedThenSucceed",
       "test.type" : "test",
       "test_session.name" : "session-name"
@@ -118,6 +118,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.module" : "junit-4.10",
@@ -168,6 +169,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.is_retry" : "true",
@@ -220,6 +222,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.is_retry" : "true",

dd-java-agent/instrumentation/junit/junit-4.10/src/test/resources/test-retry-parameterized/events.ftl

@@ -118,6 +118,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.module" : "junit-4.10",
@@ -169,6 +170,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.is_retry" : "true",
@@ -222,6 +224,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.is_retry" : "true",
@@ -275,6 +278,7 @@
       "runtime-id" : ${content_meta_runtime_id},
       "span.kind" : "test",
       "test.codeowners" : "[\"owner1\",\"owner2\"]",
+      "test.failure_suppressed" : "true",
       "test.framework" : "junit4",
       "test.framework_version" : ${content_meta_test_framework_version},
       "test.is_retry" : "true",