Skip to content

Commit 71baee2

Browse files
committed
Update test
1 parent 7b17dff commit 71baee2

1 file changed

Lines changed: 6 additions & 11 deletions

File tree

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

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -150,15 +150,19 @@ suite('PythonEnvironmentApiImpl - capability queries', () => {
150150
assert.ok(envProbe.notCalled);
151151
});
152152

153-
test('missing environment manager is unavailable immediately and can be queried after registration', async () => {
153+
test('availability tracks the registration lifecycle rather than caching a stale result', async () => {
154154
const support = await api.getEnvironmentManagerCapability(ownerId, 'environments.list');
155155
assert.ok(!support.supported && support.reason.includes(ownerId));
156156
assert.ok(envProbe.notCalled);
157157
assert.strictEqual(clock.countTimers(), 0);
158-
registerOwner();
158+
const registration = registerOwner();
159159
assert.deepStrictEqual(await api.getEnvironmentManagerCapability(ownerId, 'environments.list'), {
160160
supported: true,
161161
});
162+
envProbe.resetHistory();
163+
registration.dispose();
164+
assert.strictEqual((await api.getEnvironmentManagerCapability(ownerId, 'environments.list')).supported, false);
165+
assert.ok(envProbe.notCalled);
162166
});
163167

164168
test('provider probe failures reach the public caller', async () => {
@@ -262,13 +266,4 @@ suite('PythonEnvironmentApiImpl - capability queries', () => {
262266
sinon.stub(managers, 'getEnvironmentManager').throws(failure);
263267
await assert.rejects(api.getEnvironmentManagerCapability(ownerId, 'environments.list'), (e) => e === failure);
264268
});
265-
266-
test('unregistered environment managers are unavailable rather than cached', async () => {
267-
const registration = registerOwner();
268-
assert.strictEqual((await api.getEnvironmentManagerCapability(ownerId, 'environments.list')).supported, true);
269-
envProbe.resetHistory();
270-
registration.dispose();
271-
assert.strictEqual((await api.getEnvironmentManagerCapability(ownerId, 'environments.list')).supported, false);
272-
assert.ok(envProbe.notCalled);
273-
});
274269
});

0 commit comments

Comments
 (0)