Skip to content

Commit 89359e3

Browse files
Use ID instead of query name in error message
Co-authored-by: Robert <[email protected]>
1 parent 09bae13 commit 89359e3

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

extensions/ql-vscode/src/remote-queries/variant-analysis-manager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -289,7 +289,7 @@ export class VariantAnalysisManager extends DisposableObject implements VariantA
289289
}
290290

291291
if (!variantAnalysis.actionsWorkflowRunId) {
292-
throw new Error(`No workflow run id for variant analysis ${variantAnalysis.query.name}`);
292+
throw new Error(`No workflow run id for variant analysis with id: ${variantAnalysis.id}`);
293293
}
294294

295295
const credentials = await Credentials.initialize(this.ctx);

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -574,7 +574,7 @@ describe('Variant Analysis Manager', async function() {
574574
try {
575575
await variantAnalysisManager.cancelVariantAnalysis(variantAnalysis.id);
576576
} catch (error: any) {
577-
expect(error.message).to.equal('No workflow run id for variant analysis a-query-name');
577+
expect(error.message).to.equal(`No workflow run id for variant analysis with id: ${variantAnalysis.id}`);
578578
}
579579
});
580580

0 commit comments

Comments
 (0)