Fix test case

Author: MarcialRosales

deps/oauth2_client/include/types.hrl

@@ -22,7 +22,7 @@
 
 -record(proxy_options, {
     https :: boolean(),
-    hostname :: uri_string:uri_string(),
+    host :: uri_string:uri_string(),
     port :: integer(),
     username :: option(string() | binary()),
     password :: option(string() | binary())

deps/oauth2_client/src/oauth2_client.erl

@@ -409,15 +409,15 @@ lookup_root_oauth_provider() ->
 extract_proxy_options_from_url(URL) when is_binary(URL) ->
     extract_proxy_options_from_url(binary_to_list(URL));
 extract_proxy_options_from_url(URL) when is_list(URL) ->
-    Parsed = uri_string:parse(URL),
+    #{host := Host, port := Port, scheme := Scheme} = uri_string:parse(URL),
     #proxy_options{
         https = 
-            case maps:get("scheme", Parsed, "http") of 
+            case Scheme of 
                 "http" -> false;
                 "https" -> true
             end,
-        hostname = maps:get("host", Parsed, undefined),
-        port = maps:get("port", Parsed, undefined)
+        host = Host,
+        port = Port
     }.
 
 -spec extract_proxy_options(#{atom() => any()}|list()) -> proxy_options() | undefined.
@@ -583,7 +583,7 @@ get_proxy_if_any(ProxyOptions) ->
         undefined -> 
             []; 
         Proxy -> 
-            P = {Proxy#proxy_options.hostname, Proxy#proxy_options.port},
+            P = {Proxy#proxy_options.host, Proxy#proxy_options.port},
             case Proxy#proxy_options.https of 
                 true -> [{https_proxy, P}];
                 false -> [{proxy, P}]
@@ -705,7 +705,7 @@ format_proxy_options(ProxyOptions) ->
     lists:flatten(io_lib:format("{https: ~p, hostname: ~p, port: ~p, username: ~p, " ++
         "password: ~p }", [
         ProxyOptions#proxy_options.https,
-        ProxyOptions#proxy_options.hostname,
+        ProxyOptions#proxy_options.host,
         ProxyOptions#proxy_options.port,
         ProxyOptions#proxy_options.username,
         ProxyOptions#proxy_options.password])).

deps/rabbitmq_auth_backend_oauth2/test/rabbit_oauth2_provider_SUITE.erl

@@ -474,23 +474,13 @@ get_oauth_provider_has_jwks_uri(Config) ->
 get_oauth_provider_has_proxy(Config) ->
     {ok, OAuthProvider} = get_oauth_provider(
         ?config(oauth_provider_id, Config), [jwks_uri]),    
-    ct:log("key_config: ~p",
-        [ application:get_all_env(rabbitmq_auth_backend_oauth2)]),
-    ct:log("oauthprovider: ~p", [OAuthProvider]),
     ?assertEqual(false, 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.https),
 
-    ct:log("Parsed : ~p", [uri_string:parse("http://idp:8080")]),
-
-    Options = oauth2_client:extract_proxy_options_from_url("http://idp:8080"),
-    ct:log("Options1: ~p", [Options]),
-    Options2 = oauth2_client:extract_proxy_options_from_url(<<"http://idp:8080">>),
-    ct:log("Options2: ~p", [Options2]),
-
     ?assertEqual(?config(proxy_port, Config), 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.port),
     ?assertEqual(?config(proxy_hostname, Config), 
-        OAuthProvider#oauth_provider.proxy_options#proxy_options.hostname),
+        OAuthProvider#oauth_provider.proxy_options#proxy_options.host),
     ?assertEqual(?config(proxy_username, Config), 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.username),
     ?assertEqual(?config(proxy_password, Config), 
@@ -500,16 +490,13 @@ get_oauth_provider_has_proxy(Config) ->
 get_oauth_provider_has_https_proxy(Config) ->
     {ok, OAuthProvider} = get_oauth_provider(
         ?config(oauth_provider_id, Config), [jwks_uri]),    
-    ct:log("key_config: ~p",
-        [ application:get_all_env(rabbitmq_auth_backend_oauth2)]),
-    ct:log("oauthprovider: ~p", [OAuthProvider]),
     ?assertEqual(true, 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.https),
 
     ?assertEqual(?config(proxy_port, Config), 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.port),
     ?assertEqual(?config(proxy_hostname, Config), 
-        OAuthProvider#oauth_provider.proxy_options#proxy_options.hostname),
+        OAuthProvider#oauth_provider.proxy_options#proxy_options.host),
     ?assertEqual(?config(proxy_username, Config), 
         OAuthProvider#oauth_provider.proxy_options#proxy_options.username),
     ?assertEqual(?config(proxy_password, Config),