mirror of
https://github.com/microsoft/playwright.git
synced 2024-12-13 07:35:33 +03:00
dadb5cbc30
Textual snapshot diffs were previously broken in the HTML Report. The strikethrough'd text extended beyond the intended region. HTML Report Before: <img width="693" alt="Screen Shot 2021-12-27 at 4 43 35 PM" src="https://user-images.githubusercontent.com/11915034/147518750-a60f9002-6eed-48a1-a412-20fabd076fa6.png"> HTML Report After: <img width="206" alt="Screen Shot 2021-12-27 at 4 48 37 PM" src="https://user-images.githubusercontent.com/11915034/147518762-19a4c8f9-ccc3-4a3c-a962-5a42edc6fc5d.png"> This now matches what's expected and shown in the terminal (which has always been correct): <img width="1384" alt="Screen Shot 2021-12-27 at 4 36 29 PM" src="https://user-images.githubusercontent.com/11915034/147518799-f538259e-5a45-4d6f-916c-a12ccb620c5b.png"> NB: This MR is a workaround, but not a root cause fix. It works, but I never fully got to the root cause so a bug upstream may be required. It's unclear whether it's (1) in [`colors`](https://www.npmjs.com/package/colors), (2) in [`ansi-to-html`](https://www.npmjs.com/package/ansi-to-html), or (3) Playwright's use of the two. Since the terminal output is correct, I suspect it is in `ansi-to-html`. For example: ```js const colors = require("colors"); const Convert = require('ansi-to-html'); const convert = new Convert(); // original (strike incorrectly wraps everything in the HTML) console.log(convert.toHtml(colors.strikethrough("crossed out") + ' ' + colors.red("red"))) // prints: <strike>crossed out <span style="color:#A00">red<span style="color:#FFF"></span></span></strike> // workaround console.log(convert.toHtml(colors.reset(colors.strikethrough("crossed out")) + ' ' + colors.red("red"))) // prints: <strike>crossed out</strike> <span style="color:#A00">red<span style="color:#FFF"></span></span> ``` Fixes #11116 |
||
---|---|---|
.. | ||
assets | ||
stable-test-runner | ||
access-data.spec.ts | ||
basic.spec.ts | ||
blink-diff.spec.ts | ||
config.spec.ts | ||
exit-code.spec.ts | ||
expect.spec.ts | ||
fixture-errors.spec.ts | ||
fixtures.spec.ts | ||
gitignore.spec.ts | ||
global-setup.spec.ts | ||
golden.spec.ts | ||
hooks.spec.ts | ||
list-mode.spec.ts | ||
loader.spec.ts | ||
match-grep.spec.ts | ||
max-failures.spec.ts | ||
override-timeout.spec.ts | ||
playwright-test-fixtures.ts | ||
playwright-test.config.ts | ||
playwright.artifacts.spec.ts | ||
playwright.expect.misc.spec.ts | ||
playwright.expect.text.spec.ts | ||
playwright.expect.true.spec.ts | ||
playwright.fetch.spec.ts | ||
playwright.spec.ts | ||
playwright.trace.spec.ts | ||
repeat-each.spec.ts | ||
reporter-attachment.spec.ts | ||
reporter-base.spec.ts | ||
reporter-dot.spec.ts | ||
reporter-github.spec.ts | ||
reporter-html.spec.ts | ||
reporter-json.spec.ts | ||
reporter-junit.spec.ts | ||
reporter-line.spec.ts | ||
reporter-list.spec.ts | ||
reporter-raw.spec.ts | ||
reporter.spec.ts | ||
resolver.spec.ts | ||
retry.spec.ts | ||
runner.spec.ts | ||
shard.spec.ts | ||
stdio.spec.ts | ||
test-extend.spec.ts | ||
test-ignore.spec.ts | ||
test-info.spec.ts | ||
test-modifiers.spec.ts | ||
test-output-dir.spec.ts | ||
test-parallel.spec.ts | ||
test-serial.spec.ts | ||
test-step.spec.ts | ||
test-use.spec.ts | ||
timeout.spec.ts | ||
types-2.spec.ts | ||
types.spec.ts | ||
web-server.spec.ts | ||
worker-index.spec.ts |