From 4aec2ba08cf3dcb75e07f2a66d162142382d4a4a Mon Sep 17 00:00:00 2001 From: "andre.schmid" Date: Wed, 20 Apr 2022 16:55:18 +0100 Subject: Fix unit tests with conflicting folder Fixes two tests that relies on the same output folder from ValidationConfigManager.txtReportFilePath. This was resulting in intermittent errors during the build. Change-Id: I7bafc022dd38e2fe8e648abc8ec1d113ef7346b4 Issue-ID: SDC-3975 Signed-off-by: andre.schmid --- .../artifacts/ArtifactValidationUtilsTest.java | 31 ++++++++++++------- .../impl/validator/utils/ReportManagerTest.java | 35 ++++++++++++++-------- 2 files changed, 42 insertions(+), 24 deletions(-) (limited to 'asdctool/src') diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactValidationUtilsTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactValidationUtilsTest.java index f3997edd40..44b13bd8cc 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactValidationUtilsTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/tasks/artifacts/ArtifactValidationUtilsTest.java @@ -31,15 +31,17 @@ import static org.openecomp.sdc.asdctool.impl.validator.report.ReportFileNioHelp import static org.openecomp.sdc.asdctool.impl.validator.report.ReportFileNioHelper.withTxtFile; import fj.data.Either; -import java.io.File; +import java.nio.file.Path; import java.util.ArrayList; import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.stream.IntStream; import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.io.TempDir; import org.mockito.InjectMocks; import org.mockito.Mock; import org.mockito.MockitoAnnotations; @@ -54,15 +56,14 @@ import org.openecomp.sdc.be.model.jsonjanusgraph.datamodel.TopologyTemplate; import org.openecomp.sdc.be.model.jsonjanusgraph.operations.TopologyTemplateOperation; import org.openecomp.sdc.be.model.operations.api.StorageOperationStatus; -public class ArtifactValidationUtilsTest { +class ArtifactValidationUtilsTest { private static final String ES_ID = "testEsInCassandra"; private static final String ES_ID_NOT_IN_CASS = "testEsNotInCassandra"; private static final String TASK_NAME = "testTaskName"; private static final String UNIQUE_ID = "4321"; private static final String UNIQUE_ID_VERTEX = "321"; - private static final String resourcePath = new File("src/test/resources").getAbsolutePath(); - private static final String txtReportFilePath = ValidationConfigManager.txtReportFilePath(resourcePath); + private static String txtReportFilePath; @InjectMocks private ArtifactValidationUtils testSubject; @@ -84,6 +85,14 @@ public class ArtifactValidationUtilsTest { @Mock private TopologyTemplate topologyTemplate; + @TempDir + static Path reportOutputPath; + + @BeforeAll + static void beforeAll() { + txtReportFilePath = ValidationConfigManager.txtReportFilePath(reportOutputPath.toString()); + } + @BeforeEach public void initMocks() { MockitoAnnotations.openMocks(this); @@ -98,7 +107,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testValidateArtifactsAreInCassandra() { + void testValidateArtifactsAreInCassandra() { // given Report report = Report.make(); List artifacts = new ArrayList<>(); @@ -119,7 +128,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testValidateArtifactsNotInCassandra() { + void testValidateArtifactsNotInCassandra() { // given Report report = Report.make(); List artifacts = new ArrayList<>(); @@ -148,7 +157,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testIsArtifactsInCassandra() { + void testIsArtifactsInCassandra() { // when boolean notInCass = testSubject.isArtifactInCassandra(ES_ID_NOT_IN_CASS); boolean inCass = testSubject.isArtifactInCassandra(ES_ID); @@ -159,7 +168,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testAddRelevantArtifacts() { + void testAddRelevantArtifacts() { // given Map artifactsMap = new HashMap<>(); artifactsMap.put(ES_ID_NOT_IN_CASS, artifactDataDefinitionNotInCassandra); @@ -173,7 +182,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testAddRelevantArtifactsWithNullEsId() { + void testAddRelevantArtifactsWithNullEsId() { // given Map artifactsMap = new HashMap<>(); artifactsMap.put("", artifactDataDefinitionDummy); @@ -186,7 +195,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testValidateTopologyTemplateArtifacts() { + void testValidateTopologyTemplateArtifacts() { // given Report report = Report.make(); Map artifacts = new HashMap<>(); @@ -223,7 +232,7 @@ public class ArtifactValidationUtilsTest { } @Test - public void testValidateTopologyTemplateArtifactsNotFoundToscaElement() { + void testValidateTopologyTemplateArtifactsNotFoundToscaElement() { // given Report report = Report.make(); when(topologyTemplateOperation.getToscaElement(eq(vertex.getUniqueId()), any())) diff --git a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManagerTest.java b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManagerTest.java index af056103ff..c8d524878d 100644 --- a/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManagerTest.java +++ b/asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManagerTest.java @@ -25,19 +25,21 @@ import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertNotNull; import static org.mockito.Mockito.when; -import java.io.File; +import java.nio.file.Path; import java.util.Arrays; import java.util.List; import java.util.SortedSet; import java.util.TreeSet; +import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.io.TempDir; import org.mockito.Mockito; import org.openecomp.sdc.asdctool.impl.validator.config.ValidationConfigManager; import org.openecomp.sdc.asdctool.impl.validator.report.Report; import org.openecomp.sdc.asdctool.impl.validator.report.ReportFileNioHelper; import org.openecomp.sdc.be.dao.jsongraph.GraphVertex; -public class ReportManagerTest { +class ReportManagerTest { private static final String VERTEX_1_ID = "testID1"; private static final String TASK_1_FAILED_NAME = "testFailedTask1"; @@ -64,15 +66,22 @@ public class ReportManagerTest { private final VertexResult successResult = new VertexResult(); - private final static String resourcePath = new File("src/test/resources").getAbsolutePath(); - private final static String csvReportFilePath = ValidationConfigManager - .csvReportFilePath(resourcePath, System::currentTimeMillis); - private final static String txtReportFilePath = ValidationConfigManager.txtReportFilePath(resourcePath); + private static String csvReportFilePath; + private static String txtReportFilePath; private final GraphVertex vertexScanned = Mockito.mock(GraphVertex.class); + @TempDir + static Path reportOutputPath; + + @BeforeAll + static void beforeAll() { + csvReportFilePath = ValidationConfigManager.csvReportFilePath(reportOutputPath.toString(), System::currentTimeMillis); + txtReportFilePath = ValidationConfigManager.txtReportFilePath(reportOutputPath.toString()); + } + @Test - public void testReportTaskEnd() { + void testReportTaskEnd() { // when Report report = Report.make(); report.addSuccess(VERTEX_1_ID, TASK_1_NAME, successResult); @@ -91,7 +100,7 @@ public class ReportManagerTest { } @Test - public void testAddFailedVertex() { + void testAddFailedVertex() { // when Report report = Report.make(); report.addFailure(TASK_1_NAME, VERTEX_1_ID); @@ -110,7 +119,7 @@ public class ReportManagerTest { } @Test - public void testPrintValidationTaskStatus() { + void testPrintValidationTaskStatus() { // given when(vertexScanned.getUniqueId()).thenReturn(UNIQUE_ID); @@ -128,7 +137,7 @@ public class ReportManagerTest { } @Test - public void testWriteReportLineToFile() { + void testWriteReportLineToFile() { // when List reportTxtFile = ReportFileNioHelper.withTxtFile(txtReportFilePath, file -> { file.writeReportLineToFile(DUMMY_MESSAGE); @@ -142,7 +151,7 @@ public class ReportManagerTest { } @Test - public void testReportValidatorTypeSummary() { + void testReportValidatorTypeSummary() { // when List reportTxtFile = ReportFileNioHelper.withTxtFile(txtReportFilePath, file -> { file.reportValidatorTypeSummary(VALIDATOR_NAME, failedTasksNames, successTasksNames); @@ -159,7 +168,7 @@ public class ReportManagerTest { } @Test - public void testReportStartValidatorRun() { + void testReportStartValidatorRun() { // when List reportTxtFile = ReportFileNioHelper.withTxtFile(txtReportFilePath, file -> { file.reportStartValidatorRun(VALIDATOR_NAME, COMPONENT_SUM); @@ -174,7 +183,7 @@ public class ReportManagerTest { } @Test - public void testReportStartTaskRun() { + void testReportStartTaskRun() { // given when(vertexScanned.getUniqueId()).thenReturn(UNIQUE_ID); -- cgit 1.2.3-korg