From f8d2b8499231b13be418d96907d3d52dc3951954 Mon Sep 17 00:00:00 2001
From: Christian Galsterer <christian.galsterer@gmx.de>
Date: Sat, 21 Dec 2013 18:52:54 +0100
Subject: [PATCH] [JENKINS-21014] include verify in upstream candidate
 calculation

---
 src/main/java/hudson/maven/MavenModule.java   |  8 ++-
 .../maven/MavenSnapshotTriggerTest.java       | 59 +++++++++++++++++++
 2 files changed, 65 insertions(+), 2 deletions(-)

diff --git a/src/main/java/hudson/maven/MavenModule.java b/src/main/java/hudson/maven/MavenModule.java
index f39f02441..0349c6999 100644
--- a/src/main/java/hudson/maven/MavenModule.java
+++ b/src/main/java/hudson/maven/MavenModule.java
@@ -639,17 +639,21 @@ private int getDependencyRelevancy() {
         
         for (String goal : Util.tokenize(getGoals())) {
             if ("deploy".equals(goal) || "deploy:deploy".equals(goal)) {
-                return 2;
+                return 3;
             }
             
             if ("install".equals(goal)) {
+                relevancy = 2;
+            }
+
+            if ("verify".equals(goal)) {
                 relevancy = 1;
             }
         }
         
         for (Publisher publisher : getParent().getPublishers()) {
             if (publisher instanceof RedeployPublisher) {
-                return 2;
+                return 3;
             }
         }
         
diff --git a/src/test/java/hudson/maven/MavenSnapshotTriggerTest.java b/src/test/java/hudson/maven/MavenSnapshotTriggerTest.java
index b898e29b1..e95b77c0f 100644
--- a/src/test/java/hudson/maven/MavenSnapshotTriggerTest.java
+++ b/src/test/java/hudson/maven/MavenSnapshotTriggerTest.java
@@ -1,8 +1,15 @@
 package hudson.maven;
 
+import static org.junit.Assert.assertThat;
+import static org.hamcrest.CoreMatchers.hasItems;
+
+import hudson.model.AbstractProject;
 import org.jvnet.hudson.test.ExtractResourceSCM;
 import org.jvnet.hudson.test.HudsonTestCase;
 
+import java.util.ArrayList;
+import java.util.List;
+
 /**
  * Tests that Maven jobs are triggered, when snapshot dependencies of them were build.
  *
@@ -72,4 +79,56 @@ public void testMixedTransitiveSnapshotTrigger() throws Exception {
         assertEquals("Expected most recent build of second project to be #2", 2, projB.getLastBuild().getNumber());
         assertEquals("Expected most recent build of third project to be #2", 2, projC.getLastBuild().getNumber());
     }
+
+    /**
+     * Verifies SNAPSHOT dependency resolution of multiple SNAPSHOT upstream dependencies. (JENKINS-21014)
+     * Note - has to build the projects once each first in order to get dependency info.
+     * Artifact C depends on artifacts A and B.
+     * For artifact A there exists two projects. Project A1 has install and A1 has verify in its maven goal. A1 is selected as upstream build for artifact C due to higher ranking of install over verify.
+     * For artifact B there exists two projects. Projects B1 has  compile and B2 has verify in its maven goal. B2 is preferred as upstream dependency for artifact B as the verify goal takes precedence over the name tiebreaker which would prefer project B1.
+     */
+    public void testMultipleDependencySnapshotTrigger() throws Exception {
+        configureDefaultMaven();
+
+        MavenModuleSet projA1 = createMavenProject("snap-dep-test-A1");
+        projA1.setGoals("clean install");
+        projA1.setScm(new ExtractResourceSCM(getClass().getResource("maven-dep-test-A.zip")));
+
+        MavenModuleSet projA2 = createMavenProject("snap-dep-test-A2");
+        projA2.setGoals("clean verify");
+        projA2.setScm(new ExtractResourceSCM(getClass().getResource("maven-dep-test-A.zip")));
+
+        MavenModuleSet projB1 = createMavenProject("snap-dep-test-B1");
+        projB1.setGoals("clean compile");
+        projB1.setIgnoreUpstremChanges(false);
+        projB1.setQuietPeriod(0);
+        projB1.setScm(new ExtractResourceSCM(getClass().getResource("maven-dep-test-B.zip")));
+
+        MavenModuleSet projB2 = createMavenProject("snap-dep-test-B2");
+        projB2.setGoals("clean verify");
+        projB2.setIgnoreUpstremChanges(false);
+        projB2.setQuietPeriod(0);
+        projB2.setScm(new ExtractResourceSCM(getClass().getResource("maven-dep-test-B.zip")));
+
+        MavenModuleSet projC = createMavenProject("snap-dep-test-C");
+        projC.setGoals("clean compile");
+        projC.setIgnoreUpstremChanges(false);
+        projC.setQuietPeriod(0);
+        projC.setScm(new ExtractResourceSCM(getClass().getResource("maven-dep-test-C.zip")));
+
+        // Run all project at least once so that artifacts are known by Jenkins and SNAPSHOT dependencies are determined
+        buildAndAssertSuccess(projA1);
+        buildAndAssertSuccess(projA2);
+        buildAndAssertSuccess(projB1);
+        buildAndAssertSuccess(projB2);
+        buildAndAssertSuccess(projC);
+
+        final List<String> upstreamProjectNames = new ArrayList<String>();
+        for (AbstractProject project : projC.getUpstreamProjects()) {
+            upstreamProjectNames.add(project.getName());
+        }
+
+        assertEquals("Expected number of upstream project for project 'snap-dep-test-down' to be #2", 2, upstreamProjectNames.size());
+        assertThat("Expected 'snap-dep-test-A1' and 'snap-dep-test-B2' as upstream project for project 'snap-dep-test-C'", upstreamProjectNames, hasItems("snap-dep-test-A1", "snap-dep-test-B2"));
+    }
 }