yujunjun/playwright
1
0
Fork 0
mirror of https://github.com/microsoft/playwright.git synced 2025-06-26 21:40:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

chore(tracing): remove extra fs.access call (#21855)

Browse Source
This commit is contained in:
Dmitry Gozman 2023-03-21 19:01:09 -07:00 committed by GitHub
parent d9a9bb5a8c
commit 295a8ec92b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
packages/playwright-core/src/server/trace/recorder/tracing.ts
View File

@ -421,14 +421,10 @@ export class Tracing extends SdkObject implements InstrumentationListener, Snaps
this._allResources.add(sha1); this._allResources.add(sha1);
const resourcePath = path.join(this._state!.resourcesDir, sha1); const resourcePath = path.join(this._state!.resourcesDir, sha1);
this._appendTraceOperation(async () => { this._appendTraceOperation(async () => {
try { // Note: 'wx' flag only writes when the file does not exist.
// Perhaps we've already written this resource? // See https://nodejs.org/api/fs.html#file-system-flags.
await fs.promises.access(resourcePath); // This way tracing never have to write the same resource twice.
} catch (e) { await fs.promises.writeFile(resourcePath, buffer, { flag: 'wx' }).catch(() => {});
// If not, let's write! Note that async access is safe because we
// never remove resources until the very end.
await fs.promises.writeFile(resourcePath, buffer).catch(() => {});
}
}); });
} }