From 1813f259076f2342f0d18560c464c0f86f86e422 Mon Sep 17 00:00:00 2001 From: alfonsomthd Date: Tue, 13 Aug 2019 12:15:38 +0200 Subject: [PATCH] mgr/dashboard: fix mgr module API tests Signed-off-by: alfonsomthd (cherry picked from commit 6e6711a1e217042390b379f98c0ec064ab4e2702) --- qa/tasks/mgr/dashboard/test_mgr_module.py | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/qa/tasks/mgr/dashboard/test_mgr_module.py b/qa/tasks/mgr/dashboard/test_mgr_module.py index 4f9722469b957..154700e0b70ea 100644 --- a/qa/tasks/mgr/dashboard/test_mgr_module.py +++ b/qa/tasks/mgr/dashboard/test_mgr_module.py @@ -12,11 +12,6 @@ logger = logging.getLogger(__name__) class MgrModuleTestCase(DashboardTestCase): MGRS_REQUIRED = 1 - @classmethod - def tearDownClass(cls): - cls._ceph_cmd(['mgr', 'module', 'disable', 'telemetry']) - super(MgrModuleTestCase, cls).tearDownClass() - def wait_until_rest_api_accessible(self): """ Wait until the REST API is accessible. @@ -37,7 +32,7 @@ class MgrModuleTestCase(DashboardTestCase): class MgrModuleTest(MgrModuleTestCase): def test_list_disabled_module(self): - self._ceph_cmd(['mgr', 'module', 'disable', 'telemetry']) + self._ceph_cmd(['mgr', 'module', 'disable', 'iostat']) self.wait_until_rest_api_accessible() data = self._get('/api/mgr/module') self.assertStatus(200) @@ -65,12 +60,12 @@ class MgrModuleTest(MgrModuleTestCase): 'tags': JList(str) })) }))) - module_info = self.find_object_in_list('name', 'telemetry', data) + module_info = self.find_object_in_list('name', 'iostat', data) self.assertIsNotNone(module_info) self.assertFalse(module_info['enabled']) def test_list_enabled_module(self): - self._ceph_cmd(['mgr', 'module', 'enable', 'telemetry']) + self._ceph_cmd(['mgr', 'module', 'enable', 'iostat']) self.wait_until_rest_api_accessible() data = self._get('/api/mgr/module') self.assertStatus(200) @@ -98,7 +93,7 @@ class MgrModuleTest(MgrModuleTestCase): 'tags': JList(str) })) }))) - module_info = self.find_object_in_list('name', 'telemetry', data) + module_info = self.find_object_in_list('name', 'iostat', data) self.assertIsNotNone(module_info) self.assertTrue(module_info['enabled']) @@ -110,6 +105,7 @@ class MgrModuleTelemetryTest(MgrModuleTestCase): self.assertSchema( data, JObj( + allow_unknown=True, sub_elems={ 'channel_basic': bool, 'channel_ident': bool, @@ -119,6 +115,7 @@ class MgrModuleTelemetryTest(MgrModuleTestCase): 'description': str, 'enabled': bool, 'interval': int, + 'last_opt_revision': int, 'leaderboard': bool, 'organization': str, 'proxy': str, -- 2.39.5