diff options
author | Petr Viktorin <pviktori@redhat.com> | 2012-03-09 09:41:16 -0500 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2012-03-19 16:33:35 +0100 |
commit | c14a2d824557b06fd20bac77ddfc9e09001e2a92 (patch) | |
tree | 0f8f012dc47ab7ecb8574a53eeeba12706edc336 /TODO | |
parent | 35521ad6bb92057d5faefa2059d7d800bebb1af0 (diff) | |
download | freeipa-c14a2d824557b06fd20bac77ddfc9e09001e2a92.tar.gz freeipa-c14a2d824557b06fd20bac77ddfc9e09001e2a92.tar.xz freeipa-c14a2d824557b06fd20bac77ddfc9e09001e2a92.zip |
Use nose tools to check for exceptions
Some of our tests checked for exceptions using an error-prone
try block: they allowed the expected exception to pass, but sometimes
forgot an else block, so the test passed when an exception wasn't
thrown.
This changes the tests to use the appropriate nose tools (raises,
assert_raises).
For consistency, tests that had a correct else block are also changed.
Also fix some test problems that were hidden by the above:
- in some sudorule and HBAC tests, change the *_add_user argument name
from `users` to `user`
- don't remove HBAC testing data while it was still used
Diffstat (limited to 'TODO')
0 files changed, 0 insertions, 0 deletions