Commit 9cef72d
test: remove the third string literal argument from assert.strictEqual()
Third string literal argument in assert.strictEqual() needs to be
removed. Otherwise, on AssertionError it would not display the
values that failed the check -- this hinders debugging.
The string literals are added as comments above the check.
PR-URL: #21406
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>1 parent 2d6b337 commit 9cef72dCopy full SHA for 9cef72d
File tree
Expand file treeCollapse file tree
1 file changed
+4
-2
lines changedOpen diff view settings
Filter options
- test/parallel
Expand file treeCollapse file tree
1 file changed
+4
-2
lines changedOpen diff view settings
Collapse file
test/parallel/test-string-decoder-end.js
Copy file name to clipboardExpand all lines: test/parallel/test-string-decoder-end.js+4-2Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
103 | 103 | |
104 | 104 | |
105 | 105 | |
106 | | - |
107 | | - |
| 106 | + |
| 107 | + |
| 108 | + |
| 109 | + |
108 | 110 | |
109 | 111 | |
110 | 112 | |
|
0 commit comments