test: refactor test-worker

himself65 · targos · commit 4ffa138c8131 · 2020-04-11T10:06:06.000+02:00
PR-URL: #32509 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
diff --git a/test/parallel/test-worker.js b/test/parallel/test-worker.js
@@ -3,15 +3,17 @@ const common = require('../common');
 const assert = require('assert');
 const { Worker, isMainThread, parentPort } = require('worker_threads');
 
+const kTestString = 'Hello, world!';
+
 if (isMainThread) {
   const w = new Worker(__filename);
   w.on('message', common.mustCall((message) => {
-    assert.strictEqual(message, 'Hello, world!');
+    assert.strictEqual(message, kTestString);
   }));
 } else {
   setImmediate(() => {
     process.nextTick(() => {
-      parentPort.postMessage('Hello, world!');
+      parentPort.postMessage(kTestString);
     });
   });
 }
diff --git a/test/parallel/test-worker.mjs b/test/parallel/test-worker.mjs
@@ -2,17 +2,17 @@ import { mustCall } from '../common/index.mjs';
 import assert from 'assert';
 import { Worker, isMainThread, parentPort } from 'worker_threads';
 
-const TEST_STRING = 'Hello, world!';
+const kTestString = 'Hello, world!';
 
 if (isMainThread) {
   const w = new Worker(new URL(import.meta.url));
   w.on('message', mustCall((message) => {
-    assert.strictEqual(message, TEST_STRING);
+    assert.strictEqual(message, kTestString);
   }));
 } else {
   setImmediate(() => {
     process.nextTick(() => {
-      parentPort.postMessage(TEST_STRING);
+      parentPort.postMessage(kTestString);
     });
   });
 }
