yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #31830
[Bug 1384683] Re: Dashboard tests register admin.aggregates as side effect
** Changed in: horizon
Status: Fix Committed => Fix Released
--
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/1384683
Title:
Dashboard tests register admin.aggregates as side effect
Status in OpenStack Dashboard (Horizon):
Fix Released
Bug description:
The mocks that have recently been added to the setUp method of the
Openstack Dashboard TestCase have a side effect of registering the
admin.aggregates panel, even if it has been disabled in the
configuration. That's because Mock has to import the module in order
to mock anything inside it, and importing the panel module has a side
effect of registering the panel (the register command is at the top
level of the module code).
This doesn't affect the Dashboard tests, as this panel is already
registered in them. However, in Tuskar-UI we disable all dashboards
and their panels except for the infrastructure dashboard that we are
adding. This leads to url resolution errors when a panel is being
registered in a dashboard that doesn't exist.
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1384683/+subscriptions
References