fix: resolve CI test failures for extension tests

- Fix configuration property tests to handle package.json configuration
  as an array of sections instead of a single object
- Fix toggleTreeViewMode test to get fresh config object after toggle
- Fix performance test to create .vscode directory if it doesn't exist
- Fix addPartiallyAudited test to use non-zero end character to prevent
  line decrement behavior

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: fegge

test/extension/suite/decorations.test.ts

@@ -74,10 +74,12 @@ suite("Editor Decorations", () => {
         const relativePath = "src/sample.ts";
 
         // Use lines that are not already partially audited
+        // Note: When selection ends at character 0, the extension decrements endLine by 1
+        // So we set the end position with a non-zero character to preserve the exact line
         const startLine = 30;
         const endLine = 35;
         const start = new vscode.Position(startLine, 0);
-        const end = new vscode.Position(endLine, 0);
+        const end = new vscode.Position(endLine, 1);
         editor.selection = new vscode.Selection(start, end);
 
         // Get the initial state
@@ -126,9 +128,23 @@ suite("Editor Decorations", () => {
         assert.ok(extension, "Extension should be present");
 
         const packageJson = extension.packageJSON;
-        const configProps = packageJson?.contributes?.configuration?.properties;
+        const configuration = packageJson?.contributes?.configuration;
+
+        assert.ok(configuration, "Configuration should exist");
+
+        // Configuration can be an array or an object - combine all properties
+        let configProps: Record<string, unknown> = {};
+        if (Array.isArray(configuration)) {
+            for (const section of configuration) {
+                if (section.properties) {
+                    configProps = { ...configProps, ...section.properties };
+                }
+            }
+        } else if (configuration?.properties) {
+            configProps = configuration.properties;
+        }
 
-        assert.ok(configProps, "Configuration properties should exist");
+        assert.ok(Object.keys(configProps).length > 0, "Configuration properties should exist");
         assert.ok(configProps["weAudit.ownFindingColor"], "ownFindingColor should be configured");
         assert.ok(configProps["weAudit.otherFindingColor"], "otherFindingColor should be configured");
         assert.ok(configProps["weAudit.ownNoteColor"], "ownNoteColor should be configured");

test/extension/suite/performance.test.ts

@@ -89,6 +89,10 @@ suite("Performance", () => {
         const perfTestFile = path.join(vscodeDir, "perftest.weaudit");
 
         try {
+            // Ensure the .vscode directory exists
+            if (!fs.existsSync(vscodeDir)) {
+                fs.mkdirSync(vscodeDir, { recursive: true });
+            }
             fs.writeFileSync(perfTestFile, JSON.stringify(testData, null, 2));
 
             const startTime = Date.now();

test/extension/suite/treeViews.test.ts

@@ -43,15 +43,14 @@ suite("Tree View Integration", () => {
         this.timeout(10000);
 
         // Get the current mode from configuration
-        const config = vscode.workspace.getConfiguration("weAudit");
-        const modeBefore = config.get<string>("general.treeViewMode");
+        const modeBefore = vscode.workspace.getConfiguration("weAudit").get<string>("general.treeViewMode");
 
         // Toggle the mode
         await vscode.commands.executeCommand("weAudit.toggleTreeViewMode");
         await new Promise((resolve) => setTimeout(resolve, 300));
 
-        // Verify the configuration changed
-        const modeAfter = config.get<string>("general.treeViewMode");
+        // Get a fresh configuration object to see the updated value
+        const modeAfter = vscode.workspace.getConfiguration("weAudit").get<string>("general.treeViewMode");
         assert.notStrictEqual(modeAfter, modeBefore, "Tree view mode should change after toggle");
 
         // Verify it's one of the valid values

test/extension/suite/workspace.test.ts

@@ -29,9 +29,23 @@ suite("Workspace Operations", () => {
         assert.ok(extension, "Extension should be present");
 
         const packageJson = extension.packageJSON;
-        const configProps = packageJson?.contributes?.configuration?.properties;
+        const configuration = packageJson?.contributes?.configuration;
+
+        assert.ok(configuration, "Configuration should be defined");
+
+        // Configuration can be an array or an object - combine all properties
+        let configProps: Record<string, unknown> = {};
+        if (Array.isArray(configuration)) {
+            for (const section of configuration) {
+                if (section.properties) {
+                    configProps = { ...configProps, ...section.properties };
+                }
+            }
+        } else if (configuration?.properties) {
+            configProps = configuration.properties;
+        }
 
-        assert.ok(configProps, "Configuration properties should be defined");
+        assert.ok(Object.keys(configProps).length > 0, "Configuration properties should be defined");
 
         // Verify essential configuration properties exist
         const requiredProps = ["weAudit.general.treeViewMode", "weAudit.general.username"];