From 83d0e649b2c77283f5be3abac4bd3296d6f46f9a Mon Sep 17 00:00:00 2001
From: Morten Tokle <mortent@yahooinc.com>
Date: Wed, 13 Dec 2023 14:48:50 +0100
Subject: Propagate more useful error message to vespa log

---
 .../yahoo/vespa/config/server/MockSecretStore.java |  3 +-
 .../tenant/SecretStoreExternalIdRetrieverTest.java | 39 ++++++++++++++++++++++
 2 files changed, 41 insertions(+), 1 deletion(-)
 create mode 100644 configserver/src/test/java/com/yahoo/vespa/config/server/tenant/SecretStoreExternalIdRetrieverTest.java

(limited to 'configserver/src/test/java/com/yahoo')

diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/MockSecretStore.java b/configserver/src/test/java/com/yahoo/vespa/config/server/MockSecretStore.java
index bd469cb8f0b..ac97fe0ba05 100644
--- a/configserver/src/test/java/com/yahoo/vespa/config/server/MockSecretStore.java
+++ b/configserver/src/test/java/com/yahoo/vespa/config/server/MockSecretStore.java
@@ -1,6 +1,7 @@
 // Copyright Vespa.ai. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.config.server;
 
+import com.yahoo.container.jdisc.secretstore.SecretNotFoundException;
 import com.yahoo.container.jdisc.secretstore.SecretStore;
 
 import java.util.HashMap;
@@ -13,7 +14,7 @@ public class MockSecretStore implements SecretStore {
     public String getSecret(String key) {
         if(secrets.containsKey(key))
             return secrets.get(key).get(0);
-        throw new RuntimeException("Key not found: " + key);
+        throw new SecretNotFoundException("Key not found: " + key);
     }
 
     @Override
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/SecretStoreExternalIdRetrieverTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/SecretStoreExternalIdRetrieverTest.java
new file mode 100644
index 00000000000..96c7d9e6957
--- /dev/null
+++ b/configserver/src/test/java/com/yahoo/vespa/config/server/tenant/SecretStoreExternalIdRetrieverTest.java
@@ -0,0 +1,39 @@
+// Copyright Vespa.ai. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
+
+package com.yahoo.vespa.config.server.tenant;
+
+import com.yahoo.config.model.api.TenantSecretStore;
+import com.yahoo.config.provision.SystemName;
+import com.yahoo.config.provision.TenantName;
+import com.yahoo.vespa.config.server.MockSecretStore;
+import com.yahoo.vespa.config.server.http.InvalidApplicationException;
+import org.junit.jupiter.api.Test;
+
+import java.util.List;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+/**
+ * @author mortent
+ */
+public class SecretStoreExternalIdRetrieverTest {
+    private final MockSecretStore secretStore = new MockSecretStore();
+    private final TenantName tenantName = TenantName.from("myTenant");
+    private final TenantSecretStore tenantSecretStore = new TenantSecretStore("name", "123456789012", "role");
+
+    @Test
+    public void fills_external_ids() {
+        secretStore.put(SecretStoreExternalIdRetriever.secretName(tenantName, SystemName.PublicCd, "name"), "externalId");
+
+        List<TenantSecretStore> tenantSecretStores = SecretStoreExternalIdRetriever.populateExternalId(secretStore, tenantName, SystemName.PublicCd, List.of(tenantSecretStore));
+        assertEquals(1, tenantSecretStores.size());
+        assertEquals("externalId", tenantSecretStores.get(0).getExternalId().get());
+    }
+
+    @Test
+    public void reports_application_package_error_when_external_id_not_found() {
+        InvalidApplicationException exception = assertThrows(InvalidApplicationException.class, () -> SecretStoreExternalIdRetriever.populateExternalId(secretStore, tenantName, SystemName.PublicCd, List.of(tenantSecretStore)));
+        assertEquals("Could not find externalId for secret store: name", exception.getMessage());
+    }
+}
-- 
cgit v1.2.3