diff options
author | Stan Hu <stanhu@gmail.com> | 2017-12-09 15:45:01 -0800 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2017-12-12 15:07:25 -0800 |
commit | 8b939bfab769810ba56f5f4ca18632fd7ae435db (patch) | |
tree | fcde1e78639873980fc6a0ef944df191445a4e30 /spec/lib/gitlab/action_rate_limiter_spec.rb | |
parent | ef78f67f4a2e19d204f5f4d4770649be1fe7bee9 (diff) | |
download | gitlab-ce-8b939bfab769810ba56f5f4ca18632fd7ae435db.tar.gz |
Add spec for ActionRateLimiter
Diffstat (limited to 'spec/lib/gitlab/action_rate_limiter_spec.rb')
-rw-r--r-- | spec/lib/gitlab/action_rate_limiter_spec.rb | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/spec/lib/gitlab/action_rate_limiter_spec.rb b/spec/lib/gitlab/action_rate_limiter_spec.rb new file mode 100644 index 00000000000..84661605623 --- /dev/null +++ b/spec/lib/gitlab/action_rate_limiter_spec.rb @@ -0,0 +1,27 @@ +require 'spec_helper' + +describe Gitlab::ActionRateLimiter do + let(:redis) { double('redis') } + let(:key) { 'user:1' } + let(:cache_key) { "action_rate_limiter:test_action:#{key}" } + + subject { described_class.new(action: :test_action, expiry_time: 100) } + + before do + allow(Gitlab::Redis::Cache).to receive(:with).and_yield(redis) + end + + it 'increases the throttle count and sets the expire time' do + expect(redis).to receive(:incr).with(cache_key).and_return(1) + expect(redis).to receive(:expire).with(cache_key, 100) + + expect(subject.throttled?(key, 1)).to be false + end + + it 'returns true if the key is throttled' do + expect(redis).to receive(:incr).with(cache_key).and_return(2) + expect(redis).not_to receive(:expire) + + expect(subject.throttled?(key, 1)).to be true + end +end |