Skip to content

Commit

Permalink
test: fix arguments order in assert.strictEqual
Browse files Browse the repository at this point in the history
In the test test/parallel/test-http-client-upload.js test the
actual and expected arguments in assert.strictEqual() calls
were in the wrong order. Switched them around so the returned
value by the function is the first argument and the literal
value is the second argument.

PR-URL: #24143
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
  • Loading branch information
szabolcsit authored and BridgeAR committed Nov 13, 2018
1 parent 5510bec commit a1f5179
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-http-client-upload.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ const assert = require('assert');
const http = require('http');

const server = http.createServer(common.mustCall(function(req, res) {
assert.strictEqual('POST', req.method);
assert.strictEqual(req.method, 'POST');
req.setEncoding('utf8');

let sent_body = '';
Expand All @@ -36,7 +36,7 @@ const server = http.createServer(common.mustCall(function(req, res) {
});

req.on('end', common.mustCall(function() {
assert.strictEqual('1\n2\n3\n', sent_body);
assert.strictEqual(sent_body, '1\n2\n3\n');
console.log('request complete from server');
res.writeHead(200, { 'Content-Type': 'text/plain' });
res.write('hello\n');
Expand Down

0 comments on commit a1f5179

Please sign in to comment.