From ad27555ff8bce5bdca3ca970f9469993c8f33f02 Mon Sep 17 00:00:00 2001
From: Nigel Kibodeaux <nigel@team.about.me>
Date: Thu, 1 Dec 2016 10:18:15 -0600
Subject: [PATCH] test: use assert.strictEqual in test-cli-eval

PR-URL: https://github.com/nodejs/node/pull/9919
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 test/parallel/test-cli-eval.js | 35 +++++++++++++++++-----------------
 1 file changed, 18 insertions(+), 17 deletions(-)

diff --git a/test/parallel/test-cli-eval.js b/test/parallel/test-cli-eval.js
index e6a47a3175172e..0dc3cea6a243fd 100644
--- a/test/parallel/test-cli-eval.js
+++ b/test/parallel/test-cli-eval.js
@@ -19,15 +19,15 @@ var filename = __filename.replace(/\\/g, '/');
 // assert that nothing is written to stdout
 child.exec(nodejs + ' --eval 42',
     function(err, stdout, stderr) {
-      assert.equal(stdout, '');
-      assert.equal(stderr, '');
+      assert.strictEqual(stdout, '');
+      assert.strictEqual(stderr, '');
     });
 
 // assert that "42\n" is written to stderr
 child.exec(nodejs + ' --eval "console.error(42)"',
     function(err, stdout, stderr) {
-      assert.equal(stdout, '');
-      assert.equal(stderr, '42\n');
+      assert.strictEqual(stdout, '');
+      assert.strictEqual(stderr, '42\n');
     });
 
 // assert that the expected output is written to stdout
@@ -36,27 +36,27 @@ child.exec(nodejs + ' --eval "console.error(42)"',
 
   child.exec(cmd + '42',
       function(err, stdout, stderr) {
-        assert.equal(stdout, '42\n');
-        assert.equal(stderr, '');
+        assert.strictEqual(stdout, '42\n');
+        assert.strictEqual(stderr, '');
       });
 
   child.exec(cmd + "'[]'", common.mustCall(
       function(err, stdout, stderr) {
-        assert.equal(stdout, '[]\n');
-        assert.equal(stderr, '');
+        assert.strictEqual(stdout, '[]\n');
+        assert.strictEqual(stderr, '');
       }));
 });
 
 // assert that module loading works
 child.exec(nodejs + ' --eval "require(\'' + filename + '\')"',
     function(status, stdout, stderr) {
-      assert.equal(status.code, 42);
+      assert.strictEqual(status.code, 42);
     });
 
 // module path resolve bug, regression test
 child.exec(nodejs + ' --eval "require(\'./test/parallel/test-cli-eval.js\')"',
     function(status, stdout, stderr) {
-      assert.equal(status.code, 42);
+      assert.strictEqual(status.code, 42);
     });
 
 // Missing argument should not crash
@@ -67,26 +67,27 @@ child.exec(nodejs + ' -e', common.mustCall(function(status, stdout, stderr) {
 
 // empty program should do nothing
 child.exec(nodejs + ' -e ""', function(status, stdout, stderr) {
-  assert.equal(stdout, '');
-  assert.equal(stderr, '');
+  assert.strictEqual(stdout, '');
+  assert.strictEqual(stderr, '');
 });
 
 // "\\-42" should be interpreted as an escaped expression, not a switch
 child.exec(nodejs + ' -p "\\-42"',
     function(err, stdout, stderr) {
-      assert.equal(stdout, '-42\n');
-      assert.equal(stderr, '');
+      assert.strictEqual(stdout, '-42\n');
+      assert.strictEqual(stderr, '');
     });
 
 child.exec(nodejs + ' --use-strict -p process.execArgv',
     function(status, stdout, stderr) {
-      assert.equal(stdout, "[ '--use-strict', '-p', 'process.execArgv' ]\n");
+      assert.strictEqual(stdout,
+                         "[ '--use-strict', '-p', 'process.execArgv' ]\n");
     });
 
 // Regression test for https://github.com/nodejs/node/issues/3574
 const emptyFile = path.join(common.fixturesDir, 'empty.js');
 child.exec(nodejs + ` -e 'require("child_process").fork("${emptyFile}")'`,
     function(status, stdout, stderr) {
-      assert.equal(stdout, '');
-      assert.equal(stderr, '');
+      assert.strictEqual(stdout, '');
+      assert.strictEqual(stderr, '');
     });