Skip to content
Permalink
Browse files

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: #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>
  • Loading branch information
zzidante authored and jasnell committed Oct 12, 2018
1 parent ca24bcf commit af59b9dd02250d0b70f3273d10f4a7acc884b295
Showing with 2 additions and 2 deletions.
  1. +2 −2 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();

0 comments on commit af59b9d

Please sign in to comment.
You can’t perform that action at this time.