@powersync/service-module-postgres-storage 0.17.0 → 0.19.0

package/test/src/checkpoint_notifications.test.ts

@@ -44,7 +44,10 @@ bucket_definitions:
   const abortController = new AbortController();
   context.onTestFinished(() => abortController.abort());
   const iterator = bucketStorage
-    .watchCheckpointChanges({ user_id: 'user', signal: abortController.signal })
+    .watchCheckpointChanges({
+      user_id: 'user',
+      signal: abortController.signal
+    })
     [Symbol.asyncIterator]();
 
   await expect(
@@ -149,7 +152,10 @@ bucket_definitions:
   const abortController = new AbortController();
   context.onTestFinished(() => abortController.abort());
   const iterator = bucketStorage
-    .watchCheckpointChanges({ user_id: 'user', signal: abortController.signal })
+    .watchCheckpointChanges({
+      user_id: 'user',
+      signal: abortController.signal
+    })
     [Symbol.asyncIterator]();
 
   await expect(

package/test/src/storage.test.ts

@@ -1,12 +1,13 @@
 import { framework, storage, updateSyncRulesFromYaml } from '@powersync/service-core';
-import { bucketRequest, register, test_utils } from '@powersync/service-core-tests';
+import { bucketRequest, compactActive, register, test_utils } from '@powersync/service-core-tests';
+import { orNull } from '@powersync/service-types';
 import * as t from 'ts-codec';
 import { describe, expect, test } from 'vitest';
 import { CLEAR_BATCH_LIMIT } from '../../src/storage/PostgresSyncRulesStorage.js';
 import { POSTGRES_STORAGE_FACTORY, TEST_STORAGE_VERSIONS } from './util.js';
 
 const CheckpointRequestedAtRow = t.object({
-  checkpoint_requested_at: t.Null.or(framework.codecs.date)
+  checkpoint_requested_at: orNull(framework.codecs.date)
 });
 
 describe('Sync Bucket Validation', register.registerBucketValidationTests);
@@ -105,7 +106,12 @@ bucket_definitions:
         WHERE
           user_id = 'user2'
       `.execute();
-      await bucketStorage.compact({
+      {
+        await using activationWriter = await bucketStorage.createWriter(test_utils.BATCH_OPTIONS);
+        await activationWriter.markAllSnapshotDone('1/1');
+        await activationWriter.keepalive('1/1');
+      }
+      await compactActive(factory, {
         compactBuckets: [],
         deleteCheckpointRequestsBefore: new Date('2024-02-01T00:00:00.000Z')
       });
@@ -126,7 +132,9 @@ bucket_definitions:
             .first()
         )?.checkpoint_requested_at;
 
-      bucketStorage.setWriteCheckpointMode(storage.WriteCheckpointMode.CUSTOM);
+      bucketStorage.setWriteCheckpointMode({
+        mode: storage.WriteCheckpointMode.CUSTOM
+      });
       await using writer = await bucketStorage.createWriter(test_utils.BATCH_OPTIONS);
       await writer.markAllSnapshotDone('1/1');
       const customCheckpointRequestedAt = new Date('2024-01-01T00:00:00.000Z');

package/test/src/storage_compacting.test.ts

@@ -1,5 +1,5 @@
 import { storage, updateSyncRulesFromYaml } from '@powersync/service-core';
-import { bucketRequest, register, test_utils } from '@powersync/service-core-tests';
+import { bucketRequest, compactActive, register, test_utils } from '@powersync/service-core-tests';
 import { describe, expect, test } from 'vitest';
 import { PostgresCompactor } from '../../src/storage/PostgresCompactor.js';
 import { POSTGRES_STORAGE_FACTORY } from './util.js';
@@ -44,7 +44,7 @@ bucket_definitions:
 
     // Compact with an explicit bucket name — exercises the this.buckets
     // iteration path, NOT the compactAllBuckets discovery path.
-    await bucketStorage.compact({
+    await compactActive(factory, {
       compactBuckets: [bucketRequest(syncRulesContent, 'global[]').bucket],
       minBucketChanges: 1
     });

package/test/src/storage_sync.test.ts

@@ -17,6 +17,50 @@ function registerStorageVersionTests(storageVersion: number) {
       tableIdStrings: storageFactory.tableIdStrings
     });
 
+    test('updates source metadata on an existing resolved table', async () => {
+      await using factory = await storageFactory.factory();
+      const syncRules = await factory.updateSyncRules(
+        updateSyncRulesFromYaml(
+          `
+bucket_definitions:
+  global:
+    data:
+      - SELECT id FROM test
+`,
+          { storageVersion }
+        )
+      );
+      const bucketStorage = factory.getInstance(syncRules);
+      await using writer = await bucketStorage.createWriter(test_utils.BATCH_OPTIONS);
+      const source: storage.SourceEntityDescriptor = {
+        connectionTag: storage.SourceTable.DEFAULT_TAG,
+        objectId: 'test',
+        schema: 'public',
+        name: 'test',
+        replicaIdColumns: [{ name: 'id', type: 'VARCHAR', typeId: 25 }]
+      };
+
+      const initial = await writer.resolveTables({ connection_id: 1, source });
+      expect(initial.tables[0].sourceMetadata).toBeNull();
+
+      const sourceMetadata = { captureTableObjectId: 42 };
+      const updated = await writer.resolveTables({
+        connection_id: 1,
+        source,
+        reconcileSourceTables: ({ candidates }) => ({
+          compatibleTables: candidates.map((candidate) => candidate.withSourceMetadata(sourceMetadata)),
+          incompatibleTables: [],
+          newTableValues: { sourceMetadata }
+        })
+      });
+
+      expect(updated.tables.map((table) => table.id.toString())).toEqual(
+        initial.tables.map((table) => table.id.toString())
+      );
+      expect(updated.tables[0].sourceMetadata).toEqual(sourceMetadata);
+      expect((await writer.getSourceTableStatus(updated.tables[0]))?.sourceMetadata).toEqual(sourceMetadata);
+    });
+
     test('large batch (2)', async () => {
       // Test syncing a batch of data that is small in count,
       // but large enough in size to be split over multiple returned chunks.

package/test/src/util.ts

@@ -36,4 +36,4 @@ export const POSTGRES_STORAGE_SETUP = postgresTestSetup({
 export const POSTGRES_STORAGE_FACTORY = POSTGRES_STORAGE_SETUP;
 export const POSTGRES_REPORT_STORAGE_FACTORY = POSTGRES_STORAGE_SETUP.reportFactory;
 
-export const TEST_STORAGE_VERSIONS = SUPPORTED_STORAGE_VERSIONS;
+export const TEST_STORAGE_VERSIONS = SUPPORTED_STORAGE_VERSIONS.filter((v) => v < 3);