Test translation function of scope_aliases

Author: MarcialRosales

deps/rabbitmq_auth_backend_oauth2/src/rabbit_oauth2_schema.erl

@@ -251,7 +251,7 @@ extract_oauth_providers_signing_keys(Settings) ->
     KeyFun = fun extract_key_as_binary/1,
 
     IndexedSigningKeys = [{Name, {list_to_binary(Kid), list_to_binary(V)}} ||
-        {[?AUTH_OAUTH2, ?OAUTH_PROVIDERS, Name, "signing_keys", Kid], V} 
+        {[?AUTH_OAUTH2, ?OAUTH_PROVIDERS, Name, ?SIGNING_KEYS, Kid], V} 
             <- Settings ],
     maps:map(fun(_K,V)-> [{signing_keys, translate_list_of_signing_keys(V)}] end,
         maps:groups_from_list(KeyFun, fun({_, V}) -> V end, IndexedSigningKeys)).

deps/rabbitmq_auth_backend_oauth2/test/rabbit_oauth2_schema_SUITE.erl

@@ -15,7 +15,8 @@
 -import(rabbit_oauth2_schema, [
     translate_endpoint_params/2, 
     translate_oauth_providers/1,
-    translate_resource_servers/1
+    translate_resource_servers/1,
+    translate_scope_aliases/1
 ]).
 
 all() ->
@@ -37,7 +38,8 @@ all() ->
         test_with_many_resource_servers,
         test_resource_servers_attributes,
         test_invalid_oauth_providers_endpoint_params,
-        test_without_oauth_providers_with_endpoint_params
+        test_without_oauth_providers_with_endpoint_params,
+        test_scope_aliases
 
     ].
 
@@ -97,10 +99,11 @@ test_without_oauth_providers_with_endpoint_params(_) ->
 
 test_with_one_oauth_provider(_) ->
     Conf = [
-        {["auth_oauth2","oauth_providers","keycloak","issuer"],"https://rabbit"}
+        {["auth_oauth2","oauth_providers","keycloak","issuer"],
+            "https://rabbit"}
     ],
-    #{<<"keycloak">> := [{issuer, "https://rabbit"}]
-    } = translate_oauth_providers(Conf).
+    #{<<"keycloak">> := [{issuer, <<"https://rabbit">>}]
+    } = rabbit_oauth2_schema:translate_oauth_providers(Conf).
 
 test_with_one_resource_server(_) ->
     Conf = [
@@ -118,7 +121,7 @@ test_with_many_oauth_providers(_) ->
         {["auth_oauth2","oauth_providers","uaa","discovery_endpoint_path"],
             "/some-path"}
     ],
-    #{<<"keycloak">> := [{issuer, "https://keycloak"}
+    #{<<"keycloak">> := [{issuer, <<"https://keycloak">>}
                         ],
       <<"uaa">> := [{issuer, "https://uaa"},
                     {discovery_endpoint_path, "/some-path"}
@@ -128,8 +131,10 @@ test_with_many_oauth_providers(_) ->
 
 test_with_many_resource_servers(_) ->
     Conf = [
-        {["auth_oauth2","resource_servers","rabbitmq1","id"], "rabbitmq1"},
-        {["auth_oauth2","resource_servers","rabbitmq2","id"], "rabbitmq2"}
+        {["auth_oauth2","resource_servers","rabbitmq1","id"],
+            "rabbitmq1"},
+        {["auth_oauth2","resource_servers","rabbitmq2","id"],
+            "rabbitmq2"}
     ],
     #{<<"rabbitmq1">> := [{id, <<"rabbitmq1">>}
                         ],
@@ -268,7 +273,7 @@ test_oauth_providers_signing_keys(Conf) ->
         {["auth_oauth2","oauth_providers","keycloak","signing_keys","1"], 
             cert_filename(Conf)}
     ],
-    #{<<"keycloak">> := [{issuer, "https://keycloak"},
+    #{<<"keycloak">> := [{issuer, <<"https://keycloak">>},
                          {signing_keys, SigningKeys}
                          ]
     } = sort_settings(translate_oauth_providers(CuttlefishConf)),
@@ -277,6 +282,23 @@ test_oauth_providers_signing_keys(Conf) ->
       <<"2">> := {pem, <<"I'm not a certificate">>}
     } = SigningKeys.
 
+test_scope_aliases(_) ->
+    CuttlefishConf = [
+        {["auth_oauth2","scope_aliases","1","alias"],
+            "admin"},
+        {["auth_oauth2","scope_aliases","1","scope"], 
+            "rabbitmq.tag:administrator"},
+        {["auth_oauth2","scope_aliases","2","alias"],
+            "developer"},
+        {["auth_oauth2","scope_aliases","2","scope"], 
+            "rabbitmq.tag:management rabbitmq.read:*/*"}        
+    ],
+    #{
+        <<"admin">> := [<<"rabbitmq.tag:administrator">>],
+        <<"developer">> := [<<"rabbitmq.tag:management">>, <<"rabbitmq.read:*/*">>]                         
+    } = translate_scope_aliases(CuttlefishConf).
+    
+
 cert_filename(Conf) ->
     string:concat(?config(data_dir, Conf), "certs/cert.pem").