Rename: getCurrentRefreshPromise => waitForCurrentRefresh

Author: robertbrignull

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

@@ -19,7 +19,7 @@ export abstract class Discovery<T> extends DisposableObject {
    * Returns the promise of the currently running refresh operation, if one is in progress.
    * Otherwise returns a promise that resolves immediately.
    */
-  public getCurrentRefreshPromise(): Promise<void> {
+  public waitForCurrentRefresh(): Promise<void> {
     return this.currentDiscoveryPromise ?? Promise.resolve();
   }
 

extensions/ql-vscode/test/vscode-tests/minimal-workspace/queries-panel/query-discovery.test.ts

@@ -170,7 +170,7 @@ describe("QueryDiscovery", () => {
 
       onWatcherDidChangeEvent.fire(workspace.workspaceFolders![0].uri);
 
-      await discovery.getCurrentRefreshPromise();
+      await discovery.waitForCurrentRefresh();
 
       expect(onDidChangeQueriesSpy).toHaveBeenCalledTimes(2);
     });
@@ -200,7 +200,7 @@ describe("QueryDiscovery", () => {
 
       onDidChangeWorkspaceFoldersEvent.fire({ added: [], removed: [] });
 
-      await discovery.getCurrentRefreshPromise();
+      await discovery.waitForCurrentRefresh();
 
       expect(onDidChangeQueriesSpy).toHaveBeenCalledTimes(2);
     });