Gibson Fahnestock
6f85c81f0d
test: s/assert.equal/assert.strictEqual/
Use assert.strictEqual instead of assert.equal in tests, manually
convert types where necessary.
Backport-PR-URL: https://github.com/nodejs/node/pull/11795
PR-URL: https://github.com/nodejs/node/pull/10698
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
8 years ago
Gibson Fahnestock
db61c952de
test: use eslint to fix var->const/let
Manually fix issues that eslint --fix couldn't do automatically.
Backport-PR-URL: https://github.com/nodejs/node/pull/11775
PR-URL: https://github.com/nodejs/node/pull/10685
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
8 years ago
Rich Trott
6bdc0e54b4
test: use strict equality in regression test
Replace `==` with `===` and `assert.strictEqual()` in
test-regress-GH-877.js.
PR-URL: https://github.com/nodejs/node/pull/8098
Reviewed-By: targos - Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: cjihrig - Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
9 years ago
Roman Reiss
f29762f4dd
test: enable linting for tests
Enable linting for the test directory. A number of changes was made so
all tests conform the current rules used by lib and src directories. The
only exception for tests is that unreachable (dead) code is allowed.
test-fs-non-number-arguments-throw had to be excluded from the changes
because of a weird issue on Windows CI.
PR-URL: https://github.com/nodejs/io.js/pull/1721
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
10 years ago
isaacs
3e1b1dd4a9
Remove excessive copyright/license boilerplate
The copyright and license notice is already in the LICENSE file. There
is no justifiable reason to also require that it be included in every
file, since the individual files are not individually distributed except
as part of the entire package.
10 years ago
Fedor Indutny
0e19476595
test: split test in parallel/sequential
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
PR-URL: https://github.com/iojs/io.js/pull/172
Fix : iojs/io.js#139
10 years ago
isaacs
49519f1217
http: Reuse more http/https Agent code
12 years ago
isaacs
19ecc3a4a6
test updates for streams2
12 years ago
isaacs
0cdf85e28d
Lint all the JavaScripts.
13 years ago
Andreas Madsen
4865063924
Tests: fix jslint issues
13 years ago
Ben Noordhuis
fd29448be0
test: add missing copyright headers
13 years ago
Colton Baker
87286cc737
Fixed a lot of jslint errors.
Fixes #1831
13 years ago
Mikeal Rogers
48dcb905f6
Update tests for http2.
14 years ago
Ryan Dahl
6d8b43c366
http: Fix agent id creation
Unbreaks test-regress-GH-877.js
14 years ago
Ryan Dahl
efca5456b8
Fixes #877 . Don't wait for socket pool to establish connections.
Thanks to Yann Biancheri for putting together an initial test.
14 years ago