flake8

server/api/aup.py

@@ -31,7 +31,7 @@ def agreed_aup():
     session["user"] = {**session["user"], **{"user_accepted_aup": True}}
 
     _, location = get_redirect(current_app.app_config, user_accepted_aup=True,
-                            second_factor_confirmed=session["user"]["second_factor_confirmed"])
+                               second_factor_confirmed=session["user"]["second_factor_confirmed"])
 
     ctx_logger("redirect").debug(f"Redirecting user {user_id} to {location}")
 

server/test/api/test_mfa.py

@@ -58,7 +58,7 @@ def test_ssid_scenario(self):
         self.assertEqual(302, res.status_code)
         self.assertEqual("https://foo.bar", res.location)
 
-    def test_ssid_new_user(self):
+    def test_ssid_scenario_new_user(self):
         # initiate proxy_authz call to provision user and initialize 2fa
         res = self.post("/api/users/proxy_authz", response_status_code=200,
                         body={"user_id": "urn:new_user", "service_id": self.app.app_config.oidc.sram_service_entity_id,
@@ -88,7 +88,7 @@ def test_ssid_new_user(self):
                                content_type="application/x-www-form-urlencoded")
 
         self.assertEqual(302, res.status_code)
-        self.assertEqual(self.app.app_config.base_url+"/aup", res.location)
+        self.assertEqual(self.app.app_config.base_url + "/aup", res.location)
 
         res = self.post("/api/aup/agree", with_basic_auth=False)
         self.assertEqual("https://foo.bar", res["location"])