From a8a96339680fa1c4df5577285442e902b5637631 Mon Sep 17 00:00:00 2001 From: vasraz Date: Wed, 19 Aug 2020 18:01:52 +0100 Subject: Fix Vulnerabilities reported by SONAR Change-Id: I8f4e173a4ea5111db55eebaf15be86f1583082ad Signed-off-by: Vasyl Razinkov Issue-ID: SDC-3248 --- .../sdc/be/datatypes/enums/GraphPropertyEnum.java | 58 +++------------- .../be/datatypes/enums/JsonPresentationFields.java | 36 +++------- .../be/datatypes/enums/GraphPropertyEnumTest.java | 80 ---------------------- .../enums/JsonPresentationFieldsTest.java | 50 -------------- 4 files changed, 20 insertions(+), 204 deletions(-) (limited to 'common-be/src') diff --git a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnum.java b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnum.java index 1f1c01f9a4..70f76030bd 100644 --- a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnum.java +++ b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnum.java @@ -18,8 +18,11 @@ package org.openecomp.sdc.be.datatypes.enums; import java.util.ArrayList; import java.util.List; import java.util.Map; +import lombok.AllArgsConstructor; +import lombok.Getter; - +@AllArgsConstructor +@Getter public enum GraphPropertyEnum { // field name ,class type ,unique ,indexed @@ -59,25 +62,16 @@ public enum GraphPropertyEnum { IS_ARCHIVED("isArchived", Boolean.class, false, true), IS_VSP_ARCHIVED("isVspArchived", Boolean.class, false, true), ARCHIVE_TIME("archiveTime", Long.class, false, true), - PREV_CATALOG_UPDATE_TIME ("previousUpdateTime", Long.class, false, true), - CURRENT_CATALOG_UPDATE_TIME ("currentUpdateTime", Long.class, false, true), + PREV_CATALOG_UPDATE_TIME("previousUpdateTime", Long.class, false, true), + CURRENT_CATALOG_UPDATE_TIME("currentUpdateTime", Long.class, false, true), //Healing HEALING_VERSION("healVersion", Integer.class, false, true); - - - private String property; - private Class clazz; - private boolean unique; - private boolean indexed; - - GraphPropertyEnum(String property, Class clazz, boolean unique, boolean indexed) { - this.property = property; - this.clazz = clazz; - this.unique = unique; - this.indexed = indexed; - } + private final String property; + private final Class clazz; + private final boolean unique; + private final boolean indexed; public static GraphPropertyEnum getByProperty(String property) { for (GraphPropertyEnum currProperty : GraphPropertyEnum.values()) { @@ -88,38 +82,6 @@ public enum GraphPropertyEnum { return null; } - public String getProperty() { - return property; - } - - public void setProperty(String property) { - this.property = property; - } - - public Class getClazz() { - return clazz; - } - - public void setClazz(Class clazz) { - this.clazz = clazz; - } - - public boolean isUnique() { - return unique; - } - - public void setUnique(boolean unique) { - this.unique = unique; - } - - public boolean isIndexed() { - return indexed; - } - - public void setIndexed(boolean indexed) { - this.indexed = indexed; - } - public static List getAllProperties() { List arrayList = new ArrayList<>(); diff --git a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFields.java b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFields.java index 46a3f99abc..41c04e77b2 100644 --- a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFields.java +++ b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFields.java @@ -17,7 +17,13 @@ package org.openecomp.sdc.be.datatypes.enums; import com.fasterxml.jackson.annotation.JsonCreator; import com.fasterxml.jackson.annotation.JsonValue; +import lombok.AllArgsConstructor; +import lombok.Getter; +import lombok.ToString; +@AllArgsConstructor +@Getter +@ToString(onlyExplicitlyIncluded = true) public enum JsonPresentationFields { UNIQUE_ID("uniqueId", GraphPropertyEnum.UNIQUE_ID), @@ -256,32 +262,16 @@ public enum JsonPresentationFields { TOSCA_DEFINITIONS_VERSION("tosca_definitions_version", null); - - private String presentation; - private GraphPropertyEnum storedAs; - - JsonPresentationFields(String presentation, GraphPropertyEnum storedAs) { - this.presentation = presentation; - this.storedAs = storedAs; - } + @ToString.Include + private final String presentation; + @ToString.Exclude + private final GraphPropertyEnum storedAs; @JsonValue public String getPresentation() { return presentation; } - public void setPresentation(String presentation) { - this.presentation = presentation; - } - - public GraphPropertyEnum getStoredAs() { - return storedAs; - } - - public void setStoredAs(GraphPropertyEnum storedAs) { - this.storedAs = storedAs; - } - public static String getPresentationByGraphProperty(GraphPropertyEnum property) { for (JsonPresentationFields currPresentation : JsonPresentationFields.values()) { if (currPresentation.getStoredAs() == property) { @@ -291,12 +281,6 @@ public enum JsonPresentationFields { return null; } - @Override - public String toString() { - // TODO Auto-generated method stub - return presentation; - } - @JsonCreator public static JsonPresentationFields getByPresentation(String presentation) { for (JsonPresentationFields inst : JsonPresentationFields.values()) { diff --git a/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnumTest.java b/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnumTest.java index 63d1384cda..c4165d4cc0 100644 --- a/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnumTest.java +++ b/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/GraphPropertyEnumTest.java @@ -40,86 +40,6 @@ public class GraphPropertyEnumTest { result = GraphPropertyEnum.getByProperty(GraphPropertyEnum.COMPONENT_TYPE.getProperty()); } - @Test - public void testGetProperty() throws Exception { - GraphPropertyEnum testSubject; - String result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getProperty(); - } - - @Test - public void testSetProperty() throws Exception { - GraphPropertyEnum testSubject; - String property = ""; - - // default test - testSubject = createTestSubject(); - testSubject.setProperty(property); - } - - @Test - public void testGetClazz() throws Exception { - GraphPropertyEnum testSubject; - Class result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getClazz(); - } - - @Test - public void testSetClazz() throws Exception { - GraphPropertyEnum testSubject; - Class clazz = null; - - // default test - testSubject = createTestSubject(); - testSubject.setClazz(clazz); - } - - @Test - public void testIsUnique() throws Exception { - GraphPropertyEnum testSubject; - boolean result; - - // default test - testSubject = createTestSubject(); - result = testSubject.isUnique(); - } - - @Test - public void testSetUnique() throws Exception { - GraphPropertyEnum testSubject; - boolean unique = false; - - // default test - testSubject = createTestSubject(); - testSubject.setUnique(unique); - } - - @Test - public void testIsIndexed() throws Exception { - GraphPropertyEnum testSubject; - boolean result; - - // default test - testSubject = createTestSubject(); - result = testSubject.isIndexed(); - } - - @Test - public void testSetIndexed() throws Exception { - GraphPropertyEnum testSubject; - boolean indexed = false; - - // default test - testSubject = createTestSubject(); - testSubject.setIndexed(indexed); - } - @Test public void testGetAllProperties() throws Exception { List result; diff --git a/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFieldsTest.java b/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFieldsTest.java index b0dc9aaef4..f31b6df3a1 100644 --- a/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFieldsTest.java +++ b/common-be/src/test/java/org/openecomp/sdc/be/datatypes/enums/JsonPresentationFieldsTest.java @@ -28,46 +28,6 @@ public class JsonPresentationFieldsTest { return JsonPresentationFields.API_URL; } - @Test - public void testGetPresentation() throws Exception { - JsonPresentationFields testSubject; - String result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getPresentation(); - } - - @Test - public void testSetPresentation() throws Exception { - JsonPresentationFields testSubject; - String presentation = ""; - - // default test - testSubject = createTestSubject(); - testSubject.setPresentation(presentation); - } - - @Test - public void testGetStoredAs() throws Exception { - JsonPresentationFields testSubject; - GraphPropertyEnum result; - - // default test - testSubject = createTestSubject(); - result = testSubject.getStoredAs(); - } - - @Test - public void testSetStoredAs() throws Exception { - JsonPresentationFields testSubject; - GraphPropertyEnum storedAs = null; - - // default test - testSubject = createTestSubject(); - testSubject.setStoredAs(storedAs); - } - @Test public void testGetPresentationByGraphProperty() throws Exception { GraphPropertyEnum property = null; @@ -78,16 +38,6 @@ public class JsonPresentationFieldsTest { result = JsonPresentationFields.getPresentationByGraphProperty(GraphPropertyEnum.INVARIANT_UUID); } - @Test - public void testToString() throws Exception { - JsonPresentationFields testSubject; - String result; - - // default test - testSubject = createTestSubject(); - result = testSubject.toString(); - } - @Test public void testGetByPresentation() throws Exception { String presentation = ""; -- cgit 1.2.3-korg