From a80a2e20ab5414c35a948fd58613df33d3bee3d3 Mon Sep 17 00:00:00 2001 From: "andre.schmid" Date: Thu, 10 Oct 2019 17:39:03 +0100 Subject: Validate non-mano software information artifact Change-Id: Ie9fabd6af821b830e211ff1068fb287357fd38ee Issue-ID: SDC-2619 Signed-off-by: andre.schmid --- .../src/test/java/org/openecomp/sdc/TestUtils.java | 34 ++++--------- .../SoftwareInformationArtifactYamlParserTest.java | 59 ---------------------- .../impl/SoftwareInformationBusinessLogicTest.java | 4 +- 3 files changed, 11 insertions(+), 86 deletions(-) delete mode 100644 catalog-be/src/test/java/org/openecomp/sdc/be/components/csar/SoftwareInformationArtifactYamlParserTest.java (limited to 'catalog-be/src/test/java') diff --git a/catalog-be/src/test/java/org/openecomp/sdc/TestUtils.java b/catalog-be/src/test/java/org/openecomp/sdc/TestUtils.java index 809dc7ddb0..cb0e5e8a9b 100644 --- a/catalog-be/src/test/java/org/openecomp/sdc/TestUtils.java +++ b/catalog-be/src/test/java/org/openecomp/sdc/TestUtils.java @@ -20,36 +20,20 @@ package org.openecomp.sdc; -import org.apache.commons.io.IOUtils; +import static org.junit.Assert.assertArrayEquals; -import javax.ws.rs.core.Response; -import java.io.IOException; import java.io.InputStream; - -import static org.junit.Assert.assertArrayEquals; +import javax.ws.rs.core.Response; +import org.apache.commons.io.IOUtils; public class TestUtils { - public static boolean downloadedPayloadMatchesExpected(Response response, byte[] expected) { - boolean result = false; - try (InputStream is = response.readEntity(InputStream.class)) { - byte[] body = IOUtils.toByteArray(is); + public static boolean downloadedPayloadMatchesExpected(final Response response, final byte[] expected) { + try (final InputStream is = response.readEntity(InputStream.class)) { + final byte[] body = IOUtils.toByteArray(is); assertArrayEquals(expected, body); - result = true; - } catch(Exception ex) { - result = false; - } - return result; - } - - public static InputStream getResourceAsStream(final String resourcePath) { - return TestUtils.class.getClassLoader().getResourceAsStream(resourcePath); - } - - public static byte[] getResourceAsByteArray(final String resourcePath) throws IOException { - final InputStream resourceAsStream = getResourceAsStream(resourcePath); - if (resourceAsStream == null) { - throw new IOException("Could not find file: " + resourcePath); + return true; + } catch (final Exception ignored) { + return false; } - return IOUtils.toByteArray(resourceAsStream); } } diff --git a/catalog-be/src/test/java/org/openecomp/sdc/be/components/csar/SoftwareInformationArtifactYamlParserTest.java b/catalog-be/src/test/java/org/openecomp/sdc/be/components/csar/SoftwareInformationArtifactYamlParserTest.java deleted file mode 100644 index 6f826047a4..0000000000 --- a/catalog-be/src/test/java/org/openecomp/sdc/be/components/csar/SoftwareInformationArtifactYamlParserTest.java +++ /dev/null @@ -1,59 +0,0 @@ -/* - * ============LICENSE_START======================================================= - * Copyright (C) 2019 Nordix Foundation - * ================================================================================ - * 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. - * - * SPDX-License-Identifier: Apache-2.0 - * ============LICENSE_END========================================================= - */ - -package org.openecomp.sdc.be.components.csar; - -import static org.hamcrest.CoreMatchers.equalTo; -import static org.hamcrest.CoreMatchers.hasItems; -import static org.hamcrest.CoreMatchers.is; -import static org.junit.Assert.assertThat; - -import java.io.IOException; -import java.util.Optional; -import org.junit.Test; -import org.openecomp.sdc.TestUtils; - -public class SoftwareInformationArtifactYamlParserTest { - - @Test - public void parse() throws IOException { - //given - final byte[] resourceAsByteArray = TestUtils - .getResourceAsByteArray("artifacts/pnfSoftwareInformation/pnf-sw-information.yaml"); - //when - final Optional pnfSoftwareInformation = SoftwareInformationArtifactYamlParser - .parse(resourceAsByteArray); - //then - final PnfSoftwareVersion expectedPnfSoftwareVersion1 = new PnfSoftwareVersion("version1", "first software version of PNF"); - final PnfSoftwareVersion expectedPnfSoftwareVersion2 = new PnfSoftwareVersion("version2", "second software version of PNF"); - assertThat("The software information should be parsed", pnfSoftwareInformation.isPresent(), is(true)); - pnfSoftwareInformation.ifPresent(softwareInformation -> { - assertThat("The software information provider should be as expected", - softwareInformation.getProvider(), is(equalTo("Ericsson"))); - assertThat("The software information description should be as expected", - softwareInformation.getDescription(), is(equalTo("pnf software information"))); - assertThat("The software information version should be as expected", - softwareInformation.getVersion(), is(equalTo("1.0"))); - assertThat("The software versions should contain expected versions", - softwareInformation.getSoftwareVersionSet(), - hasItems(expectedPnfSoftwareVersion1, expectedPnfSoftwareVersion2)); - }); - } - -} \ No newline at end of file diff --git a/catalog-be/src/test/java/org/openecomp/sdc/be/components/impl/SoftwareInformationBusinessLogicTest.java b/catalog-be/src/test/java/org/openecomp/sdc/be/components/impl/SoftwareInformationBusinessLogicTest.java index 2eda76fa16..99bccfc566 100644 --- a/catalog-be/src/test/java/org/openecomp/sdc/be/components/impl/SoftwareInformationBusinessLogicTest.java +++ b/catalog-be/src/test/java/org/openecomp/sdc/be/components/impl/SoftwareInformationBusinessLogicTest.java @@ -34,11 +34,11 @@ import org.junit.runner.RunWith; import org.mockito.Mock; import org.mockito.Mockito; import org.mockito.junit.MockitoJUnitRunner; -import org.openecomp.sdc.TestUtils; import org.openecomp.sdc.be.components.csar.CsarInfo; import org.openecomp.sdc.be.components.impl.exceptions.BusinessLogicException; import org.openecomp.sdc.be.model.PropertyDefinition; import org.openecomp.sdc.be.model.Resource; +import org.openecomp.sdc.be.test.util.TestResourcesHandler; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -148,7 +148,7 @@ public class SoftwareInformationBusinessLogicTest { private void mockCsarFileMap(final String softwareInformationArtifactPath) { final byte[] softwareInformationFile; try { - softwareInformationFile = TestUtils.getResourceAsByteArray(softwareInformationArtifactPath); + softwareInformationFile = TestResourcesHandler.getResourceAsByteArray(softwareInformationArtifactPath); } catch (final IOException e) { final String errorMsg = "Could not find software information artifact " + softwareInformationArtifactPath; LOGGER.error(errorMsg, e); -- cgit 1.2.3-korg