Add junit coverage to drools-pdp

Change-Id: I253d3e85a08b893e7a9b168d92752205bcc459f6
Issue-ID: POLICY-1772
Signed-off-by: Jim Hahn <jrh3@att.com>

2 files changed, 214 insertions, 5 deletions

diff --git a/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorJaxRsTest.java b/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorJaxRsTest.java
new file mode 100644
index 00000000..7200bdce
--- /dev/null
+++ b/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorJaxRsTest.java
@@ -0,0 +1,207 @@
+/*-
+ * ============LICENSE_START=======================================================
+ * ONAP
+ * ================================================================================
+ * Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * ================================================================================
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ * ============LICENSE_END=========================================================
+ */
+
+package org.onap.policy.drools.simulators;
+
+import static org.junit.Assert.assertEquals;
+import static org.mockito.Mockito.doThrow;
+
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+import java.util.concurrent.BlockingQueue;
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.LinkedBlockingQueue;
+import javax.servlet.http.HttpServletResponse;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
+import org.mockito.Mock;
+import org.mockito.MockitoAnnotations;
+
+public class DMaaPSimulatorJaxRsTest {
+    private static final String MESSAGE = "hello";
+    private static final String MESSAGE2 = "world";
+    private static final String EXPECTED_EXCEPTION = "expected exception";
+    private static final String TOPIC = "my-topic";
+    private static final int TIMEOUT_MS = 10;
+    private static final int LONG_TIMEOUT_MS = 250;
+
+    @Mock
+    private HttpServletResponse resp;
+
+    private DMaaPSimulatorJaxRs sim;
+
+    /**
+     * Initializes objects and creates the simulator.
+     */
+    @Before
+    public void setUp() {
+        MockitoAnnotations.initMocks(this);
+
+        sim = new DMaaPSimulatorJaxRs();
+    }
+
+    @After
+    public void tearDown() {
+        DMaaPSimulatorJaxRs.reset();
+    }
+
+    @Test
+    public void testSubscribe() {
+        sim.publish(TOPIC, MESSAGE);
+
+        assertEquals(MESSAGE, sim.subscribe(0, TOPIC, resp));
+    }
+
+    @Test
+    public void testSubscribe_FlushEx() throws IOException {
+        doThrow(new IOException(EXPECTED_EXCEPTION)).when(resp).flushBuffer();
+
+        assertEquals("Got an error", sim.subscribe(TIMEOUT_MS, TOPIC, resp));
+    }
+
+    @Test
+    public void testSubscribe_BadStatus_testSetResponseCode() {
+        sim.setStatus(199);
+        assertEquals("You got response code: 199", sim.subscribe(TIMEOUT_MS, TOPIC, resp));
+
+        sim.setStatus(300);
+        assertEquals("You got response code: 300", sim.subscribe(TIMEOUT_MS, TOPIC, resp));
+    }
+
+    @Test
+    public void testSubscribe_UnknownTopic_ZeroTimeout() {
+        assertEquals(DMaaPSimulatorJaxRs.NO_TOPIC_MSG, sim.subscribe(0, TOPIC, resp));
+    }
+
+    @Test
+    public void testSubscribe_UnknownTopic_NonZeroTimeout() {
+        assertEquals(DMaaPSimulatorJaxRs.NO_TOPIC_MSG, sim.subscribe(TIMEOUT_MS, TOPIC, resp));
+    }
+
+    @Test
+    public void testGetNextMessageFromQueue() {
+        sim.publish(TOPIC, MESSAGE);
+        sim.publish(TOPIC, MESSAGE2);
+
+        assertEquals(MESSAGE, sim.subscribe(0, TOPIC, resp));
+        assertEquals(MESSAGE2, sim.subscribe(0, TOPIC, resp));
+
+        // repeat - no message
+        assertEquals(DMaaPSimulatorJaxRs.NO_DATA_MSG, sim.subscribe(0, TOPIC, resp));
+    }
+
+    @Test
+    public void testGetNextMessageFromQueue_Interrupted() throws InterruptedException {
+        sim = new DMaaPSimulatorJaxRs() {
+            @Override
+            protected String poll(BlockingQueue<String> queue, int timeout) throws InterruptedException {
+                throw new InterruptedException(EXPECTED_EXCEPTION);
+            }
+        };
+
+        sim.publish(TOPIC, MESSAGE);
+
+        // put it in the background so we don't interrupt the test thread
+        BlockingQueue<String> queue = new LinkedBlockingQueue<>();
+        backgroundSubscribe(queue);
+
+        assertEquals(DMaaPSimulatorJaxRs.NO_DATA_MSG, queue.take());
+    }
+
+    @Test
+    public void testWaitForNextMessageFromQueue() throws InterruptedException {
+        BlockingQueue<String> queue = new LinkedBlockingQueue<>();
+
+        CountDownLatch latch1 = backgroundSubscribe(queue);
+        CountDownLatch latch2 = backgroundSubscribe(queue);
+
+        // wait for both threads to start
+        latch1.await();
+        latch2.await();
+
+        /*
+         * Must pause to prevent the topic from being created before subscribe() is
+         * invoked.
+         */
+        Thread.sleep(LONG_TIMEOUT_MS / 3);
+
+        // only publish one message
+        sim.publish(TOPIC, MESSAGE);
+
+        // wait for both subscribers to add their messages to the queue
+        List<String> messages = new ArrayList<>();
+        messages.add(queue.take());
+        messages.add(queue.take());
+
+        // sort them so the order is consistent
+        Collections.sort(messages);
+
+        assertEquals("[No Data, hello]", messages.toString());
+    }
+
+    @Test
+    public void testWaitForNextMessageFromQueue_Interrupted() throws InterruptedException {
+        sim = new DMaaPSimulatorJaxRs() {
+            @Override
+            protected void sleep(int timeout) throws InterruptedException {
+                throw new InterruptedException(EXPECTED_EXCEPTION);
+            }
+        };
+
+        // put it in the background so we don't interrupt the test thread
+        BlockingQueue<String> queue = new LinkedBlockingQueue<>();
+        backgroundSubscribe(queue);
+
+        assertEquals(DMaaPSimulatorJaxRs.NO_TOPIC_MSG, queue.take());
+    }
+
+    @Test
+    public void testPublish() {
+        assertEquals("", sim.publish(TOPIC, MESSAGE));
+        assertEquals(MESSAGE, sim.subscribe(0, TOPIC, resp));
+    }
+
+    @Test
+    public void testSetStatus() {
+        assertEquals("Status code set", sim.setStatus(500));
+        assertEquals("You got response code: 500", sim.subscribe(TIMEOUT_MS, TOPIC, resp));
+    }
+
+    /**
+     * Invokes subscribe() in a background thread.
+     *
+     * @param queue where to place the returned result
+     * @return a latch that will be counted down just before the background thread invokes
+     *         subscribe()
+     */
+    private CountDownLatch backgroundSubscribe(BlockingQueue<String> queue) {
+        CountDownLatch latch = new CountDownLatch(1);
+
+        new Thread(() -> {
+            latch.countDown();
+            queue.add(sim.subscribe(LONG_TIMEOUT_MS, TOPIC, resp));
+        }).start();
+
+        return latch;
+    }
+}
diff --git a/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorTest.java b/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorTest.java
index 9740ac6d..bb1b1cb9 100644
--- a/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorTest.java
+++ b/feature-simulators/src/test/java/org/onap/policy/drools/simulators/DMaaPSimulatorTest.java
@@ -43,15 +43,17 @@ import org.onap.policy.drools.utils.logging.LoggerUtil;
 
 public class DMaaPSimulatorTest {
 
-    private static final int DMAAPSIM_SERVER_PORT = 6670;
+    private static int DMAAPSIM_SERVER_PORT;
 
     /**
      * Setup the simulator.
+     * @throws IOException if a server port cannot be allocated
      */
     @BeforeClass
-    public static void setUpSimulator() {
+    public static void setUpSimulator() throws IOException {
         LoggerUtil.setLevel("ROOT", "INFO");
         LoggerUtil.setLevel("org.eclipse.jetty", "WARN");
+        DMAAPSIM_SERVER_PORT = NetworkUtil.allocPort();
         try {
             final HttpServletServer testServer = HttpServletServerFactoryInstance.getServerFactory().build("dmaapSim",
                             "localhost", DMAAPSIM_SERVER_PORT, "/", false, true);
@@ -76,7 +78,7 @@ public class DMaaPSimulatorTest {
         Pair<Integer, String> response = dmaapGet("myTopicNoData", timeout);
         assertNotNull(response);
         assertNotNull(response.first);
-        assertEquals("No topic", response.second);
+        assertEquals(DMaaPSimulatorJaxRs.NO_TOPIC_MSG, response.second);
     }
 
     @Test
@@ -148,7 +150,7 @@ public class DMaaPSimulatorTest {
         response = dmaapGet(topics[1], 1000);
         assertNotNull(response);
         assertNotNull(response.first);
-        assertEquals("No topic", response.second);
+        assertEquals(DMaaPSimulatorJaxRs.NO_TOPIC_MSG, response.second);
 
         response = dmaapPost(topics[1], data[1][0]);
         assertNotNull(response);
@@ -183,7 +185,7 @@ public class DMaaPSimulatorTest {
         response = dmaapGet(topics[0], 1000);
         assertNotNull(response);
         assertNotNull(response.first);
-        assertEquals("No Data", response.second);
+        assertEquals(DMaaPSimulatorJaxRs.NO_DATA_MSG, response.second);
     }
 
     @Test