From d5be7f8dcf4d54fd74de1a54544cad7e1d73eb7a Mon Sep 17 00:00:00 2001
From: daywalker90 <8257956+daywalker90@users.noreply.github.com>
Date: Sat, 15 Jun 2024 12:21:42 +0200
Subject: [PATCH] CI: only fail test suite on new failures

---
 .ci/test.py | 32 +++++++++++++++++++++++++++++++-
 1 file changed, 31 insertions(+), 1 deletion(-)

diff --git a/.ci/test.py b/.ci/test.py
index c85a2c280..58fc8eaf1 100644
--- a/.ci/test.py
+++ b/.ci/test.py
@@ -4,6 +4,7 @@
 import sys
 import tempfile
 import time
+import json
 from itertools import chain
 from pathlib import Path
 
@@ -281,6 +282,27 @@ def write_gather_data_file(plugin_name: str, result, workflow: str, python_versi
     return filename
 
 
+def gather_old_failures(old_failures: list, workflow: str):
+    print("Gather old failures...")
+    configure_git()
+    subprocess.run(["git", "fetch"])
+    subprocess.run(["git", "checkout", "badges"])
+
+    directory = ".badges"
+
+    for filename in os.listdir(directory):
+        if filename.endswith(f'_{workflow}.json'):
+            file_path = os.path.join(directory, filename)
+            plugin_name = filename.rsplit(f'_{workflow}.json', 1)[0]
+
+            with open(file_path, 'r') as file:
+                data = json.load(file)
+                if data["color"] == "red":
+                    old_failures.append(plugin_name)
+
+    print(f"Old failures: {old_failures}")
+    print("Done.")
+
 def run_all(workflow: str, python_version: str, update_badges: bool, plugin_names: list):
     root_path = subprocess.check_output([
         'git',
@@ -300,14 +322,22 @@ def run_all(workflow: str, python_version: str, update_badges: bool, plugin_name
     results = [(p, run_one(p)) for p in plugins]
     success = all([t[1] for t in results])
 
+    old_failures = []
+    if not success and plugin_names == []:
+        gather_old_failures(old_failures, workflow)
+
     if update_badges:
         push_gather_data(collect_gather_data(results, success), workflow, python_version)
 
     if not success:
         print("The following tests failed:")
+        has_new_failure = False
         for t in filter(lambda t: not t[1], results):
+            if t[0].name not in old_failures:
+                has_new_failure = True
             print(" - {p.name} ({p.path})".format(p=t[0]))
-        sys.exit(1)
+        if has_new_failure:
+            sys.exit(1)
     else:
         print("All tests passed.")