check if variable execResult is defined (#15012) (#15015)

zhenghao104 · mihairatoiu · web-flow · commit 8f1b221b7652 · 2021-07-01T17:27:33.000-07:00
Co-authored-by: Zheng Hao Tang &lt;zhentan@microsoft.com&gt;

Co-authored-by: mihairatoiu &lt;79838898+mihairatoiu@users.noreply.github.com&gt;
diff --git a/Tasks/UniversalPackagesV0/package.json b/Tasks/UniversalPackagesV0/package.json
@@ -1,6 +1,6 @@
 {
     "name": "universalpackages",
-    "version": "0.0.8",
+    "version": "0.0.9",
     "description": "Download or publish universal packages.",
     "main": "universalmain.js",
     "scripts": {
diff --git a/Tasks/UniversalPackagesV0/task.json b/Tasks/UniversalPackagesV0/task.json
@@ -9,7 +9,7 @@
     "category": "Package",
     "version": {
         "Major": 0,
-        "Minor": 188,
+        "Minor": 190,
         "Patch": 0
     },
     "runsOn": [
diff --git a/Tasks/UniversalPackagesV0/task.loc.json b/Tasks/UniversalPackagesV0/task.loc.json
@@ -9,7 +9,7 @@
   "category": "Package",
   "version": {
     "Major": 0,
-    "Minor": 188,
+    "Minor": 190,
     "Patch": 0
   },
   "runsOn": [
diff --git a/Tasks/UniversalPackagesV0/universalpublish.ts b/Tasks/UniversalPackagesV0/universalpublish.ts
@@ -149,7 +149,7 @@ export async function run(artifactToolPath: string): Promise<void> {
             retries++; 
         }
 
-        if (execResult.code === packageAlreadyExistsError) {
+        if (execResult != null && execResult.code === packageAlreadyExistsError) {
             telemetry.logResult("Packaging", "UniversalPackagesCommand", execResult.code);
             throw new Error(tl.loc("Error_UnexpectedErrorArtifactTool",
                 execResult.code,

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "universalpackages",`
`3`		`- "version": "0.0.8",`
	`3`	`+ "version": "0.0.9",`
`4`	`4`	`"description": "Download or publish universal packages.",`
`5`	`5`	`"main": "universalmain.js",`
`6`	`6`	`"scripts": {`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ export async function run(artifactToolPath: string): Promise<void> {`
`149`	`149`	`retries++;`
`150`	`150`	`}`
`151`	`151`
`152`		`- if (execResult.code === packageAlreadyExistsError) {`
	`152`	`+ if (execResult != null && execResult.code === packageAlreadyExistsError) {`
`153`	`153`	`telemetry.logResult("Packaging", "UniversalPackagesCommand", execResult.code);`
`154`	`154`	`throw new Error(tl.loc("Error_UnexpectedErrorArtifactTool",`
`155`	`155`	`execResult.code,`