Fix test case

And remove log statements

Author: MarcialRosales

deps/rabbitmq_auth_backend_oauth2/src/rabbit_auth_backend_oauth2.erl

@@ -108,9 +108,6 @@ check_topic_access(#auth_user{impl = DecodedTokenFun},
     with_decoded_token(DecodedTokenFun(),
         fun(Token) ->
             Scopes = get_expanded_scopes(Token, Resource),        
-            ct:log("check_topic_access Scoes: ~p ......", [Scopes]),    
-            ct:log("check_topic_access Resource: ~p Permisson: ~p Context:~p", 
-                [Resource,Permission, Context]),   
             rabbit_oauth2_scope:topic_access(Resource, Permission, Context, Scopes)
         end).
 

deps/rabbitmq_auth_backend_oauth2/src/rabbit_oauth2_scope.erl

@@ -47,7 +47,7 @@ topic_access(#resource{virtual_host = VHost, name = ExchangeName},
              #{routing_key := RoutingKey},
              Scopes) ->
     lists:any(
-        fun({VHostPattern, ExchangeNamePattern, RoutingKeyPattern, ScopeGrantedPermission}) ->
+        fun({VHostPattern, ExchangeNamePattern, RoutingKeyPattern, ScopeGrantedPermission}) ->            
             is_binary(RoutingKeyPattern) andalso
             wildcard:match(VHost, VHostPattern) andalso
             wildcard:match(ExchangeName, ExchangeNamePattern) andalso
@@ -62,10 +62,11 @@ topic_access(#resource{virtual_host = VHost, name = ExchangeName},
 get_scope_permissions(Scopes) when is_list(Scopes) ->
     lists:filtermap(
         fun(ScopeEl) ->
-            case parse_permission_pattern(ScopeEl) of
+            Ret = case parse_permission_pattern(ScopeEl) of
                 ignore -> false;
                 Perm   -> {true, Perm}
-            end
+            end,
+            Ret
         end,
         Scopes).
 
@@ -88,7 +89,7 @@ parse_permission_pattern(_Other) ->
 
 -spec parse_resource_pattern(binary(), permission()) ->
     {rabbit_types:vhost(), binary(), binary() | none, permission()} | 'ignore'.
-parse_resource_pattern(Pattern, Permission) ->
+parse_resource_pattern(Pattern, Permission) ->    
     case binary:split(Pattern, <<"/">>, [global]) of
         [VhostPattern, NamePattern] ->
             {VhostPattern, NamePattern, none, Permission};

deps/rabbitmq_auth_backend_oauth2/test/unit_SUITE.erl

@@ -785,7 +785,7 @@ test_successful_access_with_a_token_that_uses_single_scope_alias_with_var_expans
     Alias = <<"client-alias-1">>,
     set_env(scope_aliases, #{
         Alias => [
-            <<"rabbitmq.configure:{vhost}/q-{sub}/{client_id}/*">>
+            <<"rabbitmq.configure:{vhost}/q-{sub}/{client_id}**">>
         ]            
     }),