Speed up running of unit tests locally (with an optional flag) (microsoft#8565)

DonJayamanne · web-flow · commit ab0e14d00056 · 2019-11-13T12:22:52.000-08:00
* Add ability to ignore bootstrap code from tests
* Add debug config
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -158,6 +158,29 @@
             "preLaunchTask": "Compile",
             "skipFiles": ["<node_internals>/**"]
         },
+        {
+            "name": "Unit Tests (fast, without VS Code and without react/monaco, *.unit.test.ts)",
+            "type": "node",
+            "request": "launch",
+            "program": "${workspaceFolder}/node_modules/mocha/bin/_mocha",
+            "stopOnEntry": false,
+            "sourceMaps": true,
+            "args": [
+                "./out/test/**/*.unit.test.js",
+                "--require=out/test/unittests.js",
+                "--ui=tdd",
+                "--recursive",
+                "--colors",
+                //"--grep", "<suite name>",
+                "--timeout=300000",
+                "--fast"
+            ],
+            "outFiles": [
+                "${workspaceFolder}/out/**/*.js"
+            ],
+            "preLaunchTask": "Compile",
+            "skipFiles": ["<node_internals>/**"]
+        },
         {
             "name": "Functional Tests (without VS Code, *.functional.test.ts)",
             "type": "node",
diff --git a/news/3 Code Health/8564.md b/news/3 Code Health/8564.md
@@ -0,0 +1 @@
+Improve startup times of unit tests by optionally ignoring some bootstrapping required for `monaco` and `react` tests.
diff --git a/src/test/unittests.ts b/src/test/unittests.ts
@@ -57,10 +57,16 @@ const Module = require('module');
     };
 })();
 
-// nteract/transforms-full expects to run in the browser so we have to fake
-// parts of the browser here.
-setUpDomEnvironment();
+// Setting up DOM env and transpile is required for the react & monaco related tests.
+// However this takes around 40s to setup on Mac, hence slowing down testing/development.
+// Allowing ability to disable this (faster local development & testing, saving minutes).
+if (process.argv.indexOf('--fast') === -1) {
+    // nteract/transforms-full expects to run in the browser so we have to fake
+    // parts of the browser here.
+    setUpDomEnvironment();
+
+    // Also have to setup babel to get the monaco editor to work.
+    setupTranspile();
+}
 
-// Also have to setup babel to get the monaco editor to work.
-setupTranspile();
 initialize();

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Improve startup times of unit tests by optionally ignoring some bootstrapping required for `monaco` and `react` tests.