Skip to content

Commit

Permalink
Fix RateLimitingTest assertion for :by option
Browse files Browse the repository at this point in the history
Follow-up to rails#50788.

This test wasn't actually asserting that the `:by` option worked.
  • Loading branch information
jonathanhefner committed Jan 19, 2024
1 parent c8bfd68 commit 9c3ffab
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions actionpack/test/controller/rate_limiting_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ class RateLimitingTest < ActionController::TestCase
tests RateLimitedController

setup do
Thread.current[:redis_test_seggregation] = Random.hex(10)
RateLimitedController.cache_store.clear
end

Expand Down Expand Up @@ -51,7 +50,7 @@ class RateLimitingTest < ActionController::TestCase
assert_response :forbidden

get :limited_with, params: { rate_limit_key: "other" }
get :limited_with
assert_response :ok
end

test "limited with" do
Expand Down

0 comments on commit 9c3ffab

Please sign in to comment.