From cf88f63a15e4cd7a66431b360269daa09c05ce9b Mon Sep 17 00:00:00 2001 From: mojahidi Date: Wed, 13 Dec 2017 17:23:49 +0530 Subject: Fixed sonar issues - SharedResourceGuideLineVal Removed unused method parameters Used isEmpty() to check whether the collection is empty or not Change-Id: Id812b2fd4d9b16b399eb9edc0f41e5e0c5d54211 Issue-ID: SDC-343 Signed-off-by: mojahidi --- .../SharedResourceGuideLineValidator.java | 58 +++++++++++++--------- 1 file changed, 35 insertions(+), 23 deletions(-) (limited to 'openecomp-be/lib') diff --git a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/SharedResourceGuideLineValidator.java b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/SharedResourceGuideLineValidator.java index fb7d6a2c97..c3c2ef2f6f 100644 --- a/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/SharedResourceGuideLineValidator.java +++ b/openecomp-be/lib/openecomp-sdc-validation-lib/openecomp-sdc-validation-impl/src/main/java/org/openecomp/sdc/validation/impl/validators/SharedResourceGuideLineValidator.java @@ -1,3 +1,19 @@ +/* + * Copyright © 2016-2017 European Support Limited + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + package org.openecomp.sdc.validation.impl.validators; import org.apache.commons.collections4.CollectionUtils; @@ -27,12 +43,10 @@ import java.util.HashSet; import java.util.Map; import java.util.Set; -/** - * Created by TALIO on 2/15/2017. - */ public class SharedResourceGuideLineValidator implements Validator { - public static final MdcDataDebugMessage mdcDataDebugMessage = new MdcDataDebugMessage(); - private final Logger log = (Logger) LoggerFactory.getLogger(this.getClass().getName()); + private static final MdcDataDebugMessage MDC_DATA_DEBUG_MESSAGE = new MdcDataDebugMessage(); + private static final Logger LOGGER = LoggerFactory.getLogger( + SharedResourceGuideLineValidator.class); private static final ErrorMessageCode ERROR_CODE_SRG_1 = new ErrorMessageCode("SRG1"); private static final ErrorMessageCode ERROR_CODE_SRG_2 = new ErrorMessageCode("SRG2"); private static final ErrorMessageCode ERROR_CODE_SRG_3 = new ErrorMessageCode("SRG3"); @@ -46,19 +60,17 @@ public class SharedResourceGuideLineValidator implements Validator { try { manifestContent = ValidationUtil.validateManifest(globalContext); } catch (Exception exception) { - log.debug("",exception); + LOGGER.error("Fialed to check Validation PreCondition ",exception); return; } Set baseFiles = validateManifest(manifestContent, globalContext); Map fileTypeMap = ManifestUtil.getFileTypeMap(manifestContent); - Map fileEnvMap = ManifestUtil.getFileAndItsEnv(manifestContent); globalContext.getFiles().stream() .filter(fileName -> FileData .isHeatFile(fileTypeMap.get(fileName))) .forEach(fileName -> validate(fileName, - fileEnvMap.get(fileName) != null ? fileEnvMap.get(fileName).getFile() : null, fileTypeMap, baseFiles, globalContext)); @@ -66,9 +78,9 @@ public class SharedResourceGuideLineValidator implements Validator { private Set validateManifest(ManifestContent manifestContent, GlobalValidationContext globalContext) { - mdcDataDebugMessage.debugEntryMessage("file", SdcCommon.MANIFEST_NAME); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage("file", SdcCommon.MANIFEST_NAME); Set baseFiles = ManifestUtil.getBaseFiles(manifestContent); - if (baseFiles == null || baseFiles.size() == 0) { + if (baseFiles == null || baseFiles.isEmpty()) { globalContext.addMessage( SdcCommon.MANIFEST_NAME, ErrorLevel.WARNING, @@ -89,7 +101,7 @@ public class SharedResourceGuideLineValidator implements Validator { LoggerTragetServiceName.VALIDATE_BASE_FILE, LoggerErrorDescription.MULTI_BASE_HEAT); } - mdcDataDebugMessage.debugExitMessage("file", SdcCommon.MANIFEST_NAME); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", SdcCommon.MANIFEST_NAME); return baseFiles; } @@ -99,7 +111,7 @@ public class SharedResourceGuideLineValidator implements Validator { + "]"; } - private void validate(String fileName, String envFileName, Map fileTypeMap, + private void validate(String fileName, Map fileTypeMap, Set baseFiles, GlobalValidationContext globalContext) { globalContext.setMessageCode(ERROR_CODE_SRG_5); HeatOrchestrationTemplate @@ -119,17 +131,17 @@ public class SharedResourceGuideLineValidator implements Validator { GlobalValidationContext globalContext) { - mdcDataDebugMessage.debugEntryMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage("file", fileName); //if not base return if (baseFiles == null || !baseFiles.contains(fileName)) { - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); return; } //if no resources exist return if (heatOrchestrationTemplate.getResources() == null || heatOrchestrationTemplate.getResources().size() == 0) { - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); return; } @@ -156,7 +168,7 @@ public class SharedResourceGuideLineValidator implements Validator { actualExposedResources.forEach(expectedExposedResources::remove); - if (expectedExposedResources.size() > 0) { + if (CollectionUtils.isNotEmpty(expectedExposedResources)) { expectedExposedResources .stream() .forEach(name -> globalContext.addMessage( @@ -169,7 +181,7 @@ public class SharedResourceGuideLineValidator implements Validator { LoggerErrorDescription.RESOURCE_NOT_DEFINED_AS_OUTPUT)); } - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); } private void validateHeatVolumeFile(String fileName, Map fileTypeMap, @@ -177,18 +189,18 @@ public class SharedResourceGuideLineValidator implements Validator { GlobalValidationContext globalContext) { - mdcDataDebugMessage.debugEntryMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage("file", fileName); //if not heat volume return if (!fileTypeMap.get(fileName).equals(FileData.Type.HEAT_VOL)) { - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); return; } //if no resources exist return if (heatOrchestrationTemplate.getResources() == null || heatOrchestrationTemplate.getResources().size() == 0) { - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); return; } @@ -214,7 +226,7 @@ public class SharedResourceGuideLineValidator implements Validator { actualExposedResources.forEach(expectedExposedResources::remove); - if (expectedExposedResources.size() > 0) { + if (CollectionUtils.isNotEmpty(expectedExposedResources)) { expectedExposedResources .stream() .forEach(name -> globalContext.addMessage( @@ -226,7 +238,7 @@ public class SharedResourceGuideLineValidator implements Validator { LoggerErrorDescription.VOLUME_FILE_NOT_EXPOSED)); } - mdcDataDebugMessage.debugExitMessage("file", fileName); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage("file", fileName); } @@ -234,7 +246,7 @@ public class SharedResourceGuideLineValidator implements Validator { GlobalValidationContext globalContext) { Set referenceValues = HeatStructureUtil.getReferencedValuesByFunctionName(filename, ResourceReferenceFunctions.GET_RESOURCE.getFunction(), value, globalContext); - return referenceValues != null && (referenceValues.size() > 0); + return referenceValues != null && (CollectionUtils.isNotEmpty(referenceValues)); } private String getResourceIdFromPropertyValue(String filename, Object value, -- cgit 1.2.3-korg