make file structure closer to Node.js' (#6)

* make file structure closer to Node.js'

Having the file structure as close as nodejs/node one will help keeping
the code up-to-date. We can also use package imports for the require
calls to be as close as the upstream ones as possible.

* fixup!

* fixup!

* fixup!

Author: aduh95

bin/test_runner.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-// https://github.com/cjihrig/node/blob/484d35402de36d8d5756b244c8e5fbb8aa4c6afd/lib/internal/main/test_runner.js
+// https://github.com/nodejs/node/blob/484d35402de36d8d5756b244c8e5fbb8aa4c6afd/lib/internal/main/test_runner.js
 'use strict'
 const {
   ArrayFrom,
@@ -8,15 +8,18 @@ const {
   ArrayPrototypeSort,
   Promise,
   SafeSet
-} = require('../lib/primordials')
+} = require('#internal/per_context/primordials')
 const { spawn } = require('child_process')
 const { readdirSync, statSync } = require('fs')
 const {
   codes: { ERR_TEST_FAILURE }
-} = require('../lib/errors')
-const test = require('../lib/harness')
-const { kSubtestsFailed } = require('../lib/test')
-const { isSupportedFileType, doesPathMatchFilter } = require('../lib/utils')
+} = require('#internal/errors')
+const test = require('#internal/test_runner/harness')
+const { kSubtestsFailed } = require('#internal/test_runner/test')
+const {
+  isSupportedFileType,
+  doesPathMatchFilter
+} = require('#internal/test_runner/utils')
 const { basename, join, resolve } = require('path')
 
 // TODO(cjihrig): Replace this with recursive readdir once it lands.

index.js

@@ -2,7 +2,7 @@
 
 'use strict'
 
-const test = require('./lib/harness')
+const test = require('#internal/test_runner/harness')
 
 module.exports = test
 module.exports.test = test

lib/errors.js lib/internal/errors.jslib/errors.js renamed to lib/internal/errors.js

@@ -10,7 +10,7 @@ const {
   ReflectApply,
   SafeMap,
   StringPrototypeMatch
-} = require('./primordials')
+} = require('#internal/per_context/primordials')
 
 function inspectWithNoCustomRetry (obj, options) {
   const utilInspect = lazyInternalUtilInspect()

lib/options.js lib/internal/options.jslib/options.js renamed to lib/internal/options.js

@@ -3,11 +3,14 @@
 'use strict'
 
 const { createHook, executionAsyncId } = require('async_hooks')
-const { FunctionPrototypeBind, SafeMap } = require('./primordials')
+const {
+  FunctionPrototypeBind,
+  SafeMap
+} = require('#internal/per_context/primordials')
 const {
   codes: { ERR_TEST_FAILURE }
-} = require('./errors')
-const { Test } = require('./test')
+} = require('#internal/errors')
+const { Test } = require('#internal/test_runner/test')
 
 function createProcessEventHandler (eventName, rootTest, testResources) {
   return err => {

lib/primordials.js lib/internal/per_context/primordials.jslib/primordials.js renamed to lib/internal/per_context/primordials.js

@@ -3,7 +3,7 @@
 'use strict'
 
 const { Readable } = require('stream')
-const { isError } = require('./utils')
+const { isError } = require('#internal/test_runner/utils')
 const {
   ArrayPrototypeForEach,
   ArrayPrototypeJoin,
@@ -13,8 +13,8 @@ const {
   StringPrototypeReplaceAll,
   StringPrototypeSplit,
   RegExpPrototypeSymbolReplace
-} = require('./primordials')
-const { inspectWithNoCustomRetry } = require('./errors')
+} = require('#internal/per_context/primordials')
+const { inspectWithNoCustomRetry } = require('#internal/errors')
 
 const kFrameStartRegExp = /^ {4}at /
 const kLineBreakRegExp = /\n|\r\n/

lib/harness.js lib/internal/test_runner/harness.jslib/harness.js renamed to lib/internal/test_runner/harness.js

@@ -13,13 +13,13 @@ const {
   Number,
   ObjectCreate,
   SafeMap
-} = require('./primordials')
+} = require('#internal/per_context/primordials')
 const {
   codes: { ERR_TEST_FAILURE },
   kIsNodeError
-} = require('./errors')
-const { getOptionValue } = require('./options')
-const { TapStream } = require('./tap_stream')
+} = require('#internal/errors')
+const { getOptionValue } = require('#internal/options')
+const { TapStream } = require('#internal/test_runner/tap_stream')
 const { cpus } = require('os')
 const isTestRunner = getOptionValue('--test')
 const testOnlyFlag = !isTestRunner && getOptionValue('--test-only')

lib/tap_stream.js lib/internal/test_runner/tap_stream.jslib/tap_stream.js renamed to lib/internal/test_runner/tap_stream.js

@@ -1,8 +1,10 @@
 // https://github.com/nodejs/node/blob/98e9499b55fbc63b2b3db79507ee1475302f9763/lib/internal/test_runner/utils.js
 'use strict'
-const { RegExpPrototypeExec } = require('./primordials')
+const { RegExpPrototypeExec } = require('#internal/per_context/primordials')
 const { basename } = require('path')
-const { types: { isNativeError } } = require('util')
+const {
+  types: { isNativeError }
+} = require('util')
 const kSupportedFileExtensions = /\.[cm]?js$/
 const kTestFilePattern = /((^test(-.+)?)|(.+[.\-_]test))\.[cm]?js$/
 

lib/test.js lib/internal/test_runner/test.jslib/test.js renamed to lib/internal/test_runner/test.js

@@ -9,6 +9,12 @@
   "bin": {
     "node-core-test": "./bin/test_runner.js"
   },
+  "imports": {
+    "#internal/per_context/*": "./lib/internal/per_context/*.js",
+    "#internal/test_runner/*": "./lib/internal/test_runner/*.js",
+    "#internal/*": "./lib/internal/*.js",
+    "#node:test": "./index.js"
+  },
   "scripts": {
     "test": "npm run test:lint && npm run test:unit",
     "test:lint": "standard --fix",