Code review comment for lp:~coreygoldberg/autopilot/texttest-run

Revision history for this message
Corey Goldberg (coreygoldberg) wrote :

from thomi:

OK, so we need to change things. Seems like there's several things we're trying to control independantly of each other here:

1) The level of verbosity autopilot uses when logging. Ideally we'd make sure that all our log levels were sensible, and then this would control the logging framework verbosity level. 0 * '-v' => normal and higher. '-v' => info logs and higher. '-vv' => debug logs and higher. The log *always* gets attached to the test result for every test, regardless of the output file format.

2) The format we store run reports in. We have 'text', 'xml', 'subunit'. I think we need one more, which is 'verbosetext' or something similar. This causes the test log to get printed to stdoud as the test is runing. This is equivilent to the '-v' flag before this change.

3) The location of the output report. If the -o option is specified, the report is written to that file, in the format specified by the '-f' parameter.

Finally, if, after configuring all this, stdout is not used, the 'status' format should be printed to stdout as well.

review: Needs Fixing

« Back to merge proposal