[PATCH] D77987: [test-suite] Move lit test result codes into llvm/lit

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert created this revision.
jdoerfert added reviewers: arichardson, cmatthews, dlj, leandron.
Herald added a subscriber: bollu.

In D77986 <https://reviews.llvm.org/D77986> we moved these result codes into llvm/lit, now we use
those definitions directly.

Depends on: D77986 <https://reviews.llvm.org/D77986>


Repository:
  rT test-suite

https://reviews.llvm.org/D77987

Files:
  litsupport/test.py


Index: litsupport/test.py
===================================================================
--- litsupport/test.py
+++ litsupport/test.py
@@ -13,9 +13,6 @@
 import os
 
 
-SKIPPED = lit.Test.ResultCode('SKIPPED', False)
-NOEXE = lit.Test.ResultCode('NOEXE', True)
-
 
 class TestSuiteTest(lit.formats.ShTest):
     def __init__(self):
@@ -38,7 +35,7 @@
 
         # Report missing test executables.
         if not os.path.exists(context.executable):
-            return lit.Test.Result(NOEXE, "Executable '%s' is missing" %
+            return lit.Test.Result(lit.Test.NOEXE, "Executable '%s' is missing" %
                                    context.executable)
 
         # Skip unchanged tests
@@ -46,7 +43,7 @@
             litsupport.modules.hash.compute(context)
             if litsupport.modules.hash.same_as_previous(context):
                 result = lit.Test.Result(
-                        SKIPPED, 'Executable identical to previous run')
+                        lit.Test.SKIPPED, 'Executable identical to previous run')
                 val = lit.Test.toMetricValue(context.executable_hash)
                 result.addMetric('hash', val)
                 return result


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D77987.256900.patch
Type: text/x-patch
Size: 1188 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200413/c5aabe48/attachment.bin>