Skip to content

test: favor === over in == in http test #8471

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
jun-oka wants to merge 4 commits into from
Closed

test: favor === over in == in http test #8471

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fa4cc28
test: favor ’===’ over in ’==’ in http test
jun-oka Sep 9, 2016
9f3bf42
test: favor ’assert.strictEqual’ over in ’assert.equal’ in http test
jun-oka Sep 9, 2016
d52d5e1
test: favor ’const’ over in ’var’ in http test
jun-oka Sep 9, 2016
b05c4e8
test: favor `const` over in `var` for require in http test
jun-oka Sep 12, 2016
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
test: favor ’===’ over in ’==’ in http test
  • Loading branch information
@jun-oka
jun-oka committed Sep 12, 2016
commit fa4cc2807efb4cee08d6fc381d0904fe0ed814f9
4 changes: 2 additions & 2 deletions test/parallel/test-http.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ var body0 = '';
var body1 = '';

var server = http.Server(function(req, res) {
Copy link
Member

@Trott Trott Sep 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This var and some (but not all) of the others can be changed to const.

if (responses_sent == 0) {
if (responses_sent === 0) {
assert.equal('GET', req.method);
Copy link
Member

@Trott Trott Sep 9, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you update assert.equal() here and elsewhere to use assert.strictEqual() instead? Might as well get all our minor refactoring in at once. :-D

assert.equal('/hello', url.parse(req.url).pathname);

Expand All @@ -22,7 +22,7 @@ var server = http.Server(function(req, res) {
assert.equal('bar', req.headers['foo']);
}

if (responses_sent == 1) {
if (responses_sent === 1) {
assert.equal('POST', req.method);
assert.equal('/world', url.parse(req.url).pathname);
this.close();
Expand Down