Merge pull request #520 from kids-first/fix-validation-test-summary

🎨 Display test result summary as table
kids-first · Oct 22, 2020 · d8346e8 · d8346e8
2 parents a453461 + 295793b
commit d8346e8
Showing 1 changed file with 13 additions and 6 deletions.
diff --git a/kf_lib_data_ingest/validation/reporting/markdown.py b/kf_lib_data_ingest/validation/reporting/markdown.py
@@ -125,13 +125,20 @@ def _tests_section_md(self, results, test_type):
 
         # Add test section summary
         test_order_by_outcome = [FAILED, PASSED, NA]
-        summary = " ".join(
-            [
-                f"`{code.title()}: {len(test_markdowns.get(code, []))}`"
-                for code in test_order_by_outcome
-            ]
+        summary_md = self._df_to_markdown(
+            pandas.DataFrame(
+                [
+                    {
+                        "Result": f"**{RESULT_TO_EMOJI.get(error_code)} "
+                        f"{error_code.title()}**",
+                        "# of Tests": len(test_markdowns.get(error_code, [])),
+                    }
+                    for error_code in test_order_by_outcome
+                ]
+            )
         )
-        output.append(f"### {summary}")
+        output.append("### Result Summary")
+        output.append(summary_md)
 
         # Add test markdowns
         for result_code in test_order_by_outcome: