From 896962fd08d6c40fe86172f9555fe6252cdc3fc4 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Fri, 8 Feb 2019 19:13:35 +0100 Subject: [PATCH] test: add `Worker` + `--prof` regression test Fixes: https://github.com/nodejs/node/issues/24016 PR-URL: https://github.com/nodejs/node/pull/26011 Reviewed-By: Benjamin Gruenbaum Reviewed-By: Ben Noordhuis Reviewed-By: Gireesh Punathil Reviewed-By: Joyee Cheung --- test/parallel/test-worker-prof.js | 41 +++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) create mode 100644 test/parallel/test-worker-prof.js diff --git a/test/parallel/test-worker-prof.js b/test/parallel/test-worker-prof.js new file mode 100644 index 00000000000000..5b0703f5f9b5ca --- /dev/null +++ b/test/parallel/test-worker-prof.js @@ -0,0 +1,41 @@ +'use strict'; +const common = require('../common'); +const tmpdir = require('../common/tmpdir'); +const fs = require('fs'); +const assert = require('assert'); +const { spawnSync } = require('child_process'); +const { Worker } = require('worker_threads'); + +if (!common.isMainThread) + common.skip('process.chdir is not available in Workers'); + +// Test that --prof also tracks Worker threads. +// Refs: https://github.com/nodejs/node/issues/24016 + +if (process.argv[2] === 'child') { + const spin = ` + const start = Date.now(); + while (Date.now() - start < 200); + `; + new Worker(spin, { eval: true }); + eval(spin); + return; +} + +tmpdir.refresh(); +process.chdir(tmpdir.path); +spawnSync(process.execPath, ['--prof', __filename, 'child']); +const logfiles = fs.readdirSync('.').filter((name) => /\.log$/.test(name)); +assert.strictEqual(logfiles.length, 2); // Parent thread + child thread. + +for (const logfile of logfiles) { + const lines = fs.readFileSync(logfile, 'utf8').split('\n'); + const ticks = lines.filter((line) => /^tick,/.test(line)).length; + + // Test that at least 20 ticks have been recorded for both parent and child + // threads. When not tracking Worker threads, only 1 or 2 ticks would + // have been recorded. + // When running locally on x64 Linux, this number is usually at least 150 + // for both threads, so 15 seems like a very safe threshold. + assert(ticks >= 15, `${ticks} >= 15`); +}