updated e2e

Author: JeffGepiga

src/main/services/ProjectManager.js

@@ -2544,6 +2544,7 @@ class ProjectManager {
   }
 
   async updateHostsFile(project) {
+    if (process.env.PLAYWRIGHT_TEST === 'true') return;
     // Add all project domains to hosts file
     const domainsToAdd = [];
 
@@ -3088,6 +3089,7 @@ server {
 
   // Add domain to hosts file (requires admin privileges)
   async addToHostsFile(domain) {
+    if (process.env.PLAYWRIGHT_TEST === 'true') return;
     if (!domain) return;
 
     // Security: Validate domain before using in any commands
@@ -3186,6 +3188,7 @@ server {
 
   // Remove domain from hosts file
   async removeFromHostsFile(domain) {
+    if (process.env.PLAYWRIGHT_TEST === 'true') return;
     if (!domain) return;
 
     const hostsPath = process.platform === 'win32'

src/main/utils/ConfigStore.js

@@ -6,9 +6,11 @@ const fs = require('fs-extra');
 class ConfigStore {
   constructor() {
     try {
+      const defaultData = this.getDefaults();
       this.store = new Store({
         name: 'devbox-pro-config',
-        defaults: this.getDefaults(),
+        defaults: defaultData,
+        cwd: defaultData.dataPath
       });
 
       // Ensure data directory exists
@@ -27,12 +29,19 @@ class ConfigStore {
   }
 
   getDefaults() {
-    const dataPath = path.join(os.homedir(), '.devbox-pro');
-
-    // Platform-specific default projects path
-    const defaultProjectsPath = process.platform === 'win32'
-      ? 'C:/Projects'
-      : path.join(os.homedir(), 'Projects');
+    let dataPath;
+    let defaultProjectsPath;
+
+    if (process.env.PLAYWRIGHT_TEST === 'true') {
+      const baseDir = process.env.TEST_USER_DATA_DIR || os.tmpdir();
+      dataPath = path.join(baseDir, '.devbox-pro-test');
+      defaultProjectsPath = path.join(dataPath, 'Projects');
+    } else {
+      dataPath = path.join(os.homedir(), '.devbox-pro');
+      defaultProjectsPath = process.platform === 'win32'
+        ? 'C:/Projects'
+        : path.join(os.homedir(), 'Projects');
+    }
 
     return {
       dataPath,

tests/e2e/fixtures.js

@@ -13,14 +13,18 @@ export const test = base.extend({
         // Create a temporary, random user data directory for true isolation
         const tempUserDataDir = path.join(os.tmpdir(), `devbox-e2e-${crypto.randomUUID()}`);
 
+        const envArgs = {
+            ...process.env,
+            NODE_ENV: 'production',
+            PLAYWRIGHT_TEST: 'true', // Optional flag for internal mocking if needed
+            TEST_USER_DATA_DIR: tempUserDataDir, // Explicitly pass to app
+        };
+        delete envArgs.ELECTRON_RUN_AS_NODE;
+
         // Launch Electron via Playwright
         const electronApp = await electron.launch({
             args: [mainEntry, '--user-data-dir', tempUserDataDir],
-            env: {
-                ...process.env,
-                NODE_ENV: 'production',
-                PLAYWRIGHT_TEST: 'true', // Optional flag for internal mocking if needed
-            }
+            env: envArgs
         });
 
         // Pass control back to the test