Merge branch 'fix-upgrader-script' into 'master'

Fix upgrader script

This is a fix for upgrader script not guessing the latest version correctly.

Upgrader now handles versions where a version part (major/minor/patch) can have multi-digit number, also ensures that the latest version is chosen from git tags by converting tag to Gitlab::VersionInfo and than selecting the latest/greatest version.

Fixes: #1476

See merge request !695

1 files changed, 15 insertions, 0 deletions

diff --git a/spec/lib/gitlab/upgrader_spec.rb b/spec/lib/gitlab/upgrader_spec.rb
index ce3ea6c260a..baa4bd0f28f 100644
--- a/spec/lib/gitlab/upgrader_spec.rb
+++ b/spec/lib/gitlab/upgrader_spec.rb
@@ -20,5 +20,20 @@ describe Gitlab::Upgrader do
       upgrader.stub(current_version_raw: "5.3.0")
       expect(upgrader.latest_version_raw).to eq("v5.4.2")
     end
+
+    it 'should get the latest version from tags' do
+      allow(upgrader).to receive(:fetch_git_tags).and_return([
+        '6f0733310546402c15d3ae6128a95052f6c8ea96  refs/tags/v7.1.1',
+        'facfec4b242ce151af224e20715d58e628aa5e74  refs/tags/v7.1.1^{}',
+        'f7068d99c79cf79befbd388030c051bb4b5e86d4  refs/tags/v7.10.4',
+        '337225a4fcfa9674e2528cb6d41c46556bba9dfa  refs/tags/v7.10.4^{}',
+        '880e0ba0adbed95d087f61a9a17515e518fc6440  refs/tags/v7.11.1',
+        '6584346b604f981f00af8011cd95472b2776d912  refs/tags/v7.11.1^{}',
+        '43af3e65a486a9237f29f56d96c3b3da59c24ae0  refs/tags/v7.11.2',
+        'dac18e7728013a77410e926a1e64225703754a2d  refs/tags/v7.11.2^{}',
+        '0bf21fd4b46c980c26fd8c90a14b86a4d90cc950  refs/tags/v7.9.4',
+        'b10de29edbaff7219547dc506cb1468ee35065c3  refs/tags/v7.9.4^{}'])
+      expect(upgrader.latest_version_raw).to eq("v7.11.2")
+    end
   end
 end