diff options
author | Yorick Peterse <yorickpeterse@gmail.com> | 2018-05-17 22:20:15 +0300 |
---|---|---|
committer | Yorick Peterse <yorickpeterse@gmail.com> | 2018-05-17 22:23:42 +0300 |
commit | 63c58a6dd0d8a4b3db172ffe6d0e32e127cdabd1 (patch) | |
tree | 0f97dd7a71248c75528ea814c1814b28f18d29c8 /spec/lib/gitlab/database_spec.rb | |
parent | 3a402fc7fc1dd0b0397f8cdd8ed73336ac409281 (diff) |
Memoize Gitlab::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.rb | 14 |
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 |