sorb/ThreadNet-Web
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 4 Wiki Activity

Actually skip stale screenshot reporter on errors

Browse Source
This commit is contained in:
Michael Telatynski
2026-04-08 11:43:50 +01:00
parent 1107ddf695
commit 8cae8df17f
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/element-web-playwright-common/src/stale-screenshot-reporter.ts
+9 -5
View File
@@ -56,11 +56,7 @@ class StaleScreenshotReporter implements Reporter {
this.success = false; this.success = false;
} }
public async onExit(): Promise<void> { private async checkStaleScreenshots(): Promise<void> {
if (this.failing.size) {
console.error(`${this.failing.size} tests failed, skipping stale screenshot reporter.`);
}
if (!this.snapshotRoots.size) { if (!this.snapshotRoots.size) {
this.error("No snapshot directories found, did you set the snapshotDir in your Playwright config?", ""); this.error("No snapshot directories found, did you set the snapshotDir in your Playwright config?", "");
return; return;
@@ -90,6 +86,14 @@ class StaleScreenshotReporter implements Reporter {
this.error("Stale screenshot file", screenshot); this.error("Stale screenshot file", screenshot);
} }
} }
}
public async onExit(): Promise<void> {
if (this.failing.size) {
this.error(`${this.failing.size} tests failed, skipping stale screenshot reporter.`, "");
} else {
await this.checkStaleScreenshots();
}
if (!this.success) { if (!this.success) {
process.exit(1); process.exit(1);