cinder utils tests need to be converted to mock from mox
Bug #1285905 reported by
Bill Owen
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Undecided
|
Bill Owen |
Bug Description
As part of the effort to clean up cinder test, modules using mox are to be converted to use mock.
This bug is for the conversion of
cinder/
Changed in cinder: | |
assignee: | nobody → Bill Owen (billowen) |
status: | New → In Progress |
Changed in cinder: | |
milestone: | none → icehouse-rc1 |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
milestone: | icehouse-rc1 → 2014.1 |
To post a comment you must log in.
Note: in testing these changes, a conflict between the tests in test_backup_ceph and test_utils caused two tests using mock_open to fail. paste.openstack .org/show/ 70322/ and paste.openstack .org/show/ 70372/
For more details see:
http://
http://
After quite a bit of head scratching, the issue was isolated to line 135 of paste 70372. But, no straight forward fix, or relationship between the two problems was found.
To work around the problem, I modified test_utils to test_read_ cached_ file and test_read_ modified_ cached_ file to use a local impl of mock_open rather than mock.mock_open.