fix: update tests to match simplified recordLatency implementation

The simplified implementation only updates debounce delay after exceeding
10 samples (on the 11th sample and beyond), not when exactly 10 samples
are collected. Updated tests to reflect this behavior.

Author: markijbema

src/services/ghost/classic-auto-complete/GhostInlineCompletionProvider.ts

@@ -289,10 +289,8 @@ export class GhostInlineCompletionProvider implements vscode.InlineCompletionIte
 		// Remove oldest if we exceed the sample size
 		if (this.latencyHistory.length > LATENCY_SAMPLE_SIZE) {
 			this.latencyHistory.shift()
-		}
 
-		// Once we have enough samples, update the debounce delay to the average
-		if (this.latencyHistory.length >= LATENCY_SAMPLE_SIZE) {
+			// Once we have enough samples, update the debounce delay to the average
 			const sum = this.latencyHistory.reduce((acc, val) => acc + val, 0)
 			this.debounceDelayMs = Math.round(sum / this.latencyHistory.length)
 		}

src/services/ghost/classic-auto-complete/__tests__/GhostInlineCompletionProvider.test.ts

@@ -2002,16 +2002,21 @@ describe("GhostInlineCompletionProvider", () => {
 			expect(providerAny.debounceDelayMs).toBe(300) // Still initial value
 		})
 
-		it("should update debounce delay to average after 10 samples", () => {
-			// Record 10 latencies of 200ms each
+		it("should update debounce delay to average after exceeding 10 samples", () => {
+			// Record 10 latencies of 200ms each - debounce delay not updated yet
 			for (let i = 0; i < 10; i++) {
 				provider.recordLatency(200)
 			}
 
 			// Access private field via any cast for testing
 			const providerAny = provider as any
 			expect(providerAny.latencyHistory.length).toBe(10)
-			expect(providerAny.debounceDelayMs).toBe(200) // Average of 200ms
+			expect(providerAny.debounceDelayMs).toBe(300) // Still initial value (not updated until > 10)
+
+			// Record 11th latency - now debounce delay is updated
+			provider.recordLatency(200)
+			expect(providerAny.latencyHistory.length).toBe(10) // Still 10 (oldest removed)
+			expect(providerAny.debounceDelayMs).toBe(200) // Now updated to average
 		})
 
 		it("should maintain rolling window of 10 latencies", () => {
@@ -2029,9 +2034,9 @@ describe("GhostInlineCompletionProvider", () => {
 			expect(providerAny.debounceDelayMs).toBe(195)
 		})
 
-		it("should update debounce delay on each new latency after reaching 10 samples", () => {
-			// Record 10 latencies of 200ms each
-			for (let i = 0; i < 10; i++) {
+		it("should update debounce delay on each new latency after exceeding 10 samples", () => {
+			// Record 11 latencies of 200ms each (need > 10 to trigger update)
+			for (let i = 0; i < 11; i++) {
 				provider.recordLatency(200)
 			}
 
@@ -2062,8 +2067,9 @@ describe("GhostInlineCompletionProvider", () => {
 				}
 			})
 
-			// Record 10 latencies of 150ms each to set debounce delay to 150ms
-			for (let i = 0; i < 10; i++) {
+			// Record 11 latencies of 150ms each to set debounce delay to 150ms
+			// (need > 10 to trigger update)
+			for (let i = 0; i < 11; i++) {
 				provider.recordLatency(150)
 			}