Possible fix for event listener

Change-Id: Ie2eaf7f3d02bdd0b8adae964d3abf905781f996f
Signed-off-by: Patrick Brady <pb071s@att.com>
Issue-ID: APPC-859

4 files changed, 7 insertions, 46 deletions

diff --git a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/main/java/org/onap/appc/adapter/messaging/dmaap/http/CommonHttpClient.java b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/main/java/org/onap/appc/adapter/messaging/dmaap/http/CommonHttpClient.java
index 0749eb185..259972844 100644
--- a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/main/java/org/onap/appc/adapter/messaging/dmaap/http/CommonHttpClient.java
+++ b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/main/java/org/onap/appc/adapter/messaging/dmaap/http/CommonHttpClient.java
@@ -52,23 +52,22 @@ abstract class CommonHttpClient {
     }
 
     protected HttpGet getReq(URI uri, int timeoutMs) throws AuthenticationException {
-        if (authStr == null) {
-            throw new AuthenticationException("All DMaaP requests require authentication and none was provided.");
-        }
 
         HttpGet out = (uri == null) ? new HttpGet() : new HttpGet(uri);
-        out.setHeader("Authorization", String.format("Basic %s", authStr));
+        if (authStr != null) {
+            out.setHeader("Authorization", String.format("Basic %s", authStr));
+        }
+      
         out.setConfig(getConfig(timeoutMs));
         return out;
     }
 
     protected HttpPost postReq(String url) throws AuthenticationException {
-        if (authStr == null) {
-            throw new AuthenticationException("All DMaaP requests require authentication and none was provided.");
-        }
 
         HttpPost out = (url == null) ? new HttpPost() : new HttpPost(url);
-        out.setHeader("Authorization", String.format("Basic %s", authStr));
+        if (authStr != null) {
+            out.setHeader("Authorization", String.format("Basic %s", authStr));
+        }
         out.setConfig(getConfig(0));
         return out;
     }
diff --git a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestCommonHttpClient.java b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestCommonHttpClient.java
index 1d6fcfd43..660a6e4bc 100644
--- a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestCommonHttpClient.java
+++ b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestCommonHttpClient.java
@@ -69,14 +69,6 @@ public class TestCommonHttpClient {
         assertEquals(httpGet.getConfig().getSocketTimeout(), TIMEOUT + TIMEOUT_OFFSET);
     }
 
-    @Test(expected = AuthenticationException.class)
-    public void shoudNotGetHttpRequest_whenBasicAuthNotSet() throws AuthenticationException {
-
-        noBasicAuth();
-
-        commonHttpClient.getReq(URI, TIMEOUT);
-    }
-
     @Test
     public void shouldPostHttpRequest_whenSetBasicAuth() throws AuthenticationException {
 
@@ -90,14 +82,6 @@ public class TestCommonHttpClient {
         assertEquals(httpPost.getConfig().getSocketTimeout(), TIMEOUT_OFFSET);
     }
 
-    @Test(expected = AuthenticationException.class)
-    public void shoudNotPostHttpRequest_whenBasicAuthNotSet() throws AuthenticationException {
-
-        noBasicAuth();
-
-        commonHttpClient.postReq(URL);
-    }
-
     @Test
     public void shouldGetClient() {
         assertNotNull(commonHttpClient.getClient());
diff --git a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapConsumerImpl.java b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapConsumerImpl.java
index d5707c22f..ea9a5e921 100644
--- a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapConsumerImpl.java
+++ b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapConsumerImpl.java
@@ -107,17 +107,6 @@ public class TestHttpDmaapConsumerImpl {
     }
 
     @Test
-    public void shouldNotGetHttpRequest_whenCredencialsAreNotProvided() {
-
-        httpDmaapConsumer.updateCredentials(null, null);
-
-        List<String> output =  httpDmaapConsumer.fetch(TIMEOUT_MS, LIMIT);
-
-        assertTrue(output.isEmpty());
-        verifyNoMoreInteractions(httpClient, httpResponse, statusLine, entity);
-    }
-
-    @Test
     public void shouldNotBeSuccessful_whenHttpResponseIsOtherThanOk() throws Exception {
 
         when(statusLine.getStatusCode()).thenReturn(FORBIDDEN.getStatusCode());
diff --git a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapProducerImpl.java b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapProducerImpl.java
index a3d2e3afa..01e41a857 100644
--- a/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapProducerImpl.java
+++ b/appc-adapters/appc-dmaap-adapter/appc-dmaap-adapter-bundle/src/test/java/org/onap/appc/adapter/messaging/dmaap/http/TestHttpDmaapProducerImpl.java
@@ -93,17 +93,6 @@ public class TestHttpDmaapProducerImpl {
     }
 
     @Test
-    public void shouldNotPostHttpRequest_whenCredencialsAreNotProvided() {
-
-        httpDmaapProducer.updateCredentials(null, null);
-
-        boolean successful = httpDmaapProducer.post(PARTITION, DATA);
-
-        assertFalse(successful);
-        verifyNoMoreInteractions(httpClient, httpResponse, statusLine);
-    }
-
-    @Test
     public void shouldNotBeSuccessful_whenHttpResponseIsOtherThanOk() throws Exception {
 
         when(statusLine.getStatusCode()).thenReturn(FORBIDDEN.getStatusCode());