diff --git a/extensions/ql-vscode/src/remote-queries/analyses-results-manager.ts b/extensions/ql-vscode/src/remote-queries/analyses-results-manager.ts index d28e4ed27b3..5f82826158c 100644 --- a/extensions/ql-vscode/src/remote-queries/analyses-results-manager.ts +++ b/extensions/ql-vscode/src/remote-queries/analyses-results-manager.ts @@ -101,7 +101,7 @@ export class AnalysesResultsManager { const analysisResults: AnalysisResults = { nwo: analysis.nwo, status: 'InProgress', - results: [] + interpretedResults: [] }; const queryId = analysis.downloadLink.queryId; const resultsForQuery = this.internalGetAnalysesResults(queryId); @@ -123,7 +123,7 @@ export class AnalysesResultsManager { const queryResults = await this.readResults(artifactPath); newAnaysisResults = { ...analysisResults, - results: queryResults, + interpretedResults: queryResults, status: 'Completed' }; } else { diff --git a/extensions/ql-vscode/src/remote-queries/sample-data.ts b/extensions/ql-vscode/src/remote-queries/sample-data.ts index 21b8c267ff1..f0f85b8219c 100644 --- a/extensions/ql-vscode/src/remote-queries/sample-data.ts +++ b/extensions/ql-vscode/src/remote-queries/sample-data.ts @@ -99,7 +99,7 @@ export const sampleRemoteQueryResult: RemoteQueryResult = { }; -const createAnalysisResults = (n: number) => Array(n).fill( +const createAnalysisInterpretedResults = (n: number) => Array(n).fill( { message: 'This shell command depends on an uncontrolled [absolute path](1).', shortDescription: 'Shell command built from environment values', @@ -317,18 +317,18 @@ export const sampleAnalysesResultsStage1: AnalysisResults[] = [ { nwo: 'big-corp/repo1', status: 'InProgress', - results: [] + interpretedResults: [] }, { nwo: 'big-corp/repo2', status: 'InProgress', - results: [] + interpretedResults: [] }, { nwo: 'big-corp/repo3', status: 'InProgress', - results: [] + interpretedResults: [] }, // No entries for repo4 ]; @@ -337,22 +337,22 @@ export const sampleAnalysesResultsStage2: AnalysisResults[] = [ { nwo: 'big-corp/repo1', status: 'Completed', - results: createAnalysisResults(85) + interpretedResults: createAnalysisInterpretedResults(85) }, { nwo: 'big-corp/repo2', status: 'Completed', - results: createAnalysisResults(20) + interpretedResults: createAnalysisInterpretedResults(20) }, { nwo: 'big-corp/repo3', status: 'InProgress', - results: [] + interpretedResults: [] }, { nwo: 'big-corp/repo4', status: 'InProgress', - results: [] + interpretedResults: [] }, ]; @@ -360,22 +360,22 @@ export const sampleAnalysesResultsStage3: AnalysisResults[] = [ { nwo: 'big-corp/repo1', status: 'Completed', - results: createAnalysisResults(85) + interpretedResults: createAnalysisInterpretedResults(85) }, { nwo: 'big-corp/repo2', status: 'Completed', - results: createAnalysisResults(20) + interpretedResults: createAnalysisInterpretedResults(20) }, { nwo: 'big-corp/repo3', status: 'Completed', - results: createAnalysisResults(8) + interpretedResults: createAnalysisInterpretedResults(8) }, { nwo: 'big-corp/repo4', status: 'Completed', - results: createAnalysisResults(3) + interpretedResults: createAnalysisInterpretedResults(3) }, ]; @@ -383,21 +383,21 @@ export const sampleAnalysesResultsWithFailure: AnalysisResults[] = [ { nwo: 'big-corp/repo1', status: 'Completed', - results: createAnalysisResults(85) + interpretedResults: createAnalysisInterpretedResults(85) }, { nwo: 'big-corp/repo2', status: 'Completed', - results: createAnalysisResults(20) + interpretedResults: createAnalysisInterpretedResults(20) }, { nwo: 'big-corp/repo3', status: 'Failed', - results: [] + interpretedResults: [] }, { nwo: 'big-corp/repo4', status: 'Completed', - results: createAnalysisResults(3) + interpretedResults: createAnalysisInterpretedResults(3) }, ]; diff --git a/extensions/ql-vscode/src/remote-queries/shared/analysis-result.ts b/extensions/ql-vscode/src/remote-queries/shared/analysis-result.ts index 8a81ad3e1f5..ebcf63ae668 100644 --- a/extensions/ql-vscode/src/remote-queries/shared/analysis-result.ts +++ b/extensions/ql-vscode/src/remote-queries/shared/analysis-result.ts @@ -3,7 +3,7 @@ export type AnalysisResultStatus = 'InProgress' | 'Completed' | 'Failed'; export interface AnalysisResults { nwo: string; status: AnalysisResultStatus; - results: AnalysisAlert[]; + interpretedResults: AnalysisAlert[]; } export interface AnalysisAlert { diff --git a/extensions/ql-vscode/src/remote-queries/view/RemoteQueries.tsx b/extensions/ql-vscode/src/remote-queries/view/RemoteQueries.tsx index 44e7a3793cb..c1f49d83054 100644 --- a/extensions/ql-vscode/src/remote-queries/view/RemoteQueries.tsx +++ b/extensions/ql-vscode/src/remote-queries/view/RemoteQueries.tsx @@ -73,7 +73,7 @@ const openQueryTextVirtualFile = (queryResult: RemoteQueryResult) => { }; const sumAnalysesResults = (analysesResults: AnalysisResults[]) => - analysesResults.reduce((acc, curr) => acc + curr.results.length, 0); + analysesResults.reduce((acc, curr) => acc + curr.interpretedResults.length, 0); const QueryInfo = (queryResult: RemoteQueryResult) => ( <> @@ -264,13 +264,13 @@ const AnalysesResultsDescription = ({ totalAnalysesResults, totalResults }: { to const RepoAnalysisResults = (analysisResults: AnalysisResults) => { const title = <> {analysisResults.nwo} - + ; return (
    - {analysisResults.results.map((r, i) => + {analysisResults.interpretedResults.map((r, i) =>
  • @@ -297,7 +297,7 @@ const AnalysesResults = ({ analysesResults, totalResults }: { analysesResults: A totalAnalysesResults={totalAnalysesResults} totalResults={totalResults} />
      - {analysesResults.filter(a => a.results.length > 0).map(r => + {analysesResults.filter(a => a.interpretedResults.length > 0).map(r =>
    • )} diff --git a/extensions/ql-vscode/src/vscode-tests/no-workspace/remote-query-history.test.ts b/extensions/ql-vscode/src/vscode-tests/no-workspace/remote-query-history.test.ts index b8839ee38f9..3f0d12ea8e4 100644 --- a/extensions/ql-vscode/src/vscode-tests/no-workspace/remote-query-history.test.ts +++ b/extensions/ql-vscode/src/vscode-tests/no-workspace/remote-query-history.test.ts @@ -211,14 +211,14 @@ describe('Remote queries and query history manager', function() { expect(publisher.getCall(0).args[0][0]).to.include({ nwo: 'github/vscode-codeql', status: 'InProgress', - // results: ... avoid checking the results object since it is complex + // interpretedResults: ... avoid checking the interpretedResults object since it is complex }); // second time, it has the path to the sarif file. expect(publisher.getCall(1).args[0][0]).to.include({ nwo: 'github/vscode-codeql', status: 'Completed', - // results: ... avoid checking the results object since it is complex + // interpretedResults: ... avoid checking the interpretedResults object since it is complex }); expect(publisher).to.have.been.calledTwice; @@ -226,7 +226,7 @@ describe('Remote queries and query history manager', function() { expect(arm.getAnalysesResults(rawQueryHistory[0].queryId)[0]).to.include({ nwo: 'github/vscode-codeql', status: 'Completed', - // results: ... avoid checking the results object since it is complex + // interpretedResults: ... avoid checking the interpretedResults object since it is complex }); publisher.resetHistory(); @@ -242,7 +242,7 @@ describe('Remote queries and query history manager', function() { await arm.downloadAnalysesResults(analysisSummaries, undefined, publisher); const trimmed = publisher.getCalls().map(call => call.args[0]).map(args => { - args.forEach((analysisResult: any) => delete analysisResult.results); + args.forEach((analysisResult: any) => delete analysisResult.interpretedResults); return args; });