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

revert step.attachments

Browse source
This commit is contained in:
Simon Knott 2024-12-20 11:44:41 +01:00
parent bc66311a05
commit 9231cd5d8e
No known key found for this signature in database
GPG key ID: 8CEDC00028084AEC
1 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
packages/playwright/src/worker/testInfo.ts
View file

@ -32,8 +32,7 @@ import type { StackFrame } from '@protocol/channels';
import { testInfoError } from './util';
export interface TestStepInternal {
complete(result: { error?: Error | unknown, suggestedRebaseline?: string }): void;
attachments: Attachment[];
complete(result: { error?: Error | unknown, attachments?: Attachment[], suggestedRebaseline?: string }): void;
stepId: string;
title: string;
category: 'hook' | 'fixture' | 'test.step' | 'expect' | 'attach' | string;
@ -268,12 +267,10 @@ export class TestInfoImpl implements TestInfo {
}
data.location = data.location || filteredStack[0];
const attachments: Attachment[] = [];
const step: TestStepInternal = {
stepId,
...data,
steps: [],
attachments,
complete: result => {
if (step.endWallTime)
return;
@ -301,9 +298,6 @@ export class TestInfoImpl implements TestInfo {
}
}
for (const attachment of attachments ?? [])
this._attach(attachment, stepId);
const payload: StepEndPayload = {
testId: this.testId,
stepId,
@ -313,7 +307,7 @@ export class TestInfoImpl implements TestInfo {
};
this._onStepEnd(payload);
const errorForTrace = step.error ? { name: '', message: step.error.message || '', stack: step.error.stack } : undefined;
this._tracing.appendAfterActionForStep(stepId, errorForTrace, attachments);
this._tracing.appendAfterActionForStep(stepId, errorForTrace, result.attachments);
}
};
const parentStepList = parentStep ? parentStep.steps : this._steps;
@ -416,8 +410,9 @@ export class TestInfoImpl implements TestInfo {
title: `attach "${name}"`,
category: 'attach',
});
step.attachments.push(await normalizeAndSaveAttachment(this.outputPath(), name, options));
step.complete({});
const attachment = await normalizeAndSaveAttachment(this.outputPath(), name, options);
this._attach(attachment, step.stepId);
step.complete({ attachments: [attachment] });
}
private _attach(attachment: TestInfo['attachments'][0], stepId: string | undefined) {