Repository navigation
Commit f91e19c
committed
test(webview): remove the no-explicit-any suppression from the parallelMode spec
Address the CodeRabbit maintainability finding: drop the blanket
no-explicit-any suppression (137) for ClineProvider.parallelMode.spec.ts
and type the spec properly instead.
- Private member access moves from (provider as any).x to bracket
notation (provider["x"]); public members (saveViewState, setValue,
setValues, handleModeSwitch, resolveWebviewView, log) drop the cast
entirely and keep their native generics.
- MockContextProxy now takes vscode.ExtensionContext; memento and mock
callbacks use unknown instead of any; the webview structural double is
cast once as unknown as vscode.WebviewView.
- Key/value casts are removed where the key is a valid RooCodeSettings
key; the one genuine exception (apiConfiguration is a GlobalState key
outside the proxy's generic) keeps a documented double assertion.
- api-configuration.spec.ts: document the as-unknown-as-ClineProvider
structural double in the new test (API.getConfiguration only reads
sidebarProvider.getValues).
check-types clean; parallelMode 49/49 and api-configuration 3/3 green;
eslint --prune-suppressions clean with the parallelMode entry removed
from eslint-suppressions.json and every other count unchanged.1 parent 8c86718 commit f91e19c
3 files changed
Lines changed: 166 additions & 165 deletions
File tree
- src
- core/webview/__tests__
- extension/__tests__
0 commit comments