build: Removed jsx-remove-data-test-id usage from code for multi-buil…

…d-variant testing (apache#15386) * enabled customized production build with testIds * fix for docker * changed substitution * changed substitution
QAlexBall · Aug 9, 2021 · aa8ad6e · aa8ad6e
1 parent da7fe97
commit aa8ad6e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/superset-frontend/babel.config.js b/superset-frontend/babel.config.js
@@ -82,5 +82,8 @@ module.exports = {
         ],
       ],
     },
+    testableProduction: {
+      plugins: [],
+    },
   },
 };
diff --git a/superset-frontend/package.json b/superset-frontend/package.json
@@ -17,7 +17,7 @@
     "prod": "npm run build",
     "build-dev": "cross-env NODE_OPTIONS=--max_old_space_size=8192 NODE_ENV=development webpack --mode=development --colors",
     "build-instrumented": "cross-env NODE_ENV=development BABEL_ENV=instrumented webpack --mode=development --colors",
-    "build": "cross-env NODE_OPTIONS=--max_old_space_size=8192 NODE_ENV=production BABEL_ENV=production webpack --mode=production --colors",
+    "build": "cross-env NODE_OPTIONS=--max_old_space_size=8192 NODE_ENV=production BABEL_ENV=\"${BABEL_ENV:=production}\" webpack --mode=production --colors",
     "lint": "eslint --ignore-path=.eslintignore --ext .js,.jsx,.ts,.tsx . && npm run type",
     "prettier-check": "prettier --check '{src,stylesheets}/**/*.{css,less,sass,scss}'",
     "lint-fix": "eslint --fix --ignore-path=.eslintignore --ext .js,.jsx,.ts,tsx . && npm run clean-css && npm run type",
-Original file line number
+Diff line change
@@ Expand Up / @@ -82,5 +82,8 @@ module.exports = { @@
             ],
           ],
         },
+        testableProduction: {
+          plugins: [],
+        },
       },
     };