Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2026-04-25 02:58:56 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #263623 from rwoll/fix-blockOnResponse

Browse Source 
skip flakey blockOnResponse tests
This commit is contained in:
Daniel Imms
2025-08-27 10:49:55 -07:00
committed by GitHub
parent 06de701485 87b4d2a251
commit 832ae085a4
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
extensions/vscode-api-tests/src/singlefolder-tests/chat.test.ts
View File

@@ -125,7 +125,10 @@ suite('chat', () => {
assert.strictEqual(request3.context.history.length, 2); // request + response = 2
});
test('workbench.action.chat.open.blockOnResponse defaults to non-blocking for backwards compatibility', async () => {
// fixme(rwoll): workbench.action.chat.open.blockOnResponse tests are flaking in CI:
// * https://github.com/microsoft/vscode/issues/263572
// * https://github.com/microsoft/vscode/issues/263575
test.skip('workbench.action.chat.open.blockOnResponse defaults to non-blocking for backwards compatibility', async () => {
const toolRegistration = lm.registerTool<void>('requires_confirmation_tool', {
invoke: async (_options, _token) => null, prepareInvocation: async (_options, _token) => {
return { invocationMessage: 'Invoking', pastTenseMessage: 'Invoked', confirmationMessages: { title: 'Confirm', message: 'Are you sure?' } };
@@ -146,7 +149,7 @@ suite('chat', () => {
assert.strictEqual(result, undefined);
});
test('workbench.action.chat.open.blockOnResponse resolves when waiting for user confirmation to run a tool', async () => {
test.skip('workbench.action.chat.open.blockOnResponse resolves when waiting for user confirmation to run a tool', async () => {
const toolRegistration = lm.registerTool<void>('requires_confirmation_tool', {
invoke: async (_options, _token) => null, prepareInvocation: async (_options, _token) => {
return { invocationMessage: 'Invoking', pastTenseMessage: 'Invoked', confirmationMessages: { title: 'Confirm', message: 'Are you sure?' } };
@@ -167,7 +170,7 @@ suite('chat', () => {
assert.strictEqual(result?.type, 'confirmation');
});
test('workbench.action.chat.open.blockOnResponse resolves when an error is hit', async () => {
test.skip('workbench.action.chat.open.blockOnResponse resolves when an error is hit', async () => {
const participant = chat.createChatParticipant('api-test.participant', async (_request, _context, _progress, _token) => {
return { errorDetails: { code: 'rate_limited', message: `You've been rate limited. Try again later!` } };
});