Add start time to VariantAnalysis type (#1587)
This commit is contained in:
@@ -14,6 +14,7 @@ export interface VariantAnalysis {
|
||||
repositoryLists?: string[],
|
||||
repositoryOwners?: string[],
|
||||
},
|
||||
executionStartTime: number;
|
||||
status: VariantAnalysisStatus,
|
||||
actionsWorkflowRunId?: number,
|
||||
failureReason?: VariantAnalysisFailureReason,
|
||||
|
||||
@@ -184,7 +184,8 @@ export class VariantAnalysisManager extends DisposableObject implements VariantA
|
||||
filePath: `variant_analysis_${variantAnalysisId}.ql`,
|
||||
language: variantAnalysisResponse.query_language as VariantAnalysisQueryLanguage,
|
||||
},
|
||||
databases: {}
|
||||
databases: {},
|
||||
executionStartTime: 0,
|
||||
}, variantAnalysisResponse);
|
||||
|
||||
void commands.executeCommand('codeQL.openVariantAnalysisView', processedVariantAnalysis.id);
|
||||
|
||||
@@ -30,11 +30,12 @@ export function processVariantAnalysis(
|
||||
language: submission.query.language
|
||||
},
|
||||
databases: submission.databases,
|
||||
executionStartTime: submission.startTime
|
||||
}, response);
|
||||
}
|
||||
|
||||
export function processUpdatedVariantAnalysis(
|
||||
previousVariantAnalysis: Pick<VariantAnalysis, 'query' | 'databases'>,
|
||||
previousVariantAnalysis: Pick<VariantAnalysis, 'query' | 'databases' | 'executionStartTime'>,
|
||||
response: ApiVariantAnalysis
|
||||
): VariantAnalysis {
|
||||
let scannedRepos: VariantAnalysisScannedRepository[] = [];
|
||||
@@ -57,6 +58,7 @@ export function processUpdatedVariantAnalysis(
|
||||
language: previousVariantAnalysis.query.language
|
||||
},
|
||||
databases: previousVariantAnalysis.databases,
|
||||
executionStartTime: previousVariantAnalysis.executionStartTime,
|
||||
status: processApiStatus(response.status),
|
||||
actionsWorkflowRunId: response.actions_workflow_run_id,
|
||||
scannedRepos: scannedRepos,
|
||||
|
||||
@@ -19,6 +19,7 @@ describe(VariantAnalysisAnalyzedRepos.name, () => {
|
||||
language: VariantAnalysisQueryLanguage.Javascript,
|
||||
},
|
||||
databases: {},
|
||||
executionStartTime: 1611234567890,
|
||||
status: VariantAnalysisStatus.InProgress,
|
||||
scannedRepos: [
|
||||
{
|
||||
|
||||
@@ -18,6 +18,7 @@ describe(VariantAnalysisOutcomePanels.name, () => {
|
||||
language: VariantAnalysisQueryLanguage.Javascript,
|
||||
},
|
||||
databases: {},
|
||||
executionStartTime: 1611234567890,
|
||||
status: VariantAnalysisStatus.InProgress,
|
||||
scannedRepos: [
|
||||
{
|
||||
|
||||
@@ -35,6 +35,7 @@ describe('Variant Analysis processor', function() {
|
||||
'databases': {
|
||||
'repositories': ['1', '2', '3'],
|
||||
},
|
||||
'executionStartTime': mockSubmission.startTime,
|
||||
'status': 'succeeded',
|
||||
'actionsWorkflowRunId': mockApiResponse.actions_workflow_run_id,
|
||||
'scannedRepos': [
|
||||
|
||||
@@ -25,6 +25,7 @@ export function createMockVariantAnalysis(
|
||||
databases: {
|
||||
repositories: ['1', '2', '3'],
|
||||
},
|
||||
executionStartTime: faker.datatype.number(),
|
||||
status: status,
|
||||
actionsWorkflowRunId: faker.datatype.number(),
|
||||
scannedRepos: scannedRepos,
|
||||
|
||||
Reference in New Issue
Block a user