[llvm] [ci] Handle the case where all reported tests pass but the build is still a failure (PR #120264)

Lucile Rose Nihlen via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 10 07:10:31 PST 2025


================
@@ -70,10 +72,51 @@ def test_no_failures(self):
             ),
         )
 
+    def test_no_failures_build_failed(self):
+        self.assertEqual(
+            _generate_report(
+                "Foo",
+                1,
+                [
+                    junit_from_xml(
+                        dedent(
+                            """\
+          <?xml version="1.0" encoding="UTF-8"?>
+          <testsuites time="0.00">
+          <testsuite name="Passed" tests="1" failures="0" skipped="0" time="0.00">
+          <testcase classname="Bar/test_1" name="test_1" time="0.00"/>
+          </testsuite>
+          </testsuites>"""
----------------
lnihlen wrote:

Is this brittle? I understand it's being converted to some junit format, but what is the potential for assertion failures due to signals other than test failure?

https://github.com/llvm/llvm-project/pull/120264


More information about the llvm-commits mailing list