Fix Unit Tests (#497)

stevensJourney · web-flow · commit 2770df4330ad · 2025-02-20T13:35:50.000+02:00
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -40,6 +40,11 @@ jobs:
       - name: Install dependencies
         run: pnpm install
 
+      - name: Configure Playwright
+        run: |
+          pnpm exec playwright install-deps
+          pnpm exec playwright install
+
       - name: Build
         run: pnpm build:packages
 
diff --git a/README.md b/README.md
@@ -2,7 +2,7 @@
   <a href="https://www.powersync.com" target="_blank"><img src="https://github.com/powersync-ja/.github/assets/7372448/d2538c43-c1a0-4c47-9a76-41462dba484f"/></a>
 </p>
 
-*[PowerSync](https://www.powersync.com) is a sync engine for building local-first apps with instantly-responsive UI/UX and simplified state transfer. Syncs between SQLite on the client-side and Postgres, MongoDB or MySQL on the server-side.*
+_[PowerSync](https://www.powersync.com) is a sync engine for building local-first apps with instantly-responsive UI/UX and simplified state transfer. Syncs between SQLite on the client-side and Postgres, MongoDB or MySQL on the server-side._
 
 # PowerSync JavaScript SDKs
 
@@ -100,6 +100,15 @@ Build packages
 pnpm build:packages
 ```
 
+## Tests
+
+Some packages use [Playwright](https://www.npmjs.com/package/playwright) for testing. Install Playwright dependencies with:
+
+```bash
+pnpm exec playwright install-deps
+pnpm exec playwright install
+```
+
 ## Versioning
 
 ### Development Packages
diff --git a/package.json b/package.json
@@ -33,8 +33,12 @@
     "@actions/core": "^1.10.1",
     "@changesets/cli": "2.27.2",
     "@pnpm/workspace.find-packages": "^4.0.2",
+    "@vitest/browser": "^3.0.5",
     "husky": "^9.0.11",
     "lint-staged": "^15.2.2",
-    "prettier": "^3.2.5"
+    "playwright": "^1.50.1",
+    "prettier": "^3.2.5",
+    "typescript": "^5.7.2",
+    "vitest": "^3.0.5"  
   }
 }
diff --git a/packages/attachments/package.json b/packages/attachments/package.json
@@ -32,14 +32,9 @@
     "@powersync/common": "workspace:^1.18.1"
   },
   "devDependencies": {
+    "@powersync/common": "workspace:*",
     "@types/node": "^20.17.6",
-    "@vitest/browser": "^2.1.8",
-    "ts-loader": "^9.5.1",
-    "ts-node": "^10.9.2",
-    "typescript": "^5.7.2",
-    "vite": "^5.4.11",
-    "vite-plugin-top-level-await": "^1.4.4",
-    "vitest": "^2.1.8",
-    "webdriverio": "^9.4.5"
+    "vite": "^6.1.0",
+    "vite-plugin-top-level-await": "^1.4.4"
   }
 }
diff --git a/packages/attachments/vitest.config.ts b/packages/attachments/vitest.config.ts
@@ -10,8 +10,12 @@ const config: UserConfigExport = {
     browser: {
       enabled: true,
       headless: true,
-      provider: 'webdriverio',
-      name: 'chrome' // browser name is required
+      provider: 'playwright',
+      instances: [
+        {
+          browser: 'chromium'
+        }
+      ]
     }
   }
 };
diff --git a/packages/common/package.json b/packages/common/package.json
@@ -58,8 +58,6 @@
     "rsocket-core": "1.0.0-alpha.3",
     "rsocket-websocket-client": "1.0.0-alpha.3",
     "text-encoding": "^0.7.0",
-    "typescript": "^5.5.3",
-    "vitest": "^2.0.5",
     "web-streams-polyfill": "3.2.1"
   }
 }
diff --git a/packages/common/src/client/sync/stream/AbstractRemote.ts b/packages/common/src/client/sync/stream/AbstractRemote.ts
@@ -5,20 +5,21 @@ import { type fetch } from 'cross-fetch';
 import Logger, { ILogger } from 'js-logger';
 import { RSocket, RSocketConnector, Requestable } from 'rsocket-core';
 import { WebsocketClientTransport } from 'rsocket-websocket-client';
+import PACKAGE from '../../../../package.json' with { type: 'json' };
 import { AbortOperation } from '../../../utils/AbortOperation.js';
 import { DataStream } from '../../../utils/DataStream.js';
 import { PowerSyncCredentials } from '../../connection/PowerSyncCredentials.js';
 import { StreamingSyncLine, StreamingSyncRequest } from './streaming-sync-types.js';
 
-import { version as POWERSYNC_JS_VERSION } from '../../../../package.json';
-
 export type BSONImplementation = typeof BSON;
 
-const POWERSYNC_TRAILING_SLASH_MATCH = /\/+$/;
 export type RemoteConnector = {
   fetchCredentials: () => Promise<PowerSyncCredentials | null>;
 };
 
+const POWERSYNC_TRAILING_SLASH_MATCH = /\/+$/;
+const POWERSYNC_JS_VERSION = PACKAGE.version;
+
 // Refresh at least 30 sec before it expires
 const REFRESH_CREDENTIALS_SAFETY_PERIOD_MS = 30_000;
 const SYNC_QUEUE_REQUEST_LOW_WATER = 5;
diff --git a/packages/common/tsconfig.json b/packages/common/tsconfig.json
@@ -8,8 +8,8 @@
     "outDir": "./lib",
     "lib": ["esnext"],
     "declaration": true,
-    "module": "node16",
-    "moduleResolution": "node16",
+    "module": "NodeNext",
+    "moduleResolution": "nodenext",
     "preserveConstEnums": true,
     "esModuleInterop": false,
     "skipLibCheck": false,
diff --git a/packages/drizzle-driver/package.json b/packages/drizzle-driver/package.json
@@ -33,15 +33,9 @@
     "@powersync/web": "workspace:*",
     "@journeyapps/wa-sqlite": "^1.2.0",
     "@types/node": "^20.17.6",
-    "@vitest/browser": "^2.1.4",
     "drizzle-orm": "^0.35.2",
-    "ts-loader": "^9.5.1",
-    "ts-node": "^10.9.2",
-    "typescript": "^5.6.3",
-    "vite": "^5.4.10",
+    "vite": "^6.1.0",
     "vite-plugin-top-level-await": "^1.4.4",
-    "vite-plugin-wasm": "^3.3.0",
-    "vitest": "^2.1.4",
-    "webdriverio": "^9.2.8"
+    "vite-plugin-wasm": "^3.3.0"
   }
 }
diff --git a/packages/drizzle-driver/vitest.config.ts b/packages/drizzle-driver/vitest.config.ts
@@ -1,5 +1,5 @@
-import wasm from 'vite-plugin-wasm';
 import topLevelAwait from 'vite-plugin-top-level-await';
+import wasm from 'vite-plugin-wasm';
 import { defineConfig, UserConfigExport } from 'vitest/config';
 
 const config: UserConfigExport = {
@@ -20,8 +20,12 @@ const config: UserConfigExport = {
     browser: {
       enabled: true,
       headless: true,
-      provider: 'webdriverio',
-      name: 'chrome' // browser name is required
+      provider: 'playwright',
+      instances: [
+        {
+          browser: 'chromium'
+        }
+      ]
     }
   }
 };
diff --git a/packages/kysely-driver/package.json b/packages/kysely-driver/package.json
@@ -35,14 +35,8 @@
     "@powersync/web": "workspace:*",
     "@journeyapps/wa-sqlite": "^1.2.0",
     "@types/node": "^20.17.6",
-    "@vitest/browser": "^2.1.4",
-    "ts-loader": "^9.5.1",
-    "ts-node": "^10.9.2",
-    "typescript": "^5.6.3",
-    "vite": "^5.4.10",
+    "vite": "^6.1.0",
     "vite-plugin-top-level-await": "^1.4.4",
-    "vite-plugin-wasm": "^3.3.0",
-    "vitest": "^2.1.4",
-    "webdriverio": "^9.2.8"
+    "vite-plugin-wasm": "^3.3.0"
   }
 }
diff --git a/packages/kysely-driver/vitest.config.ts b/packages/kysely-driver/vitest.config.ts
@@ -1,5 +1,5 @@
-import wasm from 'vite-plugin-wasm';
 import topLevelAwait from 'vite-plugin-top-level-await';
+import wasm from 'vite-plugin-wasm';
 import { defineConfig, UserConfigExport } from 'vitest/config';
 
 const config: UserConfigExport = {
@@ -20,8 +20,12 @@ const config: UserConfigExport = {
     browser: {
       enabled: true,
       headless: true,
-      provider: 'webdriverio',
-      name: 'chrome' // browser name is required
+      provider: 'playwright',
+      instances: [
+        {
+          browser: 'chromium'
+        }
+      ]
     }
   }
 };
diff --git a/packages/powersync-op-sqlite/package.json b/packages/powersync-op-sqlite/package.json
@@ -87,8 +87,7 @@
     "react": "18.3.1",
     "react-native": "0.75.3",
     "react-native-builder-bob": "^0.30.2",
-    "turbo": "^1.10.7",
-    "typescript": "^5.2.2"
+    "turbo": "^1.10.7"
   },
   "eslintIgnore": [
     "node_modules/",
diff --git a/packages/react-native/package.json b/packages/react-native/package.json
@@ -62,7 +62,6 @@
     "react-native-fetch-api": "^3.0.0",
     "rollup": "4.14.3",
     "text-encoding": "^0.7.0",
-    "typescript": "^5.5.3",
     "web-streams-polyfill": "3.2.1"
   },
   "keywords": [
diff --git a/packages/react/package.json b/packages/react/package.json
@@ -33,11 +33,11 @@
     "react": "*"
   },
   "devDependencies": {
+    "@powersync/common": "workspace:*",
     "@testing-library/react": "^15.0.2",
     "@types/react": "^18.2.34",
     "jsdom": "^24.0.0",
     "react": "18.2.0",
-    "react-error-boundary": "^4.1.0",
-    "typescript": "^5.5.3"
+    "react-error-boundary": "^4.1.0"
   }
 }
diff --git a/packages/react/tests/useQuery.test.tsx b/packages/react/tests/useQuery.test.tsx
@@ -106,7 +106,7 @@ describe('useQuery', () => {
 
     await waitFor(
       async () => {
-        expect(result.current.error).toEqual(Error('PowerSync failed to fetch data: some error'));
+        expect(result.current.error?.message).equal('PowerSync failed to fetch data: some error');
       },
       { timeout: 100 }
     );
@@ -131,7 +131,7 @@ describe('useQuery', () => {
 
     await waitFor(
       async () => {
-        expect(result.current.error).toEqual(Error('PowerSync failed to fetch data: some error'));
+        expect(result.current.error?.message).equals('PowerSync failed to fetch data: some error');
       },
       { timeout: 100 }
     );
diff --git a/packages/tanstack-react-query/package.json b/packages/tanstack-react-query/package.json
@@ -41,7 +41,6 @@
     "@testing-library/react": "^15.0.2",
     "@types/react": "^18.2.34",
     "jsdom": "^24.0.0",
-    "react": "18.2.0",
-    "typescript": "^5.5.3"
+    "react": "18.2.0"
   }
 }
diff --git a/packages/vue/package.json b/packages/vue/package.json
@@ -33,10 +33,9 @@
     "@powersync/common": "workspace:^1.21.0"
   },
   "devDependencies": {
+    "@powersync/common": "workspace:*",
     "flush-promises": "^1.0.2",
     "jsdom": "^24.0.0",
-    "typescript": "^5.5.3",
-    "vitest": "^1.5.1",
     "vue": "3.4.21"
   }
 }
diff --git a/packages/vue/tests/useQuery.test.ts b/packages/vue/tests/useQuery.test.ts
@@ -24,7 +24,7 @@ describe('useQuery', () => {
 
     const [{ data, isLoading, isFetching, error }] = withSetup(() => useQuery('SELECT * from lists'));
 
-    expect(error.value).toEqual(Error('PowerSync not configured.'));
+    expect(error.value?.message).toEqual('PowerSync not configured.');
     expect(isFetching.value).toEqual(false);
     expect(isLoading.value).toEqual(false);
     expect(data.value).toEqual([]);
@@ -110,7 +110,7 @@ describe('useQuery', () => {
     const [{ error }] = withSetup(() => useQuery('SELECT * from lists', [], { runQueryOnce: true }));
     await flushPromises();
 
-    expect(error.value).toEqual(Error('PowerSync failed to fetch data: some error'));
+    expect(error.value?.message).toEqual('PowerSync failed to fetch data: some error');
   });
 
   it('should set error when error occurs', async () => {
@@ -125,7 +125,7 @@ describe('useQuery', () => {
     const [{ error }] = withSetup(() => useQuery('SELECT * from lists', []));
     await flushPromises();
 
-    expect(error.value).toEqual(Error('PowerSync failed to fetch data: some error'));
+    expect(error.value?.message).toEqual('PowerSync failed to fetch data: some error');
   });
 
   it('should accept compilable queries', async () => {
@@ -163,8 +163,8 @@ describe('useQuery', () => {
       useQuery({ execute: () => [] as any, compile: () => ({ sql: 'SELECT * from lists', parameters: [] }) }, ['x'])
     );
 
-    expect(error.value).toEqual(
-      Error('PowerSync failed to fetch data: You cannot pass parameters to a compiled query.')
+    expect(error.value?.message).toEqual(
+      'PowerSync failed to fetch data: You cannot pass parameters to a compiled query.'
     );
   });
 });
diff --git a/packages/web/package.json b/packages/web/package.json
@@ -74,20 +74,16 @@
   "devDependencies": {
     "@journeyapps/wa-sqlite": "^1.2.0",
     "@types/uuid": "^9.0.6",
-    "@vitest/browser": "^2.1.4",
     "crypto-browserify": "^3.12.0",
     "p-defer": "^4.0.1",
     "source-map-loader": "^5.0.0",
     "stream-browserify": "^3.0.0",
     "terser-webpack-plugin": "^5.3.9",
-    "typescript": "^5.5.3",
     "uuid": "^9.0.1",
-    "vite": "^5.4.10",
-    "vite-plugin-top-level-await": "^1.4.1",
+    "vite": "^6.1.0",
+    "vite-plugin-top-level-await": "^1.4.4",
     "vite-plugin-wasm": "^3.3.0",
-    "vitest": "^2.1.4",
     "vm-browserify": "^1.1.2",
-    "webdriverio": "^8.40.6",
     "webpack": "^5.90.1",
     "webpack-cli": "^5.1.4",
     "webpack-node-externals": "^3.0.0"
diff --git a/packages/web/tests/crud.test.ts b/packages/web/tests/crud.test.ts
@@ -79,7 +79,7 @@ describe('CRUD Tests', () => {
     expect(await powersync.get('SELECT count(*) AS count FROM assets')).deep.equals({ count: 1 });
 
     // Make sure uniqueness is enforced
-    expect(powersync.execute('INSERT INTO assets(id, description) VALUES(?, ?)', [testId, 'test3'])).rejects.toThrow(
+    await expect(powersync.execute('INSERT INTO assets(id, description) VALUES(?, ?)', [testId, 'test3'])).rejects.toThrow(
       /UNIQUE constraint failed/
     );
   });
@@ -154,7 +154,7 @@ describe('CRUD Tests', () => {
 
   it('UPSERT not supported', async () => {
     // Just shows that we cannot currently do this
-    expect(
+    await expect(
       powersync.execute('INSERT INTO assets(id, description) VALUES(?, ?) ON CONFLICT DO UPDATE SET description = ?', [
         testId,
         'test2',
diff --git a/packages/web/tests/open.test.ts b/packages/web/tests/open.test.ts
diff --git a/packages/web/tests/stream.test.ts b/packages/web/tests/stream.test.ts
diff --git a/packages/web/tests/uploads.test.ts b/packages/web/tests/uploads.test.ts
diff --git a/packages/web/tests/utils/generateConnectedDatabase.ts b/packages/web/tests/utils/generateConnectedDatabase.ts
diff --git a/packages/web/vitest.config.ts b/packages/web/vitest.config.ts
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -33,8 +33,12 @@`
`33`	`33`	`"@actions/core": "^1.10.1",`
`34`	`34`	`"@changesets/cli": "2.27.2",`
`35`	`35`	`"@pnpm/workspace.find-packages": "^4.0.2",`
	`36`	`+ "@vitest/browser": "^3.0.5",`
`36`	`37`	`"husky": "^9.0.11",`
`37`	`38`	`"lint-staged": "^15.2.2",`
`38`		`- "prettier": "^3.2.5"`
	`39`	`+ "playwright": "^1.50.1",`
	`40`	`+ "prettier": "^3.2.5",`
	`41`	`+ "typescript": "^5.7.2",`
	`42`	`+ "vitest": "^3.0.5"`
`39`	`43`	`}`
`40`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -58,8 +58,6 @@`
`58`	`58`	`"rsocket-core": "1.0.0-alpha.3",`
`59`	`59`	`"rsocket-websocket-client": "1.0.0-alpha.3",`
`60`	`60`	`"text-encoding": "^0.7.0",`
`61`		`- "typescript": "^5.5.3",`
`62`		`- "vitest": "^2.0.5",`
`63`	`61`	`"web-streams-polyfill": "3.2.1"`
`64`	`62`	`}`
`65`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -33,11 +33,11 @@`
`33`	`33`	`"react": "*"`
`34`	`34`	`},`
`35`	`35`	`"devDependencies": {`
	`36`	`+ "@powersync/common": "workspace:*",`
`36`	`37`	`"@testing-library/react": "^15.0.2",`
`37`	`38`	`"@types/react": "^18.2.34",`
`38`	`39`	`"jsdom": "^24.0.0",`
`39`	`40`	`"react": "18.2.0",`
`40`		`- "react-error-boundary": "^4.1.0",`
`41`		`- "typescript": "^5.5.3"`
	`41`	`+ "react-error-boundary": "^4.1.0"`
`42`	`42`	`}`
`43`	`43`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,6 @@`
`41`	`41`	`"@testing-library/react": "^15.0.2",`
`42`	`42`	`"@types/react": "^18.2.34",`
`43`	`43`	`"jsdom": "^24.0.0",`
`44`		`- "react": "18.2.0",`
`45`		`- "typescript": "^5.5.3"`
	`44`	`+ "react": "18.2.0"`
`46`	`45`	`}`
`47`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -33,10 +33,9 @@`
`33`	`33`	`"@powersync/common": "workspace:^1.21.0"`
`34`	`34`	`},`
`35`	`35`	`"devDependencies": {`
	`36`	`+ "@powersync/common": "workspace:*",`
`36`	`37`	`"flush-promises": "^1.0.2",`
`37`	`38`	`"jsdom": "^24.0.0",`
`38`		`- "typescript": "^5.5.3",`
`39`		`- "vitest": "^1.5.1",`
`40`	`39`	`"vue": "3.4.21"`
`41`	`40`	`}`
`42`	`41`	`}`