Fixed task hang issue for vstest task (#9309)

Author: ShreyasRmsft

Tasks/VsTestV2/cieventlogger.ts

@@ -15,9 +15,15 @@ function getDefaultProps() {
 export function publishEvent(properties: { [key: string]: any }): void {
     try {
         tl.assertAgent('2.125.0');
-        tl.publishTelemetry(area, feature, Object.assign(getDefaultProps(), properties));
+        publishTelemetry(area, feature, Object.assign(getDefaultProps(), properties));
 
     } catch (err) {
         tl.debug('Unable to publish telemetry due to lower agent version.');
     }
+}
+
+export function publishTelemetry(area: string, feature: string, properties: { [key: string]: any }): void {
+    const data = JSON.stringify(properties);
+    tl.debug('telemetry area: ' + area + ' feature: ' + feature + ' data: ' + data);
+    tl.command('telemetry.publish', { 'area': area, 'feature': feature }, data);
 }

Tasks/VsTestV2/distributedtest.ts

@@ -29,15 +29,15 @@ export class DistributedTest {
 
             if (exitCode !== 0) {
                 tl.debug('Modules/DTAExecutionHost.exe process exited with code ' + exitCode);
-                tl.setResult(tl.TaskResult.Failed, 'Modules/DTAExecutionHost.exe process exited with code ' + exitCode);
+                tl.setResult(tl.TaskResult.Failed, 'Modules/DTAExecutionHost.exe process exited with code ' + exitCode, true);
             } else {
                 tl.debug('Modules/DTAExecutionHost.exe exited');
-                tl.setResult(tl.TaskResult.Succeeded, 'Task succeeded');
+                tl.setResult(tl.TaskResult.Succeeded, 'Task succeeded', true);
             }
         } catch (error) {
             ci.publishEvent({ environmenturi: this.inputDataContract.RunIdentifier, error: error });
             tl.error(error);
-            tl.setResult(tl.TaskResult.Failed, error);
+            tl.setResult(tl.TaskResult.Failed, error, true);
         }
     }
 

Tasks/VsTestV2/nondistributedtest.ts

@@ -36,19 +36,19 @@ export class NonDistributedTest {
 
             if (exitCode !== 0 && !this.inputDataContract.ExecutionSettings.IgnoreTestFailures) {
                 tl.debug('Modules/DTAExecutionHost.exe process exited with code ' + exitCode);
-                tl.setResult(tl.TaskResult.Failed, tl.loc('VstestFailed'));
+                tl.setResult(tl.TaskResult.Failed, tl.loc('VstestFailed'), true);
                 return;
             } else {
                 if (exitCode !== 0) {
                     console.log('Task marked as success because IgnoreTestFailures is enabled');
                 }
                 tl.debug(`Modules/DTAExecutionHost.exe exited with code ${exitCode}`);
-                tl.setResult(tl.TaskResult.Succeeded, 'Task succeeded');
+                tl.setResult(tl.TaskResult.Succeeded, 'Task succeeded', true);
             }
 
         } catch (err) {
             tl.error(err);
-            tl.setResult(tl.TaskResult.Failed, tl.loc('VstestFailedReturnCode'));
+            tl.setResult(tl.TaskResult.Failed, tl.loc('VstestFailedReturnCode'), true);
         }
     }
 

Tasks/VsTestV2/package-lock.json

@@ -19,7 +19,7 @@
     "string": "3.3.1",
     "uuid": "3.1.0",
     "vso-node-api": "6.0.4-preview",
-    "vsts-task-lib": "2.0.6",
+    "vsts-task-lib": "2.7.0",
     "xml2js": "0.4.16",
     "@types/node": "^6.0.101",
     "@types/q": "^1.5.0"

Tasks/VsTestV2/package.json

@@ -17,7 +17,7 @@
     "version": {
         "Major": 2,
         "Minor": 144,
-        "Patch": 11
+        "Patch": 12
     },
     "demands": [
         "vstest"

Tasks/VsTestV2/task.json

@@ -17,7 +17,7 @@
   "version": {
     "Major": 2,
     "Minor": 144,
-    "Patch": 11
+    "Patch": 12
   },
   "demands": [
     "vstest"