From 411e602d134aba05a8d876da41e8abe1b04c374c Mon Sep 17 00:00:00 2001 From: Sergey Avseyev Date: Mon, 25 May 2015 21:16:35 +0300 Subject: Introduce pedantic mode for tester Motivation ---------- External tools need to know whether or not any test in suite failed. For example buildbot, or packaging tool would like to stop the execution and mark it as failed if some tests does not pass. Modification ------------ Add `--pedantic` switch to tester program which will force it to quit with non-zero exit code if at least one failure detected. Also update `tests()` proc in koch to inspect result from tester and propagate it to user. Result ------ Nothing has changed in default behaviour. But following invocations will exit with non-zero code if there failed tests: ./koch tests --pedantic all ./tests/testament/tester --pedantic all --- tests/testament/tester.nim | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/testament/tester.nim b/tests/testament/tester.nim index 0308ce940..62dafee80 100644 --- a/tests/testament/tester.nim +++ b/tests/testament/tester.nim @@ -31,6 +31,7 @@ Arguments: Options: --print also print results to the console --failing only show failing/ignored tests + --pedantic return non-zero status code if there are failures """ % resultsFile type @@ -310,12 +311,14 @@ proc main() = backend.open() var optPrintResults = false var optFailing = false + var optPedantic = false var p = initOptParser() p.next() while p.kind == cmdLongoption: case p.key.string.normalize of "print", "verbose": optPrintResults = true of "failing": optFailing = true + of "pedantic": optPedantic = true else: quit Usage p.next() if p.kind != cmdArgument: quit Usage @@ -348,8 +351,10 @@ proc main() = if action == "html": openDefaultBrowser(resultsFile) else: echo r, r.data backend.close() + if optPedantic: + var failed = r.total - r.passed - r.skipped + if failed > 0 : quit(QuitFailure) if paramCount() == 0: quit Usage main() - -- cgit 1.4.1-2-gfad0