Revert "[lit][unit] makes sure to print stderr when no test failures are found"
authorYuanfang Chen <yuanfang.chen@sony.com>
Fri, 15 Jul 2022 20:21:01 +0000 (13:21 -0700)
committerYuanfang Chen <yuanfang.chen@sony.com>
Fri, 15 Jul 2022 20:26:30 +0000 (13:26 -0700)
This reverts commit 65769429c0644c81d271e9d971f5b64191503f6e.

This patch is incomplete.

llvm/utils/lit/lit/formats/googletest.py

index 6878ebc..5feefaa 100644 (file)
@@ -163,8 +163,6 @@ class GoogleTest(TestFormat):
                     res.append(l)
             assert False, f'gtest did not report the result for ' + test_name
 
-        found_failed_test = False
-
         with open(test.gtest_json_file, encoding='utf-8') as f:
             jf = json.load(f)
 
@@ -181,7 +179,6 @@ class GoogleTest(TestFormat):
                     header = f"Script:\n--\n%s --gtest_filter=%s\n--\n" % (
                         ' '.join(cmd), testname)
                     if 'failures' in testinfo:
-                        found_failed_test = True
                         output += header
                         test_out = get_test_stdout(testname)
                         if test_out:
@@ -192,12 +189,6 @@ class GoogleTest(TestFormat):
                     elif result != 'COMPLETED':
                         output += header
                         output += 'unresolved test result\n'
-
-        # In some situations, like running tests with sanitizers, all test passes but
-        # the shard could still fail due to memory issues.
-        if not found_failed_test:
-            output += f"\n{out}\n--\nexit: {exitCode}\n--\n"
-
         return lit.Test.FAIL, output
 
     def prepareCmd(self, cmd):