[JENKINS-38794] make @WithPlugins check required version when 'upload…

…Plugins=false'
jenkinsci · Oct 6, 2016 · 1d64704 · 1d64704
1 parent 8b79df4
commit 1d64704
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/src/main/java/org/jenkinsci/test/acceptance/po/PluginManager.java b/src/main/java/org/jenkinsci/test/acceptance/po/PluginManager.java
@@ -148,6 +148,12 @@ public boolean isInstalled(String... specs) {
         return true;
     }
 
+    private VersionNumber getAvailableVersionForPlugin(String pluginName) {
+        // assuming we are on 'available' page
+        String v = find(by.xpath("//input[starts-with(@name,'plugin.%s.')]/../../td[last()]", pluginName)).getText();
+        return new VersionNumber(v);
+    }
+
     /**
      * Installs specified plugins.
      *
@@ -224,6 +230,13 @@ public boolean installPlugins(final String... specs) throws UnableToResolveDepen
                 for (int attempt = 0; attempt < 2; attempt++) {// # of installations attempted, considering retries
                     visit("available");
                     check(find(by.xpath("//input[starts-with(@name,'plugin.%s.')]", n)));
+                    VersionNumber availableVersion = getAvailableVersionForPlugin(n);
+                    VersionNumber requiredVersion = new VersionNumber(candidates.get(n));
+                    if (availableVersion.isOlderThan(requiredVersion)) {
+                        throw new AssumptionViolatedException(
+                                String.format("Version '%s' of '%s' is required, but available version is '%s'", 
+                                        requiredVersion, n, availableVersion));
+                    }
 
                     clickButton("Install");