[BUG] Unable change snapshot naming for webkit

Context:

• Playwright Version: 1.22
• Operating System: linux ‘docker’
• Node.js version: 14.6
• Browser: webkit

Code Snippet

test.describe('Landing page Test Suite', () => {

    test.beforeEach(async ({ page, landingPage },testinfo) => {
        testinfo.snapshotSuffix = ''
        await page.goto(path.en.landingPage, { waitUntil: "load", timeout: 30000 })
    });

    test('Landing page is loading properly with all exact contents in EN lang', async ({ browserName, page }) => {
        expect(await page.screenshot({ fullPage: true, timeout: 4000 })).toMatchSnapshot(`${browserName}-EN.png`,{maxDiffPixels: 30,});
    });

Describe the bug

Test is failing in beforeEach hook for webkit when using testinfo.snapshotSuffix = ‘’

[rwoll]

Repro for Bisect: https://github.com/rwoll/playwright-repro-14236 Bad Commit: 41db08eca567098857d574e06691b3b43e7d6afd: feat(webkit): roll to r1631 (https://github.com/microsoft/playwright/pull/13612) WebKit roll: https://github.com/microsoft/playwright/commit/d44cfb93cb66b21ba1df1f758a2f629530cac7c2 Environment: Docker container running on an M1 Mac

Since it’s a WK roll, looks like we’ll need to bisect WK itself. (NB: I also manually rolled ToT (432c52d31aaeeb279f4bd8b59de70a15ad1cab07) to r1647, and tested; it’s still failing on that revision.)

cc: @aslushnikov @mxschmitt @yury-s

[amrsa1]

Thanks @amrsa1! It looks like I’ve found a standalone repro: https://github.com/rwoll/playwright-repro-14236

The error message is not identical, but it’s also WK+Linux/Docker-only fail. (My gut says they are related.)

Yes seems so, also extra info Im using macOs m1, running test inside docker container with param platform:linux/amd64 otherwise it wont work