From 49a9b70cb1f5f9c1490bab0a7358225c832c9a6e Mon Sep 17 00:00:00 2001 From: Justin Ridgewell Date: Mon, 8 Mar 2021 16:58:30 -0500 Subject: [PATCH] Optionally enable external-helpers in tests (#12982) * Optionally enable external-helpers in tests https://github.com/babel/babel/pull/12911 was a breaking change that now requires you to keep `babel-plugin-external-helpers` in your `package.json`. It'll will probably be unnoticeable for most, because another dependency could transitively depend on it, but AMP recently hit it when cleaning up our deps. * Cast to boolean * Fix package name --- packages/babel-helper-fixtures/src/index.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/babel-helper-fixtures/src/index.ts b/packages/babel-helper-fixtures/src/index.ts index 2f8beb723497..a9c58508fd4a 100644 --- a/packages/babel-helper-fixtures/src/index.ts +++ b/packages/babel-helper-fixtures/src/index.ts @@ -135,7 +135,9 @@ function pushTask(taskName, taskDir, suite, suiteName) { ? taskOpts.BABEL_8_BREAKING === false : taskOpts.BABEL_8_BREAKING === true), options: taskOpts, - externalHelpers: taskOpts.externalHelpers ?? true, + externalHelpers: + taskOpts.externalHelpers ?? + !!tryResolve("@babel/plugin-external-helpers"), validateLogs: taskOpts.validateLogs, ignoreOutput: taskOpts.ignoreOutput, stdout: { loc: stdoutLoc, code: readFile(stdoutLoc) },