Skip to content

Commit

Permalink
fix: rename haveRequestHeaders of ppt and pw helpers (#3937)
Browse files Browse the repository at this point in the history
* fix: rename haveRequestHeaders of ppt and pw helpers

* Update Playwright.types.ts

* Update PlaywrightTs.types.ts
  • Loading branch information
kobenguyent committed Oct 28, 2023
1 parent bfe8d8f commit f78c027
Show file tree
Hide file tree
Showing 6 changed files with 34 additions and 34 deletions.
28 changes: 14 additions & 14 deletions docs/helpers/Playwright.md
Original file line number Diff line number Diff line change
Expand Up @@ -1377,20 +1377,6 @@ I.waitForFile('avatar.jpg', 5);

Returns **[Promise][9]<void>**

### haveRequestHeaders

Set headers for all next requests

```js
I.haveRequestHeaders({
'X-Sent-By': 'CodeceptJS',
});
```

#### Parameters

- `customHeaders` **[object][5]** headers to set

### makeApiRequest

Performs [api request][25] using
Expand Down Expand Up @@ -2077,6 +2063,20 @@ I.setCookie([

Returns **[Promise][9]<void>** automatically synchronized promise through #recorder

### setPlaywrightRequestHeaders

Set headers for all next requests

```js
I.setPlaywrightRequestHeaders({
'X-Sent-By': 'CodeceptJS',
});
```

#### Parameters

- `customHeaders` **[object][5]** headers to set

### startRecordingTraffic

Starts recording the network traffics.
Expand Down
28 changes: 14 additions & 14 deletions docs/helpers/Puppeteer.md
Original file line number Diff line number Diff line change
Expand Up @@ -1227,20 +1227,6 @@ I.seeFile('avatar.jpg');
- `downloadPath` **[string][6]** change this parameter to set another directory for saving
### haveRequestHeaders
Set headers for all next requests
```js
I.haveRequestHeaders({
'X-Sent-By': 'CodeceptJS',
});
```
#### Parameters
- `customHeaders` **[object][4]** headers to set
### moveCursorTo
Moves cursor to element matched by locator.
Expand Down Expand Up @@ -1830,6 +1816,20 @@ I.setCookie([
Returns **[Promise][7]<void>** automatically synchronized promise through #recorder
### setPuppeteerRequestHeaders
Set headers for all next requests
```js
I.setPuppeteerRequestHeaders({
'X-Sent-By': 'CodeceptJS',
});
```
#### Parameters
- `customHeaders` **[object][4]** headers to set
### switchTo
Switches frame or in case of null locator reverts to parent.
Expand Down
4 changes: 2 additions & 2 deletions lib/helper/Playwright.js
Original file line number Diff line number Diff line change
Expand Up @@ -942,14 +942,14 @@ class Playwright extends Helper {
* Set headers for all next requests
*
* ```js
* I.haveRequestHeaders({
* I.setPlaywrightRequestHeaders({
* 'X-Sent-By': 'CodeceptJS',
* });
* ```
*
* @param {object} customHeaders headers to set
*/
async haveRequestHeaders(customHeaders) {
async setPlaywrightRequestHeaders(customHeaders) {
if (!customHeaders) {
throw new Error('Cannot send empty headers.');
}
Expand Down
4 changes: 2 additions & 2 deletions lib/helper/Puppeteer.js
Original file line number Diff line number Diff line change
Expand Up @@ -682,14 +682,14 @@ class Puppeteer extends Helper {
* Set headers for all next requests
*
* ```js
* I.haveRequestHeaders({
* I.setPuppeteerRequestHeaders({
* 'X-Sent-By': 'CodeceptJS',
* });
* ```
*
* @param {object} customHeaders headers to set
*/
async haveRequestHeaders(customHeaders) {
async setPuppeteerRequestHeaders(customHeaders) {
if (!customHeaders) {
throw new Error('Cannot send empty headers.');
}
Expand Down
2 changes: 1 addition & 1 deletion typings/tests/helpers/Playwright.types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ playwright._createContextPage(); // $ExpectType void
playwright._createContextPage({}); // $ExpectType void
playwright.amOnPage(str); // $ExpectType Promise<void>
playwright.resizeWindow(num, num); // $ExpectType Promise<void>
playwright.haveRequestHeaders(str); // $ExpectType void
playwright.setPlaywrightRequestHeaders(str); // $ExpectType void
playwright.moveCursorTo(str, num, num); // $ExpectType Promise<void>
playwright.dragAndDrop(str); // $ExpectError
playwright.dragAndDrop(str, str); // $ExpectType Promise<void>
Expand Down
2 changes: 1 addition & 1 deletion typings/tests/helpers/PlaywrightTs.types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ playwright._getPageUrl(); // $ExpectType Promise<any>
playwright.grabPopupText(); // $ExpectType Promise<string | null>
playwright.amOnPage(str); // $ExpectType Promise<void>
playwright.resizeWindow(num, num); // $ExpectType Promise<void>
playwright.haveRequestHeaders(str); // $ExpectType Promise<any>
playwright.setPlaywrightRequestHeaders(str); // $ExpectType Promise<any>
playwright.moveCursorTo(str, num, num); // $ExpectType Promise<void>
playwright.dragAndDrop(str); // $ExpectError
playwright.dragAndDrop(str, str); // $ExpectType Promise<void>
Expand Down

0 comments on commit f78c027

Please sign in to comment.