Fix attribute_mapping.py

Author: Ric Janus Sapasap

src/satosa/attribute_mapping.py

@@ -57,7 +57,8 @@ def to_internal_filter(self, attribute_profile, external_attribute_names):
         :type case_insensitive: bool
         :rtype: list[str]
 
-        :param attribute_profile: From which external type to convert (ex: oidc, saml, ...)
+        :param attribute_profile: From which external type to convert
+            (ex: oidc, saml, ...)
         :param external_attribute_names: A list of attribute names
         :param case_insensitive: Create a case insensitive filter
         :return: A list of attribute names in the internal format
@@ -92,27 +93,32 @@ def to_internal(self, attribute_profile, external_dict):
         :type external_dict: dict[str, str]
         :rtype: dict[str, str]
 
-        :param attribute_profile: From which external type to convert (ex: oidc, saml, ...)
+        :param attribute_profile: From which external type to convert
+            (ex: oidc, saml, ...)
         :param external_dict: Attributes in the external format
         :return: Attributes in the internal format
         """
         internal_dict = {}
 
         for internal_attribute_name, mapping in self.from_internal_attributes.items():
             if attribute_profile not in mapping:
-                logline = "no attribute mapping found for internal attribute {internal} the attribute profile {attribute}".format(
-                    internal=internal_attribute_name, attribute=attribute_profile
-                )
+                logline = (
+                    "no attribute mapping found for internal attribute"
+                    " {internal} the attribute profile {attribute}"
+                ).format(internal=internal_attribute_name, attribute=attribute_profile)
                 logger.debug(logline)
-                # skip this internal attribute if we have no mapping in the specified profile
+                # skip this internal attribute if we have no mapping in
+                # the specified profile
                 continue
 
             external_attribute_name = mapping[attribute_profile]
             attribute_values = self._collate_attribute_values_by_priority_order(
                 external_attribute_name, external_dict
             )
             if attribute_values:  # Only insert key if it has some values
-                logline = "backend attribute {external} mapped to {internal} ({value})".format(
+                logline = (
+                    "backend attribute {external} mapped to {internal}" " ({value})"
+                ).format(
                     external=external_attribute_name,
                     internal=internal_attribute_name,
                     value=attribute_values,
@@ -158,7 +164,8 @@ def _handle_template_attributes(self, attribute_profile, internal_dict):
 
         for internal_attribute_name, mapping in self.template_attributes.items():
             if attribute_profile not in mapping:
-                # skip this internal attribute if we have no mapping in the specified profile
+                # skip this internal attribute if we have no mapping in
+                # the specified profile
                 continue
 
             external_attribute_name = mapping[attribute_profile]
@@ -211,7 +218,8 @@ def from_internal(self, attribute_profile, internal_dict):
         :type internal_dict: dict[str, str]
         :rtype: dict[str, str]
 
-        :param attribute_profile: To which external type to convert (ex: oidc, saml, ...)
+        :param attribute_profile: From which external type to convert
+            (ex: oidc, saml, ...)
         :param internal_dict: attributes to map
         :return: attribute values and names in the specified "profile"
         """
@@ -231,10 +239,12 @@ def from_internal(self, attribute_profile, internal_dict):
                 continue
 
             if attribute_profile not in attribute_mapping:
-                # skip this internal attribute if we have no mapping in the specified profile
-                logline = "no mapping found for '{internal}' in attribute profile '{attribute}'".format(
-                    internal=internal_attribute_name, attribute=attribute_profile
-                )
+                # skip this internal attribute if we have no mapping in
+                # the specified profile
+                logline = (
+                    "no mapping found for '{internal}' in attribute profile"
+                    " '{attribute}'"
+                ).format(internal=internal_attribute_name, attribute=attribute_profile)
                 logger.debug(logline)
                 continue