Skip to content

fix incorrect expectation order #23466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
sunburntkamel wants to merge 3 commits into from
Closed

fix incorrect expectation order #23466

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8285bdb
Tests: fix incorrect expectation order
Oct 12, 2018
5423022
tests: fix incorrect expectation order
Oct 12, 2018
710dca7
Merge branch 'master' of https://github.com/sunburntkamel/node
Oct 12, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Tests: fix incorrect expectation order 
test-http-expect-handling.js has an instance of the test value and the expected value being reversed

Refs: https://nodejs.org/api/assert.html
  • Loading branch information
Amie committed Oct 12, 2018
commit 8285bdbdc3a3143200ace0dc26f79fda4f5df2c0
2 changes: 1 addition & 1 deletion test/parallel/test-http-expect-handling.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,5 +51,5 @@ function nextTest() {


process.on('exit', function() {
assert.strictEqual(2, testsComplete);
assert.strictEqual(testsComplete, 2);
});