openstack/horizon
Code Issues Proposed changes

Adding static check for panel plugin tests

Browse Source 
Currently, we have tests for plugin but they are not comprehensive enough.
Considering that we are adding more logic to javascript, we need to ensure
that they are properly injected in these plugin.

Change-Id: Ie9186c445962e3142f2b33027b6556f9eb3b34a5
Closes-bug: #1468547
This commit is contained in:
Thai Tran 2015-06-25 16:02:21 -07:00
parent 60294f6aa6
commit 41d5fa36c3
4 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

0
openstack_dashboard/test/test_panels/plugin_panel/static/plugin_panel/plugin.scss Normal file
View File

0
openstack_dashboard/test/test_panels/plugin_panel/static/plugin_panel/plugin.spec.js Normal file
View File

6
openstack_dashboard/test/test_plugins/panel_config/_10_admin_add_panel.py
View File

@ -17,3 +17,9 @@ ADD_ANGULAR_MODULES = ['testAngularModule']
# A list of javascript files to be included in the compressed set of files # A list of javascript files to be included in the compressed set of files
ADD_JS_FILES = ['plugin_panel/plugin_module.js'] ADD_JS_FILES = ['plugin_panel/plugin_module.js']
# A list of scss files to be included in the compressed set of files
ADD_JS_SPEC_FILES = ['plugin_panel/plugin.spec.js']
# A list of scss files to be included in the compressed set of files
ADD_SCSS_FILES = ['plugin_panel/plugin.scss']

11
openstack_dashboard/test/test_plugins/panel_tests.py
View File

@ -21,7 +21,7 @@ from openstack_dashboard.dashboards.admin.info import panel as info_panel
from openstack_dashboard.test import helpers as test from openstack_dashboard.test import helpers as test
from openstack_dashboard.test.test_panels.plugin_panel \ from openstack_dashboard.test.test_panels.plugin_panel \
import panel as plugin_panel import panel as plugin_panel
import openstack_dashboard.test.test_plugins.panel_config from openstack_dashboard.test.test_plugins import panel_config
from openstack_dashboard.utils import settings as util_settings from openstack_dashboard.utils import settings as util_settings
@ -33,9 +33,7 @@ INSTALLED_APPS = list(settings.INSTALLED_APPS)
HORIZON_CONFIG.pop('dashboards', None) HORIZON_CONFIG.pop('dashboards', None)
HORIZON_CONFIG.pop('default_dashboard', None) HORIZON_CONFIG.pop('default_dashboard', None)
util_settings.update_dashboards([ util_settings.update_dashboards([panel_config,], HORIZON_CONFIG, INSTALLED_APPS)
openstack_dashboard.test.test_plugins.panel_config,
], HORIZON_CONFIG, INSTALLED_APPS)
@override_settings(HORIZON_CONFIG=HORIZON_CONFIG, @override_settings(HORIZON_CONFIG=HORIZON_CONFIG,
@ -50,6 +48,11 @@ class PanelPluginTests(test.PluginTestCase):
# Check that the panel is in its configured panel group. # Check that the panel is in its configured panel group.
self.assertIn(plugin_panel.PluginPanel, self.assertIn(plugin_panel.PluginPanel,
[p.__class__ for p in panel_group]) [p.__class__ for p in panel_group])
# Ensure that static resources are properly injected
pc = panel_config._10_admin_add_panel
self.assertEquals(pc.ADD_JS_FILES, HORIZON_CONFIG['js_files'])
self.assertEquals(pc.ADD_JS_SPEC_FILES, HORIZON_CONFIG['js_spec_files'])
self.assertEquals(pc.ADD_SCSS_FILES, HORIZON_CONFIG['scss_files'])
def test_remove_panel(self): def test_remove_panel(self):
dashboard = horizon.get_dashboard("admin") dashboard = horizon.get_dashboard("admin")