All proposed patches have been merged. Let's mark it as Fix Released.
** Changed in: horizon
Status: In Progress => Fix Released
** Changed in: horizon
Milestone: None => rocky-1
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1752917
Title:
inappropriate mocking in
openstack_dashboard/test/unit/api/rest/test_*.py
Status in OpenStack Dashboard (Horizon):
Fix Released
Bug description:
Tests under openstack_dashboard/test/unit/api/rest mocks the whole API
modules like @mock.patch.patch(api, 'neutron').
It is not an appropriate mocking. We should mock only methods we really
expect to call.
In addition, there are several points to be improved in these tests.
- Unnecessary usage of test.mock_factory(). Test data can be used directly.
- Related to the above, setUp() is unnecessary because they are used to
prepare data with mock_factory.
- Test data should be accessed via something like self.networks.list() rather
than calling TestData directly.
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1752917/+subscriptions
--
Mailing list: https://launchpad.net/~yahoo-eng-team
Post to : [email protected]
Unsubscribe : https://launchpad.net/~yahoo-eng-team
More help : https://help.launchpad.net/ListHelp
