Remove codeQL.copyVariantAnalysisRepoListQueryHistory command and instead call copyRepoListToClipboard directly

Author: robertbrignull

extensions/ql-vscode/src/common/commands.ts

@@ -244,10 +244,6 @@ export type VariantAnalysisCommands = {
     scannedRepo: VariantAnalysisScannedRepository,
     variantAnalysisSummary: VariantAnalysis,
   ) => Promise<void>;
-  "codeQL.copyVariantAnalysisRepoListQueryHistory": (
-    variantAnalysisId: number,
-    filterSort?: RepositoriesFilterSortStateWithIds,
-  ) => Promise<void>;
   "codeQL.copyVariantAnalysisRepoListView": (
     variantAnalysisId: number,
     filterSort?: RepositoriesFilterSortStateWithIds,

extensions/ql-vscode/src/query-history/query-history-manager.ts

@@ -952,8 +952,7 @@ export class QueryHistoryManager extends DisposableObject {
       return;
     }
 
-    await this.app.commands.execute(
-      "codeQL.copyVariantAnalysisRepoListQueryHistory",
+    await this.variantAnalysisManager.copyRepoListToClipboard(
       item.variantAnalysis.id,
     );
   }

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

@@ -148,8 +148,6 @@ export class VariantAnalysisManager
     return {
       "codeQL.autoDownloadVariantAnalysisResult":
         this.enqueueDownload.bind(this),
-      "codeQL.copyVariantAnalysisRepoListQueryHistory":
-        this.copyRepoListToClipboard.bind(this),
       "codeQL.copyVariantAnalysisRepoListView":
         this.copyRepoListToClipboard.bind(this),
       "codeQL.loadVariantAnalysisRepoResults": this.loadResults.bind(this),

extensions/ql-vscode/test/vscode-tests/no-workspace/query-history/query-history-manager.test.ts

@@ -710,11 +710,12 @@ describe("QueryHistoryManager", () => {
 
     it("should copy repo list for a single variant analysis", async () => {
       queryHistoryManager = await createMockQueryHistory(allHistory);
+      queryHistoryManager.handleCopyRepoList = jest.fn();
 
       const item = variantAnalysisHistory[1];
       await queryHistoryManager.handleCopyRepoList(item);
-      expect(executeCommand).toBeCalledWith(
-        "codeQL.copyVariantAnalysisRepoListQueryHistory",
+
+      expect(queryHistoryManager.handleCopyRepoList).toBeCalledWith(
         item.variantAnalysis.id,
       );
     });