Merge lp:~rvb/maas-test/add-log-files into lp:maas-test
Proposed by
Raphaël Badin
Status: | Merged |
---|---|
Merged at revision: | 52 |
Proposed branch: | lp:~rvb/maas-test/add-log-files |
Merge into: | lp:maas-test |
Diff against target: |
136 lines (+68/-6) 3 files modified
maastest/maasfixture.py (+28/-0) maastest/main.py (+2/-0) maastest/tests/test_maasfixture.py (+38/-6) |
To merge this branch: | bzr merge lp:~rvb/maas-test/add-log-files |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jeroen T. Vermeulen (community) | Approve | ||
Review via email: mp+196252@code.launchpad.net |
Commit message
Include logs in the report when tests fail.
Description of the change
When tests fail, having the logs will help us debug issues.
The changes to test_import_
To post a comment you must log in.
Nice. This'll help. Ingenious trick for getting the files into the log... Not the prettiest presentation perhaps, but it'll get the job done.
FWIW the check_call parameter to the run_command functions defaults to False, so if you want to avoid breaking up that line in collect_logs, that might help. Not that I'm saying you should; you may have left it deliberately explicit.