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

[Chaoren Lin via lldb-commits]

chaoren added inline comments.

================
Comment at: test/dosep.py:142
@@ +141,3 @@
+    if exit_status == 0:
+        # stdout does not have any useful information from 'dotest.py', only stderr does.
+        report_test_pass(name, output[1])
----------------
To clarify, I mean something like:

```
with output_lock:
    if output_on_success:
        print >> sys.stderr, output
        print >> sys.stderr, "[%s] PASSED" % name # silent
    update_progress()
```

in `report_test_pass`.

and

```
sys.stderr.write("\r%*d out of %d test suites processed - [%s]\n" %
            (len(str(total_tests)), test_counter.value, total_tests), name)
```
in `update_progress`.


http://reviews.llvm.org/D11816