fix: update tests to match new function signatures after merge

Author: daniel-lxs

src/api/providers/bedrock.ts

@@ -224,8 +224,8 @@ export class AwsBedrockHandler extends BaseProvider implements SingleCompletionH
 
 		if (this.options.awsUseApiKey && this.options.awsApiKey) {
 			// Use API key/token-based authentication if enabled and API key is set
-			;(clientConfig as any).token = { token: this.options.awsApiKey }
-			;(clientConfig as any).authSchemePreference = ["httpBearerAuth"] // Otherwise there's no end of credential problems.
+			clientConfig.token = { token: this.options.awsApiKey }
+			clientConfig.authSchemePreference = ["httpBearerAuth"] // Otherwise there's no end of credential problems.
 		} else if (this.options.awsUseProfile && this.options.awsProfile) {
 			// Use profile-based credentials if enabled and profile is set
 			clientConfig.credentials = fromIni({

src/core/mentions/__tests__/processUserContentMentions.spec.ts

@@ -52,6 +52,8 @@ describe("processUserContentMentions", () => {
 				mockFileContextTracker,
 				mockRooIgnoreController,
 				true,
+				true, // includeDiagnosticMessages
+				50, // maxDiagnosticMessages
 				100,
 			)
 		})
@@ -79,6 +81,8 @@ describe("processUserContentMentions", () => {
 				mockFileContextTracker,
 				mockRooIgnoreController,
 				true,
+				true, // includeDiagnosticMessages
+				50, // maxDiagnosticMessages
 				undefined,
 			)
 		})
@@ -107,6 +111,8 @@ describe("processUserContentMentions", () => {
 				mockFileContextTracker,
 				mockRooIgnoreController,
 				true,
+				true, // includeDiagnosticMessages
+				50, // maxDiagnosticMessages
 				-1,
 			)
 		})
@@ -309,6 +315,8 @@ describe("processUserContentMentions", () => {
 				mockFileContextTracker,
 				undefined,
 				true, // showRooIgnoredFiles should default to true
+				true, // includeDiagnosticMessages
+				50, // maxDiagnosticMessages
 				undefined,
 			)
 		})
@@ -336,6 +344,8 @@ describe("processUserContentMentions", () => {
 				mockFileContextTracker,
 				undefined,
 				false,
+				true, // includeDiagnosticMessages
+				50, // maxDiagnosticMessages
 				undefined,
 			)
 		})

src/integrations/misc/__tests__/extract-text-large-files.spec.ts

@@ -152,18 +152,15 @@ describe("extractTextFromFile - Large File Handling", () => {
 		expect(result).toContain("[File truncated: showing 500 of 10000 total lines")
 	})
 
-	it("should handle maxReadFileLine of 0 by not truncating", async () => {
+	it("should handle maxReadFileLine of 0 by throwing an error", async () => {
 		const fileContent = "Line 1\nLine 2\nLine 3"
 
 		mockedFs.readFile.mockResolvedValue(fileContent as any)
 
-		const result = await extractTextFromFile("/test/file.ts", 0)
-
-		// maxReadFileLine of 0 or negative means no limit
-		expect(result).toContain("1 | Line 1")
-		expect(result).toContain("2 | Line 2")
-		expect(result).toContain("3 | Line 3")
-		expect(result).not.toContain("[File truncated:")
+		// maxReadFileLine of 0 should throw an error
+		await expect(extractTextFromFile("/test/file.ts", 0)).rejects.toThrow(
+			"Invalid maxReadFileLine: 0. Must be a positive integer or -1 for unlimited.",
+		)
 	})
 
 	it("should handle negative maxReadFileLine by treating as undefined", async () => {