mirror of
				https://github.com/microsoft/playwright.git
				synced 2025-06-26 21:40:17 +00:00 
			
		
		
		
	 6491e5b415
			
		
	
	
		6491e5b415
		
			
		
	
	
	
	
		
			
			The following actions keep `noWaitAfter` option: `click`, `selectOption` and `press`. All other actions that used to have `noWaitAfter` now behave like it was set to true, not waiting for follow-up navigations. In the docs, this option is marked as completely ignored. A small logic change was made to compensate for this behavior: when waiting for the `hitTargetInterceptor`, we now race it against navigations to avoid stalling when navigation stalls. Previously, waiting for the interceptor was disabled when `noWaitAfter` was passed, and since it's impossible to pass this option now, we mitigate by never stalling instead. Fixes #31469.
		
			
				
	
	
		
			142 lines
		
	
	
		
			5.5 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
			
		
		
	
	
			142 lines
		
	
	
		
			5.5 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
| /**
 | |
|  * Copyright 2018 Google Inc. All rights reserved.
 | |
|  * Modifications copyright (c) Microsoft Corporation.
 | |
|  *
 | |
|  * Licensed under the Apache License, Version 2.0 (the "License");
 | |
|  * you may not use this file except in compliance with the License.
 | |
|  * You may obtain a copy of the License at
 | |
|  *
 | |
|  *     http://www.apache.org/licenses/LICENSE-2.0
 | |
|  *
 | |
|  * Unless required by applicable law or agreed to in writing, software
 | |
|  * distributed under the License is distributed on an "AS IS" BASIS,
 | |
|  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 | |
|  * See the License for the specific language governing permissions and
 | |
|  * limitations under the License.
 | |
|  */
 | |
| 
 | |
| import type { TestServer } from '../config/testserver';
 | |
| import { test as it, expect } from './pageTest';
 | |
| 
 | |
| function initServer(server: TestServer): string[] {
 | |
|   const messages = [];
 | |
|   server.setRoute('/empty.html', async (req, res) => {
 | |
|     messages.push('route');
 | |
|     res.setHeader('Content-Type', 'text/html');
 | |
|     res.end(`<link rel='stylesheet' href='./one-style.css'>`);
 | |
|   });
 | |
|   return messages;
 | |
| }
 | |
| 
 | |
| it('should await navigation when clicking anchor', async ({ page, server }) => {
 | |
|   const messages = initServer(server);
 | |
|   await page.setContent(`<a id="anchor" href="${server.EMPTY_PAGE}">empty.html</a>`);
 | |
|   await Promise.all([
 | |
|     page.click('a').then(() => messages.push('click')),
 | |
|     page.waitForEvent('framenavigated').then(() => messages.push('navigated')),
 | |
|   ]);
 | |
|   expect(messages.join('|')).toBe('route|navigated|click');
 | |
| });
 | |
| 
 | |
| it('should not stall on JS navigation link', async ({ page, browserName }) => {
 | |
|   await page.setContent(`<a href="javascript:console.log(1)">console.log</a>`);
 | |
|   await page.click('a');
 | |
| });
 | |
| 
 | |
| it('should await cross-process navigation when clicking anchor', async ({ page, server }) => {
 | |
|   const messages = initServer(server);
 | |
|   await page.setContent(`<a href="${server.CROSS_PROCESS_PREFIX + '/empty.html'}">empty.html</a>`);
 | |
| 
 | |
|   await Promise.all([
 | |
|     page.click('a').then(() => messages.push('click')),
 | |
|     page.waitForEvent('framenavigated').then(() => messages.push('navigated')),
 | |
|   ]);
 | |
|   expect(messages.join('|')).toBe('route|navigated|click');
 | |
| });
 | |
| 
 | |
| it('should await form-get on click', async ({ page, server }) => {
 | |
|   const messages = [];
 | |
|   server.setRoute('/empty.html?foo=bar', async (req, res) => {
 | |
|     messages.push('route');
 | |
|     res.setHeader('Content-Type', 'text/html');
 | |
|     res.end(`<link rel='stylesheet' href='./one-style.css'>`);
 | |
|   });
 | |
| 
 | |
|   await page.setContent(`
 | |
|     <form action="${server.EMPTY_PAGE}" method="get">
 | |
|       <input name="foo" value="bar">
 | |
|       <input type="submit" value="Submit">
 | |
|     </form>`);
 | |
| 
 | |
|   await Promise.all([
 | |
|     page.click('input[type=submit]').then(() => messages.push('click')),
 | |
|     page.waitForEvent('framenavigated').then(() => messages.push('navigated')),
 | |
|   ]);
 | |
|   expect(messages.join('|')).toBe('route|navigated|click');
 | |
| });
 | |
| 
 | |
| it('should await form-post on click', async ({ page, server }) => {
 | |
|   const messages = initServer(server);
 | |
|   await page.setContent(`
 | |
|     <form action="${server.EMPTY_PAGE}" method="post">
 | |
|       <input name="foo" value="bar">
 | |
|       <input type="submit" value="Submit">
 | |
|     </form>`);
 | |
| 
 | |
|   await Promise.all([
 | |
|     page.click('input[type=submit]').then(() => messages.push('click')),
 | |
|     page.waitForEvent('framenavigated').then(() => messages.push('navigated')),
 | |
|   ]);
 | |
|   expect(messages.join('|')).toBe('route|navigated|click');
 | |
| });
 | |
| 
 | |
| it('should work with noWaitAfter: true', async ({ page, server }) => {
 | |
|   server.setRoute('/empty.html', async () => {});
 | |
|   await page.setContent(`<a id="anchor" href="${server.EMPTY_PAGE}">empty.html</a>`);
 | |
|   await page.click('a', { noWaitAfter: true });
 | |
| });
 | |
| 
 | |
| it('should work with dblclick without noWaitAfter when navigation is stalled', async ({ page, server }) => {
 | |
|   server.setRoute('/empty.html', async () => {});
 | |
|   await page.setContent(`<a id="anchor" href="${server.EMPTY_PAGE}">empty.html</a>`);
 | |
|   await page.dblclick('a');
 | |
| });
 | |
| 
 | |
| it('should work with waitForLoadState(load)', async ({ page, server }) => {
 | |
|   const messages = initServer(server);
 | |
|   await page.setContent(`<a id="anchor" href="${server.EMPTY_PAGE}">empty.html</a>`);
 | |
|   await Promise.all([
 | |
|     page.click('a').then(() => page.waitForLoadState('load')).then(() => messages.push('clickload')),
 | |
|     page.waitForEvent('load').then(() => messages.push('load')),
 | |
|   ]);
 | |
|   expect(messages.join('|')).toBe('route|load|clickload');
 | |
| });
 | |
| 
 | |
| it('should work with goto following click', async ({ page, server }) => {
 | |
|   server.setRoute('/login.html', async (req, res) => {
 | |
|     res.setHeader('Content-Type', 'text/html');
 | |
|     res.end(`You are logged in`);
 | |
|   });
 | |
| 
 | |
|   await page.setContent(`
 | |
|     <form action="${server.PREFIX}/login.html" method="get">
 | |
|       <input type="text">
 | |
|       <input type="submit" value="Submit">
 | |
|     </form>`);
 | |
| 
 | |
|   await page.fill('input[type=text]', 'admin');
 | |
|   await page.click('input[type=submit]');
 | |
|   await page.goto(server.EMPTY_PAGE);
 | |
| });
 | |
| 
 | |
| it('should report navigation in the log when clicking anchor', async ({ page, server, mode }) => {
 | |
|   it.skip(mode !== 'default');
 | |
| 
 | |
|   await page.setContent(`<a href="${server.PREFIX + '/frames/one-frame.html'}">click me</a>`);
 | |
|   const __testHookAfterPointerAction = () => new Promise(f => setTimeout(f, 6000));
 | |
|   const error = await page.click('a', { timeout: 5000, __testHookAfterPointerAction } as any).catch(e => e);
 | |
|   expect(error.message).toContain('page.click: Timeout 5000ms exceeded.');
 | |
|   expect(error.message).toContain('waiting for scheduled navigations to finish');
 | |
|   expect(error.message).toContain(`navigated to "${server.PREFIX + '/frames/one-frame.html'}"`);
 | |
| });
 |