Merge pull request #40 from its-dirg/user_id_to_attr-fix

Fix implementation of 'user_id_to_attr'.

Author: RebeckaG

src/satosa/base.py

@@ -128,7 +128,7 @@ def _auth_resp_finish(self, context, internal_response):
         internal_response.user_id_hash_type = UserIdHasher.hash_type(context.state)
         user_id_to_attr = self.config["INTERNAL_ATTRIBUTES"].get("user_id_to_attr", None)
         if user_id_to_attr:
-            internal_response.attributes[user_id_to_attr] = internal_response.user_id
+            internal_response.attributes[user_id_to_attr] = [internal_response.user_id]
 
         # Hash all attributes specified in INTERNAL_ATTRIBUTES["hash]
         hash_attributes = self.config["INTERNAL_ATTRIBUTES"].get("hash", [])

tests/satosa/test_base.py

@@ -97,7 +97,7 @@ def test_auth_resp_callback_func_respects_user_id_to_attr(self, context, satosa_
         UserIdHasher.save_state(InternalRequest(UserIdHashType.transient, ""), context.state)
 
         base._auth_resp_callback_func(context, internal_resp)
-        assert internal_resp.attributes["user_id"] == internal_resp.user_id
+        assert internal_resp.attributes["user_id"] == [internal_resp.user_id]
 
     @pytest.mark.parametrize("micro_services", [
         [Mock()],