Merge pull request #3325 from Kilo-Org/mark/remove-file-based-tests

Removed old filebased ghostprovider tests

Author: markijbema

src/services/ghost/__tests__/GhostProvider.spec.ts

@@ -1,6 +1,4 @@
 import { describe, it, expect, beforeEach, vi } from "vitest"
-import * as fs from "node:fs"
-import * as path from "node:path"
 import { MockWorkspace } from "./MockWorkspace"
 import * as vscode from "vscode"
 import { GhostStreamingParser } from "../GhostStreamingParser"
@@ -123,107 +121,6 @@ describe("GhostProvider", () => {
 		return { testUri, context, mockDocument }
 	}
 
-	// Test cases directory for file-based tests
-	const TEST_CASES_DIR = path.join(__dirname, "__test_cases__")
-
-	// Helper function to run file-based tests
-	async function runFileBasedTest(testCaseName: string) {
-		const testCasePath = path.join(TEST_CASES_DIR, testCaseName)
-		const inputFilePath = path.join(testCasePath, "input.js")
-		const diffFilePath = path.join(testCasePath, "response.txt")
-		const expectedFilePath = path.join(testCasePath, "expected.js")
-
-		const initialContent = fs.readFileSync(inputFilePath, "utf8")
-		// Read the response file
-		const response = fs.readFileSync(diffFilePath, "utf8")
-		const expectedContent = fs.readFileSync(expectedFilePath, "utf8")
-
-		const { testUri, context, mockDocument } = await setupTestDocument(`${testCaseName}/input.js`, initialContent)
-		
-		// Parse and apply suggestions
-		streamingParser.initialize(context)
-		const parseResult = streamingParser.parseResponse(response, "", "")
-		
-		// Apply the changes if we have suggestions
-		if (parseResult.hasNewSuggestions) {
-			const fillInSuggestion = parseResult.suggestions.getFillInAtCursor()
-			if (fillInSuggestion) {
-				// For FIM (Fill-In-Middle) suggestions, reconstruct the full content
-				const newContent = fillInSuggestion.prefix + fillInSuggestion.text + fillInSuggestion.suffix
-				;(mockDocument as any).updateContent(newContent)
-			}
-		}
-
-		const finalContent = mockWorkspace.getDocumentContent(testUri)
-		// Compare the normalized content
-		const normalizedFinal = normalizeWhitespace(finalContent)
-		const normalizedExpected = normalizeWhitespace(expectedContent)
-
-		// For certain tests, we need special handling due to formatting differences
-		if (testCaseName === "complex-multi-group") {
-			// For complex-multi-group, normalize the function and comment order
-			const normalizedForComparison = (content: string) => {
-				// Remove all whitespace and normalize function declarations with comments
-				return content.replace(/\/\/\s*([^\n]+)\s*([a-zA-Z]+\([^)]*\))\s*{/g, "$2 { // $1").replace(/\s+/g, "")
-			}
-
-			const processedFinal = normalizedForComparison(normalizedFinal)
-			const processedExpected = normalizedForComparison(normalizedExpected)
-			expect(processedFinal).toBe(processedExpected)
-		} else if (testCaseName === "partial-mixed-operations") {
-			// For partial-mixed-operations, compare without whitespace
-			const strippedFinal = normalizedFinal.replace(/\s+/g, "")
-			const strippedExpected = normalizedExpected.replace(/\s+/g, "")
-			expect(strippedFinal).toBe(strippedExpected)
-		} else {
-			expect(normalizedFinal).toBe(normalizedExpected)
-		}
-	}
-
-	describe("File-based Suggestions", () => {
-		it("should apply a simple addition from files", async () => {
-			await runFileBasedTest("simple-addition")
-		})
-
-		it("should apply multiple line additions from files", async () => {
-			await runFileBasedTest("multiple-line-additions")
-		})
-
-		it("should apply line deletions from files", async () => {
-			await runFileBasedTest("line-deletions")
-		})
-
-		it("should apply mixed addition and deletion from files", async () => {
-			await runFileBasedTest("mixed-addition-deletion")
-		})
-
-		it("should handle empty diff response from files", async () => {
-			await runFileBasedTest("empty-diff-response")
-		})
-
-		it("should apply function rename and var to const changes from files", async () => {
-			await runFileBasedTest("function-rename-var-to-const")
-		})
-	})
-
-	describe("Sequential application", () => {
-		it("should handle an inverse individual application of mixed operations", async () => {
-			await runFileBasedTest("sequential-mixed-operations")
-		})
-
-		it("should handle sequential partial application of mixed operations", async () => {
-			await runFileBasedTest("partial-mixed-operations")
-		})
-
-		it("should handle random individual application of mixed operations", async () => {
-			await runFileBasedTest("random-mixed-operations")
-		})
-
-		it("should handle complex multi-group operations", async () => {
-			await runFileBasedTest("complex-multi-group")
-		})
-	})
-
 	describe("Error Handling", () => {
 		it("should handle empty diff responses", async () => {
 			const initialContent = `console.log('test');`