Add missing SDNC and fix JUnit 89/70089/2 1.3.0
authorPamela Dragosh <pdragosh@research.att.com>
Tue, 9 Oct 2018 16:36:01 +0000 (12:36 -0400)
committerPamela Dragosh <pdragosh@research.att.com>
Tue, 9 Oct 2018 16:38:27 +0000 (12:38 -0400)
Identified in prior review, ensure all the new actors are
in the appropriate pom.xml and minor fix to JUnit for
assertEquals vs assertTrue.

Issue-ID: POLICY-1191
Change-Id: I5c7505ad795e6e19c99a3aa9cec7ecf688da8316
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
controlloop/common/feature-controlloop-amsterdam/pom.xml
controlloop/templates/template.demo/src/test/java/org/onap/policy/template/demo/VpciControlLoopTest.java

index 273d056..3bcda64 100644 (file)
             <artifactId>sdnr</artifactId>
             <version>${project.version}</version>
         </dependency>
+        <dependency>
+            <groupId>org.onap.policy.drools-applications.controlloop.common.model-impl</groupId>
+            <artifactId>sdnc</artifactId>
+            <version>${project.version}</version>
+        </dependency>
         <dependency>
             <groupId>org.onap.policy.drools-applications.controlloop.common.model-impl</groupId>
             <artifactId>vfc</artifactId>
                 </exclusion>
             </exclusions>
         </dependency>
+        <dependency>
+            <groupId>org.onap.policy.drools-applications.controlloop.common.actors</groupId>
+            <artifactId>actor.sdnc</artifactId>
+            <version>${project.version}</version>
+            <exclusions>
+                <exclusion>
+                    <artifactId>guava</artifactId>
+                    <groupId>com.google.guava</groupId>
+                </exclusion>
+            </exclusions>
+        </dependency>
         <dependency>
             <groupId>org.onap.policy.drools-applications.controlloop.common</groupId>
             <artifactId>policy-yaml</artifactId>
index 37f55dc..06e2aca 100644 (file)
@@ -3,6 +3,7 @@
  * demo
  * ================================================================================
  * Copyright (C) 2018 Wipro Limited Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2018 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.
@@ -293,17 +294,17 @@ public class VpciControlLoopTest implements TopicListener {
             String policyName = notification.getPolicyName();
             logger.debug("Rule Fired: {}", policyName);
             if (policyName.endsWith("EVENT")) {
-                assertTrue(ControlLoopNotificationType.ACTIVE.equals(notification.getNotification()));
+                assertEquals(ControlLoopNotificationType.ACTIVE, notification.getNotification());
             } else if (policyName.endsWith("GUARD_NOT_YET_QUERIED")) {
-                assertTrue(ControlLoopNotificationType.OPERATION.equals(notification.getNotification()));
+                assertEquals(ControlLoopNotificationType.OPERATION, notification.getNotification());
                 assertNotNull(notification.getMessage());
                 assertTrue(notification.getMessage().startsWith("Sending guard query"));
             } else if (policyName.endsWith("GUARD.RESPONSE")) {
-                assertTrue(ControlLoopNotificationType.OPERATION.equals(notification.getNotification()));
+                assertEquals(ControlLoopNotificationType.OPERATION, notification.getNotification());
                 assertNotNull(notification.getMessage());
                 assertTrue(notification.getMessage().toLowerCase().endsWith("permit"));
             } else if (policyName.endsWith("GUARD_PERMITTED")) {
-                assertTrue(ControlLoopNotificationType.OPERATION.equals(notification.getNotification()));
+                assertEquals(ControlLoopNotificationType.OPERATION, notification.getNotification());
                 assertNotNull(notification.getMessage());
                 assertTrue(notification.getMessage().startsWith("actor=SDNR"));
             } else if (policyName.endsWith("OPERATION.TIMEOUT")) {
@@ -311,7 +312,7 @@ public class VpciControlLoopTest implements TopicListener {
                 logger.debug("The operation timed out");
                 fail("Operation Timed Out");
             } else if (policyName.endsWith("SDNR.RESPONSE")) {
-                assertTrue(ControlLoopNotificationType.OPERATION_SUCCESS.equals(notification.getNotification()));
+                assertEquals(ControlLoopNotificationType.OPERATION_SUCCESS, notification.getNotification());
                 assertNotNull(notification.getMessage());
                 assertTrue(notification.getMessage().startsWith("actor=SDNR"));
             } else if (policyName.endsWith("EVENT.MANAGER")) {
@@ -334,7 +335,7 @@ public class VpciControlLoopTest implements TopicListener {
              */
             PciRequestWrapper dmaapRequest = (PciRequestWrapper) obj;
             PciRequest pciRequest = dmaapRequest.getBody();
-            assertTrue(pciRequest.getCommonHeader().getSubRequestId().equals("1"));
+            assertEquals("1", pciRequest.getCommonHeader().getSubRequestId());
 
             logger.debug("\n============ SDNR received the request!!! ===========\n");
             logger.debug("\n============ dmaapRequest ===========\n {} ", dmaapRequest);