Merge lp:~smoser/curtin/trunk.subp-more-tests into lp:~curtin-dev/curtin/trunk
Proposed by
Scott Moser
Status: | Merged |
---|---|
Merged at revision: | 422 |
Proposed branch: | lp:~smoser/curtin/trunk.subp-more-tests |
Merge into: | lp:~curtin-dev/curtin/trunk |
Diff against target: |
123 lines (+81/-14) 1 file modified
tests/unittests/test_util.py (+81/-14) |
To merge this branch: | bzr merge lp:~smoser/curtin/trunk.subp-more-tests |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ryan Harper (community) | Approve | ||
Server Team CI bot | continuous-integration | Needs Fixing | |
Review via email: mp+304297@code.launchpad.net |
Commit message
tests: more subp tests
Better test util.subp, exercising its target=/ and target=/some/root, and also verifying that exception is raised if subprocess exits non-zero.
To post a comment you must log in.
FAILED: Continuous integration, rev:421 /jenkins. ubuntu. com/server/ job/curtin- ci/256/ /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= amd64/256/ console /jenkins. ubuntu. com/server/ job/curtin- ci/nodes= ppc64el/ 256/console
https:/
Executed test runs:
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/server/ job/curtin- ci/256/ rebuild
https:/