[Feature] Enable toBeOK matcher also for Response interface

[sunker]

I would like the toBeOK matcher to apply also to the Response interface.

// the following is simply more expressive 
const response = await page.waitForResponse('https://example.com/resource');
await expect(response).toBeOK();

// ..than doing this
const response = await page.waitForResponse('https://example.com/resource');
await expect(response.ok()).toBeTruthy();

It makes no sense that this matcher applies to APIResponse interface but not Response interface.

This feature request has been raised before: https://github.com/microsoft/playwright/issues/12171

[ltsuda]

Hi, is this what you are looking for?

https://playwright.dev/docs/api/class-apiresponseassertions#api-response-assertions-to-be-ok

[sunker]

The receiving type of the toBeOK matcher is currently APIResponse. I would like the toBeOK matcher to also accept Response as receiving type.

[eveltman-twinfield]

I would like this as well. The use case I just came across was a "document viewer" side panel being used on a page. The document viewer was loading a PDF document in an iframe. We cannot interact with the PDF viewer but we wanted to at least verify that the URL passed to the document viewer was correct and returning a successful response.