Skip to content

Commit b14ebc6

Browse files
Remove ExtensionContext argument from VSCodeMockGitHubApiServer
1 parent a630663 commit b14ebc6

2 files changed

Lines changed: 9 additions & 18 deletions

File tree

extensions/ql-vscode/src/extension.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -879,7 +879,7 @@ async function activateWithInstalledDistribution(
879879
const summaryLanguageSupport = new SummaryLanguageSupport(app);
880880
ctx.subscriptions.push(summaryLanguageSupport);
881881

882-
const mockServer = new VSCodeMockGitHubApiServer(ctx, app);
882+
const mockServer = new VSCodeMockGitHubApiServer(app);
883883
ctx.subscriptions.push(mockServer);
884884

885885
void extLogger.log("Registering top-level command palette commands.");

extensions/ql-vscode/src/mocks/vscode-mock-gh-api-server.ts

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,5 @@
11
import { pathExists } from "fs-extra";
2-
import {
3-
env,
4-
ExtensionContext,
5-
ExtensionMode,
6-
QuickPickItem,
7-
Uri,
8-
window,
9-
} from "vscode";
2+
import { env, QuickPickItem, Uri, window } from "vscode";
103

114
import {
125
getMockGitHubApiServerScenariosPath,
@@ -15,7 +8,8 @@ import {
158
import { DisposableObject } from "../pure/disposable-object";
169
import { MockGitHubApiServer } from "./mock-gh-api-server";
1710
import { MockGitHubApiServerCommands } from "../common/commands";
18-
import { App } from "../common/app";
11+
import { App, AppMode } from "../common/app";
12+
import path from "path";
1913

2014
/**
2115
* "Interface" to the mock GitHub API server which implements VSCode interactions, such as
@@ -27,10 +21,7 @@ export class VSCodeMockGitHubApiServer extends DisposableObject {
2721
private readonly server: MockGitHubApiServer;
2822
private readonly config: MockGitHubApiConfigListener;
2923

30-
constructor(
31-
private readonly ctx: ExtensionContext,
32-
private readonly app: App,
33-
) {
24+
constructor(private readonly app: App) {
3425
super();
3526
this.server = new MockGitHubApiServer();
3627
this.config = new MockGitHubApiConfigListener();
@@ -228,11 +219,11 @@ export class VSCodeMockGitHubApiServer extends DisposableObject {
228219
return scenariosPath;
229220
}
230221

231-
if (this.ctx.extensionMode === ExtensionMode.Development) {
232-
const developmentScenariosPath = Uri.joinPath(
233-
this.ctx.extensionUri,
222+
if (this.app.mode === AppMode.Development) {
223+
const developmentScenariosPath = path.join(
224+
this.app.extensionPath,
234225
"src/mocks/scenarios",
235-
).fsPath.toString();
226+
);
236227
if (await pathExists(developmentScenariosPath)) {
237228
return developmentScenariosPath;
238229
}

0 commit comments

Comments
 (0)