- Notifications
You must be signed in to change notification settings - Fork34
Closed
Description
storage.ts
is a large, tightly coupled file that mixes core logic with directvscode
usage, making it hard to test. Split it into small, focused modules and add unit tests for the core logic using a thinvscode
adapter/mocks.
Proposal
- Extractcore storage logic (try to remove
vscode
imports or simply mock them in tests). - Introduce simple interfaces (e.g.,
BinaryManager
,CliConfigManager
,MementoManager
,SecretsManager
). - Write unit tests for core; adapter tests can use a minimal mock.
Metadata
Metadata
Assignees
Labels
No labels