diff options
| author | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-09-20 16:52:17 +0000 |
|---|---|---|
| committer | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-09-20 16:52:17 +0000 |
| commit | 2d473adb5a031b19bc82085e7362d77b63cec7fc (patch) | |
| tree | 46193091dddbf755832b4096b4ef1333031c2d26 /nova/openstack | |
| parent | 2fb9fe92fd24ecf52720f6762c32c2056ecbe2b9 (diff) | |
| download | nova-2d473adb5a031b19bc82085e7362d77b63cec7fc.tar.gz nova-2d473adb5a031b19bc82085e7362d77b63cec7fc.tar.xz nova-2d473adb5a031b19bc82085e7362d77b63cec7fc.zip | |
Fix solidfire unit tests
A couple of tests attempted to reimplement assertRaises(), but did so
badly. self.fail() will raise AssertionError, which is a subclass of
Exception, so even if the tested exception wasn't raised, it would
silently treat it as a success.
This buggy testing code hid a failure to properly test the intended
functionality in test_delete_volume_fails_no_volume.
Change-Id: I4e31bf6cb3b042aa2d9ba2a5ccaf40319d0fc124
Diffstat (limited to 'nova/openstack')
0 files changed, 0 insertions, 0 deletions