Fix sonar issue in PdpGroups 56/85356/1
authorJim Hahn <jrh3@att.com>
Mon, 15 Apr 2019 18:21:18 +0000 (14:21 -0400)
committerJim Hahn <jrh3@att.com>
Mon, 15 Apr 2019 18:21:18 +0000 (14:21 -0400)
Extracted common string constant.

Change-Id: I5721641d8a45e7dce317cb6a1b75570a715779ea
Issue-ID: POLICY-1542
Signed-off-by: Jim Hahn <jrh3@att.com>
models-pdp/src/main/java/org/onap/policy/models/pdp/concepts/PdpGroups.java

index ab9afb0..06194ea 100644 (file)
@@ -41,6 +41,8 @@ import org.onap.policy.common.parameters.ValidationStatus;
 @Setter
 @ToString
 public class PdpGroups {
+    private static final String GROUPS_FIELD = "groups";
+
     private List<PdpGroup> groups;
 
     /**
@@ -64,9 +66,9 @@ public class PdpGroups {
      * @return the validation result
      */
     public ValidationResult validatePapRest() {
-        BeanValidationResult result = new BeanValidationResult("groups", this);
+        BeanValidationResult result = new BeanValidationResult(GROUPS_FIELD, this);
 
-        result.validateNotNullList("groups", groups, PdpGroup::validatePapRest);
+        result.validateNotNullList(GROUPS_FIELD, groups, PdpGroup::validatePapRest);
         if (!result.isValid()) {
             return result;
         }
@@ -78,6 +80,6 @@ public class PdpGroups {
         }
 
         // different sizes implies duplicates
-        return new ObjectValidationResult("groups", names, ValidationStatus.INVALID, "duplicate group names");
+        return new ObjectValidationResult(GROUPS_FIELD, names, ValidationStatus.INVALID, "duplicate group names");
     }
 }