delta/node-new.git/test/internet/test-http-https-default-ports.js, branch stabilize-modules

test: refactor http-https-default-ports

2018-03-09T15:16:31+00:00

Use arrow functions.

PR-URL: https://github.com/nodejs/node/pull/19130
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Tobias Nießen 
Reviewed-By: James M Snell 
Reviewed-By: Jon Moss 
Reviewed-By: Ruben Bridgewater

test: use internet.addresses in internet tests

2017-11-08T10:54:20+00:00

PR-URL: https://github.com/nodejs/node/pull/16390
Reviewed-By: Anna Henningsen 
Reviewed-By: Refael Ackermann 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell

test: simplify test skipping

2017-07-04T09:41:49+00:00

* Make common.skip() exit.

  Also add common.printSkipMessage() for partial skips.

* Don't make needless things before skip

PR-URL: https://github.com/nodejs/node/pull/14021
Fixes: https://github.com/nodejs/node/issues/14016
Reviewed-By: Refael Ackermann

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: use const for all require() calls

2017-01-02T23:28:18+00:00

PR-URL: https://github.com/nodejs/node/pull/10550
Reviewed-By: Rich Trott

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: abstract skip functionality to common

2016-05-12T20:43:35+00:00

The tap skipping output is so prevalent yet obscure in nature that we
ought to move it into it's own function in test/common.js

PR-URL: https://github.com/nodejs/node/pull/6697
Reviewed-By: Rich Trott 
Reviewed-By: Santiago Gimeno 
Reviewed-By: Fedor Indutny

test: changing process.exit to return while skipping tests

2015-07-20T10:20:42+00:00

This patch uses `return` statement to skip the test instead of using
`process.exit` call.

PR-URL: https://github.com/nodejs/io.js/pull/2109
Reviewed-By: Ben Noordhuis 
Reviewed-By: Johan Bergström

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

test: refactor all tests that depends on crypto

2015-03-05T01:31:41+00:00

we had a few ways versions of looking for support before executing a test. this
commit unifies them as well as add the check for all tests that previously
lacked them. found by running `./configure --without-ssl && make test`. also,
produce tap skip output if the test is skipped.

PR-URL: https://github.com/iojs/io.js/pull/1049
Reviewed-By: Ben Noordhuis 
Reviewed-By: Shigeki Ohtsu