Skip to content

Commit 37117e1

Browse files
authored
Cleanup more unused methods (#4090)
1 parent b689e88 commit 37117e1

File tree

2 files changed

+1
-21
lines changed

2 files changed

+1
-21
lines changed

packages/cloud/src/CloudService.ts

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import * as vscode from "vscode"
22

3-
import type { CloudUserInfo, TelemetryEvent, OrganizationAllowList, OrganizationSettings } from "@roo-code/types"
3+
import type { CloudUserInfo, TelemetryEvent, OrganizationAllowList } from "@roo-code/types"
44
import { TelemetryService } from "@roo-code/telemetry"
55

66
import { CloudServiceCallbacks } from "./types"
@@ -84,23 +84,13 @@ export class CloudService {
8484
return this.authService!.getState()
8585
}
8686

87-
public getSessionToken(): string | undefined {
88-
this.ensureInitialized()
89-
return this.authService!.getSessionToken()
90-
}
91-
9287
public async handleAuthCallback(code: string | null, state: string | null): Promise<void> {
9388
this.ensureInitialized()
9489
return this.authService!.handleCallback(code, state)
9590
}
9691

9792
// SettingsService
9893

99-
public getOrganizationSettings(): OrganizationSettings | undefined {
100-
this.ensureInitialized()
101-
return this.settingsService!.getSettings()
102-
}
103-
10494
public getAllowList(): OrganizationAllowList {
10595
this.ensureInitialized()
10696
return this.settingsService!.getAllowList()

packages/cloud/src/__tests__/CloudService.test.ts

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -187,11 +187,6 @@ describe("CloudService", () => {
187187
expect(result).toBe("logged-out")
188188
})
189189

190-
it("should delegate getSessionToken to AuthService", () => {
191-
cloudService.getSessionToken()
192-
expect(mockAuthService.getSessionToken).toHaveBeenCalled()
193-
})
194-
195190
it("should delegate handleAuthCallback to AuthService", async () => {
196191
await cloudService.handleAuthCallback("code", "state")
197192
expect(mockAuthService.handleCallback).toHaveBeenCalledWith("code", "state")
@@ -205,11 +200,6 @@ describe("CloudService", () => {
205200
cloudService = await CloudService.createInstance(mockContext)
206201
})
207202

208-
it("should delegate getOrganizationSettings to SettingsService", () => {
209-
cloudService.getOrganizationSettings()
210-
expect(mockSettingsService.getSettings).toHaveBeenCalled()
211-
})
212-
213203
it("should delegate getAllowList to SettingsService", () => {
214204
cloudService.getAllowList()
215205
expect(mockSettingsService.getAllowList).toHaveBeenCalled()

0 commit comments

Comments
 (0)