Mark test.xml omitted if it wasn't generated

bazelbuild · Jun 15, 2021 · 97139ce · 97139ce
1 parent 6c2b9d1
commit 97139ce
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java b/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
@@ -774,18 +774,17 @@ public TestAttemptContinuation execute()
       }
 
 
+      MetadataHandler metadataHandler = actionExecutionContext.getMetadataHandler();
+
       Path xmlOutputPath = resolvedPaths.getXmlOutputPath();
       boolean testXmlExists = xmlOutputPath.exists();
-      if (!testXmlExists) {
-        MetadataHandler metadataHandler = actionExecutionContext.getMetadataHandler();
-        if (metadataHandler != null) {
+      if (!testXmlExists && metadataHandler != null) {
           try {
             // Check whether test.xml exists. If not, an IOException will be thrown.
             metadataHandler.getMetadata(testAction.getTestXml());
             testXmlExists = true;
           } catch (IOException ignored) {
           }
-        }
       }
 
       // If the test did not create a test.xml, and --experimental_split_xml_generation is enabled,
@@ -814,6 +813,10 @@ public TestAttemptContinuation execute()
         }
       }
 
+      if (!testXmlExists && metadataHandler != null) {
+          metadataHandler.markOmitted(testAction.getTestXml());
+      }
+
       TestCase details = parseTestResult(xmlOutputPath);
       if (details != null) {
         testResultDataBuilder.setTestCase(details);