diff --git a/packages/html-reporter/src/links.css b/packages/html-reporter/src/links.css index eb9390844b..4abe8a6caa 100644 --- a/packages/html-reporter/src/links.css +++ b/packages/html-reporter/src/links.css @@ -60,11 +60,6 @@ color: var(--color-scale-orange-6); border: 1px solid var(--color-scale-orange-4); } - .label-color-gray { - background-color: var(--color-scale-gray-0); - color: var(--color-scale-gray-6); - border: 1px solid var(--color-scale-gray-4); - } } @media(prefers-color-scheme: dark) { @@ -98,11 +93,6 @@ color: var(--color-scale-orange-2); border: 1px solid var(--color-scale-orange-4); } - .label-color-gray { - background-color: var(--color-scale-gray-9); - color: var(--color-scale-gray-2); - border: 1px solid var(--color-scale-gray-4); - } } .attachment-body { diff --git a/packages/html-reporter/src/testResultView.tsx b/packages/html-reporter/src/testResultView.tsx index 64497b3bcd..6cae643dac 100644 --- a/packages/html-reporter/src/testResultView.tsx +++ b/packages/html-reporter/src/testResultView.tsx @@ -176,6 +176,7 @@ const StepTreeItem: React.FC<{ }> = ({ test, step, result, depth }) => { return {msToString(step.duration)} + {step.attachments.length > 0 && { evt.stopPropagation(); }}>{icons.attachment()}} {statusIcon(step.error || step.duration === -1 ? 'failed' : (step.skipped ? 'skipped' : 'passed'))} {step.title} {step.count > 1 && <> ✕ {step.count}} @@ -183,20 +184,6 @@ const StepTreeItem: React.FC<{ } loadChildren={step.steps.length || step.snippet ? () => { const snippet = step.snippet ? [] : []; const steps = step.steps.map((s, i) => ); - const attachments = step.attachments.map(attachmentIndex => ( - - - {icons.attachment()}{result.attachments[attachmentIndex].name} - - - )); - return snippet.concat(steps, attachments); + return snippet.concat(steps); } : undefined} depth={depth}/>; }; diff --git a/tests/playwright-test/reporter-html.spec.ts b/tests/playwright-test/reporter-html.spec.ts index 2640cb61c9..36e9817ffa 100644 --- a/tests/playwright-test/reporter-html.spec.ts +++ b/tests/playwright-test/reporter-html.spec.ts @@ -959,10 +959,9 @@ for (const useIntermediateMergeReport of [true, false] as const) { await showReport(); await page.getByRole('link', { name: 'passing' }).click(); - const attachment = page.getByTestId('attachments').getByText('foo-2', { exact: true }); + const attachment = page.getByText('foo-2', { exact: true }); await expect(attachment).not.toBeInViewport(); - await page.getByLabel('attach "foo-2"').click(); - await page.getByTitle('see "foo-2"').click(); + await page.getByLabel('attach "foo-2"').getByTitle('link to attachment').click(); await expect(attachment).toBeInViewport(); await page.reload(); @@ -989,10 +988,9 @@ for (const useIntermediateMergeReport of [true, false] as const) { await showReport(); await page.getByRole('link', { name: 'passing' }).click(); - const attachment = page.getByTestId('attachments').getByText('attachment', { exact: true }); + const attachment = page.getByText('attachment', { exact: true }); await expect(attachment).not.toBeInViewport(); - await page.getByLabel('step').click(); - await page.getByTitle('see "attachment"').click(); + await page.getByLabel('step').getByTitle('link to attachment').click(); await expect(attachment).toBeInViewport(); });