Skip to content

Commit c4b1134

Browse files
Rename variantAnalysis to be more specific
1 parent 1ec3d04 commit c4b1134

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

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

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ describe('Variant Analysis Manager', async function() {
2727
let cli: CodeQLCliServer;
2828
let cancellationTokenSource: CancellationTokenSource;
2929
let variantAnalysisManager: VariantAnalysisManager;
30-
let variantAnalysis: VariantAnalysisApiResponse;
30+
let variantAnalysisApiResponse: VariantAnalysisApiResponse;
3131
let scannedRepos: ApiVariantAnalysisScannedRepository[];
3232
let getVariantAnalysisRepoStub: sinon.SinonStub;
3333
let getVariantAnalysisRepoResultStub: sinon.SinonStub;
@@ -43,7 +43,7 @@ describe('Variant Analysis Manager', async function() {
4343
cancellationTokenSource = new CancellationTokenSource();
4444

4545
scannedRepos = createMockScannedRepos();
46-
variantAnalysis = createMockApiResponse('in_progress', scannedRepos);
46+
variantAnalysisApiResponse = createMockApiResponse('in_progress', scannedRepos);
4747

4848
try {
4949
const extension = await extensions.getExtension<CodeQLExtensionInterface | Record<string, never>>('GitHub.vscode-codeql')!.activate();
@@ -66,7 +66,7 @@ describe('Variant Analysis Manager', async function() {
6666
try {
6767
await variantAnalysisManager.autoDownloadVariantAnalysisResult(
6868
scannedRepos[0],
69-
variantAnalysis,
69+
variantAnalysisApiResponse,
7070
cancellationTokenSource.token
7171
);
7272
} catch (error: any) {
@@ -103,7 +103,7 @@ describe('Variant Analysis Manager', async function() {
103103
it('should not try to download the result', async () => {
104104
await variantAnalysisManager.autoDownloadVariantAnalysisResult(
105105
scannedRepos[0],
106-
variantAnalysis,
106+
variantAnalysisApiResponse,
107107
cancellationTokenSource.token
108108
);
109109

@@ -126,7 +126,7 @@ describe('Variant Analysis Manager', async function() {
126126

127127
await variantAnalysisManager.autoDownloadVariantAnalysisResult(
128128
scannedRepos[0],
129-
variantAnalysis,
129+
variantAnalysisApiResponse,
130130
cancellationTokenSource.token
131131
);
132132

@@ -136,7 +136,7 @@ describe('Variant Analysis Manager', async function() {
136136
it('should fetch a repo task', async () => {
137137
await variantAnalysisManager.autoDownloadVariantAnalysisResult(
138138
scannedRepos[0],
139-
variantAnalysis,
139+
variantAnalysisApiResponse,
140140
cancellationTokenSource.token
141141
);
142142

@@ -146,7 +146,7 @@ describe('Variant Analysis Manager', async function() {
146146
it('should fetch a repo result', async () => {
147147
await variantAnalysisManager.autoDownloadVariantAnalysisResult(
148148
scannedRepos[0],
149-
variantAnalysis,
149+
variantAnalysisApiResponse,
150150
cancellationTokenSource.token
151151
);
152152

@@ -156,9 +156,9 @@ describe('Variant Analysis Manager', async function() {
156156
it('should pop download tasks off the queue', async () => {
157157
const getResultsSpy = sandbox.spy(variantAnalysisManager, 'autoDownloadVariantAnalysisResult');
158158

159-
await variantAnalysisManager.enqueueDownload(scannedRepos[0], variantAnalysis, cancellationTokenSource.token);
160-
await variantAnalysisManager.enqueueDownload(scannedRepos[1], variantAnalysis, cancellationTokenSource.token);
161-
await variantAnalysisManager.enqueueDownload(scannedRepos[2], variantAnalysis, cancellationTokenSource.token);
159+
await variantAnalysisManager.enqueueDownload(scannedRepos[0], variantAnalysisApiResponse, cancellationTokenSource.token);
160+
await variantAnalysisManager.enqueueDownload(scannedRepos[1], variantAnalysisApiResponse, cancellationTokenSource.token);
161+
await variantAnalysisManager.enqueueDownload(scannedRepos[2], variantAnalysisApiResponse, cancellationTokenSource.token);
162162

163163
expect(variantAnalysisManager.downloadsQueueSize()).to.equal(0);
164164
expect(getResultsSpy).to.have.been.calledThrice;

0 commit comments

Comments
 (0)