From 502bf468d952c2d61668f35bce4d8322b5d2fb84 Mon Sep 17 00:00:00 2001
From: Øyvind Grønnesby <oyving@verizonmedia.com>
Date: Fri, 31 May 2019 13:01:41 +0200
Subject: Have Endpoint use RegionName instead of a String

---
 .../main/java/com/yahoo/config/application/api/Endpoint.java | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

(limited to 'config-model-api')

diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/Endpoint.java b/config-model-api/src/main/java/com/yahoo/config/application/api/Endpoint.java
index b505bf72faf..73328027540 100644
--- a/config-model-api/src/main/java/com/yahoo/config/application/api/Endpoint.java
+++ b/config-model-api/src/main/java/com/yahoo/config/application/api/Endpoint.java
@@ -1,8 +1,11 @@
 package com.yahoo.config.application.api;
 
+import com.yahoo.config.provision.RegionName;
+
 import java.util.Objects;
 import java.util.Optional;
 import java.util.Set;
+import java.util.stream.Collectors;
 
 /**
  * Represents a (global) endpoint in 'deployments.xml'.  It contains the name of the
@@ -14,12 +17,15 @@ import java.util.Set;
 public class Endpoint {
     private final Optional<String> endpointId;
     private final String containerId;
-    private final Set<String> regions;
+    private final Set<RegionName> regions;
 
     public Endpoint(Optional<String> endpointId, String containerId, Set<String> regions) {
         this.endpointId = endpointId;
         this.containerId = containerId;
-        this.regions = Set.copyOf(regions);
+        this.regions = Set.copyOf(
+                Objects.requireNonNull(
+                        regions.stream().map(RegionName::from).collect(Collectors.toList()),
+                        "Missing 'regions' parameter"));
     }
 
     public String endpointId() {
@@ -30,7 +36,7 @@ public class Endpoint {
         return containerId;
     }
 
-    public Set<String> regions() {
+    public Set<RegionName> regions() {
         return regions;
     }
 
-- 
cgit v1.2.3