diff --git a/packages/trace-viewer/src/ui/modelUtil.ts b/packages/trace-viewer/src/ui/modelUtil.ts index d5aa384125..41a91c5a5e 100644 --- a/packages/trace-viewer/src/ui/modelUtil.ts +++ b/packages/trace-viewer/src/ui/modelUtil.ts @@ -122,6 +122,8 @@ function mergeActions(contexts: ContextEntry[]) { existing.error = action.error; if (action.attachments) existing.attachments = action.attachments; + if (action.parentId) + existing.parentId = action.parentId; continue; } map.set(key, action); diff --git a/tests/playwright-test/ui-mode-test-progress.spec.ts b/tests/playwright-test/ui-mode-test-progress.spec.ts index bdfd8da1d9..949097bcb9 100644 --- a/tests/playwright-test/ui-mode-test-progress.spec.ts +++ b/tests/playwright-test/ui-mode-test-progress.spec.ts @@ -104,7 +104,6 @@ test('should update trace live', async ({ runUITest, server }) => { await expect(listItem).toHaveText([ /Before Hooks[\d.]+m?s/, - /browserContext.newPage[\d.]+m?s/, /page.gotohttp:\/\/localhost:\d+\/one.html/, /page.gotohttp:\/\/localhost:\d+\/two.html/, /After Hooks[\d.]+m?s/, diff --git a/tests/playwright-test/ui-mode-trace.spec.ts b/tests/playwright-test/ui-mode-trace.spec.ts index 04d7724ccd..9d96621a07 100644 --- a/tests/playwright-test/ui-mode-trace.spec.ts +++ b/tests/playwright-test/ui-mode-trace.spec.ts @@ -39,7 +39,6 @@ test('should merge trace events', async ({ runUITest, server }) => { 'action list' ).toHaveText([ /Before Hooks[\d.]+m?s/, - /browserContext.newPage[\d.]+m?s/, /page.setContent[\d.]+m?s/, /expect.toBe[\d.]+m?s/, /locator.clickgetByRole\('button'\)[\d.]+m?s/, @@ -70,7 +69,6 @@ test('should merge web assertion events', async ({ runUITest }, testInfo) => { 'action list' ).toHaveText([ /Before Hooks[\d.]+m?s/, - /browserContext.newPage[\d.]+m?s/, /page.setContent[\d.]+m?s/, /expect.toBeVisiblelocator\('button'\)[\d.]+m?s/, /After Hooks[\d.]+m?s/, @@ -100,7 +98,6 @@ test('should merge screenshot assertions', async ({ runUITest }, testInfo) => { 'action list' ).toHaveText([ /Before Hooks[\d.]+m?s/, - /browserContext.newPage[\d.]+m?s/, /page\.setContent[\d.]+m?s/, /expect\.toHaveScreenshot[\d.]+m?s/, /After Hooks/, @@ -146,7 +143,6 @@ test('should show snapshots for sync assertions', async ({ runUITest, server }) 'action list' ).toHaveText([ /Before Hooks[\d.]+m?s/, - /browserContext\.newPage[\d.]+m?s/, /page\.setContent[\d.]+m?s/, /locator\.clickgetByRole\('button'\)[\d.]+m?s/, /expect\.toBe[\d.]+m?s/,