Skip to content

More succinct paralleltest test output.#439

Merged
bgrant merged 1 commit into
masterfrom
feature/compact-paralleltest-reporting
Jun 24, 2014
Merged

More succinct paralleltest test output.#439
bgrant merged 1 commit into
masterfrom
feature/compact-paralleltest-reporting

Conversation

@kwmsmith
Copy link
Copy Markdown
Contributor

I've run into problems a few times where the first 1 or 4 parallel engines have errors, but the latter ones don't. The way the output is ordered, the latter tests make it seem like everything is fine.

This makes the output more compact to give a quick overview of the success or failure of the parallel tests; it is always possible to look at the file in .parallel_out for more details.

For example:

$ make test_engines
Running 'python -m unittest discover -s distarray/local/tests -p 'paralleltest*.py'' on each engine...
====>  .parallel_out/2.7.3-unittest.out.1.00
OK
====>  .parallel_out/2.7.3-unittest.out.1.01
OK
====>  .parallel_out/2.7.3-unittest.out.1.02
OK
====>  .parallel_out/2.7.3-unittest.out.1.03
OK
====>  .parallel_out/2.7.3-unittest.out.1.04
OK
====>  .parallel_out/2.7.3-unittest.out.1.05
OK
====>  .parallel_out/2.7.3-unittest.out.1.06
OK
====>  .parallel_out/2.7.3-unittest.out.1.07
OK
====>  .parallel_out/2.7.3-unittest.out.1.08
OK
====>  .parallel_out/2.7.3-unittest.out.1.09
OK
====>  .parallel_out/2.7.3-unittest.out.1.10
OK
====>  .parallel_out/2.7.3-unittest.out.1.11
OK

@kwmsmith kwmsmith added this to the 0.4 milestone Jun 13, 2014
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same when pulling 1d5a31b on feature/compact-paralleltest-reporting into 7a7bf40 on master.

@bgrant
Copy link
Copy Markdown
Contributor

bgrant commented Jun 24, 2014

Looks good to me.

bgrant added a commit that referenced this pull request Jun 24, 2014
…eporting

More succinct paralleltest test output.
@bgrant bgrant merged commit 266e348 into master Jun 24, 2014
@bgrant bgrant deleted the feature/compact-paralleltest-reporting branch June 24, 2014 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants