chore: fix flaky re-run step in CI

zama-ai · May 27, 2024 · e7d6c60 · e7d6c60
1 parent 9a1a35b
commit e7d6c60
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/continuous-integration.yaml b/.github/workflows/continuous-integration.yaml
@@ -735,7 +735,7 @@ jobs:
           	poetry run python ./script/actions_utils/pytest_failed_test_report.py \
             --pytest-input-report "pytest_report.json" \
             --failed-tests-report "failed_tests_report.json" \
-            --failed-tests-comment "failed_tests_comment_${{ matrix.python_version }}.txt"
+            --failed-tests-comment "failed_tests_comment_${{ matrix.python_version }}.txt" \
             --failed-tests-list "failed_tests_slack_list_${{ matrix.python_version }}.txt"
 
             # Check if all failed tests are known flaky tests

diff --git a/.gitignore b/.gitignore
@@ -53,7 +53,8 @@ coverage.xml
 global-coverage-infos.json
 pytest_report.json
 failed_tests_report.json
-failed_tests_comment.txt
+failed_tests_comment_*.txt
+failed_tests_slack_list_*.txt
 
 # Translations
 *.mo

diff --git a/script/actions_utils/pytest_failed_test_report.py b/script/actions_utils/pytest_failed_test_report.py
@@ -56,7 +56,7 @@ def write_failed_tests_comment(failed_tests_comment_path: Path, failed_tests_rep
             else:
                 f.write("## ❌ Some tests failed after rerun ❌\n\n")
                 failed_tests_header = (
-                    "At least one of the following tests initially failed. They have therefore"
+                    "At least one of the following tests initially failed. They have therefore "
                     "been rerun but failed again. See below for more details.\n\n"
                 )