diff options
author | Stan Hu <stanhu@gmail.com> | 2019-07-03 17:29:36 +0000 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-07-03 17:29:36 +0000 |
commit | 4e456d32e0b3a40e605cbb71154e5d92b551f81e (patch) | |
tree | bfb1ba171078c2f2a298ebbea4a198e9f09e724a /spec | |
parent | ea2f2ab0c7386521aa7c884f9ffcab0bd61ff87b (diff) | |
parent | d0b76d065289b50a14b151f45fbb2718e8a50f86 (diff) | |
download | gitlab-ce-4e456d32e0b3a40e605cbb71154e5d92b551f81e.tar.gz |
Merge branch 'cache-performance_bar_allowed_user_ids-locally' into 'master'
Cache PerformanceBar.allowed_user_ids list locally instead of in Redis
See merge request gitlab-org/gitlab-ce!30304
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/gitlab/performance_bar_spec.rb | 27 |
1 files changed, 26 insertions, 1 deletions
diff --git a/spec/lib/gitlab/performance_bar_spec.rb b/spec/lib/gitlab/performance_bar_spec.rb index f480376acb4..ee3c571c9c0 100644 --- a/spec/lib/gitlab/performance_bar_spec.rb +++ b/spec/lib/gitlab/performance_bar_spec.rb @@ -3,17 +3,42 @@ require 'spec_helper' describe Gitlab::PerformanceBar do shared_examples 'allowed user IDs are cached' do before do - # Warm the Redis cache + # Warm the caches described_class.enabled?(user) end it 'caches the allowed user IDs in cache', :use_clean_rails_memory_store_caching do expect do + expect(described_class.l1_cache_backend).to receive(:fetch).and_call_original + expect(described_class.l2_cache_backend).not_to receive(:fetch) expect(described_class.enabled?(user)).to be_truthy end.not_to exceed_query_limit(0) end + + it 'caches the allowed user IDs in L1 cache for 1 minute', :use_clean_rails_memory_store_caching do + Timecop.travel 2.minutes do + expect do + expect(described_class.l1_cache_backend).to receive(:fetch).and_call_original + expect(described_class.l2_cache_backend).to receive(:fetch).and_call_original + expect(described_class.enabled?(user)).to be_truthy + end.not_to exceed_query_limit(0) + end + end + + it 'caches the allowed user IDs in L2 cache for 5 minutes', :use_clean_rails_memory_store_caching do + Timecop.travel 6.minutes do + expect do + expect(described_class.l1_cache_backend).to receive(:fetch).and_call_original + expect(described_class.l2_cache_backend).to receive(:fetch).and_call_original + expect(described_class.enabled?(user)).to be_truthy + end.not_to exceed_query_limit(2) + end + end end + it { expect(described_class.l1_cache_backend).to eq(Gitlab::ThreadMemoryCache.cache_backend) } + it { expect(described_class.l2_cache_backend).to eq(Rails.cache) } + describe '.enabled?' do let(:user) { create(:user) } |