chore: improve slack report format for weekly CI

zama-ai · May 28, 2024 · f7ef473 · f7ef473
1 parent f64bb5b
commit f7ef473
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/.github/workflows/continuous-integration.yaml b/.github/workflows/continuous-integration.yaml
@@ -1190,9 +1190,7 @@ jobs:
               FAILED_TESTS_LIST=$(cat "${file_name}")
               
               {
-                echo ""
-                echo ""
-                echo "Linux failed tests (Python ${linux_python_version}):"
+                echo "Linux (Python ${linux_python_version}):"
                 echo "${FAILED_TESTS_LIST}"
               } >> "$GITHUB_ENV"
             fi

diff --git a/script/actions_utils/pytest_failed_test_report.py b/script/actions_utils/pytest_failed_test_report.py
@@ -19,16 +19,16 @@ def write_failed_tests_list(failed_tests_list_path: Path, failed_tests_report: D
         if failed_tests_report["tests_failed"]:
 
             # Write all flaky tests that initially failed as a list
-            f.write("- Known flaky tests that initially failed: \n")
+            f.write("- Known flaky tests :warning:: \n")
             for flaky_name in failed_tests_report["flaky"]:
                 f.write("    - " + flaky_name)
 
-            # Write all other tests that initially failed as a list if they were not all known
+            # Write all other tests that failed as a list if they were not all known
             # flaky tests
             if not failed_tests_report["all_failed_tests_are_flaky"]:
                 f.write("\n")
 
-                f.write("- Other tests that initially failed: \n")
+                f.write("- Other failed tests ❌: \n")
                 for non_flaky_name in failed_tests_report["non_flaky"]:
                     f.write("    - " + non_flaky_name)