[PATCH] D79064: [test-suite] Register result codes with lit.

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 28 20:31:27 PDT 2020


Meinersbur updated this revision to Diff 260830.
Meinersbur added a comment.

- Missing Executables -> Executable Missing

When reporting test codes, lit appends " Test" to the category name.
I think "Executable Missing Tests" is better than "Missing Executables Tests".


Repository:
  rT test-suite

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79064/new/

https://reviews.llvm.org/D79064

Files:
  litsupport/test.py


Index: litsupport/test.py
===================================================================
--- litsupport/test.py
+++ litsupport/test.py
@@ -13,9 +13,10 @@
 import os
 
 
-SKIPPED = lit.Test.ResultCode('SKIPPED', False)
-NOEXE = lit.Test.ResultCode('NOEXE', True)
+SKIPPED = lit.Test.SKIPPED
 
+NOEXE = lit.Test.ResultCode('NOEXE', True)
+lit.main.add_result_category(NOEXE, "Executable Missing")
 
 class TestSuiteTest(lit.formats.ShTest):
     def __init__(self):


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79064.260830.patch
Type: text/x-patch
Size: 469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200429/4ead2286/attachment.bin>


More information about the llvm-commits mailing list