[Lldb-commits] [PATCH] D85169: [test] Exit with an error if no tests are run.
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Aug 3 16:08:23 PDT 2020
JDevlieghere added inline comments.
================
Comment at: lldb/packages/Python/lldbsuite/test/dotest.py:1043
+ if configuration.suite.countTestCases() == 0:
+ print("error: did not discover any tests.")
+ exitTestSuite(1)
----------------
MaskRay wrote:
> JDevlieghere wrote:
> > Maybe `did not discover any (matching) tests` to make it clear that this might be the result of user input?
> `sys.stderr.write` or `print(..., file=sys.stderr)` (which appends a newline)
>
> Trailing full stops are not recommended for error messages if the project does not already have the convention http://llvm.org/docs/CodingStandards.html#error-and-warning-messages
>
> Looking around, several messages do not have a full stop.
Let's use `logging.error` like the other error messages if we want this to go to stderr.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85169/new/
https://reviews.llvm.org/D85169
More information about the lldb-commits
mailing list