Merge pull request #294 from appwrite/test-improve-config

test: improve vite config
appwrite · Feb 14, 2023 · 99772f4 · 99772f4 · vercel · Feb 14, 2023
2 parents c2c752e + 7772703
commit 99772f4
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/tests/unit/setup.ts b/tests/unit/setup.ts
@@ -1,15 +1,10 @@
 import { vi } from 'vitest';
-import * as svelteinternal from 'svelte/internal';
 
 beforeAll(() => {
     vi.mock('$app/environment', () => ({
         browser: true
     }));
-    vi.mock('$app/stores', () => ({
-        page: null
-    }));
     vi.mock('$app/navigation', () => ({
         goto: vi.fn()
     }));
-    vi.mock('svelte', () => svelteinternal);
 });
diff --git a/vite.config.js b/vite.config.js
@@ -17,6 +17,14 @@ const config = {
     },
     server: {
         port: 3000
+    }
+};
+
+/** @type {import('vite').UserConfig} */
+const testConfig = {
+    resolve: {
+        // hotfix for https://github.com/vitest-dev/vitest/issues/2834
+        conditions: ['browser']
     },
     test: {
         include: ['tests/**/*.test.ts'],
@@ -30,4 +38,9 @@ const config = {
     }
 };
 
-export default config;
+export default process.env.VITEST
+    ? {
+          ...config,
+          ...testConfig
+      }
+    : config;