From 822f0866b4fb0aea3146b0230272c26a02bb0a4e Mon Sep 17 00:00:00 2001
From: dotasek <david.otasek@smilecdr.com>
Date: Thu, 7 Nov 2024 18:07:23 -0500
Subject: [PATCH] Fix api-key token mixup, more tests

---
 .../http/ManagedWebAccessAuthTests.java       | 65 ++++++++++++++++---
 .../http/ManagedFhirWebAccessor.java          |  2 +-
 2 files changed, 57 insertions(+), 10 deletions(-)

diff --git a/org.hl7.fhir.r5/src/test/java/org/hl7/fhir/utilities/http/ManagedWebAccessAuthTests.java b/org.hl7.fhir.r5/src/test/java/org/hl7/fhir/utilities/http/ManagedWebAccessAuthTests.java
index 98d7a8959..94f70a3cc 100644
--- a/org.hl7.fhir.r5/src/test/java/org/hl7/fhir/utilities/http/ManagedWebAccessAuthTests.java
+++ b/org.hl7.fhir.r5/src/test/java/org/hl7/fhir/utilities/http/ManagedWebAccessAuthTests.java
@@ -60,7 +60,6 @@ public class ManagedWebAccessAuthTests {
   @Test
   public void testBasicAuthCase() throws IOException, InterruptedException {
 
-
     ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor("dummyAgent", null).withBasicAuth("dummy1", "pass1");
 
     testBasicServerAuth(builder);
@@ -91,14 +90,20 @@ public class ManagedWebAccessAuthTests {
 
   @Test
   public void testTokenAuthCase() throws IOException, InterruptedException {
+
+
+    ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor("dummyAgent", null).withToken(DUMMY_TOKEN);
+
+    testTokenAuthCase(builder);
+  }
+
+  private void testTokenAuthCase(ManagedFhirWebAccessor builder) throws IOException, InterruptedException {
     HttpUrl serverUrl = server.url("blah/blah/blah?arg=blah");
 
     server.enqueue(
       new MockResponse()
         .setBody("Dummy Response").setResponseCode(200)
     );
-
-    ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor("dummyAgent", null).withToken(DUMMY_TOKEN);
     HTTPResult result = builder.httpCall(new HTTPRequest().withUrl(serverUrl.toString()).withMethod(HTTPRequest.HttpMethod.GET));
 
     assertThat(result.getCode()).isEqualTo(200);
@@ -112,7 +117,7 @@ public class ManagedWebAccessAuthTests {
     assertThat(packageRequest.getHeader("Authorization")).isEqualTo("Bearer " + DUMMY_TOKEN);
   }
 
-  private static void assertExpectedHeaders(RecordedRequest packageRequest, String expectedUrl, String expectedHttpMethod) {
+  private void assertExpectedHeaders(RecordedRequest packageRequest, String expectedUrl, String expectedHttpMethod) {
     assertThat(packageRequest.getRequestUrl().toString()).isEqualTo(expectedUrl);
     assertThat(packageRequest.getMethod()).isEqualTo(expectedHttpMethod);
     assertThat(packageRequest.getHeader("User-Agent")).isEqualTo(DUMMY_AGENT);
@@ -120,14 +125,19 @@ public class ManagedWebAccessAuthTests {
 
   @Test
   public void testApiKeyAuthCase() throws IOException, InterruptedException {
+
+    ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor("dummyAgent", null).withApiKey(DUMMY_API_KEY);
+
+    testApiKeyAuthCase(builder);
+  }
+
+  private void testApiKeyAuthCase(ManagedFhirWebAccessor builder) throws IOException, InterruptedException {
     HttpUrl serverUrl = server.url("blah/blah/blah?arg=blah");
 
     server.enqueue(
       new MockResponse()
         .setBody("Dummy Response").setResponseCode(200)
     );
-
-    ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor("dummyAgent", null).withApiKey(DUMMY_API_KEY);
     HTTPResult result = builder.httpCall(new HTTPRequest().withUrl(serverUrl.toString()).withMethod(HTTPRequest.HttpMethod.GET));
 
     assertThat(result.getCode()).isEqualTo(200);
@@ -142,7 +152,7 @@ public class ManagedWebAccessAuthTests {
   }
 
   @Test
-  public void testServerAuthFromSettings() throws IOException, InterruptedException {
+  public void testBasicAuthFromSettings() throws IOException, InterruptedException {
     ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor(
       "dummyAgent",
       List.of(getBasicAuthServerPojo()));
@@ -151,13 +161,50 @@ public class ManagedWebAccessAuthTests {
   }
 
   private ServerDetailsPOJO getBasicAuthServerPojo() {
-    ServerDetailsPOJO pojo = new ServerDetailsPOJO(
+    return new ServerDetailsPOJO(
       server.url("").toString(),
       "basic",
       "dummyServerType",
       DUMMY_USERNAME,
       DUMMY_PASSWORD,
       null, null);
-    return pojo;
   }
+
+@Test
+public void testTokenAuthFromSettings() throws IOException, InterruptedException {
+  ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor(
+    "dummyAgent",
+    List.of(getTokenAuthServerPojo()));
+
+  testTokenAuthCase(builder);
 }
+
+  private ServerDetailsPOJO getTokenAuthServerPojo() {
+    return new ServerDetailsPOJO(
+      server.url("").toString(),
+      "token",
+      "dummyServerType",
+     null,
+      null,
+      DUMMY_TOKEN, null);
+  }
+
+  @Test
+  public void testApiKeyAuthFromSettings() throws IOException, InterruptedException {
+    ManagedFhirWebAccessor builder = new ManagedFhirWebAccessor(
+      "dummyAgent",
+      List.of(getApiKeyAuthServerPojo()));
+
+    testApiKeyAuthCase(builder);
+  }
+
+  private ServerDetailsPOJO getApiKeyAuthServerPojo() {
+    return new ServerDetailsPOJO(
+      server.url("").toString(),
+      "apikey",
+      "dummyServerType",
+      null,
+      null,
+     null, DUMMY_API_KEY);
+  }
+}
\ No newline at end of file
diff --git a/org.hl7.fhir.utilities/src/main/java/org/hl7/fhir/utilities/http/ManagedFhirWebAccessor.java b/org.hl7.fhir.utilities/src/main/java/org/hl7/fhir/utilities/http/ManagedFhirWebAccessor.java
index df5f616e7..0ae20cfff 100644
--- a/org.hl7.fhir.utilities/src/main/java/org/hl7/fhir/utilities/http/ManagedFhirWebAccessor.java
+++ b/org.hl7.fhir.utilities/src/main/java/org/hl7/fhir/utilities/http/ManagedFhirWebAccessor.java
@@ -100,7 +100,7 @@ public class ManagedFhirWebAccessor extends ManagedWebAccessorBase<ManagedFhirWe
             headers.add(new HTTPHeader("Authorization", tokenCredential));
             break;
           case "apikey":
-            String apiKeyCredential = settings.getToken();
+            String apiKeyCredential = settings.getApikey();
             headers.add(new HTTPHeader("Api-Key", apiKeyCredential));
             break;
         }