From b29518ad6136fe628d7aff79d5fc607c5e890729 Mon Sep 17 00:00:00 2001 From: Francis Toth Date: Thu, 18 Jun 2020 15:35:05 -0400 Subject: Decouple TXT Report file writing and formatting logic (4/6) This commit aims to move the reportValidatorTypeSummary function from ReportManager (deprecated) to ReportFile. Signed-off-by: Francis Toth Change-Id: I30898d69bbcdb129a2dac917c4dee3c276556e91 Issue-ID: SDC-2499 --- .../executers/TopologyTemplateValidatorExecuter.java | 2 +- .../sdc/asdctool/impl/validator/report/ReportFile.java | 16 ++++++++++++++++ .../sdc/asdctool/impl/validator/utils/ReportManager.java | 10 ---------- 3 files changed, 17 insertions(+), 11 deletions(-) (limited to 'asdctool/src/main/java/org/openecomp') 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 497e589234..52c8cb5f96 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 @@ -114,7 +114,7 @@ public class TopologyTemplateValidatorExecuter { String componentScanStatus = successAllTasks ? "success" : "failed"; log.info("Topology Template " + vertex.getUniqueId() + " Validation finished with " + componentScanStatus); } - ReportManager.reportValidatorTypeSummary(getName(), failedTasks, successTasks, outputFilePath); + reportFile.reportValidatorTypeSummary(getName(), failedTasks, successTasks); return successAllVertices; } } 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 f1f084e2a8..b22edb6cee 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 @@ -20,6 +20,8 @@ package org.openecomp.sdc.asdctool.impl.validator.report; +import java.util.Set; +import org.apache.commons.text.StrBuilder; import org.openecomp.sdc.be.dao.jsongraph.GraphVertex; /** @@ -79,6 +81,20 @@ public class ReportFile { "-----------------------" ); } + + public void reportValidatorTypeSummary( + String validatorName, + Set failedTasksNames, + Set successTasksNames + ) { + StrBuilder sb = new StrBuilder(); + sb.appendln("-----------------------ValidatorExecuter " + validatorName + + " Validation Summary-----------------------"); + sb.appendln("Failed tasks: " + failedTasksNames); + sb.appendln("Success tasks: " + successTasksNames); + writer.writeln(""); + writer.write(sb.toString()); + } } /** 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 3149d049c1..a733eec38b 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 @@ -69,16 +69,6 @@ public class ReportManager { } } - public static void reportValidatorTypeSummary(String validatorName, Set failedTasksNames, - Set successTasksNames, String outputFilePath) { - StrBuilder sb = new StrBuilder(); - sb.appendln( - "-----------------------ValidatorExecuter " + validatorName + " Validation Summary-----------------------"); - sb.appendln("Failed tasks: " + failedTasksNames); - sb.appendln("Success tasks: " + successTasksNames); - writeReportLineToFile(sb.toString(), outputFilePath); - } - public static void reportEndOfToolRun(Report report, String outputFilePath) { StrBuilder sb = new StrBuilder(); sb.appendln("-----------------------------------Validator Tool Summary-----------------------------------"); -- cgit 1.2.3-korg