From c30658c11e2c80c97cb716151db55f8e58c129bc Mon Sep 17 00:00:00 2001 From: blakehall Date: Fri, 12 Oct 2018 11:22:53 -0700 Subject: [PATCH] test: fix assert.strictEqual() parameter order in test-path-maklong.js The argument order in the strictEqual check was in the wrong order. The first argument is now the actual value and the second argument is the expected value. PR-URL: https://github.com/nodejs/node/pull/23587 Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil Reviewed-By: Trivikram Kamat Reviewed-By: Sakthipriyan Vairamani --- test/parallel/test-path-makelong.js | 23 ++++++++++++----------- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/test/parallel/test-path-makelong.js b/test/parallel/test-path-makelong.js index f3b145729bd691..01c81b47502f4e 100644 --- a/test/parallel/test-path-makelong.js +++ b/test/parallel/test-path-makelong.js @@ -29,17 +29,18 @@ if (common.isWindows) { const file = fixtures.path('a.js'); const resolvedFile = path.resolve(file); - assert.strictEqual(`\\\\?\\${resolvedFile}`, - path.toNamespacedPath(file)); - assert.strictEqual(`\\\\?\\${resolvedFile}`, - path.toNamespacedPath(`\\\\?\\${file}`)); - assert.strictEqual('\\\\?\\UNC\\someserver\\someshare\\somefile', - path.toNamespacedPath( - '\\\\someserver\\someshare\\somefile')); - assert.strictEqual('\\\\?\\UNC\\someserver\\someshare\\somefile', path - .toNamespacedPath('\\\\?\\UNC\\someserver\\someshare\\somefile')); - assert.strictEqual('\\\\.\\pipe\\somepipe', - path.toNamespacedPath('\\\\.\\pipe\\somepipe')); + assert.strictEqual(path.toNamespacedPath(file), + `\\\\?\\${resolvedFile}`); + assert.strictEqual(path.toNamespacedPath(`\\\\?\\${file}`), + `\\\\?\\${resolvedFile}`); + assert.strictEqual(path.toNamespacedPath( + '\\\\someserver\\someshare\\somefile'), + '\\\\?\\UNC\\someserver\\someshare\\somefile'); + assert.strictEqual(path.toNamespacedPath( + '\\\\?\\UNC\\someserver\\someshare\\somefile'), + '\\\\?\\UNC\\someserver\\someshare\\somefile'); + assert.strictEqual(path.toNamespacedPath('\\\\.\\pipe\\somepipe'), + '\\\\.\\pipe\\somepipe'); } assert.strictEqual(path.toNamespacedPath(null), null);