summaryrefslogtreecommitdiffstats
path: root/csarvalidation/src/main
diff options
context:
space:
mode:
authorBogumil Zebek <bogumil.zebek@nokia.com>2019-09-18 11:37:56 +0200
committerZebek Bogumil <bogumil.zebek@nokia.com>2019-09-18 11:37:56 +0200
commite0a0f45ecaf19e0122e745e8301b79ca4baf2e42 (patch)
tree9a07fefc7ac667378ab94d5fe89d121376ca0a8e /csarvalidation/src/main
parent3e0efbc34176ad3ad2e6d731a699f5684c0a527d (diff)
Sonar fix
Change-Id: I7c39a2d83d067382ffb91ed136ad840dab85ccd8 Issue-ID: VNFSDK-477 Signed-off-by: Zebek Bogumil <bogumil.zebek@nokia.com>
Diffstat (limited to 'csarvalidation/src/main')
-rw-r--r--csarvalidation/src/main/java/org/onap/cvc/csar/CSARArchive.java94
-rw-r--r--csarvalidation/src/main/java/org/onap/cvc/csar/parser/ManifestFileSplitter.java6
-rw-r--r--csarvalidation/src/main/java/org/onap/cvc/csar/parser/vnf/DefinitionMetadataParser.java133
3 files changed, 156 insertions, 77 deletions
diff --git a/csarvalidation/src/main/java/org/onap/cvc/csar/CSARArchive.java b/csarvalidation/src/main/java/org/onap/cvc/csar/CSARArchive.java
index 4bb7cf3..2055cf2 100644
--- a/csarvalidation/src/main/java/org/onap/cvc/csar/CSARArchive.java
+++ b/csarvalidation/src/main/java/org/onap/cvc/csar/CSARArchive.java
@@ -15,13 +15,19 @@
*/
package org.onap.cvc.csar;
+import com.fasterxml.jackson.core.JsonProcessingException;
+import com.fasterxml.jackson.databind.ObjectMapper;
+import org.apache.commons.io.FileUtils;
+import org.apache.commons.lang3.tuple.Pair;
+import org.onap.cvc.csar.parser.SourcesParser;
+import org.onap.cvc.csar.parser.vnf.DefinitionMetadataParser;
+import org.yaml.snakeyaml.Yaml;
+
import java.io.File;
import java.io.FileInputStream;
-import java.io.FilenameFilter;
import java.io.IOException;
import java.nio.file.Path;
import java.util.ArrayList;
-import java.util.Arrays;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
@@ -29,14 +35,6 @@ import java.util.Map;
import java.util.Objects;
import java.util.Optional;
-import org.apache.commons.io.FileUtils;
-import org.apache.commons.lang3.tuple.Pair;
-import org.onap.cvc.csar.parser.SourcesParser;
-import org.yaml.snakeyaml.Yaml;
-
-import com.fasterxml.jackson.core.JsonProcessingException;
-import com.fasterxml.jackson.databind.ObjectMapper;
-
/**
* Verify the CSAR package by following the SOL004 specifications and ONAP VNFREQS for TOSCA.
*
@@ -83,11 +81,6 @@ public class CSARArchive implements AutoCloseable {
public static final String ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_1 = "tosca_simple_yaml_1_1";
public static final String ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_2 = "tosca_simple_yaml_1_2";
- protected static final String[] Entry_Definition__tosca_definitions_versions = new String[] {
- ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_0,
- ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_1,
- ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_2
- };
public static final String ENTRY_DEFINITION_METADATA = "metadata";
public static final String ENTRY_DEFINITION_TEMPLATE_NAME = "template_name";
@@ -680,7 +673,7 @@ public class CSARArchive implements AutoCloseable {
private String templateVersion;
- public String getTempalteName() {
+ public String getTemplateName() {
return tempalteName;
}
@@ -953,67 +946,14 @@ public class CSARArchive implements AutoCloseable {
try(FileInputStream ipStream = new FileInputStream(this.definitionYamlFile)) {
Map<String, ?> yaml = (Map<String, ?>) new Yaml().load(ipStream);
- //yaml is empty or version string missing
- if (yaml == null || !yaml.keySet().contains(ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION)) {
- errors.add(
- new CSARErrorEntryMissingToscaDefinitionVersion(
- this.definitionYamlFile.getName()));
- } else {
- String version = (String) yaml.get(ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION);
- if (!Arrays.asList(Entry_Definition__tosca_definitions_versions).contains(version)) {
- errors.add(new CSARErrorInvalidEntry(ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION,
- this.definitionYamlFile.getName(), -1, "Should be " + ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_1));
- } else {
- this.definition.setToscaDefinitionVersion(version);
-
- if (this.toscaMeta.getMode().equals(Mode.WITHOUT_TOSCA_META_DIR)) {
- //metadata section should be there
- if (!yaml.keySet().contains(ENTRY_DEFINITION_METADATA)) {
- errors.add(
- new CSARErrorInvalidEntryValueToscaDefinitionVersion(
- this.definitionYamlFile.getName()));
- } else {
- Map<String, String> metadata = (Map<String, String>) yaml.get(ENTRY_DEFINITION_METADATA);
-
- for(Map.Entry<String, String> entry: metadata.entrySet()) {
- String key = entry.getKey();
- String value = entry.getValue();
-
- //continue till it reaches the metadata section
- if (key.equalsIgnoreCase(ENTRY_DEFINITION_TEMPLATE_AUTHOR)) {
- this.definition.getMetadata().setTemplateAuthor(value);
- } else if (key.equalsIgnoreCase(ENTRY_DEFINITION_TEMPLATE_NAME)) {
- this.definition.getMetadata().setTempalteName(value);
- } else if (key.equalsIgnoreCase(ENTRY_DEFINITION_TEMPLATE_VERSION)) {
- this.definition.getMetadata().setTemplateVersion(value);
- } else {
- errors.add(
- new CSARErrorIgnored(
- key,
- this.definitionYamlFile.getName(),
- -1,
- null));
- }
- }
-
- if (this.definition.getMetadata().getTemplateAuthor() == null) {
- this.errors.add(
- new CSARErrorEntryMissingToscaDefinitionMetadataTemplateAuthor(
- this.definitionYamlFile.getName()));
- }
- if (this.definition.getMetadata().getTempalteName() == null) {
- this.errors.add(new CSARErrorEntryMissingToscaDefinitionMetadataTemplateName(
- this.definitionYamlFile.getName()));
- }
-
- if (this.definition.getMetadata().getTemplateVersion() == null) {
- this.errors.add(new CSARErrorEntryMissingToscaDefinitionMetadataTemplateVersion(
- this.definitionYamlFile.getName()));
- }
- }
- }
- }
- }
+ DefinitionMetadataParser definitionMetadataParser = new DefinitionMetadataParser(
+ yaml,
+ this.definitionYamlFile.getName(),
+ this.toscaMeta.getMode()
+ );
+ final Pair<Definition, List<CSARError>> data = definitionMetadataParser.parse();
+ this.definition = data.getLeft();
+ this.errors.addAll(data.getRight());
}
}
diff --git a/csarvalidation/src/main/java/org/onap/cvc/csar/parser/ManifestFileSplitter.java b/csarvalidation/src/main/java/org/onap/cvc/csar/parser/ManifestFileSplitter.java
index 03a628c..d780e67 100644
--- a/csarvalidation/src/main/java/org/onap/cvc/csar/parser/ManifestFileSplitter.java
+++ b/csarvalidation/src/main/java/org/onap/cvc/csar/parser/ManifestFileSplitter.java
@@ -18,6 +18,9 @@
package org.onap.cvc.csar.parser;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
@@ -29,6 +32,8 @@ import java.util.stream.Stream;
public class ManifestFileSplitter {
+ private static final Logger LOG = LoggerFactory.getLogger(ManifestFileSplitter.class);
+
public ManifestFileModel split(File manifestFile) {
String fileName = manifestFile.getAbsolutePath();
List<String> data = new ArrayList<>();
@@ -39,6 +44,7 @@ public class ManifestFileSplitter {
return createManifestFileModel(data, cms, lines);
} catch (IOException e) {
+ LOG.error("Unable to process manifest file!", e);
throw new IllegalArgumentException(String.format("Unable to process manifest file! Wrong file path: '%s'", fileName));
}
}
diff --git a/csarvalidation/src/main/java/org/onap/cvc/csar/parser/vnf/DefinitionMetadataParser.java b/csarvalidation/src/main/java/org/onap/cvc/csar/parser/vnf/DefinitionMetadataParser.java
new file mode 100644
index 0000000..6743741
--- /dev/null
+++ b/csarvalidation/src/main/java/org/onap/cvc/csar/parser/vnf/DefinitionMetadataParser.java
@@ -0,0 +1,133 @@
+/*
+ * Copyright 2019 Nokia
+ * <p>
+ * 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
+ * <p>
+ * http://www.apache.org/licenses/LICENSE-2.0
+ * <p>
+ * 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.onap.cvc.csar.parser.vnf;
+
+import org.apache.commons.lang3.tuple.Pair;
+import org.onap.cvc.csar.CSARArchive;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Map;
+
+/**
+ * Code extracted from CSARArchive class.
+ */
+public class DefinitionMetadataParser {
+
+ private static final String[] ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSIONS = {
+ CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_0,
+ CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_1,
+ CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_2
+ };
+
+ private Map<String, ?> yaml;
+ private String definitionYamlFileName;
+ private CSARArchive.Mode mode;
+
+ public DefinitionMetadataParser(Map<String, ?> yaml, String definitionYamlFileName, CSARArchive.Mode mode) {
+ this.yaml = yaml;
+ this.definitionYamlFileName = definitionYamlFileName;
+ this.mode = mode;
+ }
+
+ public Pair<CSARArchive.Definition, List<CSARArchive.CSARError>> parse(){
+ List<CSARArchive.CSARError> errors = new ArrayList<>();
+ CSARArchive.Definition definition = new CSARArchive.Definition();
+
+ //yaml is empty or version string missing
+ if (yaml == null || !yaml.containsKey(CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION)) {
+ errors.add(
+ new CSARArchive.CSARErrorEntryMissingToscaDefinitionVersion(
+ this.definitionYamlFileName));
+ } else {
+ convertYamlIntoDefinitionModel(yaml, definition, errors);
+ }
+ return Pair.of(definition,errors);
+ }
+
+ private void convertYamlIntoDefinitionModel(Map<String, ?> yaml, CSARArchive.Definition definition, List<CSARArchive.CSARError> errors) {
+ String version = (String) yaml.get(CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION);
+ if (!Arrays.asList(ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSIONS).contains(version)) {
+ errors.add(new CSARArchive.CSARErrorInvalidEntry(CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION,
+ this.definitionYamlFileName, -1, "Should be " + CSARArchive.ENTRY_DEFINITION_TOSCA_DEFINITIONS_VERSION_SIMPLE_1_1));
+ } else {
+ updateDefinitionModel(definition, version, errors);
+ }
+ }
+
+ private void updateDefinitionModel(CSARArchive.Definition definition, String version, List<CSARArchive.CSARError> errors) {
+ definition.setToscaDefinitionVersion(version);
+
+ if (mode.equals(CSARArchive.Mode.WITHOUT_TOSCA_META_DIR)) {
+ //metadata section should be there
+ if (!yaml.containsKey(CSARArchive.ENTRY_DEFINITION_METADATA)) {
+ errors.add(
+ new CSARArchive.CSARErrorInvalidEntryValueToscaDefinitionVersion(
+ this.definitionYamlFileName));
+ } else {
+ Map<String, String> metadata = (Map<String, String>) yaml.get(CSARArchive.ENTRY_DEFINITION_METADATA);
+ final CSARArchive.Definition.Metadata metadataModel = definition.getMetadata();
+ populateMetadataModel(metadata, errors, metadataModel);
+ validateMetadataModel(errors, metadataModel);
+ }
+ }
+ }
+
+ private void populateMetadataModel(Map<String, String> metadata, List<CSARArchive.CSARError> errors, CSARArchive.Definition.Metadata metadataModel) {
+
+ for(Map.Entry<String, String> entry: metadata.entrySet()) {
+ String key = entry.getKey();
+ String value = entry.getValue();
+
+ //continue till it reaches the metadata section
+ if (key.equalsIgnoreCase(CSARArchive.ENTRY_DEFINITION_TEMPLATE_AUTHOR)) {
+ metadataModel.setTemplateAuthor(value);
+ } else if (key.equalsIgnoreCase(CSARArchive.ENTRY_DEFINITION_TEMPLATE_NAME)) {
+ metadataModel.setTempalteName(value);
+ } else if (key.equalsIgnoreCase(CSARArchive.ENTRY_DEFINITION_TEMPLATE_VERSION)) {
+ metadataModel.setTemplateVersion(value);
+ } else {
+ errors.add(
+ new CSARArchive.CSARErrorIgnored(
+ key,
+ this.definitionYamlFileName,
+ -1,
+ null));
+ }
+ }
+ }
+
+ private void validateMetadataModel(List<CSARArchive.CSARError> errors, CSARArchive.Definition.Metadata metadataModel) {
+ if (metadataModel.getTemplateAuthor() == null) {
+ errors.add(
+ new CSARArchive.CSARErrorEntryMissingToscaDefinitionMetadataTemplateAuthor(
+ this.definitionYamlFileName));
+ }
+ if (metadataModel.getTemplateName() == null) {
+ errors.add(new CSARArchive.CSARErrorEntryMissingToscaDefinitionMetadataTemplateName(
+ this.definitionYamlFileName));
+ }
+
+ if (metadataModel.getTemplateVersion() == null) {
+ errors.add(new CSARArchive.CSARErrorEntryMissingToscaDefinitionMetadataTemplateVersion(
+ this.definitionYamlFileName));
+ }
+ }
+
+}