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

cherry-pick(#32661): fix(trace-viewer): time delta between local and … (#32663)

Browse source 
…remote actions
This commit is contained in:
Yury Semikhatsky 2024-09-17 11:43:41 -07:00 committed by GitHub
parent 3d2ffd0fe9
commit 937424fb59
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/trace-viewer/src/ui/modelUtil.ts
View file

@ -310,7 +310,7 @@ function monotonicTimeDeltaBetweenLibraryAndRunner(nonPrimaryContexts: ContextEn
for (const action of context.actions) { for (const action of context.actions) {
if (!action.startTime) if (!action.startTime)
continue; continue;
const key = matchByStepId ? action.stepId! : `${action.apiName}@${(action as any).wallTime}`; const key = matchByStepId ? action.callId! : `${action.apiName}@${(action as any).wallTime}`;
const libraryAction = libraryActions.get(key); const libraryAction = libraryActions.get(key);
if (libraryAction) if (libraryAction)
return action.startTime - libraryAction.startTime; return action.startTime - libraryAction.startTime;

BIN
tests/assets/trace-remote-time-diff.zip Normal file
View file

Binary file not shown.

6
tests/library/trace-viewer.spec.ts
View file

@ -127,6 +127,12 @@ test('should complain about newer version of trace in old viewer', async ({ show
await expect(traceViewer.page.getByText('The trace was created by a newer version of Playwright and is not supported by this version of the viewer.')).toBeVisible(); await expect(traceViewer.page.getByText('The trace was created by a newer version of Playwright and is not supported by this version of the viewer.')).toBeVisible();
}); });
test('should properly synchronize local and remote time', async ({ showTraceViewer, asset }, testInfo) => {
const traceViewer = await showTraceViewer([asset('trace-remote-time-diff.zip')]);
// The total duration should be sub 10s, rather than 16h.
await expect(traceViewer.page.locator('.timeline-time').last()).toHaveText('8.5s');
});
test('should contain action info', async ({ showTraceViewer }) => { test('should contain action info', async ({ showTraceViewer }) => {
const traceViewer = await showTraceViewer([traceFile]); const traceViewer = await showTraceViewer([traceFile]);
await traceViewer.selectAction('locator.click'); await traceViewer.selectAction('locator.click');