From 5ad4c44e90b8195e45f7bd293cd06765f1e47953 Mon Sep 17 00:00:00 2001
From: John Mc Quillan <john.mcq1llan@gmail.com>
Date: Tue, 6 Nov 2018 16:12:21 +0000
Subject: [PATCH] test: fix assert.strictEqual argument order

The arguments to assert.strictEqual in a number of calls were in the
wrong order. It is preferred that literal values are in the second
argument.

Updates test/parallel/test-fs-readStream.js

PR-URL: https://github.com/nodejs/node/pull/24172
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
---
 test/parallel/test-fs-read-stream.js | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/test/parallel/test-fs-read-stream.js b/test/parallel/test-fs-read-stream.js
index 793b474c9b8292..2a07a09c976574 100644
--- a/test/parallel/test-fs-read-stream.js
+++ b/test/parallel/test-fs-read-stream.js
@@ -42,7 +42,7 @@ const rangeFile = fixtures.path('x.txt');
 
   file.on('open', common.mustCall(function(fd) {
     file.length = 0;
-    assert.strictEqual('number', typeof fd);
+    assert.strictEqual(typeof fd, 'number');
     assert.strictEqual(file.bytesRead, 0);
     assert.ok(file.readable);
 
@@ -91,12 +91,12 @@ const rangeFile = fixtures.path('x.txt');
   const file = fs.createReadStream(fn, { encoding: 'utf8' });
   file.length = 0;
   file.on('data', function(data) {
-    assert.strictEqual('string', typeof data);
+    assert.strictEqual(typeof data, 'string');
     file.length += data.length;
 
     for (let i = 0; i < data.length; i++) {
       // http://www.fileformat.info/info/unicode/char/2026/index.htm
-      assert.strictEqual('\u2026', data[i]);
+      assert.strictEqual(data[i], '\u2026');
     }
   });
 
@@ -162,7 +162,7 @@ common.expectsError(
   });
 
   stream.on('end', common.mustCall(function() {
-    assert.strictEqual('x', stream.data);
+    assert.strictEqual(stream.data, 'x');
   }));
 }
 
@@ -176,7 +176,7 @@ common.expectsError(
   });
 
   stream.on('end', common.mustCall(function() {
-    assert.strictEqual('xy', stream.data);
+    assert.strictEqual(stream.data, 'xy');
   }));
 }
 
@@ -197,7 +197,7 @@ if (!common.isWindows) {
     });
 
     stream.on('end', common.mustCall(function() {
-      assert.strictEqual('xy', stream.data);
+      assert.strictEqual(stream.data, 'xy');
       fs.unlinkSync(filename);
     }));
   } else {