[Lldb-commits] [PATCH] D11816: Allow dosep.py to print full output of dotest.py, even when dotest succeeds.

[Zachary Turner via lldb-commits]

zturner added inline comments.

================
Comment at: test/dosep.py:142
@@ -128,2 +141,3 @@
     passes, failures = parse_test_results(output)
-    update_status(name, command, output if exit_status != 0 else None)
+    if exit_status == 0:
+        report_test_pass(name, output[1])
----------------
chaoren wrote:
> Could you please do:
> 
> ```
> if exit_status != 0:
>     report_test_failure(...)
> elif output_on_success:
>     report_test_pass(...)
> ```
> 
> so it's completely silent on success?
I don't know that it should be completely silent.  For starters, if you do it the way you suggest, it won't end up updating the progress, which is something you definitely need to see.  But I think it's worthwhile to see the names of the individual tests that are running.  The way I've written it now, you if `output_on_success` is False, you will only see one additional line, which is the name of the test being run.  I think that's useful and does not add any noise to the output


http://reviews.llvm.org/D11816