From 8c7808d15d4daa4e9d1e984a6d73f7e6a1ff5642 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 20 Jan 2022 13:15:06 -0500 Subject: [PATCH] pybind/mgr/tests: fix style Signed-off-by: Sage Weil --- src/pybind/mgr/tests/__init__.py | 8 +++----- src/pybind/mgr/tests/test_tls.py | 3 ++- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/src/pybind/mgr/tests/__init__.py b/src/pybind/mgr/tests/__init__.py index 68b61e7ec52..633959084cc 100644 --- a/src/pybind/mgr/tests/__init__.py +++ b/src/pybind/mgr/tests/__init__.py @@ -62,7 +62,7 @@ if 'UNITTEST' in os.environ: def _ceph_get_store_prefix(self, prefix): return self.mock_store_prefix('store', prefix) - def _ceph_get_module_option(self, module, key, localized_prefix= None): + def _ceph_get_module_option(self, module, key, localized_prefix=None): try: _, val, _ = self.check_mon_command({ 'prefix': 'config get', @@ -108,7 +108,7 @@ if 'UNITTEST' in os.environ: # Mocking the config store is handy sometimes: def config_get(): who = cmd['who'].split('.') - whos = ['global'] + ['.'.join(who[:i+1]) for i in range(len(who))] + whos = ['global'] + ['.'.join(who[:i + 1]) for i in range(len(who))] for attepmt in reversed(whos): val = self.mock_store_get('config', f'{attepmt}/{cmd["key"]}', None) if val is not None: @@ -151,7 +151,7 @@ if 'UNITTEST' in os.environ: def _ceph_get_foreign_option(self, entity, name): who = entity.split('.') - whos = ['global'] + ['.'.join(who[:i+1]) for i in range(len(who))] + whos = ['global'] + ['.'.join(who[:i + 1]) for i in range(len(who))] for attepmt in reversed(whos): val = self.mock_store_get('config', f'{attepmt}/{name}', None) if val is not None: @@ -174,7 +174,6 @@ if 'UNITTEST' in os.environ: if not hasattr(self, '_store'): self._store = {} - if self.__class__ not in M_classes: # call those only once. self._register_commands('') @@ -194,7 +193,6 @@ if 'UNITTEST' in os.environ: self._ceph_dispatch_remote = lambda *_: None self._ceph_get_mgr_id = mock.MagicMock() - cm = mock.Mock() cm.BaseMgrModule = M cm.BaseMgrStandbyModule = M diff --git a/src/pybind/mgr/tests/test_tls.py b/src/pybind/mgr/tests/test_tls.py index 923d91917ca..fda463cd2f4 100644 --- a/src/pybind/mgr/tests/test_tls.py +++ b/src/pybind/mgr/tests/test_tls.py @@ -15,7 +15,8 @@ class TLSchecks(unittest.TestCase): verify_tls(crt, key) def test_invalid_RDN(self): - self.assertRaises(ValueError, create_self_signed_cert, dname={'O': 'Ceph', 'Bogus': 'testsuite'}) + self.assertRaises(ValueError, create_self_signed_cert, + dname={'O': 'Ceph', 'Bogus': 'testsuite'}) def test_invalid_key(self): crt, key = create_self_signed_cert() -- 2.39.5