Code review comment for lp:~clint-fewbar/pyjuju/add-testrunner

Revision history for this message
Jim Baker (jimbaker) wrote :

Some minor issues I observed in trying this useful utility out:

1) eatmydata and iwatch need to be installed in advance; this issue is
more pronounced for eatmydata since it will fail in the test phase. I'm
not certain what the best approach is for this juju package. Perhaps
just test installation of both tools before running, as opposed to
making this a requirement of juju itself.

2) At least on my desktop, the test is always run twice, because iwatch
seems to pick up the file change twice.

3) Users need to remember they have this utility running and not attempt
to run more tests simultaneously, otherwise both test runs will now
fail.

https://codereview.appspot.com/6624055/

« Back to merge proposal