diff options
author | Dylan Griffith <dyl.griffith@gmail.com> | 2018-02-21 08:27:05 +0300 |
---|---|---|
committer | Dylan Griffith <dyl.griffith@gmail.com> | 2018-02-21 08:27:05 +0300 |
commit | 2e216dd43bb02d741b2fcbfc1fe40042ad85a590 (patch) | |
tree | 061a3b26283b22f0aee7bc960a9f7ae1440f72ec /spec/lib/gitlab/query_limiting | |
parent | 11bf575fe64e55cc932c5629e2d8d103109e0b2b (diff) |
Do not count rails sql cache as queries in query limiting
Diffstat (limited to 'spec/lib/gitlab/query_limiting')
-rw-r--r-- | spec/lib/gitlab/query_limiting/active_support_subscriber_spec.rb | 31 |
1 files changed, 23 insertions, 8 deletions
diff --git a/spec/lib/gitlab/query_limiting/active_support_subscriber_spec.rb b/spec/lib/gitlab/query_limiting/active_support_subscriber_spec.rb index b49bc5c328c..f8faeffb935 100644 --- a/spec/lib/gitlab/query_limiting/active_support_subscriber_spec.rb +++ b/spec/lib/gitlab/query_limiting/active_support_subscriber_spec.rb @@ -1,19 +1,34 @@ require 'spec_helper' describe Gitlab::QueryLimiting::ActiveSupportSubscriber do + let(:transaction) { instance_double(Gitlab::QueryLimiting::Transaction, increment: true) } + + before do + allow(Gitlab::QueryLimiting::Transaction) + .to receive(:current) + .and_return(transaction) + end + describe '#sql' do it 'increments the number of executed SQL queries' do - transaction = double(:transaction) - - allow(Gitlab::QueryLimiting::Transaction) - .to receive(:current) - .and_return(transaction) + User.count expect(transaction) - .to receive(:increment) - .at_least(:once) + .to have_received(:increment) + .once + end - User.count + context 'when the query is actually a rails cache hit' do + it 'does not increment the number of executed SQL queries' do + ActiveRecord::Base.connection.cache do + User.count + User.count + end + + expect(transaction) + .to have_received(:increment) + .once + end end end end |