Failing case number fix, capping table payload size

unloggedio · Aug 9, 2024 · a5d6172 · a5d6172
1 parent d500619
commit a5d6172
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 3 deletions.
diff --git a/src/test/java/io/unlogged/autoexecutor/AutoExecutorCITest.java b/src/test/java/io/unlogged/autoexecutor/AutoExecutorCITest.java
@@ -108,7 +108,7 @@ public void runTests(TestConfig testconfig) {
             logger.info("\n    " + resultSummary.getMode() + " Mode ->" + "\n"
                     + "    Total number of cases run : " + resultSummary.getNumberOfCases() + "\n"
                     + "    Number of Failing cases : " + resultSummary.getFailingCasesCount() + "\n"
-                    + "    Failing tests (Row numbers) : " + resultSummary.getFailingCases().toString());
+                    + "    Failing tests (Row numbers) : " + resultSummary.getFailingCaseNumbers().toString());
             if (resultSummary.getFailingCasesCount() > 0) {
                 overallStatus = false;
             }

diff --git a/src/test/java/io/unlogged/autoexecutor/report/MarkdownReportGenerator.java b/src/test/java/io/unlogged/autoexecutor/report/MarkdownReportGenerator.java
@@ -53,8 +53,8 @@ private static String generateFailingCaseSummary(AssertionDetails assertionDetai
                 .append("<summary> Case ID : ").append(assertionDetails.getCaseId()).append("</summary>\n\n")
                 .append("| Operation Type | ").append(assertionDetails.getAssertionType()).append(" |\n")
                 .append("|----------------|------|\n")
-                .append("| Expected | ").append(assertionDetails.getExpected()).append("| \n")
-                .append("| Actual | ").append(assertionDetails.getActual()).append("| \n\n")
+                .append("| Expected | ").append(capSize(assertionDetails.getExpected())).append("| \n")
+                .append("| Actual | ").append(capSize(assertionDetails.getActual())).append("| \n\n")
                 .append("</details></li>\n\n").toString();
     }
 
@@ -91,4 +91,11 @@ private static String generatePieChart(TestResultSummary testResultSummary) {
                 .append("</details>").append("\n\n");
         return pieChartBuilder.toString();
     }
+
+    private static String capSize(String payload) {
+        if (payload.length() > 10000) {
+            return "Too large to show here, please refer to Logs";
+        }
+        return payload;
+    }
 }
diff --git a/src/test/java/io/unlogged/autoexecutor/testutils/entity/TestResultSummary.java b/src/test/java/io/unlogged/autoexecutor/testutils/entity/TestResultSummary.java
@@ -1,6 +1,7 @@
 package io.unlogged.autoexecutor.testutils.entity;
 
 import java.util.List;
+import java.util.stream.Collectors;
 
 public class TestResultSummary {
     private int numberOfCases;
@@ -40,4 +41,8 @@ public void setMode(String mode) {
     public List<AssertionDetails> getFailingCases() {
         return failingCases;
     }
+
+    public List<Integer> getFailingCaseNumbers() {
+        return failingCases.stream().map(AssertionDetails::getCaseId).collect(Collectors.toList());
+    }
 }