Do not send empty test suite spans (#5405)

DataDog · Jun 22, 2023 · 9799c17 · 9799c17
1 parent db5878f
commit 9799c17
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 10 deletions.
diff --git a/...agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/DDTestSessionImpl.java b/...agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/DDTestSessionImpl.java
@@ -80,7 +80,8 @@ public void setSkipReason(String skipReason) {
 
   @Override
   public void end(@Nullable Long endTime) {
-    span.setTag(Tags.TEST_STATUS, context.getStatus());
+    String status = context.getStatus();
+    span.setTag(Tags.TEST_STATUS, status != null ? status : CIConstants.TEST_SKIP);
     testDecorator.beforeFinish(span);
 
     if (endTime != null) {

diff --git a/...a-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/DDTestSuiteImpl.java b/...a-agent/agent-ci-visibility/src/main/java/datadog/trace/civisibility/DDTestSuiteImpl.java
@@ -143,13 +143,16 @@ public void end(@Nullable Long endTime) {
     testDecorator.beforeFinish(span);
 
     String status = context.getStatus();
-    span.setTag(Tags.TEST_STATUS, status);
-    moduleContext.reportChildStatus(status);
-
-    if (endTime != null) {
-      span.finish(endTime);
-    } else {
-      span.finish();
+    if (status != null) {
+      // do not report test suite if no execution took place
+      span.setTag(Tags.TEST_STATUS, status);
+      moduleContext.reportChildStatus(status);
+
+      if (endTime != null) {
+        span.finish(endTime);
+      } else {
+        span.finish();
+      }
     }
   }
 

diff --git a/...t-ci-visibility/src/main/java/datadog/trace/civisibility/context/AbstractTestContext.java b/...t-ci-visibility/src/main/java/datadog/trace/civisibility/context/AbstractTestContext.java
@@ -4,19 +4,24 @@
 
 abstract class AbstractTestContext implements TestContext {
 
-  private String status = CIConstants.TEST_SKIP;
+  private String status;
 
   @Override
   public synchronized void reportChildStatus(String childStatus) {
     switch (childStatus) {
       case CIConstants.TEST_PASS:
-        if (CIConstants.TEST_SKIP.equals(status)) {
+        if (status == null || CIConstants.TEST_SKIP.equals(status)) {
           status = CIConstants.TEST_PASS;
         }
         break;
       case CIConstants.TEST_FAIL:
         status = CIConstants.TEST_FAIL;
         break;
+      case CIConstants.TEST_SKIP:
+        if (status == null) {
+          status = CIConstants.TEST_SKIP;
+        }
+        break;
       default:
         break;
     }