From 57a8c14ed7b5ef6b39b7e63e76d14dfcac32fb69 Mon Sep 17 00:00:00 2001 From: aribeiro Date: Fri, 8 Nov 2019 10:53:56 +0000 Subject: Fix sonarQube issue. Issue-ID: SDC-2646 Change-Id: Ic426e1cccf50e3db40381896ee50b74e647129ee Signed-off-by: aribeiro --- .../impl/pnfd/PnfTransformationEngineTest.java | 28 ++++++++++++++++++++++ .../parser/ConversionDefinitionYamlParserTest.java | 8 +++++++ 2 files changed, 36 insertions(+) (limited to 'openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter') diff --git a/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/PnfTransformationEngineTest.java b/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/PnfTransformationEngineTest.java index be13d775e3..63a7d890d0 100644 --- a/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/PnfTransformationEngineTest.java +++ b/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/PnfTransformationEngineTest.java @@ -20,6 +20,8 @@ package org.openecomp.core.converter.impl.pnfd; import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertNull; +import static org.mockito.Mockito.spy; import java.io.IOException; import java.io.InputStream; @@ -27,17 +29,24 @@ import java.nio.file.Files; import java.nio.file.Path; import java.nio.file.Paths; import java.util.Collection; +import java.util.HashMap; +import java.util.Map; +import java.util.Optional; import java.util.stream.Collectors; import java.util.stream.Stream; +import mockit.Deencapsulation; import org.apache.commons.io.IOUtils; import org.junit.Test; import org.junit.runner.RunWith; import org.junit.runners.Parameterized; +import org.mockito.Mockito; import org.onap.sdc.tosca.datatypes.model.ServiceTemplate; import org.onap.sdc.tosca.services.ToscaExtensionYamlUtil; import org.onap.sdc.tosca.services.YamlUtil; import org.openecomp.core.converter.ServiceTemplateReaderService; +import org.openecomp.core.converter.impl.pnfd.parser.PnfdCustomNodeTypeBlockParser; import org.openecomp.core.converter.pnfd.PnfdTransformationEngine; +import org.openecomp.core.converter.pnfd.model.ConversionDefinition; import org.openecomp.core.impl.services.ServiceTemplateReaderServiceImpl; @RunWith(Parameterized.class) @@ -79,6 +88,25 @@ public class PnfTransformationEngineTest { assertEquals(expectedResult, result); } + @Test + public void testBuildParseBlock() { + final PnfdCustomNodeTypeBlockParser blockParser = spy(new PnfdCustomNodeTypeBlockParser(null)); + final ConversionDefinition conversionDefinition = Mockito.mock(ConversionDefinition.class); + final Map stringObjectMap = new HashMap<>(); + stringObjectMap.put("type", null); + stringObjectMap.put("name", null); + assertEquals(Optional.empty(), Deencapsulation.invoke(blockParser, "buildParsedBlock", + stringObjectMap, stringObjectMap, conversionDefinition)); + } + + @Test + public void testReadDefinition() { + final PnfdTransformationEngine engine = spy( + new PnfdNodeTemplateTransformationEngine(null, null, "test.txt")); + Deencapsulation.invoke(engine, "readDefinition"); + assertNull(Deencapsulation.getField(engine, "transformationDescription")); + } + private String getExpectedResultFor(final String inputFilename) throws IOException { try (final InputStream inputStream = getOutputFileResourceCorrespondingTo(inputFilename)) { final ServiceTemplate serviceTemplate = toscaExtensionYamlUtil.yamlToObject(inputStream, ServiceTemplate.class); diff --git a/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/parser/ConversionDefinitionYamlParserTest.java b/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/parser/ConversionDefinitionYamlParserTest.java index 2326318657..cb901b6f3c 100644 --- a/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/parser/ConversionDefinitionYamlParserTest.java +++ b/openecomp-be/lib/openecomp-tosca-converter-lib/openecomp-tosca-converter-core/src/test/java/org/openecomp/core/converter/impl/pnfd/parser/ConversionDefinitionYamlParserTest.java @@ -21,6 +21,7 @@ package org.openecomp.core.converter.impl.pnfd.parser; import static org.hamcrest.Matchers.equalTo; import static org.hamcrest.Matchers.notNullValue; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertThat; import static org.junit.Assert.fail; import static org.openecomp.core.converter.pnfd.model.PnfTransformationToken.QUERY; @@ -28,7 +29,9 @@ import static org.openecomp.core.converter.pnfd.model.PnfTransformationToken.TO_ import java.io.IOException; import java.io.InputStream; +import java.util.HashMap; import java.util.Map; +import java.util.Optional; import org.junit.Test; import org.openecomp.core.converter.pnfd.model.ConversionDefinition; import org.openecomp.core.converter.pnfd.model.ConversionStrategyType; @@ -51,6 +54,11 @@ public class ConversionDefinitionYamlParserTest { assertConversionDefinition(definitionYaml, conversionDefinition); } + @Test + public void shouldReturnEmpty() { + assertEquals(Optional.empty(), ConversionDefinitionYamlParser.parse(new HashMap<>())); + } + private void assertConversionDefinition(final Map definitionYaml, final ConversionDefinition conversionDefinition) { assertThat("The conversion definition should have been built" -- cgit 1.2.3-korg