[CH10267] Add ability to set apiKey after instance has been defined (setter) (#40)

* Provide setter for apiKey.

* Add test for multiple instances of client.

* Wrap in try/catch to allow exception.

* Add set key test for autocomplete, browse, recommendations + code review feedback.

* No return for key setter.

Author: sblaurock

constructorio-client/src/main/java/io/constructor/client/ConstructorIO.java

@@ -109,6 +109,13 @@ public ConstructorIO(String apiToken, String apiKey, boolean isHTTPS, String hos
       this(apiToken, apiKey, isHTTPS, host, null);
     }
 
+    /**
+     * Sets apiKey
+     */
+    public void setApiKey(String apiKey) {
+        this.apiKey = apiKey;
+    }
+
     /**
      * Verifies that an autocomplete service is working.
      *

constructorio-client/src/test/java/io/constructor/client/ConstructorIOAutocompleteTest.java

@@ -82,4 +82,16 @@ public void autocompleteShouldReturnAResultWithSearchSuggestionsOnly() throws Ex
         assertEquals("autocomplete search suggestions exist", response.getSections().get("Search Suggestions").size(), 1);
         assertTrue("autocomplete result id exists", response.getResultId() != null);
     }
+
+    @Test
+    public void autocompleteShouldReturnAResultWithNewApiKeySet() throws Exception {
+        ConstructorIO constructor = new ConstructorIO("", "thiskeydoesnotexist", true, null);
+        constructor.setApiKey("ZqXaOfXuBWD4s3XzCI1q");
+        AutocompleteRequest request = new AutocompleteRequest("Stanley");
+        request.getResultsPerSection().put("Search Suggestions", 10);
+        AutocompleteResponse response = constructor.autocomplete(request, null);
+        assertEquals("autocomplete product suggestions exist", response.getSections().get("Products").size(), 0);
+        assertEquals("autocomplete search suggestions exist", response.getSections().get("Search Suggestions").size(), 1);
+        assertTrue("autocomplete result id exists", response.getResultId() != null);
+    }
 }

constructorio-client/src/test/java/io/constructor/client/ConstructorIOBasicTest.java

@@ -49,6 +49,28 @@ public void newShouldSetHostnameDefault() throws Exception {
         assertEquals("host should be set to default", constructor.host, "ac.cnstrc.com");
     }
 
+    @Test
+    public void shouldSetNewApiKey() throws Exception {
+        ConstructorIO constructor = new ConstructorIO("boinkaToken", "doinkaKey", false, null);
+        assertEquals("key should be set to default", constructor.apiKey, "doinkaKey");
+        constructor.setApiKey("foobar");
+        assertEquals("key should be set to new value", constructor.apiKey, "foobar");
+    }
+
+    @Test
+    public void shouldSetNewApiKeyMultipleInstances() throws Exception {
+        ConstructorIO constructorA = new ConstructorIO("boinkaToken", "foo", false, null);
+        ConstructorIO constructorB = new ConstructorIO("boinkaToken", "bar", false, null);
+        assertEquals("instance 1 key should be set to default", constructorA.apiKey, "foo");
+        assertEquals("instance 2 key should be set to default", constructorB.apiKey, "bar");
+        constructorA.setApiKey("newfoo");
+        assertEquals("instance 1 key should be set to new value", constructorA.apiKey, "newfoo");
+        assertEquals("instance 2 key should be set to default", constructorB.apiKey, "bar");
+        constructorB.setApiKey("newbar");
+        assertEquals("instance 1 key should be set to new value", constructorA.apiKey, "newfoo");
+        assertEquals("instance 2 key should be set to new value", constructorB.apiKey, "newbar");
+    }
+
     @Test
     public void makeAuthorizedRequestBuilderShouldSetAuthorizationHeader() throws Exception {
         ConstructorIO constructor = new ConstructorIO("boinkaToken", "doinkaKey", true, null);

constructorio-client/src/test/java/io/constructor/client/ConstructorIOBrowseTest.java

@@ -111,4 +111,15 @@ public void BrowseShouldReturnAResultWithCollectionId() throws Exception {
         assertEquals("browse results count as expected", (int)response.getResponse().getTotalNumberOfResults(), 999);
         assertTrue("browse result id exists", response.getResultId() != null);
     }
+
+    @Test
+    public void BrowseShouldReturnAResultWithNewApiKeySet() throws Exception {
+        ConstructorIO constructor = new ConstructorIO("", "thiskeydoesnotexist", true, "betaac.cnstrc.com");
+        constructor.setApiKey("key_aXLmVpkVp4BX21Sw");
+        BrowseRequest request = new BrowseRequest("collection_id", "fresh-deals");
+        BrowseResponse response = constructor.browse(request, null);
+        assertEquals("browse results exist", response.getResponse().getResults().size(), 30);
+        assertEquals("browse results count as expected", (int)response.getResponse().getTotalNumberOfResults(), 999);
+        assertTrue("browse result id exists", response.getResultId() != null);
+    }
 }

constructorio-client/src/test/java/io/constructor/client/ConstructorIORecommendationsTest.java

@@ -61,4 +61,17 @@ public void getRecommendationsShouldReturnAResultWithNumResults() throws Excepti
         assertEquals("recommendation results exist", response.getResponse().getResults().size(), 5);
         assertTrue("recommendation result id exists", response.getResultId() != null);
     }
+
+    @Test
+    public void getRecommendationsShouldReturnAResultWithNewApiKeySet() throws Exception {
+        ConstructorIO constructor = new ConstructorIO("", "thiskeydoesnotexist", true, null);
+        constructor.setApiKey("ZqXaOfXuBWD4s3XzCI1q");
+        UserInfo userInfo = new UserInfo(3, "c62a-2a09-faie");
+        RecommendationsRequest request = new RecommendationsRequest("item_page_1");
+        request.setItemIds(Arrays.asList("power_drill", "drill"));
+        request.setNumResults(5);
+        RecommendationsResponse response = constructor.recommendations(request, userInfo);
+        assertEquals("recommendation results exist", response.getResponse().getResults().size(), 5);
+        assertTrue("recommendation result id exists", response.getResultId() != null);
+    }
 }

constructorio-client/src/test/java/io/constructor/client/ConstructorIOSearchTest.java

@@ -125,4 +125,16 @@ public void SearchShouldReturnAResultWithCollectionId() throws Exception {
         assertEquals("search results count as expected", (int)response.getResponse().getTotalNumberOfResults(), 33);
         assertTrue("search result id exists", response.getResultId() != null);
     }
+
+    @Test
+    public void SearchShouldReturnAResultWithNewApiKeySet() throws Exception {
+        ConstructorIO constructor = new ConstructorIO("", "thiskeydoesnotexist", true, "betaac.cnstrc.com");
+        constructor.setApiKey("key_aXLmVpkVp4BX21Sw");
+        SearchRequest request = new SearchRequest("bananas");
+        request.setCollectionId("fresh-deals");
+        SearchResponse response = constructor.search(request, null);
+        assertEquals("search results exist", response.getResponse().getResults().size(), 30);
+        assertEquals("search results count as expected", (int)response.getResponse().getTotalNumberOfResults(), 33);
+        assertTrue("search result id exists", response.getResultId() != null);
+    }
 }