Skip to content

Commit 9c3ffab

Browse files
Fix RateLimitingTest assertion for :by option
Follow-up to rails#50788. This test wasn't actually asserting that the `:by` option worked.
1 parent c8bfd68 commit 9c3ffab

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

actionpack/test/controller/rate_limiting_test.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ class RateLimitingTest < ActionController::TestCase
2020
tests RateLimitedController
2121

2222
setup do
23-
Thread.current[:redis_test_seggregation] = Random.hex(10)
2423
RateLimitedController.cache_store.clear
2524
end
2625

@@ -51,7 +50,7 @@ class RateLimitingTest < ActionController::TestCase
5150
assert_response :forbidden
5251

5352
get :limited_with, params: { rate_limit_key: "other" }
54-
get :limited_with
53+
assert_response :ok
5554
end
5655

5756
test "limited with" do

0 commit comments

Comments
 (0)