Merge pull request #475 from nomiclabs/test-dev-perf-enhancements

Test dev perf enhancements

Author: tmilar

.gitignore

@@ -8,8 +8,8 @@ package-lock.json
 
 .DS_Store
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -71,10 +71,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

package.json

@@ -7,8 +7,8 @@
   "devDependencies": {
     "@types/mocha": "^5.2.6",
     "@types/node": "^8.10.44",
+    "@types/shelljs": "^0.8.6",
     "chai": "^4.2.0",
-    "dotenv": "^6.2.0",
     "ganache-cli": "^6.4.3",
     "lerna": "^3.13.4",
     "mocha": "^5.2.0",

packages/buidler-core/.gitignore

@@ -20,8 +20,8 @@
 /builtin-tasks
 /internal
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -83,10 +83,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

packages/buidler-core/package.json

@@ -31,8 +31,8 @@
     "lint": "npm run lint-src && npm run lint-tests",
     "lint-tests": "node -r ts-node/register ../../node_modules/tslint/bin/tslint --config tslint.json --project ./tsconfig.json",
     "lint-src": "node -r ts-node/register ../../node_modules/tslint/bin/tslint --config tslint.json --project src/tsconfig.json",
-    "test": "node ../../node_modules/mocha/bin/mocha",
-    "coverage": "node ../../node_modules/nyc/bin/nyc.js ../../node_modules/mocha/bin/mocha",
+    "test": "node ../../node_modules/mocha/bin/mocha --recursive test/**/*.ts ",
+    "coverage": "node ../../node_modules/nyc/bin/nyc.js ../../node_modules/mocha/bin/mocha --recursive test/**/*.ts",
     "build": "node ../../node_modules/typescript/bin/tsc --build src",
     "build-test": "node ../../node_modules/typescript/bin/tsc --build .",
     "clean": "node ../../node_modules/rimraf/bin.js builtin-tasks internal *.d.ts *.map *.js build-test tsconfig.tsbuildinfo"

packages/buidler-core/test/internal/core/typescript-support.ts

@@ -17,7 +17,7 @@ describe("Typescript support", function() {
     it("Should fail if an implicit any is used and the tsconfig forbids them", function() {
       // If we run this test in transpilation only mode, it will fail
       if (process.env.TS_NODE_TRANSPILE_ONLY === "true") {
-        return;
+        this.skip();
       }
 
       assert.throws(

packages/buidler-core/test/mocha.env.js

@@ -0,0 +1,2 @@
+process.env.TS_NODE_FILES = "true";
+process.env.TS_NODE_TRANSPILE_ONLY = "true";

packages/buidler-core/test/mocha.opts

@@ -1,4 +1,5 @@
+--require test/mocha.env
 --require ts-node/register
 --require source-map-support/register
---recursive test/**/*.ts --exclude test/fixture-projects/**/*.ts --exclude test/fixture-projects/**/*.js --exclude test/helpers/**/*.ts
+--exclude test/fixture-projects/**/*.ts --exclude test/fixture-projects/**/*.js --exclude test/helpers/**/*.ts
 --timeout 25000

packages/buidler-docker/.env

@@ -9,8 +9,8 @@
 /coverage
 /.nyc_output
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -72,10 +72,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

packages/buidler-docker/.gitignore

@@ -18,7 +18,7 @@
     "lint-tests": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project ./tsconfig.json",
     "watch": "node ../../node_modules/typescript/bin/tsc -w",
     "lint-src": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project src/tsconfig.json",
-    "test": "node ./scripts/run-tests.js",
+    "test": "node ../../node_modules/mocha/bin/mocha --recursive test/**/*.ts --exit",
     "build": "node ../../node_modules/typescript/bin/tsc --build src",
     "build-test": "node ../../node_modules/typescript/bin/tsc --build .",
     "clean": "node ../../node_modules/rimraf/bin.js dist build-test"

packages/buidler-docker/package.json

@@ -0,0 +1,2 @@
+process.env.TS_NODE_FILES = "true";
+process.env.TS_NODE_TRANSPILE_ONLY = "true";

packages/buidler-docker/scripts/run-tests.js

@@ -1,5 +1,4 @@
---require dotenv/config
+--require test/mocha.env
 --require ts-node/register
 --require source-map-support/register
---recursive test/**/*.ts
 --timeout 6000

packages/buidler-docker/test/mocha.env.js

@@ -9,8 +9,8 @@
 /coverage
 /.nyc_output
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -72,10 +72,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

packages/buidler-docker/test/mocha.opts

@@ -20,7 +20,7 @@
     "lint": "npm run lint-src && npm run lint-tests",
     "lint-tests": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project ./tsconfig.json",
     "lint-src": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project src/tsconfig.json",
-    "test": "node scripts/run-tests.js",
+    "test": "node ../../node_modules/mocha/bin/mocha --recursive test/**/*.ts --exit",
     "build": "node ../../node_modules/typescript/bin/tsc --build src",
     "build-test": "node ../../node_modules/typescript/bin/tsc --build .",
     "clean": "node ../../node_modules/rimraf/bin.js dist build-test"

packages/buidler-ethers/.env

@@ -0,0 +1,5 @@
+process.env.TS_NODE_FILES = "true";
+process.env.TS_NODE_TRANSPILE_ONLY = "true";
+
+// Args string to be passed to the Ganache CLI instance for test, splitted by "," or " " (space)
+process.env.GANACHE_CLI_ARGS = "-d";

packages/buidler-ethers/.gitignore

@@ -1,5 +1,5 @@
---require dotenv/config
+--require test/mocha.env
 --require ts-node/register
 --require source-map-support/register
---recursive test/**/*.ts
+--file ../common/test/run-with-ganache
 --timeout 6000

packages/buidler-ethers/package.json

@@ -9,8 +9,8 @@
 /coverage
 /.nyc_output
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -72,10 +72,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

packages/buidler-ethers/scripts/run-tests.js

@@ -24,7 +24,7 @@
     "lint": "npm run lint-src && npm run lint-tests",
     "lint-tests": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project ./tsconfig.json",
     "lint-src": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project src/tsconfig.json",
-    "test": "node scripts/run-tests.js",
+    "test": "node ../../node_modules/mocha/bin/mocha --recursive test/**/*.ts --exit",
     "build": "node ../../node_modules/typescript/bin/tsc --build src",
     "build-test": "node ../../node_modules/typescript/bin/tsc --build .",
     "clean": "node ../../node_modules/rimraf/bin.js dist build-test"

packages/buidler-ethers/test/mocha.env.js

@@ -0,0 +1,2 @@
+process.env.TS_NODE_FILES = "true";
+process.env.TS_NODE_TRANSPILE_ONLY = "true";

packages/buidler-ethers/test/mocha.opts

@@ -1,5 +1,5 @@
---require dotenv/config
+--require test/mocha.env
 --require ts-node/register
 --require source-map-support/register
---recursive test/**/*.ts
+--file ../common/test/run-with-ganache
 --timeout 6000

packages/buidler-etherscan/.env

@@ -9,8 +9,8 @@
 /coverage
 /.nyc_output
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -72,10 +72,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache
 

packages/buidler-etherscan/.gitignore

@@ -21,7 +21,7 @@
     "lint": "npm run lint-src && npm run lint-tests",
     "lint-tests": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project ./tsconfig.json",
     "lint-src": "node ../../node_modules/tslint/bin/tslint --config tslint.json --project src/tsconfig.json",
-    "test": "node scripts/run-tests.js",
+    "test": "node ../../node_modules/mocha/bin/mocha --recursive test/*.ts --exit",
     "build": "node ../../node_modules/typescript/bin/tsc --build src",
     "build-test": "node ../../node_modules/typescript/bin/tsc --build .",
     "clean": "node ../../node_modules/rimraf/bin.js dist build-test"

packages/buidler-etherscan/package.json

@@ -0,0 +1,2 @@
+process.env.TS_NODE_FILES = "true";
+process.env.TS_NODE_TRANSPILE_ONLY = "true";

packages/buidler-etherscan/scripts/run-tests.js

@@ -1,5 +1,4 @@
---require dotenv/config
+--require test/mocha.env
 --require ts-node/register
 --require source-map-support/register
---recursive test/*.ts
---timeout 5000
+--timeout 8000

packages/buidler-etherscan/test/mocha.env.js

@@ -9,8 +9,8 @@
 /coverage
 /.nyc_output
 
-# Below is Github's node gitignore template, except for dotenv's entries as it's used by mocha to pass flags to ts-node,
-# and ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
+# Below is Github's node gitignore template,
+# ignoring the node_modules part, as it'd ignore every node_modules, and we have some for testing
 
 # Logs
 logs
@@ -72,10 +72,6 @@ typings/
 # Yarn Integrity file
 .yarn-integrity
 
-# dotenv environment variables file
-#.env
-#.env.test
-
 # parcel-bundler cache (https://parceljs.org/)
 .cache