pep8 failures do not cause the tests to fail
Bug #835952 reported by
Ewan Mellor
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Ewan Mellor | ||
Bug Description
pep8 failures are no longer causing run_tests.sh to exit with status 1.
I think this has been the case since the coverage testing was added. run_tests.sh is not set -e, so it falls through to the next bit of the script (coverage) regardless of whether pep8 succeeds.
Related branches
lp:~ewanmellor/nova/lp835952
- Vish Ishaya (community): Approve
- Brian Lamar (community): Approve
-
Diff: 41 lines (+10/-8)1 file modifiedrun_tests.sh (+10/-8)
| Changed in nova: | |
| assignee: | nobody → Ewan Mellor (ewanmellor) |
| status: | New → In Progress |
| Changed in nova: | |
| status: | In Progress → Fix Committed |
| Changed in nova: | |
| milestone: | none → diablo-rbp |
| Changed in nova: | |
| milestone: | diablo-rbp → 2011.3 |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
