1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: switch order of assertion arguments

Browse Source 
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 <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
This commit is contained in:
Mel 2018-10-12 10:04:07 -07:00 committed by Ruben Bridgewater
parent aee771c73c
commit 4b7893db94
No known key found for this signature in database
GPG Key ID: F07496B3EB3C1762
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/parallel/test-http-client-get-url.js
View File

@ -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();