Merge lp:~vila/bzr/646133-selftest-as-root into lp:bzr/2.1
Proposed by
Vincent Ladeuil
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | John A Meinel | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 4861 | ||||
Proposed branch: | lp:~vila/bzr/646133-selftest-as-root | ||||
Merge into: | lp:bzr/2.1 | ||||
Diff against target: |
127 lines (+28/-4) 6 files modified
NEWS (+3/-0) bzrlib/tests/features.py (+16/-0) bzrlib/tests/per_lock/test_lock.py (+2/-1) bzrlib/tests/test_lockdir.py (+2/-1) bzrlib/tests/test_osutils.py (+2/-0) bzrlib/tests/test_selftest.py (+3/-2) |
||||
To merge this branch: | bzr merge lp:~vila/bzr/646133-selftest-as-root | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John A Meinel | Approve | ||
Review via email: mp+36470@code.launchpad.net |
Commit message
Skip chmod bits dependent tests when running as root
Description of the change
This fix the remaining failures in the test suite for 2.1 when running as root (bug #646133).
Some tests are creating files with 000 mode bits or 400 to exercise the code that tries to read or write them. 'root' doesn't care, so there is no point trying to run them.
I also fixed an old bug in test_selftest which has already been fixed in later releases. This is not strictly related but... doesn't deserve a dedicated merge proposal either.
To post a comment you must log in.
Just to clarify, we need to run as root because the buildd's run everything as root. (seems silly/foolish).
Anyway, the change looks good to me.