delta/node-new.git/test/parallel/test-buffer-concat.js, branch Remove-Python-3-tests

buffer: correct concat() error message

2019-08-22T05:21:19+00:00

PR-URL: https://github.com/nodejs/node/pull/29198
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater

buffer: fix concat error message

2019-04-04T12:21:11+00:00

The list argument may only be of type array, not of any other type
as it actually suggests.

PR-URL: https://github.com/nodejs/node/pull/27050
Reviewed-By: Anna Henningsen 
Reviewed-By: Luigi Pinca 
Reviewed-By: Yongsheng Zhang

lib: improve error handling

2018-03-25T01:01:45+00:00

This improves the error handling for a couple cases where the
received value would not have been handled so far or where the name
is wrong etc.

PR-URL: https://github.com/nodejs/node/pull/19445
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina 
Reviewed-By: Joyee Cheung 
Reviewed-By: Anna Henningsen

lib: always show ERR_INVALID_ARG_TYPE received part

2018-03-25T00:45:37+00:00

This makes a effort to make sure all of these errors will actually
also show the received input.
On top of that it refactors a few tests for better maintainability.
It will also change the returned type to always be a simple typeof
instead of special handling null.

PR-URL: https://github.com/nodejs/node/pull/19445
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina 
Reviewed-By: Joyee Cheung 
Reviewed-By: Anna Henningsen

lint: move eslint to new plugin system

2018-02-20T19:13:22+00:00

PR-URL: https://github.com/nodejs/node/pull/18566
Reviewed-By: Joyee Cheung 
Reviewed-By: Roman Reiss

test: replace assert.throws w/ common.expectsError

2017-12-08T21:23:32+00:00

PR-URL: https://github.com/nodejs/node/pull/17483
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: Evan Lucas 
Reviewed-By: James M Snell

lib: improve the usage of TypeError[INVALID_ARG_TYPE]

2017-11-12T19:04:06+00:00

The initials of expected in TypeError[ERR_INVALID_ARG_TYPE]
are inconsistent. This change is to unify them.

PR-URL: https://github.com/nodejs/node/pull/16401
Fixes: https://github.com/nodejs/node/issues/16383
Reviewed-By: Joyee Cheung 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Benjamin Gruenbaum

tools: add eslint rule for hasCrypto checking

2017-08-30T21:02:58+00:00

The motivation for this commit is to pick up early on missing checks for
crypto support (when Node is built --without-ssl).

There are currently usages of common.hasCrypto which are not just for
detecting if crypto support is available and then skip the test in
question. For these case we still want to have a lint error generated
which can then be disabled using an ESLint comment.

PR-URL: https://github.com/nodejs/node/pull/13813
Reviewed-By: James M Snell 
Reviewed-By: Teddy Katz

test: adjust indentation for stricter linting

2017-07-27T16:24:20+00:00

ESLint 4.x has stricter linting than previous versions. We are currently
using the legacy indentation rules in the test directory. This commit
changes the indentation of files to comply with the stricter 4.x linting
and enable stricter linting in the test directory.

PR-URL: https://github.com/nodejs/node/pull/14431
Reviewed-By: Refael Ackermann 
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Trevor Norris

errors,buffer: port errors to internal/errors

2017-07-12T21:00:30+00:00

PR-URL: https://github.com/nodejs/node/pull/13976
Reviewed-By: Refael Ackermann 
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Benjamin Gruenbaum