Skip to content

test(replay): Add additional tests for aria-label #8675

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 31, 2023
Merged

test(replay): Add additional tests for aria-label #8675

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
5695012
add blocking test case
billyvg Jul 19, 2023
47b8c05
test(replay): Add additional tests for aria-label
billyvg Jul 29, 2023
9e60461
lint
billyvg Jul 29, 2023
6fa9494
update snaps
billyvg Jul 31, 2023
007d569
Merge branch 'develop' into test-replay-add-additional-tests-for-rrweb2
billyvg Jul 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/browser-integration-tests/suites/replay/customEvents/template.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<meta charset="utf-8" />
</head>
<body>
<div role="button" id="error" class="btn btn-error" aria-label="An Error">An Error</div>
<div role="button" id="error" class="btn btn-error" aria-label="An Error in aria-label">An Error</div>
<button title="Button title">
<img id="img" alt="Alt Text" />
</button>
Expand Down
4 changes: 2 additions & 2 deletions packages/browser-integration-tests/suites/replay/customEvents/test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,12 @@ sentryTest(
expect.arrayContaining([
{
...expectedClickBreadcrumb,
message: 'body > div#error.btn.btn-error[aria-label="An Error"]',
message: 'body > div#error.btn.btn-error[aria-label="An Error in aria-label"]',
data: {
nodeId: expect.any(Number),
node: {
attributes: {
'aria-label': '** *****',
'aria-label': '** ***** ** **********',
class: 'btn btn-error',
id: 'error',
role: 'button',
Expand Down
3 changes: 3 additions & 0 deletions packages/browser-integration-tests/suites/replay/privacyBlock/template.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@
<input placeholder="Placeholder should be masked" />
<div title="Title should be masked">Title should be masked</div>
<svg style="width:200px;height:200px" viewBox="0 0 80 80"><path d=""/><area /><rect /></svg>
<div data-sentry-block>
<div>This should be blocked</div>
</div>
<svg style="width:200px;height:200px" viewBox="0 0 80 80" data-sentry-unblock><path d=""/><area /><rect /></svg>
<img style="width:100px;height:100px" src="file:///none.png" />
<img data-sentry-unblock style="width:100px;height:100px" src="file:///none.png" />
Expand Down
43 changes: 29 additions & 14 deletions ...wser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-chromium.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,21 @@
"textContent": "\n ",
"id": 28
},
{
"type": 2,
"tagName": "div",
"attributes": {
"rr_width": "[1250-1300]px",
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 29
},
{
"type": 3,
"textContent": "\n ",
"id": 30
},
{
"type": 2,
"tagName": "svg",
Expand All @@ -211,32 +226,32 @@
},
"childNodes": [],
"isSVG": true,
"id": 30
"id": 32
},
{
"type": 2,
"tagName": "area",
"attributes": {},
"childNodes": [],
"isSVG": true,
"id": 31
"id": 33
},
{
"type": 2,
"tagName": "rect",
"attributes": {},
"childNodes": [],
"isSVG": true,
"id": 32
"id": 34
}
],
"isSVG": true,
"id": 29
"id": 31
},
{
"type": 3,
"textContent": "\n ",
"id": 33
"id": 35
},
{
"type": 2,
Expand All @@ -246,12 +261,12 @@
"src": "file:///none.png"
},
"childNodes": [],
"id": 34
"id": 36
},
{
"type": 3,
"textContent": "\n ",
"id": 35
"id": 37
},
{
"type": 2,
Expand All @@ -262,12 +277,12 @@
"src": "file:///none.png"
},
"childNodes": [],
"id": 36
"id": 38
},
{
"type": 3,
"textContent": "\n ",
"id": 37
"id": 39
},
{
"type": 2,
Expand All @@ -277,12 +292,12 @@
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 38
"id": 40
},
{
"type": 3,
"textContent": "\n ",
"id": 39
"id": 41
},
{
"type": 2,
Expand All @@ -293,17 +308,17 @@
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 40
"id": 42
},
{
"type": 3,
"textContent": "\n ",
"id": 41
"id": 43
},
{
"type": 3,
"textContent": "\n\n",
"id": 42
"id": 44
}
],
"id": 8
Expand Down
43 changes: 29 additions & 14 deletions ...rowser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-webkit.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,21 @@
"textContent": "\n ",
"id": 28
},
{
"type": 2,
"tagName": "div",
"attributes": {
"rr_width": "[1250-1300]px",
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 29
},
{
"type": 3,
"textContent": "\n ",
"id": 30
},
{
"type": 2,
"tagName": "svg",
Expand All @@ -211,32 +226,32 @@
},
"childNodes": [],
"isSVG": true,
"id": 30
"id": 32
},
{
"type": 2,
"tagName": "area",
"attributes": {},
"childNodes": [],
"isSVG": true,
"id": 31
"id": 33
},
{
"type": 2,
"tagName": "rect",
"attributes": {},
"childNodes": [],
"isSVG": true,
"id": 32
"id": 34
}
],
"isSVG": true,
"id": 29
"id": 31
},
{
"type": 3,
"textContent": "\n ",
"id": 33
"id": 35
},
{
"type": 2,
Expand All @@ -246,12 +261,12 @@
"src": "file:///none.png"
},
"childNodes": [],
"id": 34
"id": 36
},
{
"type": 3,
"textContent": "\n ",
"id": 35
"id": 37
},
{
"type": 2,
Expand All @@ -262,12 +277,12 @@
"src": "file:///none.png"
},
"childNodes": [],
"id": 36
"id": 38
},
{
"type": 3,
"textContent": "\n ",
"id": 37
"id": 39
},
{
"type": 2,
Expand All @@ -277,12 +292,12 @@
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 38
"id": 40
},
{
"type": 3,
"textContent": "\n ",
"id": 39
"id": 41
},
{
"type": 2,
Expand All @@ -293,17 +308,17 @@
"rr_height": "[0-50]px"
},
"childNodes": [],
"id": 40
"id": 42
},
{
"type": 3,
"textContent": "\n ",
"id": 41
"id": 43
},
{
"type": 3,
"textContent": "\n\n",
"id": 42
"id": 44
}
],
"id": 8
Expand Down
1 change: 1 addition & 0 deletions packages/browser-integration-tests/suites/replay/privacyDefault/template.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
<div>This should be masked by default</div>
<div data-sentry-unmask>This should be unmasked due to data attribute</div>
<input placeholder="Placeholder should be masked" />
<input data-sentry-unmask placeholder="Placeholder can be unmasked" />
<div title="Title should be masked">Title should be masked</div>
<svg style="width:200px;height:200px" viewBox="0 0 80 80"><path d=""/><area /><rect /></svg>
<svg style="width:200px;height:200px" viewBox="0 0 80 80" data-sentry-unblock><path d=""/><area /><rect /></svg>
Expand Down
Loading