Skip to content

Commit 44aeaf7

Browse files
committed
Allow createMockVariantAnalysisHistoryItem to receive named params
Similarly, we want to provide params selectively to test sorting. We're also setting some defaults to play nicely with our current tests.
1 parent dd55b7e commit 44aeaf7

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

extensions/ql-vscode/src/vscode-tests/factories/remote-queries/variant-analysis-history-item.ts

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,21 @@ import { QueryStatus } from '../../../query-status';
33
import { VariantAnalysisStatus } from '../../../remote-queries/shared/variant-analysis';
44
import { createMockVariantAnalysis } from './shared/variant-analysis';
55

6-
export function createMockVariantAnalysisHistoryItem(
7-
historyItemStatus: QueryStatus = QueryStatus.InProgress,
8-
variantAnalysisStatus: VariantAnalysisStatus = VariantAnalysisStatus.Succeeded,
9-
failureReason?: string,
6+
export function createMockVariantAnalysisHistoryItem({
7+
historyItemStatus = QueryStatus.InProgress,
8+
variantAnalysisStatus = VariantAnalysisStatus.Succeeded,
9+
failureReason = undefined,
10+
resultCount = 0,
11+
userSpecifiedLabel = 'query-name',
12+
executionStartTime = undefined
13+
}: {
14+
historyItemStatus?: QueryStatus,
15+
variantAnalysisStatus?: VariantAnalysisStatus,
16+
failureReason?: string | undefined,
1017
resultCount?: number,
11-
userSpecifiedLabel?: string
12-
): VariantAnalysisHistoryItem {
18+
userSpecifiedLabel?: string,
19+
executionStartTime?: number
20+
}): VariantAnalysisHistoryItem {
1321
return ({
1422
t: 'variant-analysis',
1523
failureReason,

0 commit comments

Comments
 (0)