]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
pybind/mgr/tests: fix style 44695/head
authorSage Weil <sage@newdream.net>
Thu, 20 Jan 2022 18:15:06 +0000 (13:15 -0500)
committerSage Weil <sage@newdream.net>
Thu, 20 Jan 2022 18:16:58 +0000 (13:16 -0500)
Signed-off-by: Sage Weil <sage@newdream.net>
src/pybind/mgr/tests/__init__.py
src/pybind/mgr/tests/test_tls.py

index 68b61e7ec52330359e2ef094ae0ab561c517cc6d..633959084ccbed3158c05f091fa5b6eb4c33e60e 100644 (file)
@@ -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
index 923d91917caf27c5d18871073b77811cc828ba98..fda463cd2f4d8f376b5bd5895cff6bdb952a6f1d 100644 (file)
@@ -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()