[FIXED JENKINS-25788] RunMap.remove was always returning false, even when the...

[FIXED JENKINS-25788] RunMap.remove was always returning false, even when the Run was in fact removed.

changelog.html

@@ -55,6 +55,9 @@ Upcoming changes</a>
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
+  <li class=bug>
+    Spurious warnings in the log after deleting builds.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-25788">issue 25788</a>)
   <li class=bug>
     Master labels disappear when system configuration is updated.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-23966">issue 23966</a>)

core/src/main/java/jenkins/model/lazy/AbstractLazyLoadRunMap.java

@@ -746,7 +746,7 @@ public abstract class AbstractLazyLoadRunMap<R> extends AbstractMap<Integer,R> i
 
         entrySet.clearCache();
 
-        return unwrap(old)!=null;
+        return old != null;
     }
 
     /**

test/src/test/java/hudson/model/RunMapTest.java

@@ -9,9 +9,6 @@ import org.jvnet.hudson.test.Issue;
 import org.jvnet.hudson.test.JenkinsRule;
 import org.jvnet.hudson.test.SleepBuilder;
 
-/**
- * @author Kohsuke Kawaguchi
- */
 public class RunMapTest {
 
     @Rule public JenkinsRule r = new JenkinsRule();
@@ -73,4 +70,21 @@ public class RunMapTest {
 
     private static boolean bombed;
 
+    @Issue("JENKINS-25788")
+    @Test public void remove() throws Exception {
+        FreeStyleProject p = r.createFreeStyleProject();
+        FreeStyleBuild b1 = r.buildAndAssertSuccess(p);
+        FreeStyleBuild b2 = r.buildAndAssertSuccess(p);
+        RunMap<FreeStyleBuild> runs = p._getRuns();
+        assertEquals(2, runs.size());
+        assertTrue(runs.remove(b1));
+        assertEquals(1, runs.size());
+        assertFalse(runs.remove(b1));
+        assertEquals(1, runs.size());
+        assertTrue(runs.remove(b2));
+        assertEquals(0, runs.size());
+        assertFalse(runs.remove(b2));
+        assertEquals(0, runs.size());
+    }
+
 }