Skip to content

Commit

Permalink
runtime already has the config
Browse files Browse the repository at this point in the history
  • Loading branch information
SimenB committed Dec 4, 2020
1 parent 944f0cf commit d4600f5
Show file tree
Hide file tree
Showing 6 changed files with 9 additions and 19 deletions.
10 changes: 2 additions & 8 deletions packages/jest-circus/src/legacy-code-todo-rewrite/jestAdapter.ts
Expand Up @@ -72,21 +72,15 @@ const jestAdapter = async (
});

for (const path of config.setupFilesAfterEnv) {
const esm = runtime.unstable_shouldLoadAsEsm(
path,
config.extensionsToTreatAsEsm,
);
const esm = runtime.unstable_shouldLoadAsEsm(path);

if (esm) {
await runtime.unstable_importModule(path);
} else {
runtime.requireModule(path);
}
}
const esm = runtime.unstable_shouldLoadAsEsm(
testPath,
config.extensionsToTreatAsEsm,
);
const esm = runtime.unstable_shouldLoadAsEsm(testPath);

if (esm) {
await runtime.unstable_importModule(testPath);
Expand Down
2 changes: 0 additions & 2 deletions packages/jest-jasmine2/src/index.ts
Expand Up @@ -151,7 +151,6 @@ export default async function jasmine2(
for (const path of config.setupFilesAfterEnv) {
const esm = runtime.unstable_shouldLoadAsEsm(
path,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand All @@ -167,7 +166,6 @@ export default async function jasmine2(
}
const esm = runtime.unstable_shouldLoadAsEsm(
testPath,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand Down
2 changes: 0 additions & 2 deletions packages/jest-repl/src/cli/runtime-cli.ts
Expand Up @@ -99,7 +99,6 @@ export async function run(
for (const path of config.setupFiles) {
const esm = runtime.unstable_shouldLoadAsEsm(
path,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand All @@ -110,7 +109,6 @@ export async function run(
}
const esm = runtime.unstable_shouldLoadAsEsm(
filePath,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand Down
1 change: 0 additions & 1 deletion packages/jest-runner/src/runTest.ts
Expand Up @@ -171,7 +171,6 @@ async function runTestInternal(
for (const path of config.setupFiles) {
const esm = runtime.unstable_shouldLoadAsEsm(
path,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand Down
1 change: 0 additions & 1 deletion packages/jest-runtime/src/__mocks__/createRuntime.js
Expand Up @@ -118,7 +118,6 @@ module.exports = async function createRuntime(filename, config) {
for (const path of config.setupFiles) {
const esm = runtime.unstable_shouldLoadAsEsm(
path,
config.extensionsToTreatAsEsm,
);

if (esm) {
Expand Down
12 changes: 7 additions & 5 deletions packages/jest-runtime/src/index.ts
Expand Up @@ -363,7 +363,12 @@ export default class Runtime {
}

// unstable as it should be replaced by https://github.com/nodejs/modules/issues/393, and we don't want people to use it
unstable_shouldLoadAsEsm = Resolver.unstable_shouldLoadAsEsm;
unstable_shouldLoadAsEsm(path: Config.Path): boolean {
return Resolver.unstable_shouldLoadAsEsm(
path,
this._config.extensionsToTreatAsEsm,
);
}

private async loadEsmModule(
modulePath: Config.Path,
Expand Down Expand Up @@ -499,10 +504,7 @@ export default class Runtime {

if (
this._resolver.isCoreModule(resolved) ||
this.unstable_shouldLoadAsEsm(
resolved,
this._config.extensionsToTreatAsEsm,
)
this.unstable_shouldLoadAsEsm(resolved)
) {
return this.loadEsmModule(resolved, query, isStaticImport);
}
Expand Down

0 comments on commit d4600f5

Please sign in to comment.