From c315a02e60f22288027199ee60a4967cb1757aed Mon Sep 17 00:00:00 2001 From: Steven Nguyen Date: Fri, 11 Apr 2025 11:36:52 -0700 Subject: [PATCH] fix: don't set endpoint during build During build, the endpoint is invalid so calling setEndpoint() would result in an error: Invalid endpoint URL: undefined/v1 This PR ensures setEndpoint() isn't called during build. --- src/lib/stores/sdk.ts | 9 ++++++--- tests/unit/setup.ts | 6 ++---- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/src/lib/stores/sdk.ts b/src/lib/stores/sdk.ts index c4b3189dd..719ae04f2 100644 --- a/src/lib/stores/sdk.ts +++ b/src/lib/stores/sdk.ts @@ -24,6 +24,7 @@ import { import { Billing } from '../sdk/billing'; import { Backups } from '../sdk/backups'; import { Sources } from '$lib/sdk/sources'; +import { building } from '$app/environment'; export function getApiEndpoint(): string { if (VARS.APPWRITE_ENDPOINT) return VARS.APPWRITE_ENDPOINT; @@ -33,10 +34,12 @@ export function getApiEndpoint(): string { const endpoint = getApiEndpoint(); const clientConsole = new Client(); -clientConsole.setEndpoint(endpoint).setProject('console'); - const clientProject = new Client(); -clientProject.setEndpoint(endpoint).setMode('admin'); + +if (!building) { + clientConsole.setEndpoint(endpoint).setProject('console'); + clientProject.setEndpoint(endpoint).setMode('admin'); +} const sdkForProject = { client: clientProject, diff --git a/tests/unit/setup.ts b/tests/unit/setup.ts index 2b65f14eb..f5aeadd33 100644 --- a/tests/unit/setup.ts +++ b/tests/unit/setup.ts @@ -2,16 +2,14 @@ import '@testing-library/jest-dom/vitest'; import { beforeAll, vi } from 'vitest'; beforeAll(() => { - vi.mock('$app/environment', () => ({ - browser: true - })); vi.mock('$app/navigation', () => ({ goto: vi.fn(), beforeNavigate: vi.fn() })); vi.mock('$app/environment', () => ({ dev: true, - browser: true + browser: true, + building: true })); vi.mock('$env/static/public', () => import.meta.env); vi.mock('$env/dynamic/public', () => ({