refactor: Eliminate code duplication in test helpers

Removed duplicate createTempDocument() and deleteTempDocument()
functions from comparison test adapters by importing from the
shared test-helpers module.

Before:
- createTempDocument() defined 3 times (identical code)
- deleteTempDocument() defined 3 times (identical code)
- 66 lines of duplicated code

After:
- Both functions imported from src/test/test-helpers.ts
- Single source of truth for test document management
- 66 lines removed, DRY principle applied

All tests still passing with zero listener leak warnings:
- Main Extension: 226/226 passing
- Comparison Tests: 144/144 passing

Thanks to user for spotting this code smell!

Author: JohannesHoppe

comparison-test-harness/new-extension/adapter.ts

@@ -10,42 +10,11 @@
  * - Reference real code via relative paths (no copying!)
  */
 
-import { Uri, TextEdit, TextDocument, OutputChannel, workspace, WorkspaceEdit, commands } from 'vscode';
+import { Uri, TextEdit, TextDocument, OutputChannel, workspace, WorkspaceEdit } from 'vscode';
 import { ImportManager } from '../../src/imports/import-manager';
 import { ImportsConfig } from '../../src/configuration';
 import { ImportGroup, ImportGroupSettingParser, RemainImportGroup } from '../../src/imports/import-grouping';
-import * as fs from 'fs';
-import * as path from 'path';
-import * as os from 'os';
-
-/**
- * Create a REAL temporary file and open it as a TextDocument
- * This allows us to use workspace.applyEdit() which requires real files
- */
-async function createTempDocument(content: string): Promise<TextDocument> {
-  const tempDir = os.tmpdir();
-  const tempFile = path.join(tempDir, `test-${Date.now()}-${Math.random()}.ts`);
-  fs.writeFileSync(tempFile, content, 'utf-8');
-
-  const doc = await workspace.openTextDocument(Uri.file(tempFile));
-  return doc;
-}
-
-/**
- * Clean up temporary file after test completes
- * Closes the document in VSCode AND deletes the file to prevent listener leaks
- */
-async function deleteTempDocument(doc: TextDocument): Promise<void> {
-  try {
-    // Close the document in VSCode to release listeners
-    await commands.executeCommand('workbench.action.closeAllEditors');
-
-    // Delete the physical file
-    fs.unlinkSync(doc.uri.fsPath);
-  } catch (e) {
-    // Ignore errors - best effort cleanup
-  }
-}
+import { createTempDocument, deleteTempDocument } from '../../src/test/test-helpers';
 
 /**
  * Mock OutputChannel (same as in existing tests)

comparison-test-harness/old-extension/adapter.ts

@@ -12,46 +12,15 @@
 
 import 'reflect-metadata';
 import { TypescriptParser, TypescriptCodeGenerator, File, Generatable, GENERATORS, TypescriptGenerationOptions, MultiLineImportRule } from 'typescript-parser';
-import { Uri, TextDocument, TextEdit, window, TextEditor, workspace, WorkspaceEdit, commands } from 'vscode';
+import { Uri, TextDocument, TextEdit, window, TextEditor, workspace, WorkspaceEdit } from 'vscode';
 import { ImportManager } from '../old-typescript-hero/src/imports/import-manager';
 import { Configuration } from '../old-typescript-hero/src/configuration';
 import { ImportsConfig } from '../old-typescript-hero/src/configuration/imports-config';
 import { TypescriptCodeGeneratorFactory } from '../old-typescript-hero/src/ioc-symbols';
 import { getScriptKind } from '../old-typescript-hero/src/utilities/utility-functions';
 import { ImportGroupSettingParser } from '../old-typescript-hero/src/imports/import-grouping/import-group-setting-parser';
 import { ImportGroup, KeywordImportGroup, RegexImportGroup, RemainImportGroup } from '../old-typescript-hero/src/imports/import-grouping';
-import * as fs from 'fs';
-import * as path from 'path';
-import * as os from 'os';
-
-/**
- * Create a REAL temporary file and open it as a TextDocument
- * This allows us to use workspace.applyEdit() which requires real files
- */
-async function createTempDocument(content: string): Promise<TextDocument> {
-  const tempDir = os.tmpdir();
-  const tempFile = path.join(tempDir, `test-${Date.now()}-${Math.random()}.ts`);
-  fs.writeFileSync(tempFile, content, 'utf-8');
-
-  const doc = await workspace.openTextDocument(Uri.file(tempFile));
-  return doc;
-}
-
-/**
- * Clean up temporary file after test completes
- * Closes the document in VSCode AND deletes the file to prevent listener leaks
- */
-async function deleteTempDocument(doc: TextDocument): Promise<void> {
-  try {
-    // Close the document in VSCode to release listeners
-    await commands.executeCommand('workbench.action.closeAllEditors');
-
-    // Delete the physical file
-    fs.unlinkSync(doc.uri.fsPath);
-  } catch (e) {
-    // Ignore errors - best effort cleanup
-  }
-}
+import { createTempDocument, deleteTempDocument } from '../../src/test/test-helpers';
 
 /**
  * Mock Logger for old extension