From 051fcebef77e7f8c4536ef717a78f05ab1a002ff Mon Sep 17 00:00:00 2001 From: Francis Toth Date: Thu, 18 Jun 2020 15:30:26 -0400 Subject: Decouple TXT Report file writing and formatting logic (3/6) This commit aims to move the printValidationTaskStatus function's logic from ReportManager (deprecated) to ReportFile. Signed-off-by: Francis Toth Change-Id: Ic33a519f4c70276d404a83bf7de8a27f0007b285 Issue-ID: SDC-2499 --- .../executers/TopologyTemplateValidatorExecuter.java | 2 +- .../sdc/asdctool/impl/validator/report/ReportFile.java | 14 ++++++++++++++ .../sdc/asdctool/impl/validator/utils/ReportManager.java | 11 ----------- .../asdctool/impl/validator/utils/ReportManagerTest.java | 16 ++++++++-------- 4 files changed, 23 insertions(+), 20 deletions(-) diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java index 967a6c68e5..497e589234 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java @@ -108,7 +108,7 @@ public class TopologyTemplateValidatorExecuter { } else if (successAllTasks && vertexNum == verticesSize) { successTasks.add(task.getTaskName()); } - ReportManager.printValidationTaskStatus(vertex, task.getTaskName(), result.getStatus(), outputFilePath); + reportFile.printValidationTaskStatus(vertex, task.getTaskName(), result.getStatus()); report.addSuccess(vertex.getUniqueId(), task.getTaskName(), result); } String componentScanStatus = successAllTasks ? "success" : "failed"; diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/report/ReportFile.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/report/ReportFile.java index 936ec4b251..f1f084e2a8 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/report/ReportFile.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/report/ReportFile.java @@ -65,6 +65,20 @@ public class ReportFile { validatorName + " Validation Started, on " + componentsNum + " components---------"); } + + public void printValidationTaskStatus( + GraphVertex vertexScanned, + String taskName, + boolean success + ) { + String successStatus = success ? "success" : "failed"; + writer.writeln(""); + writer.writeln("-----------------------Vertex: " + + vertexScanned.getUniqueId() + ", Task " + + taskName + " " + successStatus + + "-----------------------" + ); + } } /** diff --git a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManager.java b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManager.java index 63e7ebe071..3149d049c1 100644 --- a/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManager.java +++ b/asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManager.java @@ -59,17 +59,6 @@ public class ReportManager { } } - public static void printValidationTaskStatus(GraphVertex vertexScanned, String taskName, boolean success, - String outputFilePath) { - String successStatus = success ? "success" : "failed"; - String line = - "-----------------------Vertex: " + vertexScanned.getUniqueId() + ", Task " + taskName + " " + successStatus - + "-----------------------"; - StrBuilder sb = new StrBuilder(); - sb.appendln(line); - writeReportLineToFile(line, outputFilePath); - } - public static void writeReportLineToFile(String message, String outputFilePath) { try { Files.write(Paths.get(outputFilePath), new StrBuilder().appendNewLine().toString().getBytes(), 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 479421e118..9a237af812 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 @@ -72,7 +72,7 @@ public class ReportManagerTest { .csvReportFilePath(resourcePath, System::currentTimeMillis); private final static String txtReportFilePath = ValidationConfigManager.txtReportFilePath(resourcePath); - private GraphVertex vertexScanned = Mockito.mock(GraphVertex.class); + private final GraphVertex vertexScanned = Mockito.mock(GraphVertex.class); @BeforeEach public void setup() { @@ -127,16 +127,16 @@ public class ReportManagerTest { when(vertexScanned.getUniqueId()).thenReturn(UNIQUE_ID); // when - ReportManager.printValidationTaskStatus(vertexScanned, TASK_1_NAME, false, txtReportFilePath); - - List reportOutputFile = ReportManagerHelper.getReportOutputFileAsList(txtReportFilePath); + List reportTxtFile = ReportFileNioHelper.withTxtFile(txtReportFilePath, file -> { + file.printValidationTaskStatus(vertexScanned, TASK_1_NAME, false); + return ReportFileNioHelper.readFileAsList(txtReportFilePath); + }); // then - assertNotNull(reportOutputFile); - - assertEquals(EXPECTED_OUTPUT_FILE_HEADER, reportOutputFile.get(0)); + assertNotNull(reportTxtFile); + assertEquals(EXPECTED_OUTPUT_FILE_HEADER, reportTxtFile.get(0)); assertEquals("-----------------------Vertex: " + UNIQUE_ID + ", Task " + TASK_1_NAME - + " failed-----------------------", reportOutputFile.get(2)); + + " failed-----------------------", reportTxtFile.get(2)); } @Test -- cgit 1.2.3-korg