[PATCH] D60553: [lit][tests][AIX] Update expected form of diagnostic messages; use `not` to normalize non-zero exit values

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 10 19:38:44 PDT 2019


hubert.reinterpretcast created this revision.
hubert.reinterpretcast added reviewers: xingxue, sfertile, jasonliu.
Herald added subscribers: jsji, delcypher.
Herald added a project: LLVM.

Various tests in the `lit` testing suite expect specific return codes and forms of diagnostic message from utility programs. As per POSIX.1-2017 XCU Section 1.4, Utility Description Defaults, "[the] format of diagnostic messages for most utilities is unspecified". The STDERR subsections of the `cat` and `wc` utilities merely indicate that "[the] standard error shall be used only for diagnostic messages". The corresponding EXIT STATUS subsections merely indicate, with regard to errors, an exit value of >0.

The affected tests are updated to accept the applicable diagnostic message as produced by the utilities on AIX. The exit value is normalized using `not` as necessary.


Repository:
  rL LLVM

https://reviews.llvm.org/D60553

Files:
  utils/lit/tests/Inputs/shtest-output-printing/basic.txt
  utils/lit/tests/shtest-format.py
  utils/lit/tests/shtest-output-printing.py


Index: utils/lit/tests/shtest-output-printing.py
===================================================================
--- utils/lit/tests/shtest-output-printing.py
+++ utils/lit/tests/shtest-output-printing.py
@@ -24,8 +24,8 @@
 # CHECK-NEXT: hi
 #
 # CHECK:      $ ":" "RUN: at line 3"
-# CHECK-NEXT: $ "wc" "missing-file"
+# CHECK-NEXT: $ "not" "not" "wc" "missing-file"
 # CHECK-NEXT: # redirected output from '{{.*(/|\\\\)}}basic.txt.tmp.out':
-# CHECK-NEXT: missing-file{{.*}} No such file or directory
+# CHECK-NEXT: {{cannot open missing-file|missing-file.* No such file or directory}}
 # CHECK:      note: command had no output on stdout or stderr
 # CHECK-NEXT: error: command failed with exit status: 1
Index: utils/lit/tests/shtest-format.py
===================================================================
--- utils/lit/tests/shtest-format.py
+++ utils/lit/tests/shtest-format.py
@@ -18,7 +18,7 @@
 # CHECK-NEXT: line 2: failed test output on stdout
 # CHECK: Command Output (stderr):
 # CHECK-NEXT: --
-# CHECK-NEXT: cat{{(\.exe)?}}: does-not-exist: No such file or directory
+# CHECK-NEXT: cat{{(\.exe)?}}: {{cannot open does-not-exist|does-not-exist: No such file or directory}}
 # CHECK: --
 
 # CHECK: FAIL: shtest-format :: external_shell/fail_with_bad_encoding.txt
Index: utils/lit/tests/Inputs/shtest-output-printing/basic.txt
===================================================================
--- utils/lit/tests/Inputs/shtest-output-printing/basic.txt
+++ utils/lit/tests/Inputs/shtest-output-printing/basic.txt
@@ -1,3 +1,3 @@
 # RUN: true
 # RUN: echo hi
-# RUN: wc missing-file &> %t.out
+# RUN: not not wc missing-file &> %t.out


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60553.194632.patch
Type: text/x-patch
Size: 1658 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190411/a138f57b/attachment.bin>


More information about the llvm-commits mailing list