NeutronSecurityGroupResourceValidator

This task is about  updating  error messages with error codes for NeutronSecurityGroupResourceValidator

Change-Id: I6279c76dae5aeff46bee0ffc9d097ba5b7ff267e
Issue-ID: SDC-572
Signed-off-by: mojahidi <mojahidul.islam@amdocs.com>

1 files changed, 25 insertions, 20 deletions

diff --git a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/heatresource/NeutronSecurityGroupResourceValidator.java b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/heatresource/NeutronSecurityGroupResourceValidator.java
index d7e16152f9..0a066f5c38 100644
--- a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/heatresource/NeutronSecurityGroupResourceValidator.java
+++ b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/heatresource/NeutronSecurityGroupResourceValidator.java
@@ -1,6 +1,7 @@
 package org.openecomp.sdc.validation.impl.validators.heatresource;
 
 import org.apache.commons.collections4.MapUtils;
+import org.openecomp.core.validation.ErrorMessageCode;
 import org.openecomp.core.validation.errors.ErrorMessagesFormatBuilder;
 import org.openecomp.core.validation.types.GlobalValidationContext;
 import org.openecomp.sdc.common.errors.Messages;
@@ -21,47 +22,51 @@ import java.util.Map;
  * Created by TALIO on 2/27/2017.
  */
 public class NeutronSecurityGroupResourceValidator implements ResourceValidator {
+  private static final ErrorMessageCode ERROR_CODE_HSG1 = new ErrorMessageCode("HSG1");
+
   @Override
   public void validate(String fileName, Map.Entry<String, Resource> resourceEntry,
                        GlobalValidationContext globalContext, ValidationContext validationContext) {
 
     HeatResourceValidationContext heatResourceValidationContext =
-        (HeatResourceValidationContext) validationContext;
-    validateSecurityGroupIsUsed(fileName, resourceEntry, heatResourceValidationContext, globalContext);
+            (HeatResourceValidationContext) validationContext;
+    validateSecurityGroupIsUsed(fileName, resourceEntry, heatResourceValidationContext,
+            globalContext);
   }
 
-  public void validateSecurityGroupIsUsed(String fileName, Map.Entry<String, Resource> resourceEntry,
-                                          HeatResourceValidationContext
-                                              heatResourceValidationContext,
-                                          GlobalValidationContext globalContext) {
+  private void validateSecurityGroupIsUsed(String fileName,
+                                           Map.Entry<String, Resource> resourceEntry,
+                                           HeatResourceValidationContext
+                                                   heatResourceValidationContext,
+                                           GlobalValidationContext globalContext) {
 
     Map<String, Map<String, List<String>>> securityGroupsPointedByOtherResources =
-        heatResourceValidationContext.getFileLevelResourceDependencies().
-            get(HeatResourcesTypes.NEUTRON_SECURITY_GROUP_RESOURCE_TYPE.getHeatResource());
+            heatResourceValidationContext.getFileLevelResourceDependencies()
+                    .get(HeatResourcesTypes.NEUTRON_SECURITY_GROUP_RESOURCE_TYPE.getHeatResource());
 
     if (MapUtils.isEmpty(securityGroupsPointedByOtherResources)) {
       return;
     }
 
     Map<String, List<String>> resourcesPointingCurrSecurityGroup =
-        securityGroupsPointedByOtherResources.get(resourceEntry.getKey());
+            securityGroupsPointedByOtherResources.get(resourceEntry.getKey());
 
-    if(isSecurityGroupNotInUse(resourcesPointingCurrSecurityGroup)){
+    if (isSecurityGroupNotInUse(resourcesPointingCurrSecurityGroup)) {
       globalContext.addMessage(
-          fileName,
-          ErrorLevel.WARNING,
-          ErrorMessagesFormatBuilder
-              .getErrorWithParameters(
-                  Messages.RESOURCE_NOT_IN_USE.getErrorMessage(),
-                  ValidatorConstants.Security_Group, resourceEntry.getKey()),
-          LoggerTragetServiceName.VALIDATE_ALL_SERVER_GROUP_OR_SECURITY_GROUP_IN_USE,
-          LoggerErrorDescription.SERVER_GROUP_SECURITY_GROUP_NOT_IN_USE);
+              fileName,
+              ErrorLevel.WARNING,
+              ErrorMessagesFormatBuilder
+                      .getErrorWithParameters(
+                              ERROR_CODE_HSG1, Messages.RESOURCE_NOT_IN_USE.getErrorMessage(),
+                              ValidatorConstants.Security_Group, resourceEntry.getKey()),
+              LoggerTragetServiceName.VALIDATE_ALL_SERVER_GROUP_OR_SECURITY_GROUP_IN_USE,
+              LoggerErrorDescription.SERVER_GROUP_SECURITY_GROUP_NOT_IN_USE);
     }
 
   }
 
-  public boolean isSecurityGroupNotInUse(Map<String, List<String>>
-                                             referencingResourcesToCurrSecurityGroup){
+  private boolean isSecurityGroupNotInUse(Map<String, List<String>>
+                                                  referencingResourcesToCurrSecurityGroup) {
     return MapUtils.isEmpty(referencingResourcesToCurrSecurityGroup);
   }
 }