mirror of
https://github.com/samsonjs/rack-attack.git
synced 2026-04-27 15:07:41 +00:00
Fix Rails test not being ran (#649)
This commit is contained in:
parent
cb82b9f873
commit
5124a172ac
5 changed files with 23 additions and 14 deletions
|
|
@ -12,9 +12,11 @@ describe "Cache store config when using allow2ban" do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "gives semantic error if no store was configured" do
|
unless defined?(Rails)
|
||||||
assert_raises(Rack::Attack::MissingStoreError) do
|
it "gives semantic error if no store was configured" do
|
||||||
get "/scarce-resource"
|
assert_raises(Rack::Attack::MissingStoreError) do
|
||||||
|
get "/scarce-resource"
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -12,9 +12,11 @@ describe "Cache store config when using fail2ban" do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "gives semantic error if no store was configured" do
|
unless defined?(Rails)
|
||||||
assert_raises(Rack::Attack::MissingStoreError) do
|
it "gives semantic error if no store was configured" do
|
||||||
get "/private-place"
|
assert_raises(Rack::Attack::MissingStoreError) do
|
||||||
|
get "/private-place"
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -9,9 +9,11 @@ describe "Cache store config when throttling without Rails" do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "gives semantic error if no store was configured" do
|
unless defined?(Rails)
|
||||||
assert_raises(Rack::Attack::MissingStoreError) do
|
it "gives semantic error if no store was configured" do
|
||||||
get "/", {}, "REMOTE_ADDR" => "1.2.3.4"
|
assert_raises(Rack::Attack::MissingStoreError) do
|
||||||
|
get "/", {}, "REMOTE_ADDR" => "1.2.3.4"
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -11,10 +11,12 @@ describe "Cache store config with Rails" do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "fails when Rails.cache is not set" do
|
unless defined?(Rails)
|
||||||
Object.stub_const(:Rails, OpenStruct.new(cache: nil)) do
|
it "fails when Rails.cache is not set" do
|
||||||
assert_raises(Rack::Attack::MissingStoreError) do
|
Object.stub_const(:Rails, OpenStruct.new(cache: nil)) do
|
||||||
get "/", {}, "REMOTE_ADDR" => "1.2.3.4"
|
assert_raises(Rack::Attack::MissingStoreError) do
|
||||||
|
get "/", {}, "REMOTE_ADDR" => "1.2.3.4"
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
||||||
|
|
@ -20,6 +20,7 @@ end
|
||||||
|
|
||||||
safe_require "connection_pool"
|
safe_require "connection_pool"
|
||||||
safe_require "dalli"
|
safe_require "dalli"
|
||||||
|
safe_require "rails"
|
||||||
safe_require "redis"
|
safe_require "redis"
|
||||||
safe_require "redis-store"
|
safe_require "redis-store"
|
||||||
|
|
||||||
|
|
@ -27,7 +28,7 @@ class Minitest::Spec
|
||||||
include Rack::Test::Methods
|
include Rack::Test::Methods
|
||||||
|
|
||||||
before do
|
before do
|
||||||
if Object.const_defined?(:Rails) && Rails.respond_to?(:cache)
|
if Object.const_defined?(:Rails) && Rails.respond_to?(:cache) && Rails.cache.respond_to?(:clear)
|
||||||
Rails.cache.clear
|
Rails.cache.clear
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue