summaryrefslogtreecommitdiff
path: root/spec/lib/gitlab/database_spec.rb
diff options
context:
space:
mode:
authorYorick Peterse <yorickpeterse@gmail.com>2018-05-17 21:20:15 +0200
committerYorick Peterse <yorickpeterse@gmail.com>2018-05-17 21:23:42 +0200
commit63c58a6dd0d8a4b3db172ffe6d0e32e127cdabd1 (patch)
tree0f97dd7a71248c75528ea814c1814b28f18d29c8 /spec/lib/gitlab/database_spec.rb
parent3a402fc7fc1dd0b0397f8cdd8ed73336ac409281 (diff)
downloadgitlab-ce-63c58a6dd0d8a4b3db172ffe6d0e32e127cdabd1.tar.gz
Memoize Gitlab::Database.versionmemoize-database-version
This removes the need for running a database query every time we want to check the database version.
Diffstat (limited to 'spec/lib/gitlab/database_spec.rb')
-rw-r--r--spec/lib/gitlab/database_spec.rb14
1 files changed, 14 insertions, 0 deletions
diff --git a/spec/lib/gitlab/database_spec.rb b/spec/lib/gitlab/database_spec.rb
index 1fe1d3926ad..8ac36ae8bab 100644
--- a/spec/lib/gitlab/database_spec.rb
+++ b/spec/lib/gitlab/database_spec.rb
@@ -32,6 +32,12 @@ describe Gitlab::Database do
end
describe '.version' do
+ around do |example|
+ described_class.instance_variable_set(:@version, nil)
+ example.run
+ described_class.instance_variable_set(:@version, nil)
+ end
+
context "on mysql" do
it "extracts the version number" do
allow(described_class).to receive(:database_version)
@@ -49,6 +55,14 @@ describe Gitlab::Database do
expect(described_class.version).to eq '9.4.4'
end
end
+
+ it 'memoizes the result' do
+ count = ActiveRecord::QueryRecorder
+ .new { 2.times { described_class.version } }
+ .count
+
+ expect(count).to eq(1)
+ end
end
describe '.join_lateral_supported?' do