playwright/tests/playwright-test/exit-code.spec.ts
Max Schmitt 4431a8366d
fix: collect stdio from forked processes (#27074)
A summary about the attempts we tried in
https://github.com/microsoft/playwright/pull/26931 for
https://github.com/microsoft/playwright/issues/26859 and
https://github.com/microsoft/playwright/issues/24591.

1. attempt: collect all the stdio from the worker host.
- this made it hard to capture the live console logs for the live trace
2. attempt: collect all the stdio from the worker host, but keep
proxying `process.{stdout,stderr}.write`.
- this solved the live console logs problem, since we could still listen
on them
- we found out about a race that the stdout/stderr streams can still
buffer data while the test already had finished
- we created a best effort flush implementation but didn't like the
complexity it introduced to the project
3. attempt: keep things like they are but also listen to stdio from the
worker host
- this solves the race of console.log in the test (things stay like they
are)
    - this keeps live trace working (things stay like they are)
- a known pitfall is that it could result in logs which don't end up in
the reporter onStdOut/onStdErr, but it is already a significant
improvement over how it was before.
      - before: they were not getting displayed
- after: they are getting displayed and there is a low likelihood that
they don't end up in the reporter API if the write happens slightly
before a test finished.

Closes #26931
Fixes #26859
Fixes #24591
2023-09-14 18:34:26 +02:00

197 lines
6.6 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 } from './playwright-test-fixtures';
function monotonicTime(): number {
const [seconds, nanoseconds] = process.hrtime();
return seconds * 1000 + (nanoseconds / 1000000 | 0);
}
test('should collect stdio', async ({ runInlineTest }) => {
const { exitCode, report } = await runInlineTest({
'stdio.spec.js': `
import { test, expect } from '@playwright/test';
test('stdio', () => {
process.stdout.write('stdout text');
process.stdout.write(Buffer.from('stdout buffer'));
process.stderr.write('stderr text');
process.stderr.write(Buffer.from('stderr buffer'));
});
`
});
expect(exitCode).toBe(0);
const testResult = report.suites[0].specs[0].tests[0].results[0];
const { stdout, stderr } = testResult;
expect(stdout).toEqual([{ text: 'stdout text' }, { buffer: Buffer.from('stdout buffer').toString('base64') }]);
expect(stderr).toEqual([{ text: 'stderr text' }, { buffer: Buffer.from('stderr buffer').toString('base64') }]);
});
test('should collect stdio from forked process', async ({ runInlineTest }) => {
const { exitCode, report } = await runInlineTest({
'stdio.spec.js': `
import { test } from '@playwright/test';
import { fork } from 'child_process';
test('stdio', async () => {
const child = fork('fork.js');
await new Promise((resolve) => child.on('exit', (code) => resolve(code)));
});
`,
'fork.js': `
process.stdout.write('stdout text');
process.stdout.write(Buffer.from('stdout buffer'));
process.stderr.write('stderr text');
process.stderr.write(Buffer.from('stderr buffer'));
`
});
expect(exitCode).toBe(0);
const testResult = report.suites[0].specs[0].tests[0].results[0];
const { stdout, stderr } = testResult;
expect(stdout.map(e => Buffer.from((e as any).buffer, 'base64').toString()).join('')).toEqual('stdout textstdout buffer');
expect(stderr.map(e => Buffer.from((e as any).buffer, 'base64').toString()).join('')).toEqual('stderr textstderr buffer');
});
test('should work with not defined errors', async ({ runInlineTest }) => {
const result = await runInlineTest({
'is-not-defined-error.spec.ts': `
foo();
`
});
expect(result.output).toContain('foo is not defined');
expect(result.exitCode).toBe(1);
});
test('should work with typescript', async ({ runInlineTest }) => {
const result = await runInlineTest({
'global-foo.js': `
global.foo = true;
module.exports = {
abc: 123
};
`,
'typescript.spec.ts': `
import './global-foo';
import { test, expect } from '@playwright/test';
test('should find global foo', () => {
expect(global['foo']).toBe(true);
});
test('should work with type annotations', () => {
const x: number = 5;
expect(x).toBe(5);
});
`
});
expect(result.exitCode).toBe(0);
});
test('should repeat each', async ({ runInlineTest }) => {
const { exitCode, report, passed } = await runInlineTest({
'one-success.spec.js': `
import { test, expect } from '@playwright/test';
test('succeeds', () => {
expect(1 + 1).toBe(2);
});
`
}, { 'repeat-each': 3 });
expect(exitCode).toBe(0);
expect(passed).toBe(3);
expect(report.suites.length).toBe(1);
expect(report.suites[0].specs.length).toBe(1);
expect(report.suites[0].specs[0].tests.length).toBe(3);
});
test('should allow flaky', async ({ runInlineTest }) => {
const result = await runInlineTest({
'a.test.js': `
import { test, expect } from '@playwright/test';
test('flake', async ({}, testInfo) => {
expect(testInfo.retry).toBe(1);
});
`,
}, { retries: 1 });
expect(result.exitCode).toBe(0);
expect(result.flaky).toBe(1);
});
test('should fail on unexpected pass', async ({ runInlineTest }) => {
const { exitCode, failed, output } = await runInlineTest({
'unexpected-pass.spec.js': `
import { test, expect } from '@playwright/test';
test('succeeds', () => {
test.fail();
expect(1 + 1).toBe(2);
});
`
});
expect(exitCode).toBe(1);
expect(failed).toBe(1);
expect(output).toContain('Expected to fail, but passed');
});
test('should respect global timeout', async ({ runInlineTest }) => {
const now = monotonicTime();
const { exitCode, output } = await runInlineTest({
'one-timeout.spec.js': `
import { test, expect } from '@playwright/test';
test('timeout', async () => {
await new Promise(f => setTimeout(f, 10000));
});
`
}, { 'timeout': 100000, 'global-timeout': 3000 });
expect(exitCode).toBe(1);
expect(output).toContain('Timed out waiting 3s for the test suite to run');
expect(monotonicTime() - now).toBeGreaterThan(2900);
});
test('should exit with code 1 if the specified folder does not exist', async ({ runInlineTest }) => {
const result = await runInlineTest({
'playwright.config.ts': `
module.exports = { testDir: '111111111111.js' };
`,
});
expect(result.exitCode).toBe(1);
expect(result.output).toContain(`No tests found`);
});
test('should exit with code 1 if passed a file name', async ({ runInlineTest }) => {
const result = await runInlineTest({
'playwright.config.ts': `
module.exports = { testDir: 'test.spec.js' };
`,
'test.spec.js': `
`,
});
expect(result.exitCode).toBe(1);
expect(result.output).toContain(`No tests found`);
});
test('should exit with code 0 with --pass-with-no-tests', async ({ runInlineTest }) => {
const result = await runInlineTest({
'playwright.config.ts': `
module.exports = { testDir: 'unknown' };
`,
}, undefined, undefined, { additionalArgs: ['--pass-with-no-tests'] });
expect(result.exitCode).toBe(0);
});
test('should exit with code 1 when config is not found', async ({ runInlineTest }) => {
const result = await runInlineTest({ 'my.config.js': '' }, { 'config': 'foo.config.js' });
expect(result.exitCode).toBe(1);
expect(result.output).toContain(`foo.config.js does not exist`);
});