Skip to content

Commit b96cfde

Browse files
ErickWendeljuanarbol
authored andcommitted
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 1d1ca20 commit b96cfde

2 files changed

Lines changed: 21 additions & 0 deletions

File tree

‎test/parallel/test-worker-environmentdata.js‎

Lines changed: 5 additions & 0 deletions
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)
Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
'use strict';
2+
3+
constcommon=require('../common');
4+
constassert=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+
constworker=newWorker('setInterval(() => {}, 1000);',{eval: true});
11+
awaitonce(worker,'online');
12+
conststream=awaitworker.getHeapSnapshot();
13+
assert.ok(JSON.parse(stream.read()));
14+
15+
awaitworker.terminate();
16+
})().then(common.mustCall());

0 commit comments

Comments
 (0)