Convert handleShowQueryText to use createSingleSelectionCommand

Author: robertbrignull

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

@@ -268,7 +268,10 @@ export class QueryHistoryManager extends DisposableObject {
         this.handleShowQueryLog.bind(this),
         "query",
       ),
-      "codeQLQueryHistory.showQueryText": this.handleShowQueryText.bind(this),
+      "codeQLQueryHistory.showQueryText": createSingleSelectionCommand(
+        this.handleShowQueryText.bind(this),
+        "query",
+      ),
       "codeQLQueryHistory.openQueryDirectory": createSingleSelectionCommand(
         this.handleOpenQueryDirectory.bind(this),
         "query",
@@ -811,31 +814,20 @@ export class QueryHistoryManager extends DisposableObject {
     await Promise.all(results);
   }
 
-  async handleShowQueryText(
-    singleItem: QueryHistoryInfo,
-    multiSelect: QueryHistoryInfo[] = [],
-  ) {
-    if (!this.assertSingleQuery(multiSelect)) {
-      return;
-    }
-
-    if (singleItem.t === "variant-analysis") {
-      await this.variantAnalysisManager.openQueryText(
-        singleItem.variantAnalysis.id,
-      );
+  async handleShowQueryText(item: QueryHistoryInfo) {
+    if (item.t === "variant-analysis") {
+      await this.variantAnalysisManager.openQueryText(item.variantAnalysis.id);
       return;
     }
 
     const params = new URLSearchParams({
       isQuickEval: String(
-        !!(
-          singleItem.t === "local" && singleItem.initialInfo.quickEvalPosition
-        ),
+        !!(item.t === "local" && item.initialInfo.quickEvalPosition),
       ),
-      queryText: encodeURIComponent(getQueryText(singleItem)),
+      queryText: encodeURIComponent(getQueryText(item)),
     });
 
-    const queryId = getQueryId(singleItem);
+    const queryId = getQueryId(item);
 
     const uri = Uri.parse(`codeql:${queryId}.ql?${params.toString()}`, true);
     const doc = await workspace.openTextDocument(uri);

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

@@ -183,7 +183,7 @@ describe("Variant Analyses and QueryHistoryManager", () => {
 
   it("should get the query text", async () => {
     await qhm.readQueryHistory();
-    await qhm.handleShowQueryText(qhm.treeDataProvider.allHistory[0], []);
+    await qhm.handleShowQueryText(qhm.treeDataProvider.allHistory[0]);
 
     expect(openQueryTextSpy).toHaveBeenCalledWith(
       rawQueryHistory[0].variantAnalysis.id,