@@ -495,9 +495,9 @@ def test_excluded_user_bypasses_user_rate_limit():
495495
496496 # Excluded user should never be blocked, even past maxRequests
497497 for _ in range (5 ):
498- assert should_ratelimit_request (route_metadata , "1.2.3.4" , { "id" : "user123" }, cm ) == {
499- "block" : False
500- }
498+ assert should_ratelimit_request (
499+ route_metadata , "1.2.3.4" , { "id" : "user123" }, cm
500+ ) == { "block" : False }
501501
502502
503503def test_non_excluded_user_still_rate_limited ():
@@ -514,10 +514,18 @@ def test_non_excluded_user_still_rate_limited():
514514 cm = create_connection_manager ([endpoint ], excluded_uids = ["other_user" ])
515515 route_metadata = create_route_metadata ()
516516
517- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm ) == {"block" : False }
518- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm ) == {"block" : False }
519- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm ) == {"block" : False }
520- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm ) == {
517+ assert should_ratelimit_request (
518+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm
519+ ) == {"block" : False }
520+ assert should_ratelimit_request (
521+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm
522+ ) == {"block" : False }
523+ assert should_ratelimit_request (
524+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm
525+ ) == {"block" : False }
526+ assert should_ratelimit_request (
527+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm
528+ ) == {
521529 "block" : True ,
522530 "trigger" : "user" ,
523531 }
@@ -537,11 +545,19 @@ def test_excluded_user_still_blocked_by_group_rate_limit():
537545 cm = create_connection_manager ([endpoint ], excluded_uids = ["user123" ])
538546 route_metadata = create_route_metadata ()
539547
540- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1" ) == {"block" : False }
541- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1" ) == {"block" : False }
542- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1" ) == {"block" : False }
548+ assert should_ratelimit_request (
549+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1"
550+ ) == {"block" : False }
551+ assert should_ratelimit_request (
552+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1"
553+ ) == {"block" : False }
554+ assert should_ratelimit_request (
555+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1"
556+ ) == {"block" : False }
543557 # Group rate limit still applies even for excluded users
544- assert should_ratelimit_request (route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1" ) == {
558+ assert should_ratelimit_request (
559+ route_metadata , "1.2.3.4" , {"id" : "user123" }, cm , "group1"
560+ ) == {
545561 "block" : True ,
546562 "trigger" : "group" ,
547563 }
0 commit comments