fix: cleanup

Author: qvalentin

package.json

@@ -81,7 +81,10 @@
     "languages": [
       {
         "id": "helm",
-        "aliases": ["helm-template", "helm"],
+        "aliases": [
+          "helm-template",
+          "helm"
+        ],
         "filenamePatterns": [
           "**/templates/*.yaml",
           "**/templates/*.yml",
@@ -101,8 +104,8 @@
     "compile-tests": "tsc -p . --outDir out",
     "watch-tests": "tsc -p . -w --outDir out",
     "pretest": "npm run compile-tests && npm run compile",
-    "lint": "ESLINT_USE_FLAT_CONFIG=false eslint src --ext ts",
-    "test": "vscode-test --extensionDevelopmentPath=. --extensionTestsPath=./out/test",
+    "lint": "eslint src --ext ts",
+    "test": "vscode-test",
     "release": "release-it"
   },
   "dependencies": {

src/test/extension.test.ts

@@ -6,7 +6,10 @@ import * as vscode from "vscode";
  * Helper function to wait for extension activation with timeout
  * Extension should activate automatically when Helm documents are opened
  */
-async function waitForExtensionActivation(extensionId: string, timeout: number = 10000): Promise<vscode.Extension<any>> {
+async function waitForExtensionActivation(
+  extensionId: string,
+  timeout: number = 1000,
+): Promise<vscode.Extension<any>> {
   const startTime = Date.now();
 
   while (Date.now() - startTime < timeout) {
@@ -15,112 +18,133 @@ async function waitForExtensionActivation(extensionId: string, timeout: number =
       return ext;
     }
 
-    await new Promise(resolve => setTimeout(resolve, 100));
+    await new Promise((resolve) => setTimeout(resolve, 100));
   }
 
-  throw new Error(`Extension ${extensionId} failed to activate within ${timeout}ms`);
+  throw new Error(
+    `Extension ${extensionId} failed to activate within ${timeout}ms`,
+  );
 }
 
 /**
  * Helper function to open a Helm template file and wait for language server
  */
-async function openHelmDocument(workspaceFolder: vscode.WorkspaceFolder): Promise<{ document: vscode.TextDocument; uri: vscode.Uri }> {
+async function openHelmDocument(
+  workspaceFolder: vscode.WorkspaceFolder,
+): Promise<{ document: vscode.TextDocument; uri: vscode.Uri }> {
   const docPath = path.join(
     workspaceFolder.uri.fsPath,
-    'src', 'test', 'fixtures', 'testChart', 'templates', 'deployment.yaml'
+    "src",
+    "test",
+    "fixtures",
+    "testChart",
+    "templates",
+    "deployment.yaml",
   );
   const docUri = vscode.Uri.file(docPath);
   const document = await vscode.workspace.openTextDocument(docUri);
   await vscode.window.showTextDocument(document);
 
   // Give language server a moment to initialize
-  await new Promise(resolve => setTimeout(resolve, 1000));
+  await new Promise((resolve) => setTimeout(resolve, 1000));
 
   return { document, uri: docUri };
 }
 
 suite("Extension Test Suite", () => {
-  vscode.window.showInformationMessage('Starting Helm LS extension tests');
+  vscode.window.showInformationMessage("Starting Helm LS extension tests");
 
   test("Extension loads and activates when opening Helm documents", async function () {
-    this.timeout(30000);
+    this.timeout(3000);
 
     const workspaceFolder = vscode.workspace.workspaceFolders?.[0];
-    assert.ok(workspaceFolder, 'No workspace folder found');
+    assert.ok(workspaceFolder, "No workspace folder found");
 
     // Extension should not be active initially
-    const extBefore = vscode.extensions.getExtension('helm-ls.helm-ls');
-    assert.ok(extBefore, 'Extension should be installed');
+    const extBefore = vscode.extensions.getExtension("helm-ls.helm-ls");
+    assert.ok(extBefore, "Extension should be installed");
 
     // Open a Helm file - this should trigger activation via onLanguage:helm
     await openHelmDocument(workspaceFolder);
 
     // Wait for extension to activate automatically
-    const ext = await waitForExtensionActivation('helm-ls.helm-ls');
-    assert.ok(ext.isActive, 'Extension should be activated after opening Helm document');
-    assert.strictEqual(ext.id, 'helm-ls.helm-ls');
+    const ext = await waitForExtensionActivation("helm-ls.helm-ls");
+    assert.ok(
+      ext.isActive,
+      "Extension should be activated after opening Helm document",
+    );
+    assert.strictEqual(ext.id, "helm-ls.helm-ls");
   });
 
   test("Hover support for Helm templates", async function () {
-    this.timeout(30000);
+    this.timeout(3000);
 
     const workspaceFolder = vscode.workspace.workspaceFolders?.[0];
-    assert.ok(workspaceFolder, 'No workspace folder found');
+    assert.ok(workspaceFolder, "No workspace folder found");
 
     // Ensure extension is activated
-    await waitForExtensionActivation('helm-ls.helm-ls');
+    await waitForExtensionActivation("helm-ls.helm-ls");
 
     const { uri: docUri } = await openHelmDocument(workspaceFolder);
 
     // Test hover on .Values.replicaCount (line 9, assuming it's around column 25)
     const helmPosition = new vscode.Position(8, 25);
     const helmHovers = await vscode.commands.executeCommand<vscode.Hover[]>(
-      'vscode.executeHoverProvider',
+      "vscode.executeHoverProvider",
       docUri,
-      helmPosition
+      helmPosition,
     );
 
-    assert.ok(helmHovers && helmHovers.length > 0, 'Should have hover information for Helm property');
+    assert.ok(
+      helmHovers && helmHovers.length > 0,
+      "Should have hover information for Helm property",
+    );
 
     const helmHoverContent = helmHovers[0].contents[0] as vscode.MarkdownString;
-    assert.ok(helmHoverContent, 'Hover content should exist');
+    assert.ok(helmHoverContent, "Hover content should exist");
 
     // Check for expected content (replicaCount value from values.yaml)
     assert.ok(
-      helmHoverContent.value.includes('1') || helmHoverContent.value.includes('replicaCount'),
-      `Hover should show replicaCount value or reference. Got: ${helmHoverContent.value}`
+      helmHoverContent.value.includes("1") ||
+        helmHoverContent.value.includes("replicaCount"),
+      `Hover should show replicaCount value or reference. Got: ${helmHoverContent.value}`,
     );
   });
 
   test("Hover support for YAML schema", async function () {
-    this.timeout(30000);
+    this.timeout(3000);
 
     const workspaceFolder = vscode.workspace.workspaceFolders?.[0];
-    assert.ok(workspaceFolder, 'No workspace folder found');
+    assert.ok(workspaceFolder, "No workspace folder found");
 
     // Ensure extension is activated
-    await waitForExtensionActivation('helm-ls.helm-ls');
+    await waitForExtensionActivation("helm-ls.helm-ls");
 
     const { uri: docUri } = await openHelmDocument(workspaceFolder);
 
     // Test hover on 'spec' property (line 7)
     const yamlPosition = new vscode.Position(6, 3);
     const yamlHovers = await vscode.commands.executeCommand<vscode.Hover[]>(
-      'vscode.executeHoverProvider',
+      "vscode.executeHoverProvider",
       docUri,
-      yamlPosition
+      yamlPosition,
     );
 
-    assert.ok(yamlHovers && yamlHovers.length > 0, 'Should have hover information for YAML property');
+    assert.ok(
+      yamlHovers && yamlHovers.length > 0,
+      "Should have hover information for YAML property",
+    );
 
     const yamlHoverContent = yamlHovers[0].contents[0] as vscode.MarkdownString;
-    assert.ok(yamlHoverContent, 'YAML hover content should exist');
+    assert.ok(yamlHoverContent, "YAML hover content should exist");
 
     // Check for Kubernetes documentation
     const content = yamlHoverContent.value.toLowerCase();
     assert.ok(
-      content.includes('deployment') || content.includes('spec') || content.includes('specification'),
-      `Hover should show Kubernetes schema info. Got: ${yamlHoverContent.value}`
+      content.includes("deployment") ||
+        content.includes("spec") ||
+        content.includes("specification"),
+      `Hover should show Kubernetes schema info. Got: ${yamlHoverContent.value}`,
     );
   });
 });