[Lldb-commits] [Diffusion] rL238467: Refactor test runner to print sub-test-case pass/fail rate.

[Zachary Turner]

zturner added a comment.

(Resending this through Phab since my last "reply" to email went to the old list.

It looks like the problem here is that previously we relied on the individual subprocesses of dotest.py to print their own stdout and stderr.  Now dosep gets the stdout and stderr through a pipe, and it doesn't always print the full output of the process.  So in some cases the line "Ran N tests in M seconds" isn't appearing.

I can fix this, but I guess this is a good time to remind people of the dangers of log scraping.  I don't think we should guarantee stability of the details of the test output format, except for perhaps a small summary printed at the end of the test run.  So I would think about getting whatever numbers you need into this summary section, and then relying only on that for your output


Users:
  zturner (Author)
  dawn (Auditor)

http://reviews.llvm.org/rL238467