mirror of
				https://github.com/microsoft/playwright.git
				synced 2025-06-26 21:40:17 +00:00 
			
		
		
		
	fix(test runner): more firendly test duration Reported test duration now does not include time spent in `beforeAll`, `afterAll` and fixtures that have a separate timeout. This is to avoid different reported test execution time, depending on the test execution order.
		
			
				
	
	
		
			381 lines
		
	
	
		
			12 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
			
		
		
	
	
			381 lines
		
	
	
		
			12 KiB
		
	
	
	
		
			TypeScript
		
	
	
	
	
	
/**
 | 
						|
 * 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 { test, expect, stripAnsi } from './playwright-test-fixtures';
 | 
						|
 | 
						|
test('should run fixture teardown on timeout', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'helper.ts': `
 | 
						|
      export const test = pwt.test.extend({
 | 
						|
        foo: async ({}, run, testInfo) => {
 | 
						|
          await run();
 | 
						|
          console.log('STATUS:' + testInfo.status);
 | 
						|
        }
 | 
						|
      });
 | 
						|
    `,
 | 
						|
    'c.spec.ts': `
 | 
						|
      import { test } from './helper';
 | 
						|
      test('works', async ({ foo }) => {
 | 
						|
        await new Promise(f => setTimeout(f, 100000));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 1000 });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.output).toContain('STATUS:timedOut');
 | 
						|
});
 | 
						|
 | 
						|
test('should respect test.setTimeout', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const { test } = pwt;
 | 
						|
      test('fails', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 1500));
 | 
						|
      });
 | 
						|
      test('passes', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 500));
 | 
						|
        test.setTimeout(2000);
 | 
						|
        await new Promise(f => setTimeout(f, 1000));
 | 
						|
      });
 | 
						|
 | 
						|
      test.describe('suite', () => {
 | 
						|
        test.beforeEach(() => {
 | 
						|
          test.setTimeout(2000);
 | 
						|
        });
 | 
						|
        test('passes2', async ({}, testInfo) => {
 | 
						|
          expect(testInfo.timeout).toBe(2000);
 | 
						|
          await new Promise(f => setTimeout(f, 1500));
 | 
						|
        });
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 1000 });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.passed).toBe(2);
 | 
						|
  expect(result.output).toContain('Test timeout of 1000ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('should respect test.setTimeout outside of the test', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const { test } = pwt;
 | 
						|
 | 
						|
      test.setTimeout(500);
 | 
						|
      test('fails', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 1000));
 | 
						|
      });
 | 
						|
      test('passes', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 100));
 | 
						|
      });
 | 
						|
 | 
						|
      test.describe('suite', () => {
 | 
						|
        test.setTimeout(50);
 | 
						|
        test('fails', async ({}) => {
 | 
						|
          await new Promise(f => setTimeout(f, 100));
 | 
						|
        });
 | 
						|
        test('passes', async ({}) => {
 | 
						|
        });
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(2);
 | 
						|
  expect(result.passed).toBe(2);
 | 
						|
  expect(result.output).toContain('Test timeout of 500ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('should timeout when calling test.setTimeout too late', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const { test } = pwt;
 | 
						|
      test('fails', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 500));
 | 
						|
        test.setTimeout(100);
 | 
						|
        await new Promise(f => setTimeout(f, 1));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 1000 });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.passed).toBe(0);
 | 
						|
  expect(result.output).toContain('Test timeout of 100ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('should respect test.slow', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const { test } = pwt;
 | 
						|
      test('fails', async ({}) => {
 | 
						|
        await new Promise(f => setTimeout(f, 1500));
 | 
						|
      });
 | 
						|
      test('passes', async ({}) => {
 | 
						|
        test.slow();
 | 
						|
        await new Promise(f => setTimeout(f, 1500));
 | 
						|
      });
 | 
						|
 | 
						|
      test.describe('suite', () => {
 | 
						|
        test.slow();
 | 
						|
        test('passes2', async ({}, testInfo) => {
 | 
						|
          expect(testInfo.timeout).toBe(3000);
 | 
						|
          await new Promise(f => setTimeout(f, 1500));
 | 
						|
        });
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 1000 });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.passed).toBe(2);
 | 
						|
  expect(result.output).toContain('Test timeout of 1000ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('should ignore test.setTimeout when debugging', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: async ({}, use) => {
 | 
						|
          test.setTimeout(100);
 | 
						|
          await new Promise(f => setTimeout(f, 200));
 | 
						|
          await use('hey');
 | 
						|
        },
 | 
						|
      });
 | 
						|
      test('my test', async ({ fixture }) => {
 | 
						|
        test.setTimeout(1000);
 | 
						|
        await new Promise(f => setTimeout(f, 2000));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 0 });
 | 
						|
  expect(result.exitCode).toBe(0);
 | 
						|
  expect(result.passed).toBe(1);
 | 
						|
});
 | 
						|
 | 
						|
test('should respect fixture timeout', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
        }, { timeout: 1000 }],
 | 
						|
        noTimeout: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
        }, { timeout: 0 }],
 | 
						|
        slowSetup: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 2000));
 | 
						|
          await use('hey');
 | 
						|
        }, { timeout: 500, _title: 'custom title' }],
 | 
						|
        slowTeardown: [async ({}, use) => {
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 2000));
 | 
						|
        }, { timeout: 400 }],
 | 
						|
      });
 | 
						|
      test('test ok', async ({ fixture, noTimeout }) => {
 | 
						|
        await new Promise(f => setTimeout(f, 1000));
 | 
						|
      });
 | 
						|
      test('test setup', async ({ slowSetup }) => {
 | 
						|
      });
 | 
						|
      test('test teardown', async ({ slowTeardown }) => {
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.passed).toBe(1);
 | 
						|
  expect(result.failed).toBe(2);
 | 
						|
  expect(result.output).toContain('Fixture "custom title" timeout of 500ms exceeded.');
 | 
						|
  expect(result.output).toContain('Fixture "slowTeardown" timeout of 400ms exceeded.');
 | 
						|
  expect(stripAnsi(result.output)).toContain('> 5 |       const test = pwt.test.extend({');
 | 
						|
});
 | 
						|
 | 
						|
test('should respect test.setTimeout in the worker fixture', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
        }, { scope: 'worker', timeout: 1000 }],
 | 
						|
        noTimeout: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
        }, { scope: 'worker', timeout: 0 }],
 | 
						|
        slowSetup: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 2000));
 | 
						|
          await use('hey');
 | 
						|
        }, { scope: 'worker', timeout: 500 }],
 | 
						|
        slowTeardown: [async ({}, use) => {
 | 
						|
          await use('hey');
 | 
						|
          await new Promise(f => setTimeout(f, 2000));
 | 
						|
        }, { scope: 'worker', timeout: 400, _title: 'custom title' }],
 | 
						|
      });
 | 
						|
      test('test ok', async ({ fixture, noTimeout }) => {
 | 
						|
        await new Promise(f => setTimeout(f, 1000));
 | 
						|
      });
 | 
						|
      test('test setup', async ({ slowSetup }) => {
 | 
						|
      });
 | 
						|
      test('test teardown', async ({ slowTeardown }) => {
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.passed).toBe(2);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.output).toContain('Fixture "slowSetup" timeout of 500ms exceeded.');
 | 
						|
  expect(result.output).toContain('Fixture "custom title" timeout of 400ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('fixture time in beforeAll hook should not affect test', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 500));
 | 
						|
          await use('hey');
 | 
						|
        },
 | 
						|
      });
 | 
						|
      test.beforeAll(async ({ fixture }) => {
 | 
						|
        // Nothing to see here.
 | 
						|
      });
 | 
						|
      test('test ok', async ({}) => {
 | 
						|
        test.setTimeout(1000);
 | 
						|
        await new Promise(f => setTimeout(f, 800));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(0);
 | 
						|
  expect(result.passed).toBe(1);
 | 
						|
});
 | 
						|
 | 
						|
test('fixture timeout in beforeAll hook should not affect test', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 500));
 | 
						|
          await use('hey');
 | 
						|
        }, { timeout: 800 }],
 | 
						|
      });
 | 
						|
      test.beforeAll(async ({ fixture }) => {
 | 
						|
        // Nothing to see here.
 | 
						|
      });
 | 
						|
      test('test ok', async ({}) => {
 | 
						|
        test.setTimeout(1000);
 | 
						|
        await new Promise(f => setTimeout(f, 800));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(0);
 | 
						|
  expect(result.passed).toBe(1);
 | 
						|
});
 | 
						|
 | 
						|
test('fixture time in beforeEach hook should affect test', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        fixture: async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 500));
 | 
						|
          await use('hey');
 | 
						|
        },
 | 
						|
      });
 | 
						|
      test.beforeEach(async ({ fixture }) => {
 | 
						|
        // Nothing to see here.
 | 
						|
      });
 | 
						|
      test('test ok', async ({}) => {
 | 
						|
        test.setTimeout(1000);
 | 
						|
        await new Promise(f => setTimeout(f, 800));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.output).toContain('Test timeout of 1000ms exceeded.');
 | 
						|
});
 | 
						|
 | 
						|
test('test timeout should still run hooks before fixtures teardown', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'a.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        auto: [async ({}, use) => {
 | 
						|
          console.log('\\n%%before-auto');
 | 
						|
          await use('hey');
 | 
						|
          console.log('\\n%%after-auto');
 | 
						|
        }, { auto: true }]
 | 
						|
      });
 | 
						|
      test.afterAll(async () => {
 | 
						|
        console.log('\\n%%afterAll-1');
 | 
						|
        await new Promise(f => setTimeout(f, 500));
 | 
						|
        console.log('\\n%%afterAll-2');
 | 
						|
      });
 | 
						|
      test('test fail', async ({}) => {
 | 
						|
        test.setTimeout(100);
 | 
						|
        console.log('\\n%%test');
 | 
						|
        await new Promise(f => setTimeout(f, 800));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  });
 | 
						|
  expect(result.exitCode).toBe(1);
 | 
						|
  expect(result.failed).toBe(1);
 | 
						|
  expect(result.output).toContain('Test timeout of 100ms exceeded.');
 | 
						|
  expect(result.output.split('\n').filter(line => line.startsWith('%%'))).toEqual([
 | 
						|
    '%%before-auto',
 | 
						|
    '%%test',
 | 
						|
    '%%afterAll-1',
 | 
						|
    '%%afterAll-2',
 | 
						|
    '%%after-auto',
 | 
						|
  ]);
 | 
						|
});
 | 
						|
 | 
						|
test('should not include fixtures with own timeout and beforeAll in test duration', async ({ runInlineTest }) => {
 | 
						|
  const result = await runInlineTest({
 | 
						|
    'c.spec.ts': `
 | 
						|
      const test = pwt.test.extend({
 | 
						|
        foo: [async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 1000));
 | 
						|
          await use('foo');
 | 
						|
        }, { timeout: 0 }],
 | 
						|
 | 
						|
        bar: async ({}, use) => {
 | 
						|
          await new Promise(f => setTimeout(f, 300));
 | 
						|
          await use('bar');
 | 
						|
        },
 | 
						|
      });
 | 
						|
 | 
						|
      test.beforeAll(async () => {
 | 
						|
        await new Promise(f => setTimeout(f, 1000));
 | 
						|
      });
 | 
						|
 | 
						|
      test.beforeEach(async () => {
 | 
						|
        await new Promise(f => setTimeout(f, 300));
 | 
						|
      });
 | 
						|
 | 
						|
      test.afterEach(async () => {
 | 
						|
        await new Promise(f => setTimeout(f, 300));
 | 
						|
      });
 | 
						|
 | 
						|
      test('works', async ({ foo, bar }) => {
 | 
						|
        await new Promise(f => setTimeout(f, 300));
 | 
						|
      });
 | 
						|
    `
 | 
						|
  }, { timeout: 5000 });
 | 
						|
  expect(result.exitCode).toBe(0);
 | 
						|
  const duration = result.results[0].duration;
 | 
						|
  expect(duration).toBeGreaterThanOrEqual(300 * 4);  // Includes test, beforeEach, afterEach and bar.
 | 
						|
  expect(duration).toBeLessThan(300 * 4 + 1000);  // Does not include beforeAll and foo.
 | 
						|
});
 |