summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Marius Venstad <venstad@gmail.com>2022-03-31 15:18:49 +0200
committerJon Marius Venstad <venstad@gmail.com>2022-03-31 15:18:49 +0200
commitc2293d362552d0e304944c45279ce261326abc41 (patch)
tree0d550b0358e0957e2450dd0d953c2bfa922afcb2
parent9765bf389468f02f328f5cd80f6b8b9b61d47027 (diff)
Fix tests
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/CuratorDatabaseClient.java5
-rw-r--r--node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java3
-rw-r--r--zkfacade/src/main/java/com/yahoo/vespa/curator/mock/MockCurator.java2
3 files changed, 7 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/CuratorDatabaseClient.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/CuratorDatabaseClient.java
index 379bb2566df..a9abc352d8c 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/CuratorDatabaseClient.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/persistence/CuratorDatabaseClient.java
@@ -43,8 +43,8 @@ import java.util.logging.Logger;
import java.util.stream.Collectors;
import java.util.stream.IntStream;
+import static com.yahoo.stream.CustomCollectors.toLinkedMap;
import static java.util.stream.Collectors.collectingAndThen;
-import static java.util.stream.Collectors.toMap;
/**
* Client which reads and writes nodes to a curator database.
@@ -453,7 +453,8 @@ public class CuratorDatabaseClient {
.map(this::readLoadBalancer)
.filter(Optional::isPresent)
.map(Optional::get)
- .collect(collectingAndThen(toMap(LoadBalancer::id, Function.identity()),
+ .collect(collectingAndThen(toLinkedMap(LoadBalancer::id,
+ Function.identity()),
Collections::unmodifiableMap));
}
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java
index cc121ba8104..48ee23c7b60 100644
--- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java
+++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/persistence/NodeSerializerTest.java
@@ -334,6 +334,9 @@ public class NodeSerializerTest {
" \"hostname\" : \"myHostname\",\n" +
" \"ipAddresses\" : [\"127.0.0.1\"],\n" +
" \"instance\": {\n" +
+ " \"tenantId\":\"t\",\n" +
+ " \"applicationId\":\"a\",\n" +
+ " \"instanceId\":\"i\",\n" +
" \"serviceId\": \"content/myId/0/0/stateful\",\n" +
" \"wantedVespaVersion\": \"6.42.2\"\n" +
" }\n" +
diff --git a/zkfacade/src/main/java/com/yahoo/vespa/curator/mock/MockCurator.java b/zkfacade/src/main/java/com/yahoo/vespa/curator/mock/MockCurator.java
index bd3389b8d4d..7c91e54dd4b 100644
--- a/zkfacade/src/main/java/com/yahoo/vespa/curator/mock/MockCurator.java
+++ b/zkfacade/src/main/java/com/yahoo/vespa/curator/mock/MockCurator.java
@@ -36,7 +36,7 @@ public class MockCurator extends Curator {
/**
* Creates a mock curator
*
- * @param stableOrdering if true children of a node are returned in the same order each time they are queries.
+ * @param stableOrdering if true children of a node are returned in the same order each time they are queried.
* This is not what ZooKeeper does.
*/
public MockCurator(boolean stableOrdering) {