delta/node-new.git/test, branch Remove-Python-3-tests

lib: add ASCII fast path to getStringWidth()

2019-09-03T04:02:34+00:00

A lot of strings that are going to be passed to `getStringWidth()`
are ASCII strings, for which the calculation is rather easy and
calling into C++ can be skipped.

                                                       confidence improvement accuracy (*)    (**)   (***)
     misc/getstringwidth.js n=100000 type='ascii'            ***    328.99 %      ±21.73% ±29.25% ±38.77%
     misc/getstringwidth.js n=100000 type='emojiseq'                  2.94 %       ±7.66% ±10.19% ±13.26%
     misc/getstringwidth.js n=100000 type='fullwidth'                 4.70 %       ±5.64%  ±7.50%  ±9.76%

PR-URL: https://github.com/nodejs/node/pull/29301
Reviewed-By: Gus Caplan 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Minwoo Jung 
Reviewed-By: Rich Trott

stream: don't deadlock on aborted stream

2019-09-02T14:31:33+00:00

Not all streams (e.g. http.ClientRequest) will always emit
'close' after 'aborted'.

PR-URL: https://github.com/nodejs/node/pull/29376
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca

console: use getStringWidth() for character width calculation

2019-09-02T06:13:58+00:00

This is more accurate for displayed full-width characters
(e.g. CJK ones) and makes the calculations match the ones we
use in the readline module.

Fixes: https://github.com/nodejs/node/issues/29299

PR-URL: https://github.com/nodejs/node/pull/29300
Reviewed-By: Gus Caplan 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Yongsheng Zhang 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Tobias Nießen 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell

test: remove Windows skipping of http keepalive request GC test

2019-08-30T14:16:19+00:00

Remove code to skip Windows in test-http-server-keepalive-req-gc. The
test now works reliably on Windows (I think).

PR-URL: https://github.com/nodejs/node/pull/29354
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
Reviewed-By: Сковорода Никита Андреевич 
Reviewed-By: David Carlier

util: do not throw when inspecting detached ArrayBuffer

2019-08-29T03:52:05+00:00

PR-URL: https://github.com/nodejs/node/pull/29318
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Jeremiah Senkpiel 
Reviewed-By: Gus Caplan

buffer: correct indexOf() error message

2019-08-28T19:13:51+00:00

PR-URL: https://github.com/nodejs/node/pull/29217
Reviewed-By: Ben Noordhuis 
Reviewed-By: Anna Henningsen

test: fix test-benchmark-net

2019-08-28T16:48:27+00:00

Fix test-benchmark-net to accommodate recent benchmark additions.

PR-URL: https://github.com/nodejs/node/pull/29359
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig

test: fix flaky test-http-server-keepalive-req-gc

2019-08-28T04:04:50+00:00

Use `server` to keep the event loop open until the `ongc` listener runs.

PR-URL: https://github.com/nodejs/node/pull/29347
Fixes: https://github.com/nodejs/node/issues/29344
Reviewed-By: Anna Henningsen 
Reviewed-By: Daniel Bevenius

src: allow --interpreted-frames-native-stack in NODE_OPTIONS

2019-08-27T22:59:06+00:00

PR-URL: https://github.com/nodejs/node/pull/27744
Reviewed-By: Richard Lau 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater

errors: provide defaults for unmapped uv errors

2019-08-27T18:27:33+00:00

libuv does not map 100% of errors. When an unmapped error is
encountered, the Map returns undefined, which is then
unsuccessfully destructured, causing an exception. This commit
adds a default value in the event of an unmapped error.

PR-URL: https://github.com/nodejs/node/pull/29288
Reviewed-By: Luigi Pinca 
Reviewed-By: Ben Noordhuis