test: improve to use template string

PR-URL: #18097
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>

Author: sreepurnajasti

test/message/error_exit.js

@@ -24,7 +24,7 @@ require('../common');
 const assert = require('assert');
 
 process.on('exit', function(code) {
-  console.error('Exiting with code=%d', code);
+  console.error(`Exiting with code=${code}`);
 });
 
 assert.strictEqual(1, 2);

test/message/max_tick_depth.js

@@ -25,7 +25,7 @@ require('../common');
 process.maxTickDepth = 10;
 let i = 20;
 process.nextTick(function f() {
-  console.error('tick %d', i);
+  console.error(`tick ${i}`);
   if (i-- > 0)
     process.nextTick(f);
 });

test/parallel/test-cluster-net-send.js

@@ -26,7 +26,7 @@ const fork = require('child_process').fork;
 const net = require('net');
 
 if (process.argv[2] !== 'child') {
-  console.error('[%d] master', process.pid);
+  console.error(`[${process.pid}] master`);
 
   const worker = fork(__filename, ['child']);
   let called = false;
@@ -50,7 +50,7 @@ if (process.argv[2] !== 'child') {
     assert.ok(called);
   });
 } else {
-  console.error('[%d] worker', process.pid);
+  console.error(`[${process.pid}] worker`);
 
   let socket;
   let cbcalls = 0;

test/parallel/test-domain-http-server.js

@@ -62,7 +62,7 @@ server.listen(0, next);
 
 function next() {
   const port = this.address().port;
-  console.log('listening on localhost:%d', port);
+  console.log(`listening on localhost:${port}`);
 
   let requests = 0;
   let responses = 0;
@@ -87,7 +87,7 @@ function next() {
     dom.add(req);
     req.on('response', function(res) {
       responses++;
-      console.error('requests=%d responses=%d', requests, responses);
+      console.error(`requests=${requests} responses=${responses}`);
       if (responses === requests) {
         console.error('done, closing server');
         // no more coming.

test/parallel/test-file-write-stream2.js

@@ -41,8 +41,8 @@ process.on('exit', function() {
   removeTestFile();
   if (cb_occurred !== cb_expected) {
     console.log('  Test callback events missing or out of order:');
-    console.log('    expected: %j', cb_expected);
-    console.log('    occurred: %j', cb_occurred);
+    console.log(`    expected: ${cb_expected}`);
+    console.log(`    occurred: ${cb_occurred}`);
     assert.strictEqual(
       cb_occurred, cb_expected,
       `events missing or out of order: "${cb_occurred}" !== "${cb_expected}"`);
@@ -78,7 +78,7 @@ file.on('drain', function() {
   if (countDrains === 1) {
     console.error('drain=1, write again');
     assert.strictEqual(fs.readFileSync(filepath, 'utf8'), EXPECTED);
-    console.error('ondrain write ret=%j', file.write(EXPECTED));
+    console.error(`ondrain write ret= ${file.write(EXPECTED)}`);
     cb_occurred += 'write ';
   } else if (countDrains === 2) {
     console.error('second drain, end');
@@ -102,7 +102,7 @@ file.on('error', function(err) {
 
 for (let i = 0; i < 11; i++) {
   const ret = file.write(String(i));
-  console.error('%d %j', i, ret);
+  console.error(`${i} ${ret}`);
 
   // return false when i hits 10
   assert.strictEqual(ret, i !== 10);

test/parallel/test-file-write-stream3.js

@@ -42,8 +42,8 @@ const fileDataExpected_3 = 'abcdefghij\u2026\u2026qrstuvwxyz';
 process.on('exit', function() {
   if (cb_occurred !== cb_expected) {
     console.log('  Test callback events missing or out of order:');
-    console.log('    expected: %j', cb_expected);
-    console.log('    occurred: %j', cb_occurred);
+    console.log(`    expected: ${cb_expected}`);
+    console.log(`    occurred: ${cb_occurred}`);
     assert.strictEqual(
       cb_occurred, cb_expected,
       `events missing or out of order: "${cb_occurred}" !== "${cb_expected}"`);

test/parallel/test-fs-read-stream-fd-leak.js

@@ -24,7 +24,7 @@ fs.close = function() {
 };
 
 function testLeak(endFn, callback) {
-  console.log('testing for leaks from fs.createReadStream().%s()...', endFn);
+  console.log(`testing for leaks from fs.createReadStream().${endFn}()...`);
 
   let i = 0;
   let check = 0;

test/parallel/test-fs-write-string-coerce.js

@@ -19,8 +19,8 @@ fs.open(fn, 'w', 0o644, common.mustCall(function(err, fd) {
     assert.strictEqual(Buffer.byteLength(expected), written);
     fs.closeSync(fd);
     const found = fs.readFileSync(fn, 'utf8');
-    console.log('expected: "%s"', expected);
-    console.log('found: "%s"', found);
+    console.log(`expected: "${expected}"`);
+    console.log(`found: "${found}"`);
     fs.unlinkSync(fn);
     assert.strictEqual(expected, found);
   }));

test/parallel/test-http-client-timeout-agent.js

@@ -90,7 +90,7 @@ server.listen(0, options.host, function() {
 });
 
 process.on('exit', function() {
-  console.error('done=%j sent=%j', requests_done, requests_sent);
+  console.error(`done=${requests_done} sent=${requests_sent}`);
   assert.strictEqual(requests_done, requests_sent,
                      'timeout on http request called too much');
 });

test/parallel/test-http-outgoing-finish.js

@@ -58,19 +58,19 @@ function write(out) {
   // that 'finish' isn't emitted until the stream is fully flushed.
   out.on('finish', function() {
     finishEvent = true;
-    console.error('%s finish event', name);
+    console.error(`${name} finish event`);
     process.nextTick(function() {
       assert(endCb, `${name} got finish event before endcb!`);
-      console.log('ok - %s finishEvent', name);
+      console.log(`ok - ${name} finishEvent`);
     });
   });
 
   out.end(buf, function() {
     endCb = true;
-    console.error('%s endCb', name);
+    console.error(`${name} endCb`);
     process.nextTick(function() {
       assert(finishEvent, `${name} got endCb event before finishEvent!`);
-      console.log('ok - %s endCb', name);
+      console.log(`ok - ${name} endCb`);
     });
   });
 }