From 7c35f7f60806b6d3bd2645ba6725bf8af89bb09f Mon Sep 17 00:00:00 2001 From: Mel Date: Fri, 12 Oct 2018 10:04:07 -0700 Subject: [PATCH] test: switch order of assertion arguments this updates the arguments in order to give the actual return value first and the given expected value second. PR-URL: https://github.com/nodejs/node/pull/23563 Reviewed-By: James M Snell Reviewed-By: Anna Henningsen Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil --- test/parallel/test-http-client-get-url.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-http-client-get-url.js b/test/parallel/test-http-client-get-url.js index 7f6bfb8ecaa17b..a72eea56538c4d 100644 --- a/test/parallel/test-http-client-get-url.js +++ b/test/parallel/test-http-client-get-url.js @@ -28,8 +28,8 @@ const URL = url.URL; const testPath = '/foo?bar'; const server = http.createServer(common.mustCall((req, res) => { - assert.strictEqual('GET', req.method); - assert.strictEqual(testPath, req.url); + assert.strictEqual(req.method, 'GET'); + assert.strictEqual(req.url, testPath); res.writeHead(200, { 'Content-Type': 'text/plain' }); res.write('hello\n'); res.end();