Skip to content

Commit ba5b5ac

Browse files
authored
test: improve worker_threads coverage
PR-URL: #41818 Refs: https://coverage.nodejs.org/coverage-7123a00b03a90862/lib/internal/worker.js.html#L412 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 3b338cf commit ba5b5ac

File tree

2 files changed

+21
-0
lines changed

2 files changed

+21
-0
lines changed

test/parallel/test-worker-environmentdata.js

+5
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
'use strict';
2+
// Flags: --expose-internals
23

34
require('../common');
45
const {
@@ -8,6 +9,8 @@ const {
89
threadId,
910
} = require('worker_threads');
1011

12+
const { assignEnvironmentData } = require('internal/worker');
13+
1114
const {
1215
deepStrictEqual,
1316
strictEqual,
@@ -26,6 +29,8 @@ if (!process.env.HAS_STARTED_WORKER) {
2629
strictEqual(getEnvironmentData('foo'), 'bar');
2730
deepStrictEqual(getEnvironmentData('hello'), { value: 'world' });
2831
strictEqual(getEnvironmentData(1), undefined);
32+
assignEnvironmentData(undefined); // It won't setup any key.
33+
strictEqual(getEnvironmentData(undefined), undefined);
2934

3035
// Recurse to make sure the environment data is inherited
3136
if (threadId <= 2)
+16
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
'use strict';
2+
3+
const common = require('../common');
4+
const assert = require('assert');
5+
const { Worker } = require('worker_threads');
6+
const { once } = require('events');
7+
8+
// Ensure that worker.getHeapSnapshot() returns a valid JSON
9+
(async () => {
10+
const worker = new Worker('setInterval(() => {}, 1000);', { eval: true });
11+
await once(worker, 'online');
12+
const stream = await worker.getHeapSnapshot();
13+
assert.ok(JSON.parse(stream.read()));
14+
15+
await worker.terminate();
16+
})().then(common.mustCall());

0 commit comments

Comments
 (0)