yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #45980
[Bug 1540245] Re: Navigation for plugins page structure doesn't work for tests
Reviewed: https://review.openstack.org/274666
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=f93c39258694d880b930823b5a1120114dda65bc
Submitter: Jenkins
Branch: master
commit f93c39258694d880b930823b5a1120114dda65bc
Author: Timur Sufiev <tsufiev@xxxxxxxxxxxx>
Date: Mon Feb 1 16:44:24 2016 +0300
Fix i9n tests pluggable nav structure
Since JSON converted into Python object contains only dicts and lists,
when searching for leaf-nodes in sidebar nav structure we should treat
lists the same way as tuples. Also remove 'Data Processing' section
from CORE_PAGE_STRUCTURE which hid this issue before with 2 initlal
sahara-dashboard tests.
Change-Id: I5b84fd3b769ae559cea484319b6b8956b80f99ae
Closes-Bug: #1540245
** Changed in: horizon
Status: In Progress => 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/1540245
Title:
Navigation for plugins page structure doesn't work for tests
Status in OpenStack Dashboard (Horizon):
Fix Released
Bug description:
I can't override navigation structure for sahara dashboard in tests.
Fresh sahara navigation structure looks like this (from config):
plugin_page_structure={
"Project":
{
"Data Processing":
{
"_":
(
"Clusters",
"Jobs",
"Cluster Templates",
"Node Group Templates",
"Job Templates",
"Job Binaries",
"Data Sources",
"Image Registry",
"Plugins"
)
}
}
}
In horizon (https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/pages/navigation.py#L333) this structure present as string value.
After decoding it using json.loads structure contains dicts and list, but rec function (https://github.com/openstack/horizon/blob/master/openstack_dashboard/test/integration_tests/pages/navigation.py#L320) doesn't process lists (only tuples).
To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1540245/+subscriptions
References