From 0de154678c9f612823aa6fdf3fe2faa201c5340b Mon Sep 17 00:00:00 2001 From: Alexandre Aubin Date: Wed, 20 Jan 2021 05:59:24 +0100 Subject: [PATCH] Black again, with the right version this time.. --- moulinette/interfaces/api.py | 9 ++++- moulinette/utils/filesystem.py | 56 +++++++++++++++++----------- moulinette/utils/stream.py | 9 ++--- test/src/old_slapdtest/_slapdtest.py | 14 +++++-- test/test_ldap.py | 8 ++-- 5 files changed, 61 insertions(+), 35 deletions(-) diff --git a/moulinette/interfaces/api.py b/moulinette/interfaces/api.py index b849a3ef..b2d245ef 100644 --- a/moulinette/interfaces/api.py +++ b/moulinette/interfaces/api.py @@ -288,7 +288,10 @@ class _ActionsMapPlugin(object): # Append messages route app.route( - "/messages", name="messages", callback=self.messages, skip=["actionsmap"], + "/messages", + name="messages", + callback=self.messages, + skip=["actionsmap"], ) # Append routes from the actions map @@ -815,7 +818,9 @@ class Interface(BaseInterface): """ logger.debug( - "starting the server instance in %s:%d", host, port, + "starting the server instance in %s:%d", + host, + port, ) try: diff --git a/moulinette/utils/filesystem.py b/moulinette/utils/filesystem.py index 6e7b9b19..c9fe126e 100644 --- a/moulinette/utils/filesystem.py +++ b/moulinette/utils/filesystem.py @@ -22,9 +22,11 @@ def read_file(file_path): Keyword argument: file_path -- Path to the text file """ - assert isinstance(file_path, str), ( - "Error: file_path '%s' should be a string but is of type '%s' instead" - % (file_path, type(file_path)) + assert isinstance( + file_path, str + ), "Error: file_path '%s' should be a string but is of type '%s' instead" % ( + file_path, + type(file_path), ) # Check file exists @@ -151,24 +153,30 @@ def write_to_file(file_path, data, file_mode="w"): file_mode -- Mode used when writing the file. Option meant to be used by append_to_file to avoid duplicating the code of this function. """ - assert isinstance(data, str) or isinstance(data, list), ( - "Error: data '%s' should be either a string or a list but is of type '%s'" - % (data, type(data)) + assert isinstance(data, str) or isinstance( + data, list + ), "Error: data '%s' should be either a string or a list but is of type '%s'" % ( + data, + type(data), ) assert not os.path.isdir(file_path), ( "Error: file_path '%s' point to a dir, it should be a file" % file_path ) - assert os.path.isdir(os.path.dirname(file_path)), ( - "Error: the path ('%s') base dir ('%s') is not a dir" - % (file_path, os.path.dirname(file_path),) + assert os.path.isdir( + os.path.dirname(file_path) + ), "Error: the path ('%s') base dir ('%s') is not a dir" % ( + file_path, + os.path.dirname(file_path), ) # If data is a list, check elements are strings and build a single string if not isinstance(data, str): for element in data: - assert isinstance(element, str), ( - "Error: element '%s' should be a string but is of type '%s' instead" - % (element, type(element)) + assert isinstance( + element, str + ), "Error: element '%s' should be a string but is of type '%s' instead" % ( + element, + type(element), ) data = "\n".join(data) @@ -203,20 +211,26 @@ def write_to_json(file_path, data, sort_keys=False, indent=None): """ # Assumptions - assert isinstance(file_path, str), ( - "Error: file_path '%s' should be a string but is of type '%s' instead" - % (file_path, type(file_path)) + assert isinstance( + file_path, str + ), "Error: file_path '%s' should be a string but is of type '%s' instead" % ( + file_path, + type(file_path), ) - assert isinstance(data, dict) or isinstance(data, list), ( - "Error: data '%s' should be a dict or a list but is of type '%s' instead" - % (data, type(data),) + assert isinstance(data, dict) or isinstance( + data, list + ), "Error: data '%s' should be a dict or a list but is of type '%s' instead" % ( + data, + type(data), ) assert not os.path.isdir(file_path), ( "Error: file_path '%s' point to a dir, it should be a file" % file_path ) - assert os.path.isdir(os.path.dirname(file_path)), ( - "Error: the path ('%s') base dir ('%s') is not a dir" - % (file_path, os.path.dirname(file_path),) + assert os.path.isdir( + os.path.dirname(file_path) + ), "Error: the path ('%s') base dir ('%s') is not a dir" % ( + file_path, + os.path.dirname(file_path), ) # Write dict to file diff --git a/moulinette/utils/stream.py b/moulinette/utils/stream.py index 92fa7f7c..7951497a 100644 --- a/moulinette/utils/stream.py +++ b/moulinette/utils/stream.py @@ -19,19 +19,16 @@ class LogPipe(threading.Thread): self.start() def fileno(self): - """Return the write file descriptor of the pipe - """ + """Return the write file descriptor of the pipe""" return self.fdWrite def run(self): - """Run the thread, logging everything. - """ + """Run the thread, logging everything.""" for line in iter(self.pipeReader.readline, ""): self.log_callback(line.strip("\n")) self.pipeReader.close() def close(self): - """Close the write end of the pipe. - """ + """Close the write end of the pipe.""" os.close(self.fdWrite) diff --git a/test/src/old_slapdtest/_slapdtest.py b/test/src/old_slapdtest/_slapdtest.py index a8b196ed..d65b79f7 100644 --- a/test/src/old_slapdtest/_slapdtest.py +++ b/test/src/old_slapdtest/_slapdtest.py @@ -199,7 +199,8 @@ def combined_logger( fmt=" ".join((log_name, sys_log_format)) ) my_syslog_handler = logging.handlers.SysLogHandler( - address="/dev/log", facility=SysLogHandler.LOG_DAEMON, + address="/dev/log", + facility=SysLogHandler.LOG_DAEMON, ) my_syslog_handler.setFormatter(my_syslog_formatter) new_logger.addHandler(my_syslog_handler) @@ -371,7 +372,8 @@ class SlapdObject(object): """ include_directives = "\n".join( 'include "{schema_prefix}/{schema_file}"'.format( - schema_prefix=self._schema_prefix, schema_file=schema_file, + schema_prefix=self._schema_prefix, + schema_file=schema_file, ) for schema_file in self.openldap_schema_files ) @@ -561,7 +563,13 @@ class SlapdObject(object): if ldap_uri is None: ldap_uri = self.default_ldap_uri args = ( - [ldapcommand, "-H", ldap_uri,] + self._cli_auth_args() + (extra_args or []) + [ + ldapcommand, + "-H", + ldap_uri, + ] + + self._cli_auth_args() + + (extra_args or []) ) self._log.debug("Run command: %r", " ".join(args)) proc = subprocess.Popen( diff --git a/test/test_ldap.py b/test/test_ldap.py index a28b1c27..68a7aae1 100644 --- a/test/test_ldap.py +++ b/test/test_ldap.py @@ -58,9 +58,11 @@ class TestLDAP: def test_authenticate_sasl_non_interactive_bind(self, ldap_server): self.ldap_conf["parameters"]["uri"] = ldap_server.uri - self.ldap_conf["parameters"]["user_rdn"] = ( - "gidNumber=%s+uidNumber=%s,cn=peercred,cn=external,cn=auth" - % (os.getgid(), os.getuid(),) + self.ldap_conf["parameters"][ + "user_rdn" + ] = "gidNumber=%s+uidNumber=%s,cn=peercred,cn=external,cn=auth" % ( + os.getgid(), + os.getuid(), ) ldap_interface = m_ldap.Authenticator(**self.ldap_conf)