test: return dotenv-safe back to wallet and sdk

packages/js-dash-sdk/karma/options.js

@@ -1,6 +1,6 @@
 /* eslint-disable import/no-extraneous-dependencies */
 const webpack = require('webpack');
-const dotenvResult = require('dotenv').config();
+const dotenvSafe = require('dotenv-safe');
 
 const karmaMocha = require('karma-mocha');
 const karmaMochaReporter = require('karma-mocha-reporter');
@@ -9,6 +9,15 @@ const karmaChromeLauncher = require('karma-chrome-launcher');
 const karmaWebpack = require('karma-webpack');
 const webpackBaseConfig = require('../webpack.base.config');
 
+let env = {};
+if (process.env.LOAD_ENV) {
+  const dotenvResult = dotenvSafe.config();
+  if (dotenvResult.error) {
+    throw dotenvResult.error;
+  }
+  env = dotenvResult.parsed;
+}
+
 module.exports = {
   frameworks: ['mocha', 'chai', 'webpack'],
   webpack: {
@@ -17,7 +26,7 @@ module.exports = {
     plugins: [
       ...webpackBaseConfig.plugins,
       new webpack.EnvironmentPlugin(
-        dotenvResult.parsed,
+        env,
       ),
     ],
   },

packages/js-dash-sdk/package.json

@@ -15,9 +15,9 @@
     "lint:fix": "eslint . --fix",
     "test": "yarn run test:types && yarn run test:unit && yarn run test:browsers",
     "test:browsers": "karma start ./karma/karma.conf.js --single-run",
-    "test:browsers:functional": "karma start ./karma/karma.functional.conf.js --single-run",
+    "test:browsers:functional": "LOAD_ENV=true karma start ./karma/karma.functional.conf.js --single-run",
     "test:unit": "ts-mocha -p tsconfig.mocha.json src/**/*.spec.ts",
-    "test:functional": "yarn run build && mocha --recursive tests/functional/**/*.js",
+    "test:functional": "yarn run build:ts && LOAD_ENV=true mocha --recursive tests/functional/**/*.js",
     "test:types": "yarn pnpify tsd",
     "prepublishOnly": "yarn run build",
     "prepare": "yarn run build"
@@ -72,7 +72,7 @@
     "chance": "^1.1.6",
     "crypto-browserify": "^3.12.0",
     "dirty-chai": "^2.0.1",
-    "dotenv": "^8.6.0",
+    "dotenv-safe": "^8.2.0",
     "eslint": "^7.32.0",
     "eslint-config-airbnb-base": "^14.2.1",
     "eslint-config-airbnb-typescript": "^17.0.0",

packages/js-dash-sdk/src/test/bootstrap.js

@@ -1,15 +1,16 @@
-const dotenv = require('dotenv');
 const path = require('path');
-
+const dotenvSafe = require('dotenv-safe');
 const sinon = require('sinon');
 const sinonChai = require('sinon-chai');
 const { use } = require('chai');
 const dirtyChai = require('dirty-chai');
 const chaiAsPromised = require('chai-as-promised');
 
-dotenv.config({
-  path: path.resolve(__dirname, '..', '..', '.env'),
-});
+if (process.env.LOAD_ENV === 'true') {
+  dotenvSafe.config({
+    path: path.resolve(__dirname, '..', '..', '.env'),
+  });
+}
 
 use(dirtyChai);
 use(sinonChai);

packages/wallet-lib/karma/options.js

@@ -1,6 +1,6 @@
 /* eslint-disable import/no-extraneous-dependencies */
 const webpack = require('webpack');
-const dotenvResult = require('dotenv').config();
+const dotenvSafe = require('dotenv-safe');
 
 const karmaMocha = require('karma-mocha');
 const karmaMochaReporter = require('karma-mocha-reporter');
@@ -11,6 +11,15 @@ const karmaWebpack = require('karma-webpack');
 
 const webpackConfig = require('../webpack.config');
 
+let env = {};
+if (process.env.LOAD_ENV) {
+  const dotenvResult = dotenvSafe.config();
+  if (dotenvResult.error) {
+    throw dotenvResult.error;
+  }
+  env = dotenvResult.parsed;
+}
+
 module.exports = {
   client: {
     mocha: {
@@ -24,7 +33,7 @@ module.exports = {
     plugins: [
       ...webpackConfig.plugins,
       new webpack.EnvironmentPlugin(
-        dotenvResult.parsed,
+        env,
       ),
     ],
     resolve: webpackConfig.resolve,

packages/wallet-lib/package.json

@@ -10,10 +10,10 @@
     "lint": "eslint .",
     "lint:fix": "eslint . --fix",
     "test:unit": "nyc yarn mocha 'src/**/*.spec.js'",
-    "test:functional": "nyc yarn run mocha 'tests/functional/*.js'",
+    "test:functional": "LOAD_ENV=true nyc yarn run mocha 'tests/functional/*.js'",
     "test:integration": "nyc yarn run mocha 'tests/integration/**/*.spec.js'",
     "test:browsers": "karma start ./karma/karma.conf.js --single-run",
-    "test:browsers:functional": "karma start ./karma/karma.functional.conf.js --single-run",
+    "test:browsers:functional": "LOAD_ENV=true karma start ./karma/karma.functional.conf.js --single-run",
     "test:types": "yarn pnpify tsd",
     "test": "yarn run test:types && yarn run test:unit && yarn run test:integration && yarn run test:browsers",
     "prepublishOnly": "yarn run build:web"
@@ -66,7 +66,7 @@
     "chai-as-promised": "^7.1.1",
     "crypto-browserify": "^3.12.0",
     "dirty-chai": "^2.0.1",
-    "dotenv": "^8.6.0",
+    "dotenv-safe": "^8.2.0",
     "eslint": "^7.32.0",
     "eslint-config-airbnb-base": "^14.2.1",
     "eslint-plugin-import": "^2.24.2",

packages/wallet-lib/src/test/bootstrap.js

@@ -1,7 +1,7 @@
 const { use } = require('chai');
 const { expect } = require('chai');
+const dotenvSafe = require('dotenv-safe');
 const path = require('path');
-const dotenv = require('dotenv');
 const sinon = require('sinon');
 const sinonChai = require('sinon-chai');
 const dirtyChai = require('dirty-chai');
@@ -11,9 +11,11 @@ use(sinonChai);
 use(dirtyChai);
 use(chaiAsPromised);
 
-dotenv.config({
-  path: path.resolve(__dirname, '..', '..', '.env'),
-});
+if (process.env.LOAD_ENV === 'true') {
+  dotenvSafe.config({
+    path: path.resolve(__dirname, '..', '..', '.env'),
+  });
+}
 
 beforeEach(function beforeEach() {
   if (!this.sinon) {

yarn.lock

@@ -1874,7 +1874,7 @@ __metadata:
     crypto-browserify: ^3.12.0
     crypto-js: ^4.0.0
     dirty-chai: ^2.0.1
-    dotenv: ^8.6.0
+    dotenv-safe: ^8.2.0
     eslint: ^7.32.0
     eslint-config-airbnb-base: ^14.2.1
     eslint-plugin-import: ^2.24.2
@@ -6117,7 +6117,7 @@ __metadata:
     chance: ^1.1.6
     crypto-browserify: ^3.12.0
     dirty-chai: ^2.0.1
-    dotenv: ^8.6.0
+    dotenv-safe: ^8.2.0
     eslint: ^7.32.0
     eslint-config-airbnb-base: ^14.2.1
     eslint-config-airbnb-typescript: ^17.0.0