Skip to content

Commit 04d7b12

Browse files
committed
Extract "getAnalysisResultCount" to helper file
1 parent 3e33b00 commit 04d7b12

2 files changed

Lines changed: 10 additions & 6 deletions

File tree

extensions/ql-vscode/src/pure/sarif-utils.ts

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as Sarif from 'sarif';
2-
import { HighlightedRegion } from '../remote-queries/shared/analysis-result';
2+
import { AnalysisResults, HighlightedRegion } from '../remote-queries/shared/analysis-result';
33
import { ResolvableLocationValue } from './bqrs-cli-types';
44

55
export interface SarifLink {
@@ -236,3 +236,11 @@ export function parseHighlightedLine(
236236

237237
return { plainSection1, highlightedSection, plainSection2 };
238238
}
239+
240+
/**
241+
* Returns the number of (raw + interpreted) results for an analysis.
242+
*/
243+
export const getAnalysisResultCount = (analysisResults: AnalysisResults): number => {
244+
const rawResultCount = analysisResults.rawResults?.resultSet.rows.length || 0;
245+
return analysisResults.interpretedResults.length + rawResultCount;
246+
};

extensions/ql-vscode/src/remote-queries/view/RemoteQueries.tsx

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ import { AlertIcon, CodeSquareIcon, FileCodeIcon, RepoIcon, TerminalIcon } from
2020
import AnalysisAlertResult from './AnalysisAlertResult';
2121
import RawResultsTable from './RawResultsTable';
2222
import RepositoriesSearch from './RepositoriesSearch';
23+
import { getAnalysisResultCount } from '../../pure/sarif-utils';
2324

2425
const numOfReposInContractedMode = 10;
2526

@@ -67,11 +68,6 @@ const openQueryTextVirtualFile = (queryResult: RemoteQueryResult) => {
6768
});
6869
};
6970

70-
const getAnalysisResultCount = (analysisResults: AnalysisResults): number => {
71-
const rawResultCount = analysisResults.rawResults?.resultSet.rows.length || 0;
72-
return analysisResults.interpretedResults.length + rawResultCount;
73-
};
74-
7571
const sumAnalysesResults = (analysesResults: AnalysisResults[]) =>
7672
analysesResults.reduce((acc, curr) => acc + getAnalysisResultCount(curr), 0);
7773

0 commit comments

Comments
 (0)