Skip to content

Commit f1b6c07

Browse files
edvilmeCopilot
andcommitted
Remove registry lookup error test that only verified default throw propagation
The test stubbed getEnvironmentManager to throw and asserted the error propagated through getEnvironmentManagerCapability, but the implementation has no try/catch around that call. The test confirmed default JS/TS throw semantics rather than any capability-specific behavior, and had no equivalent counterpart for the package-manager registry lookup path. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 39ba576 commit f1b6c07

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

‎src/test/extensionApi.capabilities.unit.test.ts‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -215,10 +215,4 @@ suite('PythonEnvironmentApiImpl - capability queries', () => {
215215
assert.ok(pkgProbe.notCalled);
216216
assert.ok(scopedProbe.notCalled);
217217
});
218-
219-
test('registry lookup errors reject', async () => {
220-
const failure = new Error('registry lookup failed');
221-
sinon.stub(managers, 'getEnvironmentManager').throws(failure);
222-
await assert.rejects(api.getEnvironmentManagerCapability(ownerId, 'environments.list'), (e) => e === failure);
223-
});
224218
});

0 commit comments

Comments
 (0)