From a086604f8f78f6b7096a0d5590a415c8a9575bf3 Mon Sep 17 00:00:00 2001
From: hectorcoronado <hectorcoronado.developer@gmail.com>
Date: Sun, 15 Jul 2018 18:14:37 -0700
Subject: [PATCH] test: remove 3rd arg from to assert.strictEqual()
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

prevents AssertionError from reporting string literal,
instead displays values of first 2 args passed to
assert.strictEqual()

PR-URL: https://github.com/nodejs/node/pull/21828
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 .../test-stream-transform-final-sync.js       | 30 ++++++++++++-------
 1 file changed, 20 insertions(+), 10 deletions(-)

diff --git a/test/parallel/test-stream-transform-final-sync.js b/test/parallel/test-stream-transform-final-sync.js
index 7dbd06d60c3625..666c1dce9996d1 100644
--- a/test/parallel/test-stream-transform-final-sync.js
+++ b/test/parallel/test-stream-transform-final-sync.js
@@ -59,42 +59,52 @@ The order things are called
 const t = new stream.Transform({
   objectMode: true,
   transform: common.mustCall(function(chunk, _, next) {
-    assert.strictEqual(++state, chunk, 'transformCallback part 1');
+    // transformCallback part 1
+    assert.strictEqual(++state, chunk);
     this.push(state);
-    assert.strictEqual(++state, chunk + 2, 'transformCallback part 2');
+    // transformCallback part 2
+    assert.strictEqual(++state, chunk + 2);
     process.nextTick(next);
   }, 3),
   final: common.mustCall(function(done) {
     state++;
-    assert.strictEqual(state, 10, 'finalCallback part 1');
+    // finalCallback part 1
+    assert.strictEqual(state, 10);
     state++;
-    assert.strictEqual(state, 11, 'finalCallback part 2');
+    // finalCallback part 2
+    assert.strictEqual(state, 11);
     done();
   }, 1),
   flush: common.mustCall(function(done) {
     state++;
-    assert.strictEqual(state, 12, 'flushCallback part 1');
+    // fluchCallback part 1
+    assert.strictEqual(state, 12);
     process.nextTick(function() {
       state++;
-      assert.strictEqual(state, 15, 'flushCallback part 2');
+      // fluchCallback part 2
+      assert.strictEqual(state, 15);
       done();
     });
   }, 1)
 });
 t.on('finish', common.mustCall(function() {
   state++;
-  assert.strictEqual(state, 13, 'finishListener');
+  // finishListener
+  assert.strictEqual(state, 13);
 }, 1));
 t.on('end', common.mustCall(function() {
   state++;
-  assert.strictEqual(state, 16, 'end event');
+  // endEvent
+  assert.strictEqual(state, 16);
 }, 1));
 t.on('data', common.mustCall(function(d) {
-  assert.strictEqual(++state, d + 1, 'dataListener');
+  // dataListener
+  assert.strictEqual(++state, d + 1);
 }, 3));
 t.write(1);
 t.write(4);
 t.end(7, common.mustCall(function() {
   state++;
-  assert.strictEqual(state, 14, 'endMethodCallback');
+  // endMethodCallback
+  assert.strictEqual(state, 14);
 }, 1));