fix(test): fix the route test (#14890)

This commit is contained in:
Pavel Feldman 2022-06-15 15:15:45 -08:00 committed by GitHub
parent 59fa61aef1
commit 225ab68d1c
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 389 additions and 7 deletions

View file

@ -702,6 +702,13 @@ export class Page extends ChannelOwner<channels.PageChannel> implements api.Page
return this; return this;
} }
override prependListener(event: string | symbol, listener: Listener): this {
if (event === Events.Page.FileChooser && !this.listenerCount(event))
this._channel.setFileChooserInterceptedNoReply({ intercepted: true });
super.prependListener(event, listener);
return this;
}
override off(event: string | symbol, listener: Listener): this { override off(event: string | symbol, listener: Listener): this {
super.off(event, listener); super.off(event, listener);
if (event === Events.Page.FileChooser && !this.listenerCount(event)) if (event === Events.Page.FileChooser && !this.listenerCount(event))

View file

@ -1525,6 +1525,207 @@ export interface Page {
*/ */
off(event: 'worker', listener: (worker: Worker) => void): this; off(event: 'worker', listener: (worker: Worker) => void): this;
/**
* Emitted when the page closes.
*/
prependListener(event: 'close', listener: (page: Page) => void): this;
/**
* Emitted when JavaScript within the page calls one of console API methods, e.g. `console.log` or `console.dir`. Also
* emitted if the page throws an error or a warning.
*
* The arguments passed into `console.log` appear as arguments on the event handler.
*
* An example of handling `console` event:
*
* ```js
* page.on('console', async msg => {
* const values = [];
* for (const arg of msg.args())
* values.push(await arg.jsonValue());
* console.log(...values);
* });
* await page.evaluate(() => console.log('hello', 5, {foo: 'bar'}));
* ```
*
*/
prependListener(event: 'console', listener: (consoleMessage: ConsoleMessage) => void): this;
/**
* Emitted when the page crashes. Browser pages might crash if they try to allocate too much memory. When the page crashes,
* ongoing and subsequent operations will throw.
*
* The most common way to deal with crashes is to catch an exception:
*
* ```js
* try {
* // Crash might happen during a click.
* await page.click('button');
* // Or while waiting for an event.
* await page.waitForEvent('popup');
* } catch (e) {
* // When the page crashes, exception message contains 'crash'.
* }
* ```
*
*/
prependListener(event: 'crash', listener: (page: Page) => void): this;
/**
* Emitted when a JavaScript dialog appears, such as `alert`, `prompt`, `confirm` or `beforeunload`. Listener **must**
* either [dialog.accept([promptText])](https://playwright.dev/docs/api/class-dialog#dialog-accept) or
* [dialog.dismiss()](https://playwright.dev/docs/api/class-dialog#dialog-dismiss) the dialog - otherwise the page will
* [freeze](https://developer.mozilla.org/en-US/docs/Web/JavaScript/EventLoop#never_blocking) waiting for the dialog, and
* actions like click will never finish.
*
* ```js
* page.on('dialog', dialog => {
* dialog.accept();
* });
* ```
*
* > NOTE: When no [page.on('dialog')](https://playwright.dev/docs/api/class-page#page-event-dialog) listeners are present,
* all dialogs are automatically dismissed.
*/
prependListener(event: 'dialog', listener: (dialog: Dialog) => void): this;
/**
* Emitted when the JavaScript [`DOMContentLoaded`](https://developer.mozilla.org/en-US/docs/Web/Events/DOMContentLoaded)
* event is dispatched.
*/
prependListener(event: 'domcontentloaded', listener: (page: Page) => void): this;
/**
* Emitted when attachment download started. User can access basic file operations on downloaded content via the passed
* [Download] instance.
*/
prependListener(event: 'download', listener: (download: Download) => void): this;
/**
* Emitted when a file chooser is supposed to appear, such as after clicking the `<input type=file>`. Playwright can
* respond to it via setting the input files using
* [fileChooser.setFiles(files[, options])](https://playwright.dev/docs/api/class-filechooser#file-chooser-set-files) that
* can be uploaded after that.
*
* ```js
* page.on('filechooser', async (fileChooser) => {
* await fileChooser.setFiles('/tmp/myfile.pdf');
* });
* ```
*
*/
prependListener(event: 'filechooser', listener: (fileChooser: FileChooser) => void): this;
/**
* Emitted when a frame is attached.
*/
prependListener(event: 'frameattached', listener: (frame: Frame) => void): this;
/**
* Emitted when a frame is detached.
*/
prependListener(event: 'framedetached', listener: (frame: Frame) => void): this;
/**
* Emitted when a frame is navigated to a new url.
*/
prependListener(event: 'framenavigated', listener: (frame: Frame) => void): this;
/**
* Emitted when the JavaScript [`load`](https://developer.mozilla.org/en-US/docs/Web/Events/load) event is dispatched.
*/
prependListener(event: 'load', listener: (page: Page) => void): this;
/**
* Emitted when an uncaught exception happens within the page.
*
* ```js
* // Log all uncaught errors to the terminal
* page.on('pageerror', exception => {
* console.log(`Uncaught exception: "${exception}"`);
* });
*
* // Navigate to a page with an exception.
* await page.goto('data:text/html,<script>throw new Error("Test")</script>');
* ```
*
*/
prependListener(event: 'pageerror', listener: (error: Error) => void): this;
/**
* Emitted when the page opens a new tab or window. This event is emitted in addition to the
* [browserContext.on('page')](https://playwright.dev/docs/api/class-browsercontext#browser-context-event-page), but only
* for popups relevant to this page.
*
* The earliest moment that page is available is when it has navigated to the initial url. For example, when opening a
* popup with `window.open('http://example.com')`, this event will fire when the network request to "http://example.com" is
* done and its response has started loading in the popup.
*
* ```js
* // Note that Promise.all prevents a race condition
* // between evaluating and waiting for the popup.
* const [popup] = await Promise.all([
* // It is important to call waitForEvent first.
* page.waitForEvent('popup'),
* // Opens the popup.
* page.evaluate(() => window.open('https://example.com')),
* ]);
* console.log(await popup.evaluate('location.href'));
* ```
*
* > NOTE: Use
* [page.waitForLoadState([state, options])](https://playwright.dev/docs/api/class-page#page-wait-for-load-state) to wait
* until the page gets to a particular state (you should not need it in most cases).
*/
prependListener(event: 'popup', listener: (page: Page) => void): this;
/**
* Emitted when a page issues a request. The [request] object is read-only. In order to intercept and mutate requests, see
* [page.route(url, handler[, options])](https://playwright.dev/docs/api/class-page#page-route) or
* [browserContext.route(url, handler[, options])](https://playwright.dev/docs/api/class-browsercontext#browser-context-route).
*/
prependListener(event: 'request', listener: (request: Request) => void): this;
/**
* Emitted when a request fails, for example by timing out.
*
* ```js
* page.on('requestfailed', request => {
* console.log(request.url() + ' ' + request.failure().errorText);
* });
* ```
*
* > NOTE: HTTP Error responses, such as 404 or 503, are still successful responses from HTTP standpoint, so request will
* complete with [page.on('requestfinished')](https://playwright.dev/docs/api/class-page#page-event-request-finished) event
* and not with [page.on('requestfailed')](https://playwright.dev/docs/api/class-page#page-event-request-failed). A request
* will only be considered failed when the client cannot get an HTTP response from the server, e.g. due to network error
* net::ERR_FAILED.
*/
prependListener(event: 'requestfailed', listener: (request: Request) => void): this;
/**
* Emitted when a request finishes successfully after downloading the response body. For a successful response, the
* sequence of events is `request`, `response` and `requestfinished`.
*/
prependListener(event: 'requestfinished', listener: (request: Request) => void): this;
/**
* Emitted when [response] status and headers are received for a request. For a successful response, the sequence of events
* is `request`, `response` and `requestfinished`.
*/
prependListener(event: 'response', listener: (response: Response) => void): this;
/**
* Emitted when [WebSocket] request is sent.
*/
prependListener(event: 'websocket', listener: (webSocket: WebSocket) => void): this;
/**
* Emitted when a dedicated [WebWorker](https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API) is spawned by the
* page.
*/
prependListener(event: 'worker', listener: (worker: Worker) => void): this;
accessibility: Accessibility; accessibility: Accessibility;
/** /**
@ -6555,6 +6756,93 @@ export interface BrowserContext {
*/ */
off(event: 'serviceworker', listener: (worker: Worker) => void): this; off(event: 'serviceworker', listener: (worker: Worker) => void): this;
/**
* > NOTE: Only works with Chromium browser's persistent context.
*
* Emitted when new background page is created in the context.
*
* ```js
* const backgroundPage = await context.waitForEvent('backgroundpage');
* ```
*
*/
prependListener(event: 'backgroundpage', listener: (page: Page) => void): this;
/**
* Emitted when Browser context gets closed. This might happen because of one of the following:
* - Browser context is closed.
* - Browser application is closed or crashed.
* - The [browser.close()](https://playwright.dev/docs/api/class-browser#browser-close) method was called.
*/
prependListener(event: 'close', listener: (browserContext: BrowserContext) => void): this;
/**
* The event is emitted when a new Page is created in the BrowserContext. The page may still be loading. The event will
* also fire for popup pages. See also [page.on('popup')](https://playwright.dev/docs/api/class-page#page-event-popup) to
* receive events about popups relevant to a specific page.
*
* The earliest moment that page is available is when it has navigated to the initial url. For example, when opening a
* popup with `window.open('http://example.com')`, this event will fire when the network request to "http://example.com" is
* done and its response has started loading in the popup.
*
* ```js
* const [newPage] = await Promise.all([
* context.waitForEvent('page'),
* page.click('a[target=_blank]'),
* ]);
* console.log(await newPage.evaluate('location.href'));
* ```
*
* > NOTE: Use
* [page.waitForLoadState([state, options])](https://playwright.dev/docs/api/class-page#page-wait-for-load-state) to wait
* until the page gets to a particular state (you should not need it in most cases).
*/
prependListener(event: 'page', listener: (page: Page) => void): this;
/**
* Emitted when a request is issued from any pages created through this context. The [request] object is read-only. To only
* listen for requests from a particular page, use
* [page.on('request')](https://playwright.dev/docs/api/class-page#page-event-request).
*
* In order to intercept and mutate requests, see
* [browserContext.route(url, handler[, options])](https://playwright.dev/docs/api/class-browsercontext#browser-context-route)
* or [page.route(url, handler[, options])](https://playwright.dev/docs/api/class-page#page-route).
*/
prependListener(event: 'request', listener: (request: Request) => void): this;
/**
* Emitted when a request fails, for example by timing out. To only listen for failed requests from a particular page, use
* [page.on('requestfailed')](https://playwright.dev/docs/api/class-page#page-event-request-failed).
*
* > NOTE: HTTP Error responses, such as 404 or 503, are still successful responses from HTTP standpoint, so request will
* complete with
* [browserContext.on('requestfinished')](https://playwright.dev/docs/api/class-browsercontext#browser-context-event-request-finished)
* event and not with
* [browserContext.on('requestfailed')](https://playwright.dev/docs/api/class-browsercontext#browser-context-event-request-failed).
*/
prependListener(event: 'requestfailed', listener: (request: Request) => void): this;
/**
* Emitted when a request finishes successfully after downloading the response body. For a successful response, the
* sequence of events is `request`, `response` and `requestfinished`. To listen for successful requests from a particular
* page, use [page.on('requestfinished')](https://playwright.dev/docs/api/class-page#page-event-request-finished).
*/
prependListener(event: 'requestfinished', listener: (request: Request) => void): this;
/**
* Emitted when [response] status and headers are received for a request. For a successful response, the sequence of events
* is `request`, `response` and `requestfinished`. To listen for response events from a particular page, use
* [page.on('response')](https://playwright.dev/docs/api/class-page#page-event-response).
*/
prependListener(event: 'response', listener: (response: Response) => void): this;
/**
* > NOTE: Service workers are only supported on Chromium-based browsers.
*
* Emitted when new service worker is created in the context.
*/
prependListener(event: 'serviceworker', listener: (worker: Worker) => void): this;
/** /**
* Adds cookies into this browser context. All pages within this context will have these cookies installed. Cookies can be * Adds cookies into this browser context. All pages within this context will have these cookies installed. Cookies can be
* obtained via * obtained via
@ -7231,6 +7519,11 @@ export interface Worker {
*/ */
off(event: 'close', listener: (worker: Worker) => void): this; off(event: 'close', listener: (worker: Worker) => void): this;
/**
* Emitted when this dedicated [WebWorker](https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API) is terminated.
*/
prependListener(event: 'close', listener: (worker: Worker) => void): this;
url(): string;} url(): string;}
/** /**
@ -11029,6 +11322,17 @@ export interface ElectronApplication {
*/ */
off(event: 'window', listener: (page: Page) => void): this; off(event: 'window', listener: (page: Page) => void): this;
/**
* This event is issued when the application closes.
*/
prependListener(event: 'close', listener: () => void): this;
/**
* This event is issued for every window that is created **and loaded** in Electron. It contains a [Page] that can be used
* for Playwright automation.
*/
prependListener(event: 'window', listener: (page: Page) => void): this;
/** /**
* Returns the BrowserWindow object that corresponds to the given Playwright page. * Returns the BrowserWindow object that corresponds to the given Playwright page.
* @param page Page to retrieve the window for. * @param page Page to retrieve the window for.
@ -11313,6 +11617,11 @@ export interface AndroidDevice {
*/ */
off(event: 'webview', listener: (androidWebView: AndroidWebView) => void): this; off(event: 'webview', listener: (androidWebView: AndroidWebView) => void): this;
/**
* Emitted when a new WebView instance is detected.
*/
prependListener(event: 'webview', listener: (androidWebView: AndroidWebView) => void): this;
/** /**
* Disconnects from the device. * Disconnects from the device.
*/ */
@ -12033,6 +12342,16 @@ export interface AndroidSocket {
*/ */
off(event: 'data', listener: (buffer: Buffer) => void): this; off(event: 'data', listener: (buffer: Buffer) => void): this;
/**
* Emitted when the socket is closed.
*/
prependListener(event: 'close', listener: () => void): this;
/**
* Emitted when data is available to read from the socket.
*/
prependListener(event: 'data', listener: (buffer: Buffer) => void): this;
/** /**
* Closes the socket. * Closes the socket.
*/ */
@ -12075,6 +12394,11 @@ export interface AndroidWebView {
*/ */
off(event: 'close', listener: () => void): this; off(event: 'close', listener: () => void): this;
/**
* Emitted when the WebView is closed.
*/
prependListener(event: 'close', listener: () => void): this;
/** /**
* Connects to the WebView and returns a regular Playwright [Page] to interact with. * Connects to the WebView and returns a regular Playwright [Page] to interact with.
*/ */
@ -12848,6 +13172,13 @@ export interface Browser extends EventEmitter {
*/ */
off(event: 'disconnected', listener: (browser: Browser) => void): this; off(event: 'disconnected', listener: (browser: Browser) => void): this;
/**
* Emitted when Browser gets disconnected from the browser application. This might happen because of one of the following:
* - Browser application is closed or crashed.
* - The [browser.close()](https://playwright.dev/docs/api/class-browser#browser-close) method was called.
*/
prependListener(event: 'disconnected', listener: (browser: Browser) => void): this;
/** /**
* Get the browser type (chromium, firefox or webkit) that the browser belongs to. * Get the browser type (chromium, firefox or webkit) that the browser belongs to.
*/ */
@ -13342,6 +13673,11 @@ export interface BrowserServer {
*/ */
off(event: 'close', listener: () => void): this; off(event: 'close', listener: () => void): this;
/**
* Emitted when the browser server closes.
*/
prependListener(event: 'close', listener: () => void): this;
/** /**
* Closes the browser gracefully and makes sure the process is terminated. * Closes the browser gracefully and makes sure the process is terminated.
*/ */
@ -15430,6 +15766,36 @@ export interface WebSocket {
*/ */
off(event: 'socketerror', listener: (string: String) => void): this; off(event: 'socketerror', listener: (string: String) => void): this;
/**
* Fired when the websocket closes.
*/
prependListener(event: 'close', listener: (webSocket: WebSocket) => void): this;
/**
* Fired when the websocket receives a frame.
*/
prependListener(event: 'framereceived', listener: (data: {
/**
* frame payload
*/
payload: string|Buffer;
}) => void): this;
/**
* Fired when the websocket sends a frame.
*/
prependListener(event: 'framesent', listener: (data: {
/**
* frame payload
*/
payload: string|Buffer;
}) => void): this;
/**
* Fired when the websocket has an error.
*/
prependListener(event: 'socketerror', listener: (string: String) => void): this;
/** /**
* Indicates that the web socket has been closed. * Indicates that the web socket has been closed.
*/ */

View file

@ -830,11 +830,11 @@ it('should contain raw response header after fulfill', async ({ page, server })
for (const method of ['fulfill', 'continue', 'fallback', 'abort'] as const) { for (const method of ['fulfill', 'continue', 'fallback', 'abort'] as const) {
it(`route.${method} should throw if called twice`, async ({ page, server }) => { it(`route.${method} should throw if called twice`, async ({ page, server }) => {
const routePromise = new Promise<Route>(async resolve => { let resolve;
await page.route('**/*', resolve); const resolvePromise = new Promise<Route>(f => resolve = f);
}); await page.route('**/*', resolve);
page.goto(server.PREFIX + '/empty.html').catch(() => {}); page.goto(server.PREFIX + '/empty.html').catch(() => {});
const route = await routePromise; const route = await resolvePromise;
await route[method](); await route[method]();
const e = await route[method]().catch(e => e); const e = await route[method]().catch(e => e);
expect(e.message).toContain('Route is already handled!'); expect(e.message).toContain('Route is already handled!');

View file

@ -169,6 +169,15 @@ it('should emit event once', async ({ page, server }) => {
expect(chooser).toBeTruthy(); expect(chooser).toBeTruthy();
}); });
it('should emit event via prepend', async ({ page, server }) => {
await page.setContent(`<input type=file>`);
const [chooser] = await Promise.all([
new Promise(f => page.prependListener('filechooser', f)),
page.click('input'),
]);
expect(chooser).toBeTruthy();
});
it('should emit event for iframe', async ({ page, server, browserName }) => { it('should emit event for iframe', async ({ page, server, browserName }) => {
it.skip(browserName === 'firefox'); it.skip(browserName === 'firefox');
const frame = await attachFrame(page, 'frame1', server.EMPTY_PAGE); const frame = await attachFrame(page, 'frame1', server.EMPTY_PAGE);

View file

@ -109,7 +109,7 @@ class TypesGenerator {
return ''; return '';
this.handledMethods.add(`${className}.${methodName}#${overloadIndex}`); this.handledMethods.add(`${className}.${methodName}#${overloadIndex}`);
if (!method) { if (!method) {
if (new Set(['on', 'addListener', 'off', 'removeListener', 'once']).has(methodName)) if (new Set(['on', 'addListener', 'off', 'removeListener', 'once', 'prependListener']).has(methodName))
return ''; return '';
throw new Error(`Unknown override method "${className}.${methodName}"`); throw new Error(`Unknown override method "${className}.${methodName}"`);
} }
@ -269,9 +269,9 @@ class TypesGenerator {
once: 'Adds an event listener that will be automatically removed after it is triggered once. See `addListener` for more information about this event.' once: 'Adds an event listener that will be automatically removed after it is triggered once. See `addListener` for more information about this event.'
} }
const indent = exportMembersAsGlobals ? '' : ' '; const indent = exportMembersAsGlobals ? '' : ' ';
for (const method of ['on', 'once', 'addListener', 'removeListener', 'off']) { for (const method of ['on', 'once', 'addListener', 'removeListener', 'off', 'prependListener']) {
for (const { eventName, params, comment } of eventDescriptions) { for (const { eventName, params, comment } of eventDescriptions) {
if ((method === 'on' || method === 'addListener') && comment) if ((method === 'on' || method === 'addListener' || method === 'prependListener') && comment)
parts.push(this.writeComment(comment, indent)); parts.push(this.writeComment(comment, indent));
else else
parts.push(this.writeComment(commentForMethod[method], indent)); parts.push(this.writeComment(commentForMethod[method], indent));