From 0f5e135224e50dd07242e0a656d448c3da21a21b Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Wed, 8 Jan 2025 09:15:20 +0000
Subject: [PATCH] Ignore failing tests in stale-screenshot-reporter.ts to avoid
 confusing errors

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
---
 playwright/stale-screenshot-reporter.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/playwright/stale-screenshot-reporter.ts b/playwright/stale-screenshot-reporter.ts
index 3e38f78ca9..dc934827c1 100644
--- a/playwright/stale-screenshot-reporter.ts
+++ b/playwright/stale-screenshot-reporter.ts
@@ -23,6 +23,7 @@ class StaleScreenshotReporter implements Reporter {
     private success = true;
 
     public onTestEnd(test: TestCase): void {
+        if (!test.ok()) return;
         for (const annotation of test.annotations) {
             if (annotation.type === "_screenshot") {
                 this.screenshots.add(annotation.description);