Merge pull request #38 from its-dirg/consent-api

Update consent micro service to work with new API.

Author: RebeckaG

src/satosa/micro_services/consent.py

@@ -99,7 +99,7 @@ def _approve_new_consent(self, context, internal_response, id_hash):
             internal_response.attributes = {}
             return self._end_consent(context, internal_response)
 
-        consent_redirect = "%s?ticket=%s" % (self.redirect_url, ticket)
+        consent_redirect = "%s/%s" % (self.redirect_url, ticket)
         return Redirect(consent_redirect)
 
     def process(self, context, internal_response):

tests/satosa/micro_services/test_consent.py

@@ -1,7 +1,6 @@
 import json
 import re
 from collections import Counter
-from unittest.mock import Mock
 from urllib.parse import urlparse, parse_qs
 
 import pytest
@@ -65,10 +64,8 @@ def consent_registration_endpoint_regex(self):
     def assert_redirect(self, redirect_resp, expected_ticket):
         assert isinstance(redirect_resp, Redirect)
 
-        parsed_url = parse_qs(urlparse(redirect_resp.message).query)
-        assert len(parsed_url["ticket"]) == 1
-        ticket = parsed_url["ticket"][0]
-        assert ticket == expected_ticket
+        path = urlparse(redirect_resp.message).path
+        assert path == "/consent/" + expected_ticket
 
     def assert_registration_req(self, request, internal_response, sign_key_path, base_url, requester_name):
         split_path = request.path_url.lstrip("/").split("/")