Reduce Technical Debt on drools-applications 97/33797/1
authorliamfallon <liam.fallon@ericsson.com>
Fri, 2 Mar 2018 14:24:23 +0000 (14:24 +0000)
committerliamfallon <liam.fallon@ericsson.com>
Fri, 2 Mar 2018 14:29:10 +0000 (14:29 +0000)
This commit mops up the technical debt issues in
drools-applications that are reaistic to fix witout
major surgery on the code.

note: warnings on non compliant Java class names that are used in
yaml serializaiton and deserailization are ignored.

note: warnings on complexity figures for equals() methods are ignored.

Issue-ID: POLICY-455
Change-Id: I99ccda4bd9932ceb5ccbcfd2778730234b55d7a5
Signed-off-by: liamfallon <liam.fallon@ericsson.com>
18 files changed:
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAIGETVnfResponseTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAIGETVserverResponseTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQCloudRegionTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQExtraPropertyTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQGenericVNFTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQInventoryResponseItemsTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQServiceInstanceTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQTenantTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQVServerTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/AAINQVfModuleTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/PNFInstanceTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/RelationshipDataTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/RelationshipListTest.java
controlloop/common/model-impl/aai/src/test/java/org/onap/policy/aai/RelationshipTest.java
controlloop/common/model-impl/vfc/src/main/java/org/onap/policy/vfc/VFCResponseDescriptor.java
controlloop/common/model-impl/vfc/src/test/java/org/onap/policy/vfc/TestDemo.java
controlloop/common/simulators/src/main/java/org/onap/policy/simulators/SoSimulatorJaxRs.java
controlloop/common/simulators/src/main/java/org/onap/policy/simulators/Util.java

index b5de93c..9a4050f 100644 (file)
@@ -44,15 +44,15 @@ public class AAIGETVnfResponseTest {
         response.setIsClosedLoopDisabled("false");
                response.setResourceVersion("1494001988835");
                response.setModelInvariantId("f18be3cd-d446-456e-9109-121d9b62feaa");
-               assertEquals(response.getVnfID(), "83f674e8-7555-44d7-9a39-bdc3770b0491"); 
-        assertEquals(response.getVnfName(), "lll_vnf_010317");
-        assertEquals(response.getVnfType(), "Basa-122216-Service/VidVsamp12BaseVolume 1");
-        assertEquals(response.getServiceId(), "a9a77d5a-123e-4ca2-9eb9-0b015d2ee0fb");
-        assertEquals(response.getOrchestrationStatus(), "Created");
-        assertEquals(response.getInMaint(), "false");
-        assertEquals(response.getIsClosedLoopDisabled(), "false");
-        assertEquals(response.getResourceVersion(), "1494001988835");
-               assertEquals(response.getModelInvariantId(), "f18be3cd-d446-456e-9109-121d9b62feaa");
+               assertEquals("83f674e8-7555-44d7-9a39-bdc3770b0491", response.getVnfID()); 
+        assertEquals("lll_vnf_010317", response.getVnfName());
+        assertEquals("Basa-122216-Service/VidVsamp12BaseVolume 1", response.getVnfType());
+        assertEquals("a9a77d5a-123e-4ca2-9eb9-0b015d2ee0fb", response.getServiceId());
+        assertEquals("Created", response.getOrchestrationStatus());
+        assertEquals("false", response.getInMaint());
+        assertEquals("false", response.getIsClosedLoopDisabled());
+        assertEquals("1494001988835", response.getResourceVersion());
+               assertEquals("f18be3cd-d446-456e-9109-121d9b62feaa", response.getModelInvariantId());
                  
                RelationshipList relationshipList = new RelationshipList();
                Relationship     relationship = new Relationship();
@@ -76,8 +76,8 @@ public class AAIGETVnfResponseTest {
                item.setPropertyKey("service-instance.service-instance-name"); 
                item.setPropertyValue("lll_svc_010317"); 
                relatedToProperty.getRelatedTo().add(item);
-               assertEquals(item.getPropertyKey(), "service-instance.service-instance-name"); 
-               assertEquals(item.getPropertyValue(), "lll_svc_010317"); 
+               assertEquals("service-instance.service-instance-name", item.getPropertyKey()); 
+               assertEquals("lll_svc_010317", item.getPropertyValue()); 
                
         relationship.setRelatedTo("service-instance");
         relationship.setRelatedLink("/aai/v11/business/customers/customer/MSO_1610_ST/service-subscriptions/service-subscription/MSO-dev-service-type/service-instances/service-instance/e1e9c97c-02c0-4919-9b4c-eb5d5ef68970");
index 7cf0696..65c2cdc 100644 (file)
@@ -33,45 +33,47 @@ public class AAIGETVserverResponseTest {
 
        @Test
        public void test() {
-               AAIGETVserverResponse response = new AAIGETVserverResponse(); 
+               AAIGETVserverResponse response = new AAIGETVserverResponse();
                
-               response.setVserverID("d0668d4f-c25e-4a1b-87c4-83845c01efd8"); 
-               response.setVserverName("USMSO1SX7NJ0103UJZZ01-vjunos0"); 
-               response.setVserverName2("vjunos0"); 
-               response.setVserverSelflink("https://aai-ext1.test.aaa.com:8443/aai/v7/cloud-infrastructure/cloud-regions/cloud-region/aaa-aic/AAIAIC25/tenants/tenant/USMSO1SX7NJ0103UJZZ01%3A%3AuCPE-VMS/vservers/vserver/d0668d4f-c25e-4a1b-87c4-83845c01efd8"); 
-               response.setInMaint("false"); 
-               response.setIsClosedLoopDisabled("false"); 
-               response.setResourceVersion("1494001931513"); 
+               response.setVserverID("d0668d4f-c25e-4a1b-87c4-83845c01efd8");
+               response.setVserverName("USMSO1SX7NJ0103UJZZ01-vjunos0");
+               response.setVserverName2("vjunos0");
+               response.setVserverSelflink("https://aai-ext1.test.aaa.com:8443/aai/v7/cloud-infrastructure/cloud-regions/cloud-region/aaa-aic/AAIAIC25/tenants/tenant/USMSO1SX7NJ0103UJZZ01%3A%3AuCPE-VMS/vservers/vserver/d0668d4f-c25e-4a1b-87c4-83845c01efd8");
+               response.setInMaint("false");
+               response.setIsClosedLoopDisabled("false");
+               response.setResourceVersion("1494001931513");
                  
-               assertEquals(response.getVserverID(), "d0668d4f-c25e-4a1b-87c4-83845c01efd8"); 
-               assertEquals(response.getVserverName(), "USMSO1SX7NJ0103UJZZ01-vjunos0"); 
-               assertEquals(response.getVserverName2(), "vjunos0"); 
-               assertEquals(response.getVserverSelflink(), "https://aai-ext1.test.aaa.com:8443/aai/v7/cloud-infrastructure/cloud-regions/cloud-region/aaa-aic/AAIAIC25/tenants/tenant/USMSO1SX7NJ0103UJZZ01%3A%3AuCPE-VMS/vservers/vserver/d0668d4f-c25e-4a1b-87c4-83845c01efd8"); 
-               assertEquals(response.getInMaint(), "false"); 
-               assertEquals(response.getIsClosedLoopDisabled(), "false"); 
-               assertEquals(response.getResourceVersion(), "1494001931513"); 
+               assertEquals("d0668d4f-c25e-4a1b-87c4-83845c01efd8", response.getVserverID());
+               assertEquals("USMSO1SX7NJ0103UJZZ01-vjunos0", response.getVserverName());
+               assertEquals("vjunos0", response.getVserverName2());
+               assertEquals("https://aai-ext1.test.aaa.com:8443/aai/v7/cloud-infrastructure/cloud-regions/cloud-region/"
+                               + "aaa-aic/AAIAIC25/tenants/tenant/USMSO1SX7NJ0103UJZZ01%3A%3AuCPE-VMS/vservers/vserver/d0668d4f-c25e-4a1b-87c4-83845c01efd8",
+                               response.getVserverSelflink());
+               assertEquals("false", response.getInMaint());
+               assertEquals("false", response.getIsClosedLoopDisabled());
+               assertEquals("1494001931513", response.getResourceVersion());
 
                RelationshipList relationshipList = new RelationshipList();
                Relationship     relationship = new Relationship();
                RelationshipData relationshipData = new RelationshipData();
                RelationshipDataItem relationshipDataItem = new RelationshipDataItem();
                
-               relationshipDataItem.setRelationshipKey("customer.global-customer-id"); 
-               relationshipDataItem.setRelationshipValue("MSO_1610_ST"); 
+               relationshipDataItem.setRelationshipKey("customer.global-customer-id");
+               relationshipDataItem.setRelationshipValue("MSO_1610_ST");
                relationshipData.getRelationshipData().add(relationshipDataItem);
                
-               relationshipDataItem.setRelationshipKey("service-subscription.service-type"); 
-               relationshipDataItem.setRelationshipValue("MSO-dev-service-type"); 
+               relationshipDataItem.setRelationshipKey("service-subscription.service-type");
+               relationshipDataItem.setRelationshipValue("MSO-dev-service-type");
                relationshipData.getRelationshipData().add(relationshipDataItem);
                
-               relationshipDataItem.setRelationshipKey("service-instance.service-instance-id"); 
-               relationshipDataItem.setRelationshipValue("e1e9c97c-02c0-4919-9b4c-eb5d5ef68970"); 
+               relationshipDataItem.setRelationshipKey("service-instance.service-instance-id");
+               relationshipDataItem.setRelationshipValue("e1e9c97c-02c0-4919-9b4c-eb5d5ef68970");
                relationshipData.getRelationshipData().add(relationshipDataItem);
                
                RelatedToProperty relatedToProperty = new RelatedToProperty();
-               RelatedToPropertyItem item = new RelatedToPropertyItem();  
-               item.setPropertyKey("service-instance.service-instance-name"); 
-               item.setPropertyValue("lll_svc_010317"); 
+               RelatedToPropertyItem item = new RelatedToPropertyItem();
+               item.setPropertyKey("service-instance.service-instance-name");
+               item.setPropertyValue("lll_svc_010317");
                relatedToProperty.getRelatedTo().add(item);
                
         relationship.setRelatedTo("service-instance");
@@ -80,12 +82,12 @@ public class AAIGETVserverResponseTest {
         relationship.setRelatedToProperty(relatedToProperty);
        
         relationshipList.getRelationshipList().add(relationship);
-        response.setRelationshipList(relationshipList); 
-        assertEquals(response.getRelationshipList(), relationshipList); 
+        response.setRelationshipList(relationshipList);
+        assertEquals(response.getRelationshipList(), relationshipList);
         response.setRequestError(null);
         assertNull(response.getRequestError());
         
         logger.info(Serialization.gsonPretty.toJson(response));
        }
 
-}
\ No newline at end of file
+}
index f5c5940..5d1fe27 100644 (file)
@@ -43,11 +43,11 @@ public class AAINQCloudRegionTest {
                aaiNQCloudRegion.setCloudRegionVersion("v1");
                aaiNQCloudRegion.setComplexName("SharedNode"); 
                aaiNQCloudRegion.setResourceVersion("1504789196021");
-        assertNotNull(aaiNQCloudRegion);               
-               assertEquals(aaiNQCloudRegion.getCloudOwner(), "Rackspace");
-               assertEquals(aaiNQCloudRegion.getCloudRegionId(), "DFW");
-               assertEquals(aaiNQCloudRegion.getCloudRegionVersion(), "v1");
-               assertEquals(aaiNQCloudRegion.getComplexName(), "SharedNode"); 
-               assertEquals(aaiNQCloudRegion.getResourceVersion(), "1504789196021");
+               assertNotNull(aaiNQCloudRegion);                
+               assertEquals("Rackspace", aaiNQCloudRegion.getCloudOwner());
+               assertEquals("DFW", aaiNQCloudRegion.getCloudRegionId());
+               assertEquals("v1", aaiNQCloudRegion.getCloudRegionVersion());
+               assertEquals("SharedNode", aaiNQCloudRegion.getComplexName()); 
+               assertEquals("1504789196021", aaiNQCloudRegion.getResourceVersion());
        }
 }
index 2b6dc74..78189c8 100644 (file)
@@ -42,7 +42,7 @@ public class AAINQExtraPropertyTest {
                aaiNQExtraProperty.setPropertyValue("service-instance");                
                assertNotNull(aaiNQExtraProperty); 
 
-               assertEquals(aaiNQExtraProperty.getPropertyName(), "model.model-name"); 
-               assertEquals(aaiNQExtraProperty.getPropertyValue(), "service-instance");                
+               assertEquals("model.model-name", aaiNQExtraProperty.getPropertyName()); 
+               assertEquals("service-instance", aaiNQExtraProperty.getPropertyValue());                
        }
 }
index 7bd7d02..fba01bc 100644 (file)
@@ -38,42 +38,42 @@ public class AAINQGenericVNFTest {
        @Test
        public void test() {
                AAINQGenericVNF aaiNQGenericVNF = new AAINQGenericVNF();
-               aaiNQGenericVNF.setEncrypedAccessFlag(true); 
-               aaiNQGenericVNF.setInMaint(false); 
-               aaiNQGenericVNF.setIpv4Loopback0Address("aa"); 
-               aaiNQGenericVNF.setIpv4OamAddress("oamAddress"); 
-               aaiNQGenericVNF.setIsClosedLoopDisabled(false); 
-               aaiNQGenericVNF.setModelInvariantId("653d2caa-7e47-4614-95b3-26c8d82755b8"); 
-               aaiNQGenericVNF.setModelVersionId("98f410f6-4c63-447b-97d2-42508437cec0"); 
+               aaiNQGenericVNF.setEncrypedAccessFlag(true);
+               aaiNQGenericVNF.setInMaint(false);
+               aaiNQGenericVNF.setIpv4Loopback0Address("aa");
+               aaiNQGenericVNF.setIpv4OamAddress("oamAddress");
+               aaiNQGenericVNF.setIsClosedLoopDisabled(false);
+               aaiNQGenericVNF.setModelInvariantId("653d2caa-7e47-4614-95b3-26c8d82755b8");
+               aaiNQGenericVNF.setModelVersionId("98f410f6-4c63-447b-97d2-42508437cec0");
                aaiNQGenericVNF.setOperationalState("active");
                aaiNQGenericVNF.setPersonaModelId("653d2caa-7e47-4614-95b3-26c8d82755b8");
                aaiNQGenericVNF.setPersonaModelVersion("98f410f6-4c63-447b-97d2-42508437cec0");
                aaiNQGenericVNF.setProvStatus("complete");
                aaiNQGenericVNF.setResourceVersion("1505056714553");
                aaiNQGenericVNF.setServiceId("e8cb8968-5411-478b-906a-f28747de72cd");
-               aaiNQGenericVNF.setVnfID("ed8b2bce-6b27-4089-992c-4a2c66024bcd"); 
+               aaiNQGenericVNF.setVnfID("ed8b2bce-6b27-4089-992c-4a2c66024bcd");
                aaiNQGenericVNF.setVnfName("vCPEInfraVNF14a");
                aaiNQGenericVNF.setVnfName2("malumabb12");
                aaiNQGenericVNF.setVnfType("vCPEInfraService10/vCPEInfraService10 0");
-        assertNotNull(aaiNQGenericVNF); 
-        
-               assertEquals(aaiNQGenericVNF.getEncrypedAccessFlag(), true); 
-               assertEquals(aaiNQGenericVNF.getInMaint(), false); 
-               assertEquals(aaiNQGenericVNF.getIpv4Loopback0Address(), "aa"); 
-               assertEquals(aaiNQGenericVNF.getIpv4OamAddress(), "oamAddress"); 
-               assertEquals(aaiNQGenericVNF.getIsClosedLoopDisabled(), false); 
-               assertEquals(aaiNQGenericVNF.getModelInvariantId(), "653d2caa-7e47-4614-95b3-26c8d82755b8"); 
-               assertEquals(aaiNQGenericVNF.getModelVersionId(), "98f410f6-4c63-447b-97d2-42508437cec0"); 
-               assertEquals(aaiNQGenericVNF.getOperationalState(), "active");
-               assertEquals(aaiNQGenericVNF.getPersonaModelId(), "653d2caa-7e47-4614-95b3-26c8d82755b8");
-               assertEquals(aaiNQGenericVNF.getPersonaModelVersion(), "98f410f6-4c63-447b-97d2-42508437cec0");
-               assertEquals(aaiNQGenericVNF.getProvStatus(), "complete");
-               assertEquals(aaiNQGenericVNF.getResourceVersion(), "1505056714553");
-               assertEquals(aaiNQGenericVNF.getServiceId(), "e8cb8968-5411-478b-906a-f28747de72cd");
-               assertEquals(aaiNQGenericVNF.getVnfID(), "ed8b2bce-6b27-4089-992c-4a2c66024bcd"); 
-               assertEquals(aaiNQGenericVNF.getVnfName(), "vCPEInfraVNF14a");
-               assertEquals(aaiNQGenericVNF.getVnfName2(), "malumabb12");
-               assertEquals(aaiNQGenericVNF.getVnfType(), "vCPEInfraService10/vCPEInfraService10 0");
+               assertNotNull(aaiNQGenericVNF);
+
+               assertEquals(true, aaiNQGenericVNF.getEncrypedAccessFlag());
+               assertEquals(false, aaiNQGenericVNF.getInMaint());
+               assertEquals("aa", aaiNQGenericVNF.getIpv4Loopback0Address());
+               assertEquals("oamAddress", aaiNQGenericVNF.getIpv4OamAddress());
+               assertEquals(false, aaiNQGenericVNF.getIsClosedLoopDisabled());
+               assertEquals("653d2caa-7e47-4614-95b3-26c8d82755b8", aaiNQGenericVNF.getModelInvariantId());
+               assertEquals("98f410f6-4c63-447b-97d2-42508437cec0", aaiNQGenericVNF.getModelVersionId());
+               assertEquals("active", aaiNQGenericVNF.getOperationalState());
+               assertEquals("653d2caa-7e47-4614-95b3-26c8d82755b8", aaiNQGenericVNF.getPersonaModelId());
+               assertEquals("98f410f6-4c63-447b-97d2-42508437cec0", aaiNQGenericVNF.getPersonaModelVersion());
+               assertEquals("complete", aaiNQGenericVNF.getProvStatus());
+               assertEquals("1505056714553", aaiNQGenericVNF.getResourceVersion());
+               assertEquals("e8cb8968-5411-478b-906a-f28747de72cd", aaiNQGenericVNF.getServiceId());
+               assertEquals("ed8b2bce-6b27-4089-992c-4a2c66024bcd", aaiNQGenericVNF.getVnfID());
+               assertEquals("vCPEInfraVNF14a", aaiNQGenericVNF.getVnfName());
+               assertEquals("malumabb12", aaiNQGenericVNF.getVnfName2());
+               assertEquals("vCPEInfraService10/vCPEInfraService10 0", aaiNQGenericVNF.getVnfType());
        }
 
 }
index 4228243..caeec79 100644 (file)
@@ -46,7 +46,7 @@ public class AAINQInventoryResponseItemsTest {
                AAINQInventoryResponseItems aaiNQInventoryResponseItems = new AAINQInventoryResponseItems();
         AAINQInventoryResponseItem serviceItem = new AAINQInventoryResponseItem();
         serviceItem.setModelName("service-instance");
-        assertEquals(serviceItem.getModelName(), "service-instance");
+        assertEquals("service-instance", serviceItem.getModelName());
 
         serviceItem.setServiceInstance(new AAINQServiceInstance());
         serviceItem.getServiceInstance().setServiceInstanceID("dhv-test-vhnfportal-service-instance-id");
index dcb19a4..3b3d13c 100644 (file)
@@ -51,14 +51,14 @@ public class AAINQServiceInstanceTest {
                aaiNQServiceInstance.setModelInvariantId("SomeID");
                aaiNQServiceInstance.setModelVersionId("SomeVersion");
                assertNotNull(aaiNQServiceInstance);
-               assertEquals(aaiNQServiceInstance.getServiceInstanceID(), "dhv-test-vhnfportal-service-instance-id");
-               assertEquals(aaiNQServiceInstance.getServiceInstanceName(), "dhv-test-service-instance-name1");
-               assertEquals(aaiNQServiceInstance.getPersonaModelId(), "82194af1-3c2c-485a-8f44-420e22a9eaa4");
-               assertEquals(aaiNQServiceInstance.getPersonaModelVersion(), "1.0");
-               assertEquals(aaiNQServiceInstance.getServiceInstanceLocationId(), "dhv-test-service-instance-location-id1");
-               assertEquals(aaiNQServiceInstance.getResourceVersion(), "1485366092");
-               assertEquals(aaiNQServiceInstance.getModelInvariantId(), "SomeID");
-               assertEquals(aaiNQServiceInstance.getModelVersionId(), "SomeVersion");
+               assertEquals("dhv-test-vhnfportal-service-instance-id", aaiNQServiceInstance.getServiceInstanceID());
+               assertEquals("dhv-test-service-instance-name1", aaiNQServiceInstance.getServiceInstanceName());
+               assertEquals("82194af1-3c2c-485a-8f44-420e22a9eaa4", aaiNQServiceInstance.getPersonaModelId());
+               assertEquals("1.0", aaiNQServiceInstance.getPersonaModelVersion());
+               assertEquals("dhv-test-service-instance-location-id1", aaiNQServiceInstance.getServiceInstanceLocationId());
+               assertEquals("1485366092", aaiNQServiceInstance.getResourceVersion());
+               assertEquals("SomeID", aaiNQServiceInstance.getModelInvariantId());
+               assertEquals("SomeVersion", aaiNQServiceInstance.getModelVersionId());
            logger.info(Serialization.gsonPretty.toJson(aaiNQServiceInstance));         
        }
 
index c40a462..b696c7b 100644 (file)
@@ -46,9 +46,9 @@ public class AAINQTenantTest {
                aaiNQTenant.setTenantName("dhv-test-tenant-name");
                aaiNQTenant.setResourceVersion("1485366334");
            assertNotNull(aaiNQTenant); 
-               assertEquals(aaiNQTenant.getTenantId(), "dhv-test-tenant");
-               assertEquals(aaiNQTenant.getTenantName(), "dhv-test-tenant-name");
-               assertEquals(aaiNQTenant.getResourceVersion(), "1485366334");
+           assertEquals("dhv-test-tenant", aaiNQTenant.getTenantId());
+           assertEquals("dhv-test-tenant-name", aaiNQTenant.getTenantName());
+           assertEquals("1485366334", aaiNQTenant.getResourceVersion());
 
                logger.info(Serialization.gsonPretty.toJson(aaiNQTenant));
        }
index b928e86..556d82c 100644 (file)
@@ -52,14 +52,14 @@ public class AAINQVServerTest {
                aaiNQVServer.setIsClosedLoopDisabled(false);
                aaiNQVServer.setResourceVersion("1485366417");
            assertNotNull(aaiNQVServer); 
-               assertEquals(aaiNQVServer.getVserverId(), "dhv-test-vserver");
-               assertEquals(aaiNQVServer.getVserverName(), "dhv-test-vserver-name");
-               assertEquals(aaiNQVServer.getVserverName2(), "dhv-test-vserver-name2");
-               assertEquals(aaiNQVServer.getProvStatus(), "PREPROV");
-               assertEquals(aaiNQVServer.getVserverSelflink(), "dhv-test-vserver-selflink");
-               assertEquals(aaiNQVServer.getInMaint(), false);
-               assertEquals(aaiNQVServer.getIsClosedLoopDisabled(), false);
-               assertEquals(aaiNQVServer.getResourceVersion(), "1485366417");
+           assertEquals("dhv-test-vserver", aaiNQVServer.getVserverId());
+           assertEquals("dhv-test-vserver-name", aaiNQVServer.getVserverName());
+           assertEquals("dhv-test-vserver-name2", aaiNQVServer.getVserverName2());
+           assertEquals("PREPROV", aaiNQVServer.getProvStatus());
+           assertEquals("dhv-test-vserver-selflink", aaiNQVServer.getVserverSelflink());
+           assertEquals(false, aaiNQVServer.getInMaint());
+           assertEquals(false, aaiNQVServer.getIsClosedLoopDisabled());
+               assertEquals("1485366417", aaiNQVServer.getResourceVersion());
            logger.info(Serialization.gsonPretty.toJson(aaiNQVServer));
        }
 
index 21f13d5..7595f88 100644 (file)
@@ -16,7 +16,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  * ============LICENSE_END=========================================================
-*/
+ */
 package org.onap.policy.aai;
 
 import static org.junit.Assert.*;
@@ -56,21 +56,21 @@ public class AAINQVfModuleTest {
                aaiNQVfModule.setContrailServiceInstanceFqdn("example-contrail-service-instance-fqdn-val-86796");
                aaiNQVfModule.setModelInvariantId("SomeId");
                aaiNQVfModule.setModelVersionId("SomeVersion");
-           assertNotNull(aaiNQVfModule); 
-               assertEquals(aaiNQVfModule.getVfModuleId(), "example-vf-module-id-val-49261");
-               assertEquals(aaiNQVfModule.getVfModuleName(), "example-vf-module-name-val-73074");
-               assertEquals(aaiNQVfModule.getHeatStackId(), "example-heat-stack-id-val-86300");
-               assertEquals(aaiNQVfModule.getOrchestrationStatus(), "example-orchestration-status-val-56523");
-               assertEquals(aaiNQVfModule.getIsBaseVfModule(), true);
-               assertEquals(aaiNQVfModule.getResourceVersion(), "1485366450");
-               assertEquals(aaiNQVfModule.getPersonaModelId(), "ef86f9c5-2165-44f3-8fc3-96018b609ea5");
-               assertEquals(aaiNQVfModule.getPersonaModelVersion(), "1.0");
-               assertEquals(aaiNQVfModule.getWidgetModelId(), "example-widget-model-id-val-92571");
-               assertEquals(aaiNQVfModule.getWidgetModelVersion(), "example-widget-model-version-val-83317");
-               assertEquals(aaiNQVfModule.getContrailServiceInstanceFqdn(), "example-contrail-service-instance-fqdn-val-86796");
-               assertEquals(aaiNQVfModule.getModelInvariantId(), "SomeId");
-               assertEquals(aaiNQVfModule.getModelVersionId(), "SomeVersion");
-           logger.info(Serialization.gsonPretty.toJson(aaiNQVfModule));                
+               assertNotNull(aaiNQVfModule);
+               assertEquals("example-vf-module-id-val-49261", aaiNQVfModule.getVfModuleId());
+               assertEquals("example-vf-module-name-val-73074", aaiNQVfModule.getVfModuleName());
+               assertEquals("example-heat-stack-id-val-86300", aaiNQVfModule.getHeatStackId());
+               assertEquals("example-orchestration-status-val-56523", aaiNQVfModule.getOrchestrationStatus());
+               assertEquals(true, aaiNQVfModule.getIsBaseVfModule());
+               assertEquals("1485366450", aaiNQVfModule.getResourceVersion());
+               assertEquals("ef86f9c5-2165-44f3-8fc3-96018b609ea5", aaiNQVfModule.getPersonaModelId());
+               assertEquals("1.0", aaiNQVfModule.getPersonaModelVersion());
+               assertEquals("example-widget-model-id-val-92571", aaiNQVfModule.getWidgetModelId());
+               assertEquals("example-widget-model-version-val-83317", aaiNQVfModule.getWidgetModelVersion());
+               assertEquals("example-contrail-service-instance-fqdn-val-86796", aaiNQVfModule.getContrailServiceInstanceFqdn());
+               assertEquals("SomeId", aaiNQVfModule.getModelInvariantId());
+               assertEquals("SomeVersion", aaiNQVfModule.getModelVersionId());
+               logger.info(Serialization.gsonPretty.toJson(aaiNQVfModule));            
        }
 
 }
index a280f54..b4a8a51 100644 (file)
@@ -47,7 +47,7 @@ public class PNFInstanceTest {
                pnfInstance.setPNFType(PNFType.ENODEB); 
                pnfInstance.setPNFSerial("pnf-serial-test"); 
            assertNotNull(pnfInstance); 
-           assertEquals(pnfInstance.getPNFInstanceName(), "pnf-instance-name-test"); 
+           assertEquals("pnf-instance-name-test", pnfInstance.getPNFInstanceName()); 
 
            PNFInstance pnfInstanceNull = new PNFInstance(null);
            assertNotNull(pnfInstanceNull);
index b25d3e4..6b228f7 100644 (file)
@@ -46,8 +46,8 @@ public class RelationshipDataTest {
                relationshipDataItem.setRelationshipKey("relationship-key");
                relationshipDataItem.setRelationshipValue("relationship-value"); 
                assertNotNull(relationshipDataItem);
-               assertEquals(relationshipDataItem.getRelationshipKey(), "relationship-key");
-               assertEquals(relationshipDataItem.getRelationshipValue(), "relationship-value"); 
+               assertEquals("relationship-key", relationshipDataItem.getRelationshipKey());
+               assertEquals("relationship-value", relationshipDataItem.getRelationshipValue());
 
                relationshipData.getRelationshipData().add(relationshipDataItem); 
                RelationshipDataItem relationshipDataItem2 = new RelationshipDataItem(); 
index a5a7d90..7300fb8 100644 (file)
@@ -42,28 +42,28 @@ public class RelationshipListTest {
        @Test
        public void test() {
                RelationshipList relationshipList = new RelationshipList();
-               Relationship relationship = new Relationship(); 
-               relationship.setRelatedLink("related-link"); 
-               relationship.setRelatedTo("related-to"); 
-               assertEquals(relationship.getRelatedLink(), "related-link"); 
-               assertEquals(relationship.getRelatedTo(), "related-to"); 
+               Relationship relationship = new Relationship();
+               relationship.setRelatedLink("related-link");
+               relationship.setRelatedTo("related-to");
+               assertEquals("related-link", relationship.getRelatedLink());
+               assertEquals("related-to", relationship.getRelatedTo());
 
                RelatedToProperty relatedToProperty = new RelatedToProperty();
-               RelatedToPropertyItem relatedToPropertyItem = new RelatedToPropertyItem(); 
-               relatedToPropertyItem.setPropertyKey("model.model-name"); 
+               RelatedToPropertyItem relatedToPropertyItem = new RelatedToPropertyItem();
+               relatedToPropertyItem.setPropertyKey("model.model-name");
                relatedToPropertyItem.setPropertyValue("service-instance");
-               relatedToProperty.getRelatedTo().add(relatedToPropertyItem); 
-               RelatedToPropertyItem relatedToPropertyItem2 = new RelatedToPropertyItem(); 
-               relatedToPropertyItem2.setPropertyKey("model.model-name2"); 
+               relatedToProperty.getRelatedTo().add(relatedToPropertyItem);
+               RelatedToPropertyItem relatedToPropertyItem2 = new RelatedToPropertyItem();
+               relatedToPropertyItem2.setPropertyKey("model.model-name2");
                relatedToPropertyItem2.setPropertyValue("service-instance2");
                relatedToProperty.getRelatedTo().add(relatedToPropertyItem2);           
-               relationship.setRelatedToProperty(relatedToProperty); 
-               RelationshipDataItem relationshipDataItem = new RelationshipDataItem(); 
+               relationship.setRelatedToProperty(relatedToProperty);
+               RelationshipDataItem relationshipDataItem = new RelationshipDataItem();
                relationshipDataItem.setRelationshipKey("relationship-key");
-               relationshipDataItem.setRelationshipValue("relationship-value");  
-               RelationshipData relationshipData = new RelationshipData(); 
-               relationshipData.getRelationshipData().add(relationshipDataItem); 
-               relationship.setRelationshipData(relationshipData); 
+               relationshipDataItem.setRelationshipValue("relationship-value"); 
+               RelationshipData relationshipData = new RelationshipData();
+               relationshipData.getRelationshipData().add(relationshipDataItem);
+               relationship.setRelationshipData(relationshipData);
                relationshipList.getRelationshipList().add(relationship);
                
            assertNotNull(relationshipList);
index 3f899eb..81ba088 100644 (file)
@@ -42,30 +42,30 @@ public class RelationshipTest {
        @Test
        public void test() {
                Relationship relationship = new Relationship();
-               relationship.setRelatedLink("related-link"); 
-               relationship.setRelatedTo("related-to"); 
-               assertEquals(relationship.getRelatedLink(), "related-link"); 
-               assertEquals(relationship.getRelatedTo(), "related-to"); 
+               relationship.setRelatedLink("related-link");
+               relationship.setRelatedTo("related-to");
+               assertEquals("related-link", relationship.getRelatedLink());
+               assertEquals("related-to", relationship.getRelatedTo());
                
                RelatedToProperty relatedToProperty = new RelatedToProperty();
-               RelatedToPropertyItem relatedToPropertyItem = new RelatedToPropertyItem(); 
-               relatedToPropertyItem.setPropertyKey("model.model-name"); 
+               RelatedToPropertyItem relatedToPropertyItem = new RelatedToPropertyItem();
+               relatedToPropertyItem.setPropertyKey("model.model-name");
                relatedToPropertyItem.setPropertyValue("service-instance");
-               relatedToProperty.getRelatedTo().add(relatedToPropertyItem); 
-               RelatedToPropertyItem relatedToPropertyItem2 = new RelatedToPropertyItem(); 
-               relatedToPropertyItem2.setPropertyKey("model.model-name2"); 
+               relatedToProperty.getRelatedTo().add(relatedToPropertyItem);
+               RelatedToPropertyItem relatedToPropertyItem2 = new RelatedToPropertyItem();
+               relatedToPropertyItem2.setPropertyKey("model.model-name2");
                relatedToPropertyItem2.setPropertyValue("service-instance2");
                relatedToProperty.getRelatedTo().add(relatedToPropertyItem2);           
-               relationship.setRelatedToProperty(relatedToProperty); 
-               assertEquals(relationship.getRelatedToProperty(), relatedToProperty); 
-               RelationshipDataItem relationshipDataItem = new RelationshipDataItem(); 
+               relationship.setRelatedToProperty(relatedToProperty);
+               assertEquals(relationship.getRelatedToProperty(), relatedToProperty);
+               RelationshipDataItem relationshipDataItem = new RelationshipDataItem();
                relationshipDataItem.setRelationshipKey("relationship-key");
-               relationshipDataItem.setRelationshipValue("relationship-value");  
-               RelationshipData relationshipData = new RelationshipData(); 
-               relationshipData.getRelationshipData().add(relationshipDataItem); 
-               relationship.setRelationshipData(relationshipData); 
-               assertEquals(relationship.getRelationshipData(), relationshipData); 
-           assertNotNull(relationship); 
+               relationshipDataItem.setRelationshipValue("relationship-value"); 
+               RelationshipData relationshipData = new RelationshipData();
+               relationshipData.getRelationshipData().add(relationshipDataItem);
+               relationship.setRelationshipData(relationshipData);
+               assertEquals(relationship.getRelationshipData(), relationshipData);
+           assertNotNull(relationship);
            logger.info(Serialization.gsonPretty.toJson(relationship));
        }
 
index 81eb3bc..f237554 100644 (file)
@@ -42,7 +42,7 @@ public class VFCResponseDescriptor implements Serializable {
     private String responseId;
 
     @SerializedName("responseHistoryList")
-    public List<VFCResponseDescriptor> responseHistoryList;
+    private List<VFCResponseDescriptor> responseHistoryList;
 
     public VFCResponseDescriptor() {
                // Default constructor for VFCResponseDescriptor
index 96a62bf..69d3d58 100644 (file)
@@ -66,8 +66,8 @@ public class TestDemo {
         responseDescriptor.setErrorCode(null);
         responseDescriptor.setResponseId("11");
 
-           response.getResponseDescriptor().responseHistoryList = new LinkedList<>();
-        response.getResponseDescriptor().responseHistoryList.add(responseDescriptor);
+           response.getResponseDescriptor().setResponseHistoryList(new LinkedList<>());
+        response.getResponseDescriptor().getResponseHistoryList().add(responseDescriptor);
 
         body = Serialization.gsonPretty.toJson(response);
         System.out.println(body);
index e70a885..eb12362 100644 (file)
@@ -34,7 +34,7 @@ public class SoSimulatorJaxRs {
        @Path("/v5/{serviceInstanceId}/vnfs/{vnfInstanceId}/vfModules")
        @Consumes(MediaType.APPLICATION_JSON)
        @Produces("application/json")
-       public String SoPostQuery(@PathParam("serviceInstanceId") String serviceInstanceId, @PathParam("vnfInstanceId") String vnfInstanceId)
+       public String soPostQuery(@PathParam("serviceInstanceId") String serviceInstanceId, @PathParam("vnfInstanceId") String vnfInstanceId)
        {
 
                // the requestID contained in the SO Response is a newly generated requestID 
index cd45d31..05afd11 100644 (file)
@@ -26,52 +26,56 @@ import org.onap.policy.drools.http.server.HttpServletServer;
 import org.onap.policy.drools.utils.NetworkUtil;
 
 public class Util {
-  public static final String AAISIM_SERVER_NAME = "aaiSim";
-  public static final String SOSIM_SERVER_NAME = "soSim";
-  public static final String VFCSIM_SERVER_NAME = "vfcSim";
-  public static final String GUARDSIM_SERVER_NAME = "guardSim";
+       private Util() {
+               // Prevent instantiation of thic class
+       }
 
-  public static final int AAISIM_SERVER_PORT = 6666;
-  public static final int SOSIM_SERVER_PORT = 6667;
-  public static final int VFCSIM_SERVER_PORT = 6668;
-  public static final int GUARDSIM_SERVER_PORT = 6669;
+       public static final String AAISIM_SERVER_NAME = "aaiSim";
+       public static final String SOSIM_SERVER_NAME = "soSim";
+       public static final String VFCSIM_SERVER_NAME = "vfcSim";
+       public static final String GUARDSIM_SERVER_NAME = "guardSim";
 
-  public static HttpServletServer buildAaiSim() throws InterruptedException, IOException {
-    final HttpServletServer testServer = HttpServletServer.factory.build(AAISIM_SERVER_NAME,
-        "localhost", AAISIM_SERVER_PORT, "/", false, true);
-    testServer.addServletClass("/*", AaiSimulatorJaxRs.class.getName());
-    testServer.waitedStart(5000);
-    if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
-      throw new IllegalStateException("cannot connect to port " + testServer.getPort());
-    return testServer;
-  }
+       public static final int AAISIM_SERVER_PORT = 6666;
+       public static final int SOSIM_SERVER_PORT = 6667;
+       public static final int VFCSIM_SERVER_PORT = 6668;
+       public static final int GUARDSIM_SERVER_PORT = 6669;
 
-  public static HttpServletServer buildSoSim() throws InterruptedException, IOException {
-    final HttpServletServer testServer = HttpServletServer.factory.build(SOSIM_SERVER_NAME,
-        "localhost", SOSIM_SERVER_PORT, "/", false, true);
-    testServer.addServletClass("/*", SoSimulatorJaxRs.class.getName());
-    testServer.waitedStart(5000);
-    if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
-      throw new IllegalStateException("cannot connect to port " + testServer.getPort());
-    return testServer;
-  }
+       public static HttpServletServer buildAaiSim() throws InterruptedException, IOException {
+               final HttpServletServer testServer = HttpServletServer.factory.build(AAISIM_SERVER_NAME,
+                               "localhost", AAISIM_SERVER_PORT, "/", false, true);
+               testServer.addServletClass("/*", AaiSimulatorJaxRs.class.getName());
+               testServer.waitedStart(5000);
+               if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
+                       throw new IllegalStateException("cannot connect to port " + testServer.getPort());
+               return testServer;
+       }
 
-  public static HttpServletServer buildVfcSim() throws InterruptedException, IOException {
-    final HttpServletServer testServer = HttpServletServer.factory.build(VFCSIM_SERVER_NAME,
-        "localhost", VFCSIM_SERVER_PORT, "/", false, true);
-    testServer.addServletClass("/*", VfcSimulatorJaxRs.class.getName());
-    testServer.waitedStart(5000);
-    if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
-      throw new IllegalStateException("cannot connect to port " + testServer.getPort());
-    return testServer;
-  }
+       public static HttpServletServer buildSoSim() throws InterruptedException, IOException {
+               final HttpServletServer testServer = HttpServletServer.factory.build(SOSIM_SERVER_NAME,
+                               "localhost", SOSIM_SERVER_PORT, "/", false, true);
+               testServer.addServletClass("/*", SoSimulatorJaxRs.class.getName());
+               testServer.waitedStart(5000);
+               if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
+                       throw new IllegalStateException("cannot connect to port " + testServer.getPort());
+               return testServer;
+       }
 
-  public static HttpServletServer buildGuardSim() throws InterruptedException, IOException {
-    HttpServletServer testServer = HttpServletServer.factory.build(GUARDSIM_SERVER_NAME, "localhost", GUARDSIM_SERVER_PORT, "/", false, true);
-    testServer.addServletClass("/*", GuardSimulatorJaxRs.class.getName());
-    testServer.waitedStart(5000);
-    if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
-       throw new IllegalStateException("cannot connect to port " + testServer.getPort());
-    return testServer;
-  }
+       public static HttpServletServer buildVfcSim() throws InterruptedException, IOException {
+               final HttpServletServer testServer = HttpServletServer.factory.build(VFCSIM_SERVER_NAME,
+                               "localhost", VFCSIM_SERVER_PORT, "/", false, true);
+               testServer.addServletClass("/*", VfcSimulatorJaxRs.class.getName());
+               testServer.waitedStart(5000);
+               if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
+                       throw new IllegalStateException("cannot connect to port " + testServer.getPort());
+               return testServer;
+       }
+
+       public static HttpServletServer buildGuardSim() throws InterruptedException, IOException {
+               HttpServletServer testServer = HttpServletServer.factory.build(GUARDSIM_SERVER_NAME, "localhost", GUARDSIM_SERVER_PORT, "/", false, true);
+               testServer.addServletClass("/*", GuardSimulatorJaxRs.class.getName());
+               testServer.waitedStart(5000);
+               if (!NetworkUtil.isTcpPortOpen("localhost", testServer.getPort(), 5, 10000L))
+                       throw new IllegalStateException("cannot connect to port " + testServer.getPort());
+               return testServer;
+       }
 }