[Feature]: add failOnStatusCode to requestContext

[fdm-mateco]

🚀 Feature Request

When defining a (manual) API request context, one should be able to define failOnStatusCode. This should cause all requests done through this context to apply the flag (unless overridden).

const context = await request.newContext({
      baseURL: 'https://example/api/',
      extraHTTPHeaders: {
        'Content-Type': 'application/json',
        'Authorization': `Bearer ${process.env.TOKEN}`,
      },
      failOnStatusCode: true
    });

Example

No response

Motivation

It will prevent repetitive code, as we'll no longer have to add the property on every API call as is now:

await this.context.get('contactpersons', { failOnStatusCode: true });
await this.context.get('addresses', { failOnStatusCode: true });
await this.context.get('customers', { failOnStatusCode: true });

[Skn0tt]

Sounds good! I think it should be wrapped in apiRequest, like this:

const context = await request.newContext({
  baseURL: 'https://example/api/',
  extraHTTPHeaders: {
    'Content-Type': 'application/json',
    'Authorization': `Bearer ${process.env.TOKEN}`,
  },
  apiRequest: {
    failOnStatusCode: true
  }
});

Would you be open to contribute a PR for this? The relevant code areas are:

• playwright/packages/playwright-core/src/server/fetch.ts

  Line 208 in 9b58e4a

  if (params.failOnStatusCode && (fetchResponse.status < 200 || fetchResponse.status >= 400)) {

• playwright/packages/playwright-core/src/server/fetch.ts

  Line 609 in 9b58e4a

  _defaultOptions(): FetchRequestOptions {

• playwright/docs/src/api/params.md

  Line 948 in 9b58e4a

  ## shared-context-params-list-v1.8

Happy to contribute the patch myself if this isn't interesting to you.