diff --git a/test/es-module/esm-snapshot.js b/test/es-module/esm-snapshot.js deleted file mode 100644 index f52c6f3adff..00000000000 --- a/test/es-module/esm-snapshot.js +++ /dev/null @@ -1,3 +0,0 @@ -/* eslint-disable node-core/required-modules */ -'use strict'; -module.exports = 1; diff --git a/test/es-module/test-esm-basic-imports.mjs b/test/es-module/test-esm-basic-imports.mjs index 23989bddd5b..78a4106f944 100644 --- a/test/es-module/test-esm-basic-imports.mjs +++ b/test/es-module/test-esm-basic-imports.mjs @@ -1,7 +1,7 @@ // Flags: --experimental-modules import '../common'; import assert from 'assert'; -import ok from './test-esm-ok.mjs'; +import ok from '../fixtures/es-modules/test-esm-ok.mjs'; import okShebang from './test-esm-shebang.mjs'; assert(ok); diff --git a/test/es-module/test-esm-double-encoding.mjs b/test/es-module/test-esm-double-encoding.mjs index 4af0c33ad8d..c81d0530d33 100644 --- a/test/es-module/test-esm-double-encoding.mjs +++ b/test/es-module/test-esm-double-encoding.mjs @@ -3,4 +3,4 @@ import '../common'; // Assert we can import files with `%` in their pathname. -import './test-esm-double-encoding-native%2520.js'; +import '../fixtures/es-modules/test-esm-double-encoding-native%2520.js'; diff --git a/test/es-module/test-esm-dynamic-import.js b/test/es-module/test-esm-dynamic-import.js index 9c4d48aaf02..c6daa95f9a6 100644 --- a/test/es-module/test-esm-dynamic-import.js +++ b/test/es-module/test-esm-dynamic-import.js @@ -7,8 +7,8 @@ const vm = require('vm'); common.crashOnUnhandledRejection(); -const relativePath = './test-esm-ok.mjs'; -const absolutePath = require.resolve('./test-esm-ok.mjs'); +const relativePath = '../fixtures/es-modules/test-esm-ok.mjs'; +const absolutePath = require.resolve('../fixtures/es-modules/test-esm-ok.mjs'); const targetURL = new URL('file:///'); targetURL.pathname = absolutePath; diff --git a/test/es-module/test-esm-encoded-path.mjs b/test/es-module/test-esm-encoded-path.mjs index 2c6e145927a..365a425afa5 100644 --- a/test/es-module/test-esm-encoded-path.mjs +++ b/test/es-module/test-esm-encoded-path.mjs @@ -2,6 +2,6 @@ import '../common'; import assert from 'assert'; // ./test-esm-ok.mjs -import ok from './test-%65%73%6d-ok.mjs'; +import ok from '../fixtures/es-modules/test-%65%73%6d-ok.mjs'; assert(ok); diff --git a/test/es-module/test-esm-example-loader.js b/test/es-module/test-esm-example-loader.js index f7f0fd059a4..0b0001acea2 100644 --- a/test/es-module/test-esm-example-loader.js +++ b/test/es-module/test-esm-example-loader.js @@ -1,6 +1,6 @@ // Flags: --experimental-modules --loader ./test/fixtures/es-module-loaders/example-loader.mjs /* eslint-disable node-core/required-modules */ import assert from 'assert'; -import ok from './test-esm-ok.mjs'; +import ok from '../fixtures/es-modules/test-esm-ok.mjs'; assert(ok); diff --git a/test/es-module/test-esm-forbidden-globals.mjs b/test/es-module/test-esm-forbidden-globals.mjs index d92df1611f0..840a7ea5413 100644 --- a/test/es-module/test-esm-forbidden-globals.mjs +++ b/test/es-module/test-esm-forbidden-globals.mjs @@ -1,5 +1,5 @@ // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ +import '../common'; if (typeof arguments !== 'undefined') { throw new Error('not an ESM'); diff --git a/test/es-module/test-esm-json.mjs b/test/es-module/test-esm-json.mjs index 5b432e03958..a7146d19a9a 100644 --- a/test/es-module/test-esm-json.mjs +++ b/test/es-module/test-esm-json.mjs @@ -1,9 +1,8 @@ // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ -import '../common/index'; +import '../common'; import assert from 'assert'; -import ok from './test-esm-ok.mjs'; -import json from './json.json'; +import ok from '../fixtures/es-modules/test-esm-ok.mjs'; +import json from '../fixtures/es-modules/json.json'; assert(ok); assert.strictEqual(json.val, 42); diff --git a/test/es-module/test-esm-loader-dependency.mjs b/test/es-module/test-esm-loader-dependency.mjs index 260bf613a75..1ed8685a6f2 100644 --- a/test/es-module/test-esm-loader-dependency.mjs +++ b/test/es-module/test-esm-loader-dependency.mjs @@ -1,5 +1,5 @@ // Flags: --experimental-modules --loader ./test/fixtures/es-module-loaders/loader-with-dep.mjs /* eslint-disable node-core/required-modules */ -import './test-esm-ok.mjs'; +import '../fixtures/es-modules/test-esm-ok.mjs'; // We just test that this module doesn't fail loading diff --git a/test/es-module/test-esm-main-lookup.mjs b/test/es-module/test-esm-main-lookup.mjs index 12f12c845ca..ca313a1d264 100644 --- a/test/es-module/test-esm-main-lookup.mjs +++ b/test/es-module/test-esm-main-lookup.mjs @@ -1,5 +1,5 @@ // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ +import '../common'; import assert from 'assert'; import main from '../fixtures/es-modules/pjson-main'; diff --git a/test/es-module/test-esm-named-exports.mjs b/test/es-module/test-esm-named-exports.mjs index 9698ae3d9c0..3aae9230de3 100644 --- a/test/es-module/test-esm-named-exports.mjs +++ b/test/es-module/test-esm-named-exports.mjs @@ -1,9 +1,8 @@ // Flags: --experimental-modules --loader ./test/fixtures/es-module-loaders/builtin-named-exports-loader.mjs -/* eslint-disable node-core/required-modules */ -import '../common/index'; +import '../common'; import { readFile } from 'fs'; import assert from 'assert'; -import ok from './test-esm-ok.mjs'; +import ok from '../fixtures/es-modules/test-esm-ok.mjs'; assert(ok); assert(readFile); diff --git a/test/es-module/test-esm-namespace.mjs b/test/es-module/test-esm-namespace.mjs index 6a5ee28a545..04845e2b3e4 100644 --- a/test/es-module/test-esm-namespace.mjs +++ b/test/es-module/test-esm-namespace.mjs @@ -1,7 +1,5 @@ // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ - -import '../common/index'; +import '../common'; import * as fs from 'fs'; import assert from 'assert'; diff --git a/test/es-module/test-esm-ok.mjs b/test/es-module/test-esm-ok.mjs deleted file mode 100644 index 49de5c47ec2..00000000000 --- a/test/es-module/test-esm-ok.mjs +++ /dev/null @@ -1,5 +0,0 @@ -// Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ - -const isJs = true; -export default isJs; diff --git a/test/es-module/test-esm-preserve-symlinks.js b/test/es-module/test-esm-preserve-symlinks.js index e8473c36473..28cba3e7022 100644 --- a/test/es-module/test-esm-preserve-symlinks.js +++ b/test/es-module/test-esm-preserve-symlinks.js @@ -1,4 +1,3 @@ -// Flags: --experimental-modules 'use strict'; const common = require('../common'); diff --git a/test/es-module/test-esm-resolve-hook.mjs b/test/es-module/test-esm-resolve-hook.mjs index e2d20a42d42..e326d20b6d8 100644 --- a/test/es-module/test-esm-resolve-hook.mjs +++ b/test/es-module/test-esm-resolve-hook.mjs @@ -2,7 +2,7 @@ /* eslint-disable node-core/required-modules */ import { namedExport } from '../fixtures/es-module-loaders/js-as-esm.js'; import assert from 'assert'; -import ok from './test-esm-ok.mjs'; +import ok from '../fixtures/es-modules/test-esm-ok.mjs'; assert(ok); assert(namedExport); diff --git a/test/es-module/test-esm-shared-loader-dep.mjs b/test/es-module/test-esm-shared-loader-dep.mjs index 03668cbc993..5c274d835c8 100644 --- a/test/es-module/test-esm-shared-loader-dep.mjs +++ b/test/es-module/test-esm-shared-loader-dep.mjs @@ -1,7 +1,7 @@ // Flags: --experimental-modules --loader ./test/fixtures/es-module-loaders/loader-shared-dep.mjs -/* eslint-disable node-core/required-modules */ +import '../common'; import assert from 'assert'; -import './test-esm-ok.mjs'; +import '../fixtures/es-modules/test-esm-ok.mjs'; import dep from '../fixtures/es-module-loaders/loader-dep.js'; assert.strictEqual(dep.format, 'esm'); diff --git a/test/es-module/test-esm-shebang.mjs b/test/es-module/test-esm-shebang.mjs index 96cec8ea98e..d5faace4792 100644 --- a/test/es-module/test-esm-shebang.mjs +++ b/test/es-module/test-esm-shebang.mjs @@ -1,6 +1,6 @@ #! }]) // isn't js // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ +import '../common'; const isJs = true; export default isJs; diff --git a/test/es-module/test-esm-snapshot.mjs b/test/es-module/test-esm-snapshot.mjs index 4d3cf245a3c..3d4b44bbdd5 100644 --- a/test/es-module/test-esm-snapshot.mjs +++ b/test/es-module/test-esm-snapshot.mjs @@ -1,8 +1,7 @@ // Flags: --experimental-modules -/* eslint-disable node-core/required-modules */ -import '../common/index'; -import './esm-snapshot-mutator'; -import one from './esm-snapshot'; +import '../common'; +import '../fixtures/es-modules/esm-snapshot-mutator'; +import one from '../fixtures/es-modules/esm-snapshot'; import assert from 'assert'; assert.strictEqual(one, 1); diff --git a/test/es-module/esm-snapshot-mutator.js b/test/fixtures/es-modules/esm-snapshot-mutator.js similarity index 77% rename from test/es-module/esm-snapshot-mutator.js rename to test/fixtures/es-modules/esm-snapshot-mutator.js index 6c9a707306f..ee52c270f66 100644 --- a/test/es-module/esm-snapshot-mutator.js +++ b/test/fixtures/es-modules/esm-snapshot-mutator.js @@ -1,4 +1,3 @@ -/* eslint-disable node-core/required-modules */ 'use strict'; const shouldSnapshotFilePath = require.resolve('./esm-snapshot.js'); require('./esm-snapshot.js'); diff --git a/test/fixtures/es-modules/esm-snapshot.js b/test/fixtures/es-modules/esm-snapshot.js new file mode 100644 index 00000000000..329a0ca3f43 --- /dev/null +++ b/test/fixtures/es-modules/esm-snapshot.js @@ -0,0 +1,2 @@ +'use strict'; +module.exports = 1; diff --git a/test/es-module/json.json b/test/fixtures/es-modules/json.json similarity index 100% rename from test/es-module/json.json rename to test/fixtures/es-modules/json.json diff --git a/test/es-module/test-esm-double-encoding-native%20.js b/test/fixtures/es-modules/test-esm-double-encoding-native%20.js similarity index 87% rename from test/es-module/test-esm-double-encoding-native%20.js rename to test/fixtures/es-modules/test-esm-double-encoding-native%20.js index 8780cf774d2..ea1caa81bea 100644 --- a/test/es-module/test-esm-double-encoding-native%20.js +++ b/test/fixtures/es-modules/test-esm-double-encoding-native%20.js @@ -1,5 +1,4 @@ 'use strict'; -require('../common'); // Trivial test to assert we can load files with `%` in their pathname. // Imported by `test-esm-double-encoding.mjs`. diff --git a/test/fixtures/es-modules/test-esm-ok.mjs b/test/fixtures/es-modules/test-esm-ok.mjs new file mode 100644 index 00000000000..7dfc6eb5879 --- /dev/null +++ b/test/fixtures/es-modules/test-esm-ok.mjs @@ -0,0 +1,2 @@ +const isJs = true; +export default isJs; diff --git a/test/parallel/test-module-main-extension-lookup.js b/test/parallel/test-module-main-extension-lookup.js index 6f7bc2eb1db..3d20316647c 100644 --- a/test/parallel/test-module-main-extension-lookup.js +++ b/test/parallel/test-module-main-extension-lookup.js @@ -1,9 +1,11 @@ 'use strict'; require('../common'); +const fixtures = require('../common/fixtures'); const { execFileSync } = require('child_process'); const node = process.argv[0]; -execFileSync(node, ['--experimental-modules', 'test/es-module/test-esm-ok']); execFileSync(node, ['--experimental-modules', - 'test/fixtures/es-modules/noext']); + fixtures.path('es-modules', 'test-esm-ok')]); +execFileSync(node, ['--experimental-modules', + fixtures.path('es-modules', 'noext')]);