From fe565781b63fb0e2db789844f39b923bfafef2f3 Mon Sep 17 00:00:00 2001 From: Roo Code Date: Thu, 18 Dec 2025 13:49:32 +0000 Subject: [PATCH] fix: add hasNativeApiProtocol assertion in native tool calling tests --- apps/vscode-e2e/src/suite/tools/apply-diff-native.test.ts | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/apps/vscode-e2e/src/suite/tools/apply-diff-native.test.ts b/apps/vscode-e2e/src/suite/tools/apply-diff-native.test.ts index 9936390775..3d453bdf89 100644 --- a/apps/vscode-e2e/src/suite/tools/apply-diff-native.test.ts +++ b/apps/vscode-e2e/src/suite/tools/apply-diff-native.test.ts @@ -60,6 +60,14 @@ function assertNativeProtocolUsed(verification: NativeProtocolVerification, test `This indicates an API request was made.`, ) + // Check that native protocol was actually used (anthropic/openai format) + assert.strictEqual( + verification.hasNativeApiProtocol, + true, + `[${testName}] Native API protocol should be used. ` + + `Expected apiProtocol to be "anthropic" or "openai", but got: ${verification.apiProtocol}`, + ) + // Check that response doesn't contain XML tool tags assert.strictEqual( verification.responseIsNotXML,