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

fix(aria): escape leading dash in property values

Browse source
This commit is contained in:
Pavel Feldman 2025-01-06 16:30:16 -08:00
parent 4e8c83055f
commit 30acc8ae8f
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/playwright-core/src/server/injected/yaml.ts
View file

@ -59,7 +59,7 @@ function yamlStringNeedsQuotes(str: string): boolean {
return true; return true;
// Strings starting with '-' followed by a space need quotes // Strings starting with '-' followed by a space need quotes
if (/^-\s/.test(str)) if (/^-/.test(str))
return true; return true;
// Strings containing ':' or '\n' followed by a space or at the end need quotes // Strings containing ':' or '\n' followed by a space or at the end need quotes

3
tests/page/page-aria-snapshot.spec.ts
View file

@ -555,6 +555,7 @@ it('should escape special yaml values', async ({ page }) => {
<a href="#">null</a>NULL <a href="#">null</a>NULL
<a href="#">123</a>123 <a href="#">123</a>123
<a href="#">-1.2</a>-1.2 <a href="#">-1.2</a>-1.2
<a href="#">-</a>-
<input type=text value="555"> <input type=text value="555">
`); `);
@ -573,6 +574,8 @@ it('should escape special yaml values', async ({ page }) => {
- text: "123" - text: "123"
- link "-1.2" - link "-1.2"
- text: "-1.2" - text: "-1.2"
- link "-"
- text: "-"
- textbox: "555" - textbox: "555"
`); `);
}); });