diff options
author | Jared Haines <jared.haines@gmail.com> | 2018-10-12 09:17:17 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-10-13 09:09:46 -0700 |
commit | 478fe0aaff48c6dcbb586e2f82ea3a5499250985 (patch) | |
tree | d57b240fb89b2f5fa88226e9b2df0680e80fabae /test/parallel/test-http-server.js | |
parent | f0c5c962ce971213c0c055b91bb747860defd492 (diff) | |
download | android-node-v8-478fe0aaff48c6dcbb586e2f82ea3a5499250985.tar.gz android-node-v8-478fe0aaff48c6dcbb586e2f82ea3a5499250985.tar.bz2 android-node-v8-478fe0aaff48c6dcbb586e2f82ea3a5499250985.zip |
test: fix order of values in test assertions
Have actual first, expected second.
PR-URL: https://github.com/nodejs/node/pull/23450
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Diffstat (limited to 'test/parallel/test-http-server.js')
-rw-r--r-- | test/parallel/test-http-server.js | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/test/parallel/test-http-server.js b/test/parallel/test-http-server.js index 567f2b3b31..d46e25bdf6 100644 --- a/test/parallel/test-http-server.js +++ b/test/parallel/test-http-server.js @@ -37,23 +37,23 @@ const server = http.createServer(function(req, res) { req.id = request_number++; if (req.id === 0) { - assert.strictEqual('GET', req.method); - assert.strictEqual('/hello', url.parse(req.url).pathname); - assert.strictEqual('world', qs.parse(url.parse(req.url).query).hello); - assert.strictEqual('b==ar', qs.parse(url.parse(req.url).query).foo); + assert.strictEqual(req.method, 'GET'); + assert.strictEqual(url.parse(req.url).pathname, '/hello'); + assert.strictEqual(qs.parse(url.parse(req.url).query).hello, 'world'); + assert.strictEqual(qs.parse(url.parse(req.url).query).foo, 'b==ar'); } if (req.id === 1) { - assert.strictEqual('POST', req.method); - assert.strictEqual('/quit', url.parse(req.url).pathname); + assert.strictEqual(req.method, 'POST'); + assert.strictEqual(url.parse(req.url).pathname, '/quit'); } if (req.id === 2) { - assert.strictEqual('foo', req.headers['x-x']); + assert.strictEqual(req.headers['x-x'], 'foo'); } if (req.id === 3) { - assert.strictEqual('bar', req.headers['x-x']); + assert.strictEqual(req.headers['x-x'], 'bar'); this.close(); } @@ -112,8 +112,8 @@ server.on('listening', function() { }); process.on('exit', function() { - assert.strictEqual(4, request_number); - assert.strictEqual(4, requests_sent); + assert.strictEqual(request_number, 4); + assert.strictEqual(requests_sent, 4); const hello = new RegExp('/hello'); assert.ok(hello.test(server_response)); @@ -121,5 +121,5 @@ process.on('exit', function() { const quit = new RegExp('/quit'); assert.ok(quit.test(server_response)); - assert.strictEqual(true, client_got_eof); + assert.strictEqual(client_got_eof, true); }); |