aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrancis Toth <francis.toth@yoppworks.com>2020-06-18 15:30:26 -0400
committerFrancis Toth <francis.toth@yoppworks.com>2020-06-21 07:34:43 -0400
commit051fcebef77e7f8c4536ef717a78f05ab1a002ff (patch)
tree1c735113663946c6f12dde81ed1f468dd365b0d0
parent194c4a67301b88783ff7d2587feb66912de5ff94 (diff)
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 <francis.toth@yoppworks.com> Change-Id: Ic33a519f4c70276d404a83bf7de8a27f0007b285 Issue-ID: SDC-2499
-rw-r--r--asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/executers/TopologyTemplateValidatorExecuter.java2
-rw-r--r--asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/report/ReportFile.java14
-rw-r--r--asdctool/src/main/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManager.java11
-rw-r--r--asdctool/src/test/java/org/openecomp/sdc/asdctool/impl/validator/utils/ReportManagerTest.java16
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<String> reportOutputFile = ReportManagerHelper.getReportOutputFileAsList(txtReportFilePath);
+ List<String> 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