lukechilds/node - node - Gitea: Git with a cup of tea

Author	SHA1	Message	Date
Myles Borins	28e9a022df	test: wrap assert.fail when passed to callback Currently there are many instances where assert.fail is directly passed to a callback for error handling. Unfortunately this will swallow the error as it is the third argument of assert.fail that sets the message not the first. This commit adds a new function to test/common.js that simply wraps assert.fail and calls it with the provided message. Tip of the hat to @trott for pointing me in the direction of this. PR-URL: https://github.com/nodejs/node/pull/3453 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>	9 years ago
Roman Reiss	e3f9335c40	tools: re-enable comma-spacing linter rule The rule was disabled because of an eslint bug which is now resolved. All code in lib was already conforming and only test code needed a few changes to make the linter happy with this rule enabled. Ref: https://github.com/eslint/eslint/issues/2408 PR-URL: https://github.com/nodejs/io.js/pull/2072 Reviewed-By: Yosuke Furukawa <yosuke.furukawa@gmail.com> Reviewed-by: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Alex Kocharin <alex@kocharin.ru>	10 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
Andrius Bentkus	a382c9a97c	udp: make it possible to receive empty udp packets A udp packet can have 0 content. In that case nread will be equal to 0, but addr != NULL. Add test case for empty data gram packets and fixed test that checked for OOB when length == 0. Signed-off-by: Trevor Norris <trev.norris@gmail.com>	11 years ago
Trevor Norris	8130744044	dgram: send() can accept strings Strings passed to Socket#send() will be passed to Buffer and parsed as UTF8.	11 years ago
Ben Noordhuis	defa637378	dgram: fix out-of-bound memory read	13 years ago
isaacs	0cdf85e28d	Lint all the JavaScripts.	13 years ago
Nathan Rajlich	855f46677c	test: Add test case for MODULE_NOT_FOUND.	13 years ago
Ryan Dahl	55048cdf79	Update copyright headers	14 years ago
Felix Geisendörfer	66601f13d9	Do not cache modules that throw exceptions If a module throws an exception on load, it should not be cached. This patch shows the problem in a test case and also fixes it. See: https://groups.google.com/forum/#!topic/nodejs-dev/1cIrvJcADbY Closes GH-707 Closes GH-710	14 years ago