feat(test): Adds comprehensive test of SVN submission information

- Add multiple test cases to verify parsing of SVN commit information, including search for specific revisions, case-insensitive revision prefix processing, and the ability to extract multi-line submission information from improved formats.
- Handle tests on Chinese file names and contents to ensure correct parsing under different encoding conditions.
- Enhance the processing of SVN command output to ensure that submission information and differences can be displayed correctly on different platforms.

Author: ubuntutester202

src/utils/__tests__/svn.spec.ts

@@ -3,6 +3,7 @@ import { promises as fs } from "fs"
 import { exec } from "child_process"
 import { promisify } from "util"
 import * as path from "path"
+import * as os from "os"
 
 // Use vi.hoisted to ensure the mock is available at the right time
 const { mockExecAsync } = vi.hoisted(() => ({
@@ -25,9 +26,9 @@ vi.mock("vscode", () => ({
 			appendLine: vi.fn(),
 			show: vi.fn(),
 		})),
-		showErrorMessage: vi.fn(),
-		showWarningMessage: vi.fn(),
-		showInformationMessage: vi.fn(),
+		showErrorMessage: vi.fn(() => Promise.resolve("Show Output")),
+		showWarningMessage: vi.fn(() => Promise.resolve()),
+		showInformationMessage: vi.fn(() => Promise.resolve()),
 	},
 	env: {
 		openExternal: vi.fn(),
@@ -190,6 +191,82 @@ Working Copy Root Path: /test/workspace`,
 			expect(result[0].message).toBe("Test commit message")
 		})
 
+		it("should search for specific revision when query is in 'r123' format", async () => {
+			// Mock checkSvnInstalled to return true
+			mockExecAsync.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" }).mockResolvedValueOnce({
+				stdout: `<?xml version="1.0" encoding="UTF-8"?>
+<log>
+<logentry revision="456">
+<author>jane.smith</author>
+<date>2023-02-15T14:30:00.000000Z</date>
+<msg>Specific revision commit</msg>
+</logentry>
+</log>`,
+				stderr: "",
+			})
+
+			// Mock checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			const result = await searchSvnCommits("r456", "/test/workspace")
+			expect(result).toHaveLength(1)
+			expect(result[0].revision).toBe("456")
+			expect(result[0].author).toBe("jane.smith")
+			expect(result[0].message).toBe("Specific revision commit")
+		})
+
+		it("should handle case-insensitive 'r' prefix in revision search", async () => {
+			// Mock checkSvnInstalled to return true
+			mockExecAsync.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" }).mockResolvedValueOnce({
+				stdout: `<?xml version="1.0" encoding="UTF-8"?>
+<log>
+<logentry revision="789">
+<author>test.user</author>
+<date>2023-03-15T16:30:00.000000Z</date>
+<msg>Case insensitive test</msg>
+</logentry>
+</log>`,
+				stderr: "",
+			})
+
+			// Mock checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			const result = await searchSvnCommits("R789", "/test/workspace")
+			expect(result).toHaveLength(1)
+			expect(result[0].revision).toBe("789")
+		})
+
+		it("should NOT treat pure numbers as revision searches", async () => {
+			// Mock checkSvnInstalled to return true
+			mockExecAsync.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" }).mockResolvedValueOnce({
+				stdout: `<?xml version="1.0" encoding="UTF-8"?>
+<log>
+<logentry revision="123">
+<author>john.doe</author>
+<date>2023-01-15T10:30:00.000000Z</date>
+<msg>Message containing 456 number</msg>
+</logentry>
+<logentry revision="456">
+<author>jane.smith</author>
+<date>2023-02-15T14:30:00.000000Z</date>
+<msg>Another commit</msg>
+</logentry>
+</log>`,
+				stderr: "",
+			})
+
+			// Mock checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			// Search for pure number "456" should search in message content, not as specific revision
+			const result = await searchSvnCommits("456", "/test/workspace")
+			// Should find the commit with "456" in the message, NOT the commit with revision 456
+			expect(result).toHaveLength(1)
+			expect(result[0].revision).toBe("123")
+			expect(result[0].message).toContain("456")
+		})
+
 		it("should return empty array when SVN is not available", async () => {
 			mockExecAsync.mockRejectedValue(new Error("Command not found"))
 
@@ -204,6 +281,8 @@ Working Copy Root Path: /test/workspace`,
 			mockExecAsync.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" }).mockResolvedValueOnce({
 				stdout: `------------------------------------------------------------------------
 r123 | john.doe | 2023-01-15 10:30:00 +0000 (Sun, 15 Jan 2023) | 1 line
+Changed paths:
+   M /test.txt
 
 Test commit message
 ------------------------------------------------------------------------`,
@@ -218,13 +297,269 @@ Test commit message
 			expect(result).toContain("Test commit message")
 		})
 
+		it("should parse changed files information correctly", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo
+			mockFsAccess.mockResolvedValue(undefined)
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r456 | jane.smith | 2023-02-15 14:30:00 +0800 (Wed, 15 Feb 2023) | 2 lines
+Changed paths:
+   A /src/new-file.ts
+   M /src/existing-file.ts
+   D /src/old-file.ts
+
+Added new feature
+Fixed bug in existing functionality
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({
+					stdout: `Index: src/new-file.ts
+===================================================================
+--- src/new-file.ts	(nonexistent)
++++ src/new-file.ts	(revision 456)
+@@ -0,0 +1,3 @@
++export function newFunction() {
++  return 'Hello World';
++}`,
+					stderr: "",
+				})
+
+			const result = await getSvnCommitInfoForMentions("456", "/test/workspace")
+
+			// Should contain basic commit info
+			expect(result).toContain("r456 by jane.smith")
+			expect(result).toContain("Added new feature")
+			expect(result).toContain("Fixed bug in existing functionality")
+
+			// Should contain changed files section
+			expect(result).toContain("Changed files:")
+			expect(result).toContain("A /src/new-file.ts")
+			expect(result).toContain("M /src/existing-file.ts")
+			expect(result).toContain("D /src/old-file.ts")
+
+			// Should contain diff section
+			expect(result).toContain("Diff:")
+			expect(result).toContain("export function newFunction()")
+		})
+
+		it("should handle date parsing with Chinese day names gracefully", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo
+			mockFsAccess.mockResolvedValue(undefined)
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r789 | chinese.user | 2023-03-15 16:30:00 +0800 (星期三, 15 三月 2023) | 1 line
+Changed paths:
+   M /中文文件.txt
+
+测试中文提交信息
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({
+					stdout: `Index: 中文文件.txt
+===================================================================
+--- 中文文件.txt	(revision 788)
++++ 中文文件.txt	(revision 789)
+@@ -1 +1 @@
+-旧内容
++新内容`,
+					stderr: "",
+				})
+
+			const result = await getSvnCommitInfoForMentions("789", "/test/workspace")
+
+			// Should contain commit info with extracted date
+			expect(result).toContain("r789 by chinese.user")
+			expect(result).toContain("2023-03-15 16:30:00 +0800")
+			expect(result).toContain("测试中文提交信息")
+
+			// Should handle Chinese file names and content
+			expect(result).toContain("M /中文文件.txt")
+			expect(result).toContain("新内容")
+		})
+
+		it("should handle commit message extraction from improved format", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo
+			mockFsAccess.mockResolvedValue(undefined)
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r100 | developer | 2023-01-01 12:00:00 +0000 (Sun, 01 Jan 2023) | 3 lines
+Changed paths:
+   M /src/component.ts
+
+This is a multi-line commit message
+with detailed explanation
+and some additional notes
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({
+					stdout: "",
+					stderr: "",
+				})
+
+			const result = await getSvnCommitInfoForMentions("100", "/test/workspace")
+
+			// Should extract multi-line commit message correctly
+			expect(result).toContain("This is a multi-line commit message")
+			expect(result).toContain("with detailed explanation")
+			expect(result).toContain("and some additional notes")
+		})
+
+		it("should handle revision with 'r' prefix input", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo
+			mockFsAccess.mockResolvedValue(undefined)
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r555 | test.author | 2023-05-15 10:15:00 +0000 (Mon, 15 May 2023) | 1 line
+Changed paths:
+   M /test-file.txt
+
+Test with r prefix input
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({
+					stdout: "",
+					stderr: "",
+				})
+
+			const result = await getSvnCommitInfoForMentions("r555", "/test/workspace")
+			expect(result).toContain("r555 by test.author")
+			expect(result).toContain("Test with r prefix input")
+		})
+
 		it("should return error message for invalid revision", async () => {
 			// Mock checkSvnInstalled to fail
 			mockExecAsync.mockRejectedValue(new Error("Command not found"))
 
 			const result = await getSvnCommitInfoForMentions("invalid", "/test/workspace")
 			expect(result).toBe("Error: SVN not available or not an SVN repository")
 		})
+
+		it("should handle diff output with UTF-8 encoding", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			// Create a buffer with UTF-8 encoded Chinese characters
+			const utf8Buffer = Buffer.from("测试文件内容", "utf8")
+
+			// Mock the svn log and diff commands
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r123 | john.doe | 2023-01-15 10:30:00 +0000 (Sun, 15 Jan 2023) | 1 line
+Changed paths:
+   M /test.txt
+
+Test commit message
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({ stdout: utf8Buffer, stderr: "" })
+
+			const result = await getSvnCommitInfoForMentions("123", "/test/workspace")
+			expect(result).toContain("r123")
+			expect(result).toContain("john.doe")
+			expect(result).toContain("Test commit message")
+			expect(result).toContain("测试文件内容")
+		})
+
+		it("should handle diff output with problematic encoding", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			// Create a buffer with problematic encoding (contains replacement characters)
+			const problematicBuffer = Buffer.from("Test file with � characters", "utf8")
+
+			// Mock the svn log and diff commands
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r123 | john.doe | 2023-01-15 10:30:00 +0000 (Sun, 15 Jan 2023) | 1 line
+Changed paths:
+   M /test.txt
+
+Test commit message
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({ stdout: problematicBuffer, stderr: "" })
+
+			const result = await getSvnCommitInfoForMentions("123", "/test/workspace")
+			expect(result).toContain("r123")
+			expect(result).toContain("john.doe")
+			expect(result).toContain("Test commit message")
+			// Should handle the problematic encoding gracefully
+			expect(result).toContain("Test file with")
+		})
+
+		it("should handle string output from svn diff command", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			// Mock the svn log command with string output instead of Buffer
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r123 | john.doe | 2023-01-15 10:30:00 +0000 (Sun, 15 Jan 2023) | 1 line
+Changed paths:
+   M /test.txt
+
+Test commit message
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockResolvedValueOnce({
+					stdout: "Index: test.txt\n===================================================================\n--- test.txt\t(revision 122)\n+++ test.txt\t(revision 123)\n@@ -1 +1 @@\n-old content\n+new content",
+					stderr: "",
+				})
+
+			const result = await getSvnCommitInfoForMentions("123", "/test/workspace")
+			expect(result).toContain("r123")
+			expect(result).toContain("john.doe")
+			expect(result).toContain("Test commit message")
+			expect(result).toContain("new content")
+		})
+
+		it("should handle diff command failure gracefully", async () => {
+			// Mock checkSvnInstalled and checkSvnRepo to return true
+			mockFsAccess.mockResolvedValue(undefined)
+
+			// Mock the svn log command to succeed but diff to fail
+			mockExecAsync
+				.mockResolvedValueOnce({ stdout: "svn, version 1.14.0\n", stderr: "" })
+				.mockResolvedValueOnce({
+					stdout: `------------------------------------------------------------------------
+r123 | john.doe | 2023-01-15 10:30:00 +0000 (Sun, 15 Jan 2023) | 1 line
+Changed paths:
+   M /test.txt
+
+Test commit message
+------------------------------------------------------------------------`,
+					stderr: "",
+				})
+				.mockRejectedValueOnce(new Error("svn: E160013: File not found"))
+
+			const result = await getSvnCommitInfoForMentions("123", "/test/workspace")
+			expect(result).toContain("r123")
+			expect(result).toContain("john.doe")
+			expect(result).toContain("Test commit message")
+			// Should not contain diff section when diff fails
+			expect(result).not.toContain("Diff:")
+		})
 	})
 
 	describe("getWorkspaceSvnInfo", () => {