Merge pull request #13628 from vespa-engine/mpolden/fix-log-message

Log correct dispatched requests

1 files changed, 9 insertions, 1 deletions

diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/dns/NameServiceQueueTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/dns/NameServiceQueueTest.java
index d0362ae98b8..30ed9b5432c 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/dns/NameServiceQueueTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/dns/NameServiceQueueTest.java
@@ -74,13 +74,21 @@ public class NameServiceQueueTest {
         assertTrue(queue.requests().isEmpty());
         assertTrue("Removed " + r1, nameService.findRecords(Record.Type.CNAME, r1.name()).isEmpty());
 
-        // Keep n most recent requests
+        // Keep n last requests
         queue = queue.with(req1).with(req2).with(req3).with(req4).with(req6)
                      .last(2);
         assertEquals(List.of(req4, req6), List.copyOf(queue.requests()));
         assertSame(queue, queue.last(2));
         assertSame(queue, queue.last(10));
         assertTrue(queue.last(0).requests().isEmpty());
+
+        // Keep n first requests
+        queue = NameServiceQueue.EMPTY.with(req1).with(req2).with(req3).with(req4).with(req6)
+                                      .first(3);
+        assertEquals(List.of(req1, req2, req3), List.copyOf(queue.requests()));
+        assertSame(queue, queue.first(3));
+        assertSame(queue, queue.first(10));
+        assertTrue(queue.first(0).requests().isEmpty());
     }
 
 }