CI: fix jest issue with latest node 12.17

electrode-io · May 29, 2020 · 898bc6b · 898bc6b
1 parent e437011
commit 898bc6b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/.github/workflows/nodejs.yml b/.github/workflows/nodejs.yml
@@ -16,7 +16,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [10.x, 12.x]
+        node-version: [10.x, 12.16.x]
 
     steps:
     - uses: actions/checkout@v2

diff --git a/.travis.yml b/.travis.yml
@@ -1,7 +1,7 @@
 language: node_js
 node_js:
   - v10
-  - v12
+  - v12.16
 dist: trusty # needs Ubuntu Trusty
 sudo: required  # no need for virtualization.
 addons:

diff --git a/packages/xarc-app-dev/config/babel/babelrc.js b/packages/xarc-app-dev/config/babel/babelrc.js
@@ -11,6 +11,8 @@ const archetype = require("@xarc/app/config/archetype")();
 const optionalRequire = require("optional-require")(require);
 const optFlow = optionalRequire("electrode-archetype-opt-flow");
 
+const isJest = Boolean(process.env.JEST_WORKER_ID);
+
 const {
   enableTypeScript,
   flowRequireDirective,
@@ -34,7 +36,7 @@ const fileId = "xarc-app-dev:babelrc.js";
 
 const checkEnv = names => {
   names = names.filter(x => !process.env.hasOwnProperty(x));
-  if (names.length > 0) {
+  if (!isJest && names.length > 0) {
     console.error(
       ck`\n<red>Notice:</> ${fileId}: env ${names.join(", ")} not defined - default to 'false'\n`
     );
@@ -140,7 +142,9 @@ const plugins = basePlugins.concat(
 const target = isNodeTarget ? "node" : archetype.babel.target;
 
 const targets = archetype.babel.envTargets[target];
-console.log(ck`<orange>Babel preset-env compile targets: </><cyan>${JSON.stringify(targets)}</>`);
+if (!isJest) {
+  console.log(ck`<orange>Babel preset-env compile targets: </><cyan>${JSON.stringify(targets)}</>`);
+}
 
 const useBuiltIns =
   !isNodeTarget && archetype.babel.hasMultiTargets