Use new variant analysis statuses

The `status` on the variant analysis has changed to have `succeeded`,
`failed` and `cancelled` instead of `completed`.

Author: koesie10

extensions/ql-vscode/src/mocks/scenarios/problem-query-success/27-getVariantAnalysis.json

@@ -98,7 +98,7 @@
       "created_at": "2022-10-26T12:45:12Z",
       "updated_at": "2022-10-26T12:45:15Z",
       "actions_workflow_run_id": 3329095282,
-      "status": "completed",
+      "status": "succeeded",
       "completed_at": "2022-10-26T12:48:17Z",
       "scanned_repositories": [
         {

extensions/ql-vscode/src/mocks/scenarios/problem-query-warnings/51-getVariantAnalysis.json

@@ -98,7 +98,7 @@
       "created_at": "2022-10-27T07:42:59Z",
       "updated_at": "2022-10-27T07:43:02Z",
       "actions_workflow_run_id": 3335462873,
-      "status": "completed",
+      "status": "succeeded",
       "completed_at": "2022-10-27T08:18:47Z",
       "scanned_repositories": [
         {

extensions/ql-vscode/src/mocks/scenarios/query-failure/15-getVariantAnalysis.json

@@ -98,7 +98,7 @@
       "created_at": "2022-10-27T12:00:27Z",
       "updated_at": "2022-10-27T12:00:30Z",
       "actions_workflow_run_id": 3337181325,
-      "status": "completed",
+      "status": "succeeded",
       "completed_at": "2022-10-27T12:04:03Z",
       "scanned_repositories": [
         {

extensions/ql-vscode/src/remote-queries/gh-api/variant-analysis.ts

@@ -35,10 +35,13 @@ export interface VariantAnalysis {
 
 export type VariantAnalysisStatus =
   | 'in_progress'
-  | 'completed';
+  | 'succeeded'
+  | 'failed'
+  | 'cancelled';
 
 export type VariantAnalysisFailureReason =
   | 'no_repos_queried'
+  | 'actions_workflow_run_failed'
   | 'internal_error';
 
 export type VariantAnalysisRepoStatus =

extensions/ql-vscode/src/remote-queries/shared/variant-analysis.ts

@@ -56,6 +56,7 @@ export function isFinalVariantAnalysisStatus(status: VariantAnalysisStatus): boo
 
 export enum VariantAnalysisFailureReason {
   NoReposQueried = 'noReposQueried',
+  ActionsWorkflowRunFailed = 'actionsWorkflowRunFailed',
   InternalError = 'internalError',
 }
 

extensions/ql-vscode/src/remote-queries/variant-analysis-monitor.ts

@@ -4,8 +4,8 @@ import * as ghApiClient from './gh-api/gh-api-client';
 
 import {
   isFinalVariantAnalysisStatus,
+  repoHasDownloadableArtifact,
   VariantAnalysis,
-  VariantAnalysisRepoStatus,
   VariantAnalysisScannedRepository
 } from './shared/variant-analysis';
 import { VariantAnalysisMonitorResult } from './shared/variant-analysis-monitor-result';
@@ -61,7 +61,7 @@ export class VariantAnalysisMonitor extends DisposableObject {
       const downloadedRepos = this.downloadVariantAnalysisResults(variantAnalysis, scannedReposDownloaded);
       scannedReposDownloaded.push(...downloadedRepos);
 
-      if (isFinalVariantAnalysisStatus(variantAnalysis.status) || variantAnalysis.failureReason) {
+      if (isFinalVariantAnalysisStatus(variantAnalysis.status)) {
         break;
       }
 
@@ -82,7 +82,7 @@ export class VariantAnalysisMonitor extends DisposableObject {
     scannedRepo: VariantAnalysisScannedRepository,
     alreadyDownloaded: number[]
   ): boolean {
-    return !alreadyDownloaded.includes(scannedRepo.repository.id) && scannedRepo.analysisStatus === VariantAnalysisRepoStatus.Succeeded;
+    return !alreadyDownloaded.includes(scannedRepo.repository.id) && repoHasDownloadableArtifact(scannedRepo);
   }
 
   private getReposToDownload(

extensions/ql-vscode/src/remote-queries/variant-analysis-processor.ts

@@ -64,7 +64,7 @@ export function processUpdatedVariantAnalysis(
     executionStartTime: previousVariantAnalysis.executionStartTime,
     createdAt: response.created_at,
     updatedAt: response.updated_at,
-    status: processApiStatus(response.status, response.failure_reason),
+    status: processApiStatus(response.status),
     completedAt: response.completed_at,
     actionsWorkflowRunId: response.actions_workflow_run_id,
     scannedRepos: scannedRepos,
@@ -188,20 +188,26 @@ function processApiRepoStatus(analysisStatus: ApiVariantAnalysisRepoStatus): Var
   }
 }
 
-function processApiStatus(status: ApiVariantAnalysisStatus, failureReason: string | undefined): VariantAnalysisStatus {
-  if (status === 'in_progress') {
+function processApiStatus(status: ApiVariantAnalysisStatus): VariantAnalysisStatus {
+  if (status === 'succeeded') {
+    return VariantAnalysisStatus.Succeeded;
+  } else if (status === 'in_progress') {
     return VariantAnalysisStatus.InProgress;
-  } else if (failureReason !== undefined) {
+  } else if (status === 'failed') {
     return VariantAnalysisStatus.Failed;
+  } else if (status === 'cancelled') {
+    return VariantAnalysisStatus.Canceled;
   } else {
-    return VariantAnalysisStatus.Succeeded;
+    return VariantAnalysisStatus.InProgress;
   }
 }
 
 export function processFailureReason(failureReason: ApiVariantAnalysisFailureReason): VariantAnalysisFailureReason {
   switch (failureReason) {
     case 'no_repos_queried':
       return VariantAnalysisFailureReason.NoReposQueried;
+    case 'actions_workflow_run_failed':
+      return VariantAnalysisFailureReason.ActionsWorkflowRunFailed;
     case 'internal_error':
       return VariantAnalysisFailureReason.InternalError;
   }

extensions/ql-vscode/src/vscode-tests/cli-integration/remote-queries/variant-analysis-monitor.test.ts

@@ -125,7 +125,7 @@ describe('Variant Analysis Monitor', async function() {
       describe('when there are successfully scanned repos', async () => {
         beforeEach(async function() {
           scannedRepos = createMockScannedRepos(['pending', 'pending', 'in_progress', 'in_progress', 'succeeded', 'succeeded', 'succeeded']);
-          mockApiResponse = createMockApiResponse('completed', scannedRepos);
+          mockApiResponse = createMockApiResponse('succeeded', scannedRepos);
           mockGetVariantAnalysis = sandbox.stub(ghApiClient, 'getVariantAnalysis').resolves(mockApiResponse);
           succeededRepos = scannedRepos.filter(r => r.analysis_status === 'succeeded');
         });
@@ -190,7 +190,7 @@ describe('Variant Analysis Monitor', async function() {
       describe('when there are no repos to scan', async () => {
         beforeEach(async function() {
           scannedRepos = [];
-          mockApiResponse = createMockApiResponse('completed', scannedRepos);
+          mockApiResponse = createMockApiResponse('succeeded', scannedRepos);
           mockGetVariantAnalysis = sandbox.stub(ghApiClient, 'getVariantAnalysis').resolves(mockApiResponse);
         });
 

extensions/ql-vscode/src/vscode-tests/factories/remote-queries/gh-api/variant-analysis-api-response.ts

@@ -43,7 +43,7 @@ export function createFailedMockApiResponse(
   scannedRepos: VariantAnalysisScannedRepository[] = createMockScannedRepos(),
   skippedRepos: VariantAnalysisSkippedRepositories = createMockSkippedRepos(),
 ): VariantAnalysisApiResponse {
-  const variantAnalysis = createMockApiResponse('completed', scannedRepos, skippedRepos);
+  const variantAnalysis = createMockApiResponse('failed', scannedRepos, skippedRepos);
   variantAnalysis.failure_reason = 'internal_error';
   return variantAnalysis;
 }

extensions/ql-vscode/test/pure-tests/remote-queries/variant-analysis-processor.test.ts

@@ -31,7 +31,7 @@ import {
 describe(processVariantAnalysis.name, function() {
   const scannedRepos = createMockScannedRepos();
   const skippedRepos = createMockSkippedRepos();
-  const mockApiResponse = createMockApiResponse('completed', scannedRepos, skippedRepos);
+  const mockApiResponse = createMockApiResponse('succeeded', scannedRepos, skippedRepos);
   const mockSubmission = createMockSubmission();
 
   it('should process an API response and return a variant analysis', () => {