microsoft/playwright
1
1
Fork 0
mirror of https://github.com/microsoft/playwright.git synced 2024-12-14 21:53:35 +03:00
Code Issues Projects Releases Wiki Activity

test: deliver colorful debug messages, do not pipe stdio (#3477)

Browse Source
This commit is contained in:
Pavel Feldman 2020-08-14 13:42:03 -07:00 committed by GitHub
parent ae4280a12b
commit d537088af8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 26 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
test/runner/runner.js
View File

@ -184,9 +184,11 @@ class Worker extends EventEmitter {
detached: false, detached: false,
env: { env: {
FORCE_COLOR: process.stdout.isTTY ? 1 : 0, FORCE_COLOR: process.stdout.isTTY ? 1 : 0,
DEBUG_COLORS: process.stdout.isTTY ? 1 : 0,
...process.env ...process.env
}, },
stdio: ['ignore', 'pipe', 'pipe', 'ipc'] // Can't pipe since piping slows down termination for some reason.
stdio: ['ignore', 'ignore', 'ignore', 'ipc']
}); });
this.process.on('exit', () => this.emit('exit')); this.process.on('exit', () => this.emit('exit'));
this.process.on('message', message => { this.process.on('message', message => {
@ -195,18 +197,23 @@ class Worker extends EventEmitter {
}); });
this.stdout = []; this.stdout = [];
this.stderr = []; this.stderr = [];
this.process.stdout.on('data', data => { this.on('stdout', data => {
if (runner._options.dumpio) if (runner._options.dumpio)
process.stdout.write(data); process.stdout.write(data);
else else
this.stdout.push(data.toString()); this.stdout.push(data);
}); });
this.on('stderr', data => {
this.process.stderr.on('data', data => {
if (runner._options.dumpio) if (runner._options.dumpio)
process.stderr.write(data); process.stderr.write(data);
else else
this.stderr.push(data.toString()); this.stderr.push(data);
});
this.on('debug', data => {
if (runner._options.dumpio)
process.stderr.write(data + '\n');
else
this.stderr.push(data + '\n');
}); });
} }

14
test/runner/worker.js
View File

@ -14,7 +14,7 @@
* limitations under the License. * limitations under the License.
*/ */
const path = require('path'); const debug = require('debug');
const Mocha = require('mocha'); const Mocha = require('mocha');
const { fixturesUI, fixturePool } = require('./fixturesUI'); const { fixturesUI, fixturePool } = require('./fixturesUI');
const { gracefullyCloseAll } = require('../../lib/server/processLauncher'); const { gracefullyCloseAll } = require('../../lib/server/processLauncher');
@ -29,6 +29,18 @@ extendExpects();
let closed = false; let closed = false;
process.stdout.write = chunk => {
sendMessageToParent('stdout', chunk);
};
process.stderr.write = chunk => {
sendMessageToParent('stderr', chunk);
};
debug.log = data => {
sendMessageToParent('debug', data);
};
process.on('message', async message => { process.on('message', async message => {
if (message.method === 'init') if (message.method === 'init')
process.env.JEST_WORKER_ID = message.params.workerId; process.env.JEST_WORKER_ID = message.params.workerId;