Reinstate the released lock messages for freestyle builds (removed by j…

…enkinsci#673)

src/main/java/org/jenkins/plugins/lockableresources/LockableResourcesManager.java

@@ -643,19 +643,27 @@ private void freeResources(List<LockableResource> unlockResources, Run<?, ?> bui
         removeResources(toBeRemoved);
     }
 
-    public void unlockBuild(@Nullable Run<?, ?> build) {
+    @NonNull
+    public List<String> unlockBuild2(@Nullable Run<?, ?> build) {
 
         if (build == null) {
-            return;
+            return Collections.emptyList();
         }
 
-        List<String> resourcesInUse =
-                LockedResourcesBuildAction.findAndInitAction(build).getCurrentUsedResourceNames();
+        List<String> resourcesInUse = new ArrayList<>(
+                LockedResourcesBuildAction.findAndInitAction(build).getCurrentUsedResourceNames());
 
-        if (resourcesInUse.size() == 0) {
-            return;
+        if (!resourcesInUse.isEmpty()) {
+            unlockNames(resourcesInUse, build);
         }
-        unlockNames(resourcesInUse, build);
+        return resourcesInUse;
+    }
+
+    /** @deprecated Use unlockBuild2 */
+    @Deprecated
+    public void unlockBuild(@Nullable Run<?, ?> build) {
+
+        unlockBuild2(build);
     }
 
     // ---------------------------------------------------------------------------

src/main/java/org/jenkins/plugins/lockableresources/queue/LockRunListener.java

@@ -97,16 +97,29 @@ public void onCompleted(Run<?, ?> build, @NonNull TaskListener listener) {
         // Skip unlocking for multiple configuration projects,
         // only the child jobs will actually unlock resources.
         if (build instanceof MatrixBuild) return;
-        LOGGER.info(build.getFullDisplayName());
-        LockableResourcesManager.get().unlockBuild(build);
+
+        List<String> unlocked = LockableResourcesManager.get().unlockBuild2(build);
+        if (!unlocked.isEmpty()) {
+            listener.getLogger().printf("%s released lock on %s%n", LOG_PREFIX, unlocked);
+            LOGGER.info(build.getFullDisplayName()
+                + " released lock on "
+                + unlocked
+                + ", because the build has finished.");
+        }
     }
 
     @Override
     public void onDeleted(Run<?, ?> build) {
         // Skip unlocking for multiple configuration projects,
         // only the child jobs will actually unlock resources.
         if (build instanceof MatrixBuild) return;
-        LOGGER.info(build.getFullDisplayName());
-        LockableResourcesManager.get().unlockBuild(build);
+
+        List<String> unlocked = LockableResourcesManager.get().unlockBuild2(build);
+        if (!unlocked.isEmpty()) {
+            LOGGER.warning(build.getFullDisplayName()
+                + " released lock on "
+                + unlocked
+                + ", because the build has been deleted.");
+        }
     }
 }

src/test/java/org/jenkins/plugins/lockableresources/FreeStyleProjectTest.java

@@ -260,6 +260,7 @@ public boolean perform(AbstractBuild<?, ?> build, Launcher launcher, BuildListen
         QueueTaskFuture<FreeStyleBuild> fb2q = f2.scheduleBuild2(0);
 
         semaphore.release();
+        j.waitForMessage("released lock on [shared]", fb0);
         j.waitForCompletion(fb0);
         // fb1 or fb2 might run first, it shouldn't matter as long as they both get the resource
         FreeStyleBuild fb1 = fb1q.waitForStart();

src/test/java/org/jenkins/plugins/lockableresources/NodesMirrorTest.java

@@ -16,7 +16,7 @@
 
 public class NodesMirrorTest {
 
-    private static final Logger LOGGER = Logger.getLogger(NodesMirror.class.getName());
+    private static final Logger LOGGER = Logger.getLogger(NodesMirrorTest.class.getName());
 
     @Rule
     public final JenkinsRule j = new JenkinsRule();