From 0f9fae4861e86ec49eca968b71dc37e372d8e6a7 Mon Sep 17 00:00:00 2001
From: toby <smorgrav@yahoo-inc.com>
Date: Tue, 29 Aug 2017 10:47:27 +0200
Subject: Add unit tests to preferred re-allocation sorter

---
 .../provisioning/NodePrioritizerTest.java          | 86 ++++++++++++++++++++++
 1 file changed, 86 insertions(+)
 create mode 100644 node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java

diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java
new file mode 100644
index 00000000000..407101c7b8b
--- /dev/null
+++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/NodePrioritizerTest.java
@@ -0,0 +1,86 @@
+package com.yahoo.vespa.hosted.provision.provisioning;// Copyright 2017 Yahoo Holdings. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+
+import com.yahoo.component.Version;
+import com.yahoo.config.provision.ApplicationId;
+import com.yahoo.config.provision.ClusterMembership;
+import com.yahoo.config.provision.ClusterSpec;
+import com.yahoo.config.provision.Flavor;
+import com.yahoo.config.provision.NodeFlavors;
+import com.yahoo.config.provision.NodeType;
+import com.yahoo.config.provisioning.FlavorsConfig;
+import com.yahoo.vespa.hosted.provision.Node;
+import org.junit.Assert;
+import org.junit.Test;
+
+import java.time.Instant;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Optional;
+
+/**
+ * @author smorgrav
+ */
+public class NodePrioritizerTest {
+
+    private static NodeFlavors flavors = new NodeFlavors(flavorsConfig());
+
+    @Test
+    public void relocated_nodes_are_preferred() {
+        List<Node> nodes = new ArrayList<>();
+        Node parent = createParent("parent");
+        Node b = createNode(parent, "b", "d2");
+        nodes.add(b);
+
+        // Only one node - should be obvious what to prefer
+        Assert.assertTrue(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, b, parent));
+
+        // Two equal nodes - choose lexically
+        Node a = createNode(parent, "a", "d2");
+        nodes.add(a);
+        Assert.assertTrue(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, a, parent));
+        Assert.assertFalse(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, b, parent));
+
+        // Smallest node should be preferred
+        Node c = createNode(parent, "c", "d1");
+        nodes.add(c);
+        Assert.assertTrue(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, c, parent));
+
+        // Unallocated over allocated
+        ClusterSpec spec = ClusterSpec.from(ClusterSpec.Type.container, ClusterSpec.Id.from("mycluster"), ClusterSpec.Group.from(0), Version.fromString("6.142.22"));
+        c = c.allocate(ApplicationId.defaultId(), ClusterMembership.from(spec, 0), Instant.now());
+        nodes.remove(c);
+        nodes.add(c);
+        Node d = createNode(parent, "d", "d1");
+        nodes.add(d);
+        Assert.assertTrue(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, d, parent));
+        Assert.assertFalse(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, c, parent));
+
+        // Container over content
+        ClusterSpec spec2 = ClusterSpec.from(ClusterSpec.Type.content, ClusterSpec.Id.from("mycluster"), ClusterSpec.Group.from(0), Version.fromString("6.142.22"));
+        d = d.allocate(ApplicationId.defaultId(), ClusterMembership.from(spec, 0), Instant.now());
+        nodes.remove(d);
+        nodes.add(d);
+        Assert.assertTrue(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, c, parent));
+        Assert.assertFalse(NodePrioritizer.isPreferredNodeToBeReloacted(nodes, d, parent));
+    }
+
+    private static Node createNode(Node parent, String hostname, String flavor) {
+        return Node.createDockerNode("openid", Collections.singleton("127.0.0.1"), new HashSet<>(), hostname, Optional.of(parent.hostname()),
+                flavors.getFlavorOrThrow(flavor), NodeType.tenant);
+    }
+
+    private static Node createParent(String hostname) {
+        return Node.create("openid", Collections.singleton("127.0.0.1"), new HashSet<>(), hostname, Optional.empty(),
+                flavors.getFlavorOrThrow("host-large"), NodeType.host);
+    }
+
+    private static FlavorsConfig flavorsConfig() {
+        FlavorConfigBuilder b = new FlavorConfigBuilder();
+        b.addFlavor("host-large", 6., 6., 6, Flavor.Type.BARE_METAL);
+        b.addFlavor("d1", 1, 1., 1, Flavor.Type.DOCKER_CONTAINER);
+        b.addFlavor("d2", 2, 2., 2, Flavor.Type.DOCKER_CONTAINER);
+        return b.build();
+    }
+}
-- 
cgit v1.2.3