return -1 and tesT

jmock-developers · Mar 27, 2019 · ea908f8 · ea908f8
1 parent a8bd585
commit ea908f8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/jmock/src/main/java/org/jmock/lib/concurrent/internal/DeltaQueue.java b/jmock/src/main/java/org/jmock/lib/concurrent/internal/DeltaQueue.java
@@ -42,7 +42,7 @@ public long delay(T element) {
             next = next.next;
         }
         if (next == null) {
-            return 0;
+            return -1;
         }
         return ret;
     }

diff --git a/jmock/src/test/java/org/jmock/test/unit/lib/concurrent/internal/DeltaQueueTests.java b/jmock/src/test/java/org/jmock/test/unit/lib/concurrent/internal/DeltaQueueTests.java
@@ -167,4 +167,11 @@ public void testReturnsFalseIfElementAlreadyRemoved() {
 
         assertFalse(deltaQueue.remove(elementC));
     }
+
+    public void testDelayIsMinusOneWhenElementIsAlreadyRemoved() {
+        deltaQueue.add(1L, elementA);
+        deltaQueue.remove(elementA);
+
+        assertEquals("delay", -1, deltaQueue.delay(elementA));
+    }
 }