telemetry: add result field to metric #4171

      1) transformByQ
           "before all" hook for "WHEN upload payload with missing sha256 in headers THEN fails to upload":
         Error: Metric `codeTransform_jobCreateZipEndTime` was emitted without the `result` property. This property is always required. Consider using `.run()` instead of `.emit()`, which will set these properties automatically. See https://github.com/aws/aws-toolkit-vscode/blob/master/docs/telemetry.md#guidelines

Author: dhasani23

src/codewhisperer/service/transformByQHandler.ts

@@ -389,11 +389,14 @@ export async function zipCode(modulePath: string) {
         fs.rmSync(dependencyFolderPath, { recursive: true, force: true })
     }
 
+    // for now, use the pass/fail status of the maven command to determine this metric status
+    const mavenStatus = mavenFailed ? MetadataResult.Fail : MetadataResult.Pass
     telemetry.codeTransform_jobCreateZipEndTime.emit({
         codeTransformSessionId: codeTransformTelemetryState.getSessionId(),
         // TODO: A nice to have would be getting the zipUploadSize
         codeTransformTotalByteSize: 0,
         codeTransformRunTimeLatency: calculateTotalLatency(zipStartTime),
+        result: mavenStatus,
     })
     return tempFilePath
 }