Skip to content

Commit 4af2629

Browse files
committed
chore: change to snapshot testing
1 parent a99cc35 commit 4af2629

2 files changed

Lines changed: 16 additions & 3 deletions

File tree

tests/tools/pages.test.js.snapshot

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
exports[`pages > list_pages > list pages for extension pages with --category-extensions 1`] = `
2+
# list_pages response
3+
## Pages
4+
1: about:blank [selected]
5+
## Extension Pages
6+
2: chrome-extension://<extension-id>/popup.html
7+
## Extension Service Workers
8+
sw-1: chrome-extension://<extension-id>/sw.js
9+
`;

tests/tools/pages.test.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ describe('pages', () => {
4949
assert.ok(response.includePages);
5050
});
5151
});
52-
it(`list pages for extension pages with --category-extensions`, async () => {
52+
it(`list pages for extension pages with --category-extensions`, async t => {
5353
await withMcpContext(
5454
async (response, context) => {
5555
await installExtension.handler(
@@ -66,7 +66,7 @@ describe('pages', () => {
6666
response,
6767
context,
6868
);
69-
const popupTarget = await context.browser.waitForTarget(
69+
const _popupTarget = await context.browser.waitForTarget(
7070
t => t.type() === 'page' && t.url().includes('chrome-extension://'),
7171
);
7272

@@ -88,7 +88,11 @@ describe('pages', () => {
8888
};
8989
assert.ok(textContent);
9090

91-
assert.ok(textContent.text.includes(popupTarget.url()));
91+
const text = textContent.text.replaceAll(
92+
extensionId,
93+
'<extension-id>',
94+
);
95+
t.assert.snapshot?.(text);
9296
},
9397
{
9498
executablePath: process.env.CANARY_EXECUTABLE_PATH,

0 commit comments

Comments
 (0)