End tests reliabily on CI (microsoft#9348)

Kartik Raj · web-flow · commit 38122f340f06 · 2019-12-31T14:48:17.000-08:00
* End tests reliabily on CI

* Code reviews
diff --git a/src/test/debuggerTest.ts b/src/test/debuggerTest.ts
@@ -19,6 +19,9 @@ function start() {
         extensionTestsPath: path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'out', 'test', 'index'),
         launchArgs: [workspacePath],
         version: 'stable'
-    }).catch(console.error);
+    }).catch(ex => {
+        console.error('End Debugger tests (with errors)', ex);
+        process.exit(1);
+    });
 }
 start();
diff --git a/src/test/multiRootTest.ts b/src/test/multiRootTest.ts
@@ -15,6 +15,9 @@ function start() {
         extensionTestsPath: path.join(EXTENSION_ROOT_DIR_FOR_TESTS, 'out', 'test', 'index'),
         launchArgs: [workspacePath],
         version: 'stable'
-    }).catch(console.error);
+    }).catch(ex => {
+        console.error('End Multiroot tests (with errors)', ex);
+        process.exit(1);
+    });
 }
 start();
diff --git a/src/test/standardTest.ts b/src/test/standardTest.ts
@@ -18,8 +18,7 @@ function start() {
         launchArgs: [workspacePath],
         version: 'stable'
     }).catch(ex => {
-        console.error('End Standard tests (with errors)');
-        console.error(ex);
+        console.error('End Standard tests (with errors)', ex);
         if (process.env.IS_SMOKE_TEST) {
             process.exit(1);
         }
diff --git a/src/test/testBootstrap.ts b/src/test/testBootstrap.ts
@@ -108,4 +108,7 @@ async function start() {
     proc.once('close', end);
 }
 
-start().catch(ex => console.error(ex));
+start().catch(ex => {
+    console.error('File testBootstrap.ts failed with Errors', ex);
+    process.exit(1);
+});

Original file line number	Diff line number	Diff line change
`@@ -108,4 +108,7 @@ async function start() {`
`108`	`108`	`proc.once('close', end);`
`109`	`109`	`}`
`110`	`110`
`111`		`-start().catch(ex => console.error(ex));`
	`111`	`+start().catch(ex => {`
	`112`	`+ console.error('File testBootstrap.ts failed with Errors', ex);`
	`113`	`+ process.exit(1);`
	`114`	`+});`