delta/node-new.git/test/parallel/test-child-process-kill.js, branch Remove-Python-3-tests

test: replace callback with arrow function

2018-11-23T07:41:27+00:00

PR-URL: https://github.com/nodejs/node/pull/24531
Reviewed-By: Gireesh Punathil 
Reviewed-By: Colin Ihrig

test: add common.noop, default for common.mustCall()

2017-03-26T19:47:15+00:00

Export a new common.noop no-operation function for general use.
Allow using common.mustCall() without a fn argument to simplify
test cases.

Replace various non-op functions throughout tests with common.noop

PR-URL: https://github.com/nodejs/node/pull/12027
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: Richard Lau 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Teddy Katz 
Reviewed-By: Colin Ihrig 
Reviewed-By: Franziska Hinkelmann

meta: restore original copyright header

2017-03-10T19:23:48+00:00

A prior io.js era commit inappropriately removed the
original copyright statements from the source. This
restores those in any files still remaining from that
edit.

Ref: https://github.com/nodejs/TSC/issues/174
Ref: https://github.com/nodejs/node/pull/10599
PR-URL: https://github.com/nodejs/node/pull/10155

Note: This PR was required, reviewed-by and approved
by the Node.js Foundation Legal Committee and the TSC.
There is no `Approved-By:` meta data.

test: add common.mustNotCall()

2017-02-06T19:07:55+00:00

This commit adds a mustNotCall() helper for testing. This provides
an alternative to using common.fail() as a callback, or creating
a callback function for the sole purpose of calling common.fail().

PR-URL: https://github.com/nodejs/node/pull/11152
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Michael Dawson 
Reviewed-By: Santiago Gimeno 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Yuta Hiroto

test: refactor test-child-process-kill

2016-12-21T23:11:29+00:00

* var -> const
* assert.equal() -> assert.strictEqual()

PR-URL: https://github.com/nodejs/node/pull/9903
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Franziska Hinkelmann

test: use mustCall() for simple flow tracking

2016-07-18T21:14:16+00:00

Many of the tests use variables to track when callback functions
are invoked or events are emitted. These variables are then
asserted on process exit. This commit replaces this pattern in
straightforward cases with common.mustCall(). This makes the
tests easier to reason about, leads to a net reduction in lines
of code, and uncovered a few bugs in tests. This commit also
replaces some callbacks that should never be called with
common.fail().

PR-URL: https://github.com/nodejs/node/pull/7753
Reviewed-By: Wyatt Preul 
Reviewed-By: Minwoo Jung 
Reviewed-By: Ben Noordhuis

test: use common.isWindows consistently

2015-07-30T18:59:36+00:00

In the tests, we use "process.platform === 'win32'" in some places.
This patch replaces them with the "common.isWindows" for consistency.

PR-URL: https://github.com/nodejs/io.js/pull/2269
Reviewed-By: Colin Ihrig 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Jeremiah Senkpiel

test: enable linting for tests

2015-05-19T19:21:27+00:00

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

Remove excessive copyright/license boilerplate

2015-01-12T23:30:28+00:00

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.

test: split test in parallel/sequential

2014-12-17T13:45:02+00:00

Reviewed-By: Ben Noordhuis 
PR-URL: https://github.com/iojs/io.js/pull/172
Fix: iojs/io.js#139