Skip to content

Commit 4b16f78

Browse files
committed
remove browser name as it is not necessary in chromatic
1 parent 3cfcbf3 commit 4b16f78

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

tests/e2e/find-wallet.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,11 @@ test.beforeEach(async ({ page }) => {
99
})
1010

1111
test.describe("Find Wallet Page", () => {
12-
test("loads successfully", async ({ page, browserName }, testInfo) => {
12+
test("loads successfully", async ({ page }, testInfo) => {
1313
await expect(
1414
page.getByRole("heading", { name: "Choose your wallet" })
1515
).toBeVisible()
16-
await takeSnapshot(page, `find-wallet-${browserName}-initial`, testInfo)
16+
await takeSnapshot(page, "find-wallet-initial", testInfo)
1717
})
1818

1919
test("personas filter updates counter and list", async ({ page }) => {

tests/e2e/home.spec.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ test.beforeEach(async ({ page }) => {
99
})
1010

1111
test.describe("Home Page", () => {
12-
test("loads successfully", async ({ page, browserName }, testInfo) => {
12+
test("loads successfully", async ({ page }, testInfo) => {
1313
await expect(page).toHaveTitle(/Ethereum.org/)
1414

15-
await takeSnapshot(page, `home-${browserName}-initial`, testInfo)
15+
await takeSnapshot(page, "home-initial", testInfo)
1616
})
1717

1818
test("search functionality", async ({ page }) => {
@@ -41,12 +41,10 @@ test.describe("Home Page", () => {
4141
await expect(page).toHaveURL(/.*\/developers/)
4242
})
4343

44-
test("navigation menu - mobile", async ({ page, browserName }, testInfo) => {
44+
test("navigation menu - mobile", async ({ page }, testInfo) => {
4545
// Set viewport to mobile size
4646
await page.setViewportSize({ width: breakpointAsNumber.sm, height: 800 })
4747

48-
await takeSnapshot(page, `home-${browserName}-initial`, testInfo)
49-
5048
const nav = page.getByRole("navigation", { name: "Primary" })
5149
const menuButton = nav.getByRole("button", {
5250
name: /toggle menu button/i,
@@ -56,7 +54,7 @@ test.describe("Home Page", () => {
5654
// Open the mobile menu
5755
await menuButton.click()
5856

59-
await takeSnapshot(page, `home-${browserName}-menu-open`, testInfo)
57+
await takeSnapshot(page, "home-menu-open", testInfo)
6058

6159
// Check that navigation links are visible in the mobile menu
6260
const sidebar = page.getByRole("dialog", { name: /ethereum.org/i })

0 commit comments

Comments
 (0)