[Bug]: Overridden browser fixture does not work as expected about playwright

[dgozman]

@mdmundo Thank you for the repro, illustrates the question nicely!

The difference between the two fixtures is that browser is a worker-scoped fixture, while chromium is a test-scoped fixture.

So, when using the browser fixture:

• Test successfully finishes.
• Test-scoped fixtures are successfully teared down.
• Playwright initiates worker termination, which includes tearing down the browser fixture.
• Playwright ensures a 30 seconds maximum limit on the worker teardown, and kills the worker process after that. The browser is forcefully closed. Note there is a similar issue #30504 that advocates for a customizable timeout for worker fixture teardown.

When using the chromium fixture:

• Test successfully finishes.
• Teardown for test-scoped fixtures is initiated, which includes tearing down the chromium fixture.
• Since chromium never finishes, Playwright does not consider the test to be finished. Given that you set timeout: 0 in the config, Playwright waits indefinitely for the test to finish.

Given the above, it seems like things work as intended. Let me know what you think.

[mdmundo]

@dgozman should not it be able to override timeout with this? { scope: "worker", timeout: 0 }

export const test = base.extend({
  browser: [
    ({ browser }, use) => use(browser),
    { scope: "worker", timeout: 0 },
  ],
});

[dgozman]

@mdmundo Yep, this should work once #30504 is fixed. I'll close this issue, since there is no action item for Playwright. Thank you for filing!

[mdmundo]

@dgozman thanks for the explanation!