Rename decodeBqrsToExternalApiUsages => decodeBqrsToMethods

Author: robertbrignull

extensions/ql-vscode/src/model-editor/bqrs.ts

@@ -3,9 +3,7 @@ import { Call, CallClassification, Method } from "./method";
 import { ModeledMethodType } from "./modeled-method";
 import { parseLibraryFilename } from "./library";
 
-export function decodeBqrsToExternalApiUsages(
-  chunk: DecodedBqrsChunk,
-): Method[] {
+export function decodeBqrsToMethods(chunk: DecodedBqrsChunk): Method[] {
   const methodsByApiName = new Map<string, Method>();
 
   chunk?.tuples.forEach((tuple) => {

extensions/ql-vscode/src/model-editor/external-api-usage-queries.ts

@@ -15,7 +15,7 @@ import { QueryLanguage } from "../common/query-language";
 import { fetchExternalApiQueries } from "./queries";
 import { Method } from "./method";
 import { runQuery } from "../local-queries/run-query";
-import { decodeBqrsToExternalApiUsages } from "./bqrs";
+import { decodeBqrsToMethods } from "./bqrs";
 
 type RunQueryOptions = {
   cliServer: CodeQLCliServer;
@@ -132,7 +132,7 @@ export async function runExternalApiQueries(
     maxStep: externalApiQueriesProgressMaxStep,
   });
 
-  return decodeBqrsToExternalApiUsages(bqrsChunk);
+  return decodeBqrsToMethods(bqrsChunk);
 }
 
 type GetResultsOptions = {

extensions/ql-vscode/test/unit-tests/model-editor/bqrs.test.ts

@@ -1,8 +1,8 @@
-import { decodeBqrsToExternalApiUsages } from "../../../src/model-editor/bqrs";
+import { decodeBqrsToMethods } from "../../../src/model-editor/bqrs";
 import { DecodedBqrsChunk } from "../../../src/common/bqrs-cli-types";
 import { CallClassification } from "../../../src/model-editor/method";
 
-describe("decodeBqrsToExternalApiUsages", () => {
+describe("decodeBqrsToMethods", () => {
   const chunk: DecodedBqrsChunk = {
     columns: [
       { name: "usage", kind: "Entity" },
@@ -230,11 +230,11 @@ describe("decodeBqrsToExternalApiUsages", () => {
     ],
   };
 
-  it("extracts api usages", () => {
-    // Even though there are a number of usages with the same number of usages, the order returned should be stable:
+  it("extracts methods", () => {
+    // Even though there are a number of methods with the same number of usages, the order returned should be stable:
     // - Iterating over a map (as done by .values()) is guaranteed to be in insertion order
     // - Sorting the array of usages is guaranteed to be a stable sort
-    expect(decodeBqrsToExternalApiUsages(chunk)).toEqual([
+    expect(decodeBqrsToMethods(chunk)).toEqual([
       {
         library: "rt",
         libraryVersion: undefined,