delta/node-new.git/test/parallel/test-require-resolve.js, branch cleanup-getnodeversion.py

test: fix arguments ordering for assertions to match the docs

2018-10-15T13:34:45+00:00

PR-URL: https://github.com/nodejs/node/pull/23575
Reviewed-By: Hitesh Kanwathirtha 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Sakthipriyan Vairamani

module: validate request in require.resolve.paths

2018-01-29T09:29:38+00:00

PR-URL: https://github.com/nodejs/node/pull/18359
Fixes: https://github.com/nodejs/node/issues/18352
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig

module: validate request in require.resolve

2018-01-29T09:29:38+00:00

PR-URL: https://github.com/nodejs/node/pull/18359
Fixes: https://github.com/nodejs/node/issues/18352
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig

module: replace default paths in require.resolve()

2017-11-21T16:38:12+00:00

Prior to this commit, the default search paths would be included
in the require.resolve() process, even if user specified paths
were provided. This commit causes the default paths to be
omitted by using a fake parent module.

Refs: https://github.com/nodejs/node/issues/5963
PR-URL: https://github.com/nodejs/node/pull/17113
Reviewed-By: Refael Ackermann 
Reviewed-By: James M Snell

test: call toLowerCase on the resolved module

2017-10-25T17:45:36+00:00

The commit updates test-require-resolve.js to call toLowerCase on the
resolved module instead of the path. Currently this test will fail if
the path to where node exists contains uppercase letters. For example:

```
$ out/Release/node
test/parallel/test-require-resolve.js
/root/rpmbuild/BUILD/node-v8.8.0/test/parallel
module.js:515
    throw err;
    ^

Error: Cannot find module
'/root/rpmbuild/build/node-v8.8.0/test/fixtures/nested-index/one'
    at Function.Module._resolveFilename (module.js:513:15)
    at Function.resolve (internal/module.js:18:19)
    at Object.
(/root/rpmbuild/BUILD/node-v8.8.0/test/parallel/test-require-resolve.js:37:11)
    at Module._compile (module.js:612:30)
    at Object.Module._extensions..js (module.js:623:10)
    at Module.load (module.js:531:32)
    at tryModuleLoad (module.js:494:12)
    at Function.Module._load (module.js:486:3)
    at Function.Module.runMain (module.js:653:10)
    at startup (bootstrap_node.js:187:16)
```

PR-URL: https://github.com/nodejs/node/pull/16486
Reviewed-By: Luigi Pinca >
Reviewed-By: Ben Noordhuis >
Reviewed-By: Colin Ihrig >
Reviewed-By: Anna Henningsen >
Reviewed-By: James M Snell >

module: support custom paths to require.resolve()

2017-10-25T13:14:14+00:00

This commit allows custom lookup paths to be passed to
require.resolve(). It also adds require.resolve.paths()
which retrieves the default resolution paths.

Fixes: https://github.com/nodejs/node/issues/5963
Fixes: https://github.com/nodejs/node/issues/16389
PR-URL: https://github.com/nodejs/node/pull/16397
Reviewed-By: James M Snell 
Reviewed-By: Wyatt Preul 
Reviewed-By: Gireesh Punathil

test: fix regression in test-require-resolver.js

2017-10-14T11:40:02+00:00

PR-URL: https://github.com/nodejs/node/pull/16192
Refs: https://github.com/nodejs/node/pull/15984
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau

test: use fixtures.path instead of fixturesDir

2017-10-13T18:11:18+00:00

Updated parallel/test-require-resolve.js to use the fixtures module.

PR-URL: https://github.com/nodejs/node/pull/15984
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Tobias Nießen

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

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.