Fix formatting

Signed-off-by: Ivan Kanakarakis <[email protected]>

Author: c00kiemon5ter

.gitignore

@@ -14,4 +14,4 @@ _build
 build/
 dist/
 .coverage
-venv/
+venv/

src/satosa/frontends/saml2.py

@@ -1210,4 +1210,3 @@ def _metadata_endpoint(self, context, co_name):
         # authentication request dynamically create an IdP instance.
         self.idp = self._create_co_virtual_idp(context, co_name=co_name)
         return super()._metadata_endpoint(context=context);
-

tests/conftest.py

@@ -64,7 +64,7 @@ def sp_conf(cert_and_key):
 
 @pytest.fixture
 def idp_conf(cert_and_key):
-    idp_base = BASE_URL
+    idp_base = "http://idp.example.com"
 
     idpconfig = {
         "entityid": "{}/{}/proxy.xml".format(idp_base, "Saml2IDP"),

tests/satosa/frontends/test_saml2.py

@@ -298,14 +298,14 @@ def test_acr_mapping_per_idp_in_authn_response(self, context, idp_conf, sp_conf,
         ]
     )
     def test_respect_sp_entity_categories(
-            self,
-            context,
-            entity_category,
-            entity_category_module,
-            expected_attributes,
-            idp_conf,
-            sp_conf,
-            internal_response
+        self,
+        context,
+        entity_category,
+        entity_category_module,
+        expected_attributes,
+        idp_conf,
+        sp_conf,
+        internal_response
     ):
         idp_metadata_str = create_metadata_from_config_dict(idp_conf)
         idp_conf["service"]["idp"]["policy"]["default"]["entity_categories"] = [entity_category_module]
@@ -365,7 +365,7 @@ def test_metadata_endpoint(self, context, idp_conf):
         assert idp_conf["entityid"] in resp.message
 
     def test_custom_attribute_release_with_less_attributes_than_entity_category(
-            self, context, idp_conf, sp_conf, internal_response
+        self, context, idp_conf, sp_conf, internal_response
     ):
         idp_metadata_str = create_metadata_from_config_dict(idp_conf)
         idp_conf["service"]["idp"]["policy"]["default"]["entity_categories"] = ["swamid"]
@@ -387,8 +387,8 @@ def test_custom_attribute_release_with_less_attributes_than_entity_category(
         internal_response.requester = sp_conf["entityid"]
         resp = self.get_auth_response(samlfrontend, context, internal_response, sp_conf, idp_metadata_str)
         assert len(resp.ava.keys()) == (
-                len(expected_attributes)
-                - len(custom_attributes[internal_response.auth_info.issuer][internal_response.requester]["exclude"])
+            len(expected_attributes)
+            - len(custom_attributes[internal_response.auth_info.issuer][internal_response.requester]["exclude"])
         )
 
 
@@ -442,7 +442,7 @@ class TestSAMLVirtualCoFrontend(TestSAMLFrontend):
         CO_O: ["Medium Energy Synchrotron Source"],
         CO_C: ["US"],
         CO_CO: ["United States"],
-        CO_NOREDUORGACRONYM: ["MESS"]
+        CO_NOREDUORGACRONYM: ["MESS"],
     }
     KEY_SSO = "single_sign_on_service"
 
@@ -474,7 +474,7 @@ def frontend(self, idp_conf, sp_conf):
         conf = {
             "idp_config": idp_conf,
             "endpoints": ENDPOINTS,
-            "collaborative_organizations": [collab_org]
+            "collaborative_organizations": [collab_org],
         }
 
         # Use a richer set of internal attributes than what is provided
@@ -652,42 +652,40 @@ def test_co_static_attributes(self, frontend, context, internal_response,
 class TestSubjectTypeToSamlNameIdFormat:
     def test_should_default_to_persistent(self):
         assert (
-                subject_type_to_saml_nameid_format("unmatched")
-                == NAMEID_FORMAT_PERSISTENT
+            subject_type_to_saml_nameid_format("unmatched")
+            == NAMEID_FORMAT_PERSISTENT
         )
 
     def test_should_map_persistent(self):
         assert (
-                subject_type_to_saml_nameid_format(NAMEID_FORMAT_PERSISTENT)
-                == NAMEID_FORMAT_PERSISTENT
+            subject_type_to_saml_nameid_format(NAMEID_FORMAT_PERSISTENT)
+            == NAMEID_FORMAT_PERSISTENT
         )
 
     def test_should_map_transient(self):
         assert (
-                subject_type_to_saml_nameid_format(NAMEID_FORMAT_TRANSIENT)
-                == NAMEID_FORMAT_TRANSIENT
+            subject_type_to_saml_nameid_format(NAMEID_FORMAT_TRANSIENT)
+            == NAMEID_FORMAT_TRANSIENT
         )
 
     def test_should_map_emailaddress(self):
         assert (
-                subject_type_to_saml_nameid_format(NAMEID_FORMAT_EMAILADDRESS)
-                == NAMEID_FORMAT_EMAILADDRESS
+            subject_type_to_saml_nameid_format(NAMEID_FORMAT_EMAILADDRESS)
+            == NAMEID_FORMAT_EMAILADDRESS
         )
 
     def test_should_map_unspecified(self):
         assert (
-                subject_type_to_saml_nameid_format(NAMEID_FORMAT_UNSPECIFIED)
-                == NAMEID_FORMAT_UNSPECIFIED
+            subject_type_to_saml_nameid_format(NAMEID_FORMAT_UNSPECIFIED)
+            == NAMEID_FORMAT_UNSPECIFIED
         )
 
     def test_should_map_public(self):
         assert (
-                subject_type_to_saml_nameid_format("public")
-                == NAMEID_FORMAT_PERSISTENT
+            subject_type_to_saml_nameid_format("public") == NAMEID_FORMAT_PERSISTENT
         )
 
     def test_should_map_pairwise(self):
         assert (
-                subject_type_to_saml_nameid_format("pairwise")
-                == NAMEID_FORMAT_TRANSIENT
+            subject_type_to_saml_nameid_format("pairwise") == NAMEID_FORMAT_TRANSIENT
         )