Merge "Improved the UT coverage"

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmInfo.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmInfo.java
index fe1f677..c7881d1 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmInfo.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmInfo.java
@@ -17,9 +17,12 @@ package org.onap.holmes.common.api.entity;
 \r
 import com.google.gson.annotations.SerializedName;\r
 import lombok.Data;\r
+import lombok.Getter;\r
 import lombok.NoArgsConstructor;\r
+import lombok.Setter;\r
 \r
-@Data\r
+@Getter\r
+@Setter\r
 @NoArgsConstructor\r
 public class AlarmInfo {\r
 \r

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmsCorrelation.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmsCorrelation.java
index 4cbd6bf..e21f97d 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmsCorrelation.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/AlarmsCorrelation.java
@@ -18,9 +18,12 @@ package org.onap.holmes.common.api.entity;
 import com.google.gson.annotations.SerializedName;\r
 import java.util.Date;\r
 import lombok.Data;\r
+import lombok.Getter;\r
 import lombok.NoArgsConstructor;\r
+import lombok.Setter;\r
 \r
-@Data\r
+@Getter\r
+@Setter\r
 @NoArgsConstructor\r
 public class AlarmsCorrelation {\r
 \r
@@ -47,4 +50,18 @@ public class AlarmsCorrelation {
     private long reserve2 = -1;\r
 \r
     private long reserve3 = -1;\r
+\r
+    @Override\r
+    public String toString() {\r
+        StringBuilder sb = new StringBuilder("[");\r
+        sb.append("ruleId=").append(ruleId).append(",");\r
+        sb.append("ruleInfo=").append(ruleInfo).append(",");\r
+        sb.append("createTime=").append(createTime == null ? null : createTime.toString()).append(",");\r
+        sb.append("parentAlarmId=").append(parentAlarmId).append(",");\r
+        sb.append("childAlarmId=").append(childAlarmId).append(",");\r
+        sb.append("reserve1=").append(reserve1).append(",");\r
+        sb.append("reserve2=").append(reserve2).append(",");\r
+        sb.append("reserve3=").append(reserve3).append("]");\r
+        return sb.toString();\r
+    }\r
 }\r

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/CorrelationRule.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/CorrelationRule.java
index db6c19f..14a3f07 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/CorrelationRule.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/CorrelationRule.java
@@ -61,23 +61,23 @@ public class CorrelationRule implements Cloneable {
             throw new InternalError(e);\r
         }\r
 \r
-        r.rid = rid == null ? null : rid;\r
-        r.name = name == null ? null : name;\r
-        r.description = description == null ? null : description;\r
+        r.rid = rid;\r
+        r.name = name;\r
+        r.description = description;\r
         r.enabled = enabled;\r
         r.templateID = templateID;\r
-        r.engineID = engineID == null ? null : engineID;\r
-        r.engineType = engineType == null ? null : engineType;\r
-        r.creator = creator == null ? null : creator;\r
-        r.modifier = modifier  == null ? null : modifier;\r
+        r.engineID = engineID;\r
+        r.engineType = engineType;\r
+        r.creator = creator;\r
+        r.modifier = modifier;\r
         r.params = params == null ? null : (Properties) params.clone();\r
-        r.content = content  == null ? null : content;\r
-        r.vendor = vendor  == null ? null : vendor;\r
+        r.content = content;\r
+        r.vendor = vendor;\r
         r.createTime = createTime == null ? null : (Date) createTime.clone();\r
         r.updateTime = updateTime == null ? null : (Date) updateTime.clone();\r
-        r.packageName = packageName == null ? null : packageName;\r
-        r.closedControlLoopName = closedControlLoopName == null ? null : closedControlLoopName;\r
-        r.engineInstance = engineInstance == null ? null : engineInstance;\r
+        r.packageName = packageName;\r
+        r.closedControlLoopName = closedControlLoopName;\r
+        r.engineInstance = engineInstance;\r
 \r
         return r;\r
     }\r

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceEntity.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceEntity.java
index 0244f84..4bfb3cf 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceEntity.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceEntity.java
@@ -16,16 +16,14 @@
 package org.onap.holmes.common.api.entity;
 
 
-import lombok.AllArgsConstructor;
-import lombok.Data;
-import lombok.NoArgsConstructor;
+import lombok.*;
 
 import java.util.List;
 
-
-@Data
 @NoArgsConstructor
 @AllArgsConstructor
+@Getter
+@Setter
 public class ServiceEntity {
 
     private String serviceName;

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode.java
index 2c2b793..a8fc04d 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode.java
@@ -23,7 +23,6 @@ import lombok.Getter;
 import lombok.NoArgsConstructor;\r
 import lombok.Setter;\r
 \r
-@Data\r
 @NoArgsConstructor\r
 @AllArgsConstructor\r
 @Getter\r

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode4Query.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode4Query.java
index 205126d..98e5506 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode4Query.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceNode4Query.java
@@ -15,11 +15,10 @@
  */
 package org.onap.holmes.common.api.entity;
 
-import lombok.AllArgsConstructor;
-import lombok.Data;
-import lombok.NoArgsConstructor;
+import lombok.*;
 
-@Data
+@Getter
+@Setter
 @NoArgsConstructor
 @AllArgsConstructor
 public class ServiceNode4Query {

diff --git a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceRegisterEntity.java b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceRegisterEntity.java
index f5dca01..eba7544 100644 (file)
--- a/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceRegisterEntity.java
+++ b/holmes-actions/src/main/java/org/onap/holmes/common/api/entity/ServiceRegisterEntity.java
@@ -24,7 +24,6 @@ import lombok.Getter;
 import lombok.NoArgsConstructor;\r
 import lombok.Setter;\r
 \r
-@Data\r
 @NoArgsConstructor\r
 @AllArgsConstructor\r
 @Setter\r

diff --git a/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/AlarmsCorrelationTest.java b/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/AlarmsCorrelationTest.java
index dd940c4..822d74c 100644 (file)
--- a/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/AlarmsCorrelationTest.java
+++ b/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/AlarmsCorrelationTest.java
@@ -110,7 +110,6 @@ public class AlarmsCorrelationTest {
         assertThat(alarmsCorrelation.getReserve3(), equalTo(reserve3));
     }
 
-
     @Test
     public void testToString() throws Exception {
         final AlarmsCorrelation alarmsCorrelationTemp = new AlarmsCorrelation();
@@ -119,14 +118,4 @@ public class AlarmsCorrelationTest {
         alarmsCorrelation.setRuleId(ruleId);
         assertThat(alarmsCorrelation.toString(), equalTo(alarmsCorrelationTemp.toString()));
     }
-
-
-    @Test
-    public void testEqualsAndHashCode() throws Exception {
-        final AlarmsCorrelation alarmsCorrelationTemp = new AlarmsCorrelation();
-        String ruleId = "ruleId";
-        alarmsCorrelationTemp.setRuleId(ruleId);
-        alarmsCorrelation.setRuleId(ruleId);
-        assertThat(alarmsCorrelation, equalTo(alarmsCorrelationTemp));
-    }
-} 
+}

diff --git a/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/CorrelationRuleTest.java b/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/CorrelationRuleTest.java
index 1a25a35..c4b605a 100644 (file)
--- a/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/CorrelationRuleTest.java
+++ b/holmes-actions/src/test/java/org/onap/holmes/common/api/entity/CorrelationRuleTest.java
@@ -150,4 +150,19 @@ public class CorrelationRuleTest {
         correlationRule.setEngineInstance(engineInstance);
         assertThat(correlationRule.getEngineInstance(), equalTo(engineInstance));
     }
+
+    @Test
+    public void getterAndSetter4ClosedControlLoopName(){
+        final String closedControlLoopName = "closedControlLoopName";
+        correlationRule.setClosedControlLoopName(closedControlLoopName);
+        assertThat(correlationRule.getClosedControlLoopName(), equalTo(closedControlLoopName));
+    }
+
+    @Test
+    public void testClone() {
+        final String rid = "rid";
+        correlationRule.setRid(rid);
+        CorrelationRule rule = (CorrelationRule) correlationRule.clone();
+        assertThat(rule.getRid(), equalTo(rid));
+    }
 } 

diff --git a/holmes-actions/src/test/java/org/onap/holmes/common/utils/MSBRegisterUtilTest.java b/holmes-actions/src/test/java/org/onap/holmes/common/utils/MSBRegisterUtilTest.java
index 76aad81..ab75e06 100644 (file)
--- a/holmes-actions/src/test/java/org/onap/holmes/common/utils/MSBRegisterUtilTest.java
+++ b/holmes-actions/src/test/java/org/onap/holmes/common/utils/MSBRegisterUtilTest.java
@@ -35,6 +35,7 @@ public class MSBRegisterUtilTest {
 \r
     @Rule\r
     public PowerMockRule powerMockRule = new PowerMockRule();\r
+\r
     private MSBRegisterUtil msbRegisterUtil = new MSBRegisterUtil();\r
 \r
     @Test\r

diff --git a/pom.xml b/pom.xml
index c7cb10c..8135422 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -30,13 +30,13 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>\r
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>\r
         <maven.test.skip>false</maven.test.skip>\r
+        <powermock.version>1.7.1</powermock.version>\r
         <maven.test.failure.ignore>false</maven.test.failure.ignore>\r
         <finalName>${project.artifactId}-${project.version}</finalName>\r
         <release.dir>${basedir}/target</release.dir>\r
         <bundle.name>${project.artifactId}-${project.version}</bundle.name>\r
         <pkgzip.dir>${basedir}/../release/pkgzip</pkgzip.dir>\r
         <excludesFile>**/*$*</excludesFile>\r
-        <nexusproxy>https://nexus.open-o.org/content</nexusproxy>\r
 \r
         <stringtemplate.version>3.2.1</stringtemplate.version>\r
         <dropwizard.version>1.3.0</dropwizard.version>\r
@@ -46,6 +46,7 @@
         <slf4j.version>1.7.25</slf4j.version>\r
         <quartz.version>2.2.1</quartz.version>\r
     </properties>\r
+\r
     <dependencyManagement>\r
         <dependencies>\r
             <dependency>\r
@@ -121,7 +122,7 @@
             <dependency>\r
                 <groupId>org.powermock</groupId>\r
                 <artifactId>powermock-module-junit4</artifactId>\r
-                <version>1.6.5</version>\r
+                <version>${powermock.version}</version>\r
                 <scope>test</scope>\r
             </dependency>\r
             <dependency>\r
@@ -133,25 +134,25 @@
             <dependency>\r
                 <groupId>org.powermock</groupId>\r
                 <artifactId>powermock-api-easymock</artifactId>\r
-                <version>1.6.5</version>\r
+                <version>${powermock.version}</version>\r
                 <scope>test</scope>\r
             </dependency>\r
             <dependency>\r
                 <groupId>org.powermock</groupId>\r
                 <artifactId>powermock-api-mockito</artifactId>\r
-                <version>1.7.1</version>\r
+                <version>${powermock.version}</version>\r
                 <scope>test</scope>\r
             </dependency>\r
             <dependency>\r
                 <groupId>org.powermock</groupId>\r
                 <artifactId>powermock-module-junit4-rule</artifactId>\r
-                <version>1.6.5</version>\r
+                <version>${powermock.version}</version>\r
                 <scope>test</scope>\r
             </dependency>\r
             <dependency>\r
                 <groupId>org.powermock</groupId>\r
                 <artifactId>powermock-classloading-xstream</artifactId>\r
-                <version>1.6.5</version>\r
+                <version>${powermock.version}</version>\r
                 <scope>test</scope>\r
             </dependency>\r
             <dependency>\r