Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

don't copy screenshot file

Browse source
This commit is contained in:
Simon Knott 2024-12-17 10:42:29 +01:00
parent b04880d7b8
commit 1b1da1c075
No known key found for this signature in database
GPG key ID: 8CEDC00028084AEC
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
packages/playwright/src/index.ts
View file

@ -638,7 +638,7 @@ class ArtifactsRecorder {
try {
const screenshotPath = this._createScreenshotAttachmentPath();
await fs.promises.rename(file, screenshotPath);
await this._attachScreenshot(screenshotPath);
this._attachScreenshot(screenshotPath);
} catch {
}
}
@ -665,14 +665,19 @@ class ArtifactsRecorder {
if (temporary)
this._temporaryScreenshots.push(screenshotPath);
else
await this._attachScreenshot(screenshotPath);
this._attachScreenshot(screenshotPath);
} catch {
// Screenshot may fail, just ignore.
}
}
private async _attachScreenshot(screenshotPath: string) {
await this._testInfo.attach('screenshot', { path: screenshotPath, contentType: 'image/png' });
private _attachScreenshot(screenshotPath: string) {
const step = this._testInfo._addStep({
title: `attach "screenshot"`,
category: 'attach',
});
step.attachments.push({ name: 'screenshot', path: screenshotPath, contentType: 'image/png' });
step.complete({});
}
private async _screenshotOnTestFailure() {