summaryrefslogtreecommitdiffstats
path: root/service-monitor
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2024-04-12 08:30:35 +0200
committerHenning Baldersheim <balder@yahoo-inc.com>2024-04-12 08:39:04 +0200
commit76a89b62274060452022ddf24a7685ee2f380cb4 (patch)
treeef924603de22efd026f519ab31fd8f5a6ff60f2f /service-monitor
parent7e7ebf7b527be1f163d497a41898e2252d878fe7 (diff)
Replace all usages of Arrays.asList with List.of where possible.
Diffstat (limited to 'service-monitor')
-rw-r--r--service-monitor/src/main/java/com/yahoo/vespa/service/monitor/ServiceModel.java3
-rw-r--r--service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java4
-rw-r--r--service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java3
3 files changed, 4 insertions, 6 deletions
diff --git a/service-monitor/src/main/java/com/yahoo/vespa/service/monitor/ServiceModel.java b/service-monitor/src/main/java/com/yahoo/vespa/service/monitor/ServiceModel.java
index 5bafc7ac4bd..7d89b86c07b 100644
--- a/service-monitor/src/main/java/com/yahoo/vespa/service/monitor/ServiceModel.java
+++ b/service-monitor/src/main/java/com/yahoo/vespa/service/monitor/ServiceModel.java
@@ -9,7 +9,6 @@ import com.yahoo.vespa.applicationmodel.ServiceCluster;
import com.yahoo.vespa.applicationmodel.ServiceInstance;
import java.util.ArrayList;
-import java.util.Arrays;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
@@ -74,7 +73,7 @@ public class ServiceModel {
ApplicationInstance previous = applicationInstances.put(instance.hostName(), application);
if (previous != null && !previous.equals(application)) {
throw new IllegalStateException("Major assumption broken: Multiple application instances contain host " +
- instance.hostName().s() + ": " + Arrays.asList(previous, application));
+ instance.hostName().s() + ": " + List.of(previous, application));
}
serviceInstances
diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java
index a5b8054536b..00c15add762 100644
--- a/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java
+++ b/service-monitor/src/test/java/com/yahoo/vespa/service/duper/DuperModelTest.java
@@ -66,7 +66,7 @@ public class DuperModelTest {
duperModel.add(application1);
assertEquals(Optional.of(application1), duperModel.getApplicationInfo(id1));
- assertEquals(Arrays.asList(application1), duperModel.getApplicationInfos());
+ assertEquals(List.of(application1), duperModel.getApplicationInfos());
assertEquals(1, duperModel.numberOfApplications());
duperModel.registerListener(listener1);
@@ -86,7 +86,7 @@ public class DuperModelTest {
assertEquals(Optional.empty(), duperModel.getApplicationInfo(id1));
verify(listener1, times(1)).applicationRemoved(id1);
verifyNoMoreInteractions(listener1);
- assertEquals(Arrays.asList(application2), duperModel.getApplicationInfos());
+ assertEquals(List.of(application2), duperModel.getApplicationInfos());
duperModel.remove(id1);
verifyNoMoreInteractions(listener1);
diff --git a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java
index bef87a627f5..9c0c94cef27 100644
--- a/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java
+++ b/service-monitor/src/test/java/com/yahoo/vespa/service/model/ExampleModel.java
@@ -12,7 +12,6 @@ import com.yahoo.config.provision.InstanceName;
import com.yahoo.vespa.service.slobrok.SlobrokMonitor;
import java.util.ArrayList;
-import java.util.Arrays;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -151,7 +150,7 @@ public class ExampleModel {
* A bit unrealistic, but the port is the same on all hosts.
*/
ClusterBuilder addPort(int port, String... tags) {
- portInfos.add(new PortInfo(port, Arrays.asList(tags)));
+ portInfos.add(new PortInfo(port, List.of(tags)));
return this;
}