@byline/db-postgres 1.12.1 → 1.12.2

package/dist/modules/storage/tests/storage-restore.test.js

@@ -5,18 +5,7 @@
  *
  * Copyright (c) Infonomic Company Limited
  */
-/**
- * Integration test for the "make current" / restore-version flow.
- *
- * Validates that the Postgres locale='all' round-trip
- * (`getDocumentByVersion` → `createDocumentVersion`) preserves multi-locale
- * content and stable block `_id`s — the core invariant
- * `restoreDocumentVersion` in @byline/core depends on. Pure storage-layer:
- * the lifecycle wrapper, auth, and hooks are covered by unit tests in
- * `packages/core/src/services/document-lifecycle.test.node.ts`.
- */
-import assert from 'node:assert';
-import { after, before, describe, it } from 'node:test';
+import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 import { setupTestDB, teardownTestDB } from '../../../lib/test-helper.js';
 let commandBuilders;
 let queryBuilders;
@@ -45,7 +34,7 @@ const RestoreCollectionConfig = {
 };
 let testCollection = {};
 describe('Document version restore — storage round-trip', () => {
-    before(async () => {
+    beforeAll(async () => {
         const testDB = setupTestDB([RestoreCollectionConfig]);
         commandBuilders = testDB.commandBuilders;
         queryBuilders = testDB.queryBuilders;
@@ -55,7 +44,7 @@ describe('Document version restore — storage round-trip', () => {
             throw new Error('Failed to create test collection');
         testCollection = { id: collection.id };
     });
-    after(async () => {
+    afterAll(async () => {
         try {
             await commandBuilders.collections.delete(testCollection.id);
         }
@@ -96,7 +85,7 @@ describe('Document version restore — storage round-trip', () => {
         });
         const v1Sections = v1Read.fields.sections;
         const v1ItemIds = v1Sections[0].sectionItem.map((item) => item._id);
-        assert.ok(v1ItemIds.every((id) => typeof id === 'string' && id.length > 0), 'v1 items should have stable _ids');
+        expect(v1ItemIds.every((id) => typeof id === 'string' && id.length > 0), 'v1 items should have stable _ids').toBeTruthy();
         // Mutate to v2 (different content)
         const v2Data = {
             sku: v1Data.sku,
@@ -134,7 +123,7 @@ describe('Document version restore — storage round-trip', () => {
             locale: 'all',
             status: 'draft',
         });
-        assert.strictEqual(v3.document.event_type, 'restore', 'event_type should be persisted as "restore"');
+        expect(v3.document.event_type, 'event_type should be persisted as "restore"').toBe('restore');
         // Reconstruct v3. Multi-locale fields, block _ids, and per-item content
         // should match v1 — not v2.
         const v3Read = await queryBuilders.documents.getDocumentByVersion({
@@ -142,16 +131,16 @@ describe('Document version restore — storage round-trip', () => {
             locale: 'all',
         });
         const v3Fields = v3Read.fields;
-        assert.deepStrictEqual(v3Fields.title, v1Data.title, 'restored title should match v1 across all locales');
+        expect(v3Fields.title, 'restored title should match v1 across all locales').toEqual(v1Data.title);
         const v3Sections = v3Fields.sections;
         const v3ItemIds = v3Sections[0].sectionItem.map((item) => item._id);
-        assert.deepStrictEqual(v3ItemIds, v1ItemIds, 'restored block items should keep the v1 _ids verbatim (identity preserved across restore)');
-        assert.strictEqual(v3Sections[0].sectionItem.length, 2, "restored version should have v1's two items, not v2's single item");
-        assert.deepStrictEqual(v3Sections[0].sectionItem[0].heading, {
+        expect(v3ItemIds, 'restored block items should keep the v1 _ids verbatim (identity preserved across restore)').toEqual(v1ItemIds);
+        expect(v3Sections[0].sectionItem.length, "restored version should have v1's two items, not v2's single item").toBe(2);
+        expect(v3Sections[0].sectionItem[0].heading).toEqual({
             en: 'Intro EN',
             fr: 'Intro FR',
         });
-        assert.deepStrictEqual(v3Sections[0].sectionItem[1].heading, {
+        expect(v3Sections[0].sectionItem[1].heading).toEqual({
             en: 'Body EN',
             fr: 'Body FR',
         });

package/dist/modules/storage/tests/storage-store-manifest.test.js

@@ -5,8 +5,7 @@
  *
  * Copyright (c) Infonomic Company Limited
  */
-import assert from 'node:assert';
-import { describe, it } from 'node:test';
+import { describe, expect, it } from 'vitest';
 import { allStoreTypes, buildSelectList, columns, fieldTypeLiterals, UNIFIED_COLUMN_COUNT, } from '../storage-store-manifest.js';
 /**
  * Parse a generated SELECT list into individual column expressions.
@@ -36,13 +35,13 @@ describe('storage-store-manifest', () => {
     describe('column count', () => {
         it('UNIFIED_COLUMN_COUNT matches manifest', () => {
             // columns array has all columns except field_type, which is added during generation
-            assert.strictEqual(UNIFIED_COLUMN_COUNT, columns.length + 1);
+            expect(UNIFIED_COLUMN_COUNT).toBe(columns.length + 1);
         });
         for (const storeType of allStoreTypes) {
             it(`${storeType} SELECT list has ${UNIFIED_COLUMN_COUNT} columns`, () => {
                 const selectList = buildSelectList(storeType);
                 const cols = parseColumns(selectList);
-                assert.strictEqual(cols.length, UNIFIED_COLUMN_COUNT, `Expected ${UNIFIED_COLUMN_COUNT} columns for ${storeType}, got ${cols.length}:\n${cols.map((c, i) => `  ${i + 1}. ${c}`).join('\n')}`);
+                expect(cols.length, `Expected ${UNIFIED_COLUMN_COUNT} columns for ${storeType}, got ${cols.length}:\n${cols.map((c, i) => `  ${i + 1}. ${c}`).join('\n')}`).toBe(UNIFIED_COLUMN_COUNT);
             });
         }
     });
@@ -53,7 +52,7 @@ describe('storage-store-manifest', () => {
                 if (storeType === 'text')
                     continue;
                 const aliases = parseColumns(buildSelectList(storeType)).map(extractAlias);
-                assert.deepStrictEqual(aliases, referenceAliases, `Column order mismatch between text and ${storeType}`);
+                expect(aliases, `Column order mismatch between text and ${storeType}`).toEqual(referenceAliases);
             }
         });
     });
@@ -62,7 +61,7 @@ describe('storage-store-manifest', () => {
             for (const storeType of allStoreTypes) {
                 const selectList = buildSelectList(storeType);
                 const expected = fieldTypeLiterals[storeType];
-                assert.ok(selectList.includes(`'${expected}' as "field_type"`), `Expected field_type '${expected}' for ${storeType}`);
+                expect(selectList.includes(`'${expected}' as "field_type"`), `Expected field_type '${expected}' for ${storeType}`).toBe(true);
             }
         });
     });
@@ -71,7 +70,7 @@ describe('storage-store-manifest', () => {
             for (const storeType of allStoreTypes) {
                 const cols = parseColumns(buildSelectList(storeType));
                 const fieldTypeCol = cols[3];
-                assert.ok(fieldTypeCol?.includes('field_type'), `Expected field_type at index 3 for ${storeType}, got: ${fieldTypeCol}`);
+                expect(fieldTypeCol?.includes('field_type'), `Expected field_type at index 3 for ${storeType}, got: ${fieldTypeCol}`).toBe(true);
             }
         });
     });
@@ -79,20 +78,20 @@ describe('storage-store-manifest', () => {
         it('text store maps value → text_value', () => {
             const cols = parseColumns(buildSelectList('text'));
             const textValueCol = cols.find((c) => c.includes('text_value'));
-            assert.ok(textValueCol, 'text_value column not found');
-            assert.ok(textValueCol.includes('value as "text_value"'), `Expected 'value as "text_value"', got: ${textValueCol}`);
+            expect(textValueCol, 'text_value column not found').toBeTruthy();
+            expect(textValueCol?.includes('value as "text_value"'), `Expected 'value as "text_value"', got: ${textValueCol}`).toBe(true);
         });
         it('boolean store maps value → boolean_value', () => {
             const cols = parseColumns(buildSelectList('boolean'));
             const boolCol = cols.find((c) => c.includes('boolean_value'));
-            assert.ok(boolCol, 'boolean_value column not found');
-            assert.ok(boolCol.includes('value as "boolean_value"'), `Expected 'value as "boolean_value"', got: ${boolCol}`);
+            expect(boolCol, 'boolean_value column not found').toBeTruthy();
+            expect(boolCol?.includes('value as "boolean_value"'), `Expected 'value as "boolean_value"', got: ${boolCol}`).toBe(true);
         });
         it('numeric store includes number_type, value_integer, value_decimal, value_float', () => {
             const cols = parseColumns(buildSelectList('numeric'));
             const colText = cols.join(' ');
             for (const field of ['number_type', 'value_integer', 'value_decimal', 'value_float']) {
-                assert.ok(colText.includes(field), `Expected ${field} in numeric SELECT list`);
+                expect(colText.includes(field), `Expected ${field} in numeric SELECT list`).toBe(true);
             }
         });
         it('file store includes all file-specific columns', () => {
@@ -107,14 +106,14 @@ describe('storage-store-manifest', () => {
                 'storage_provider',
                 'storage_path',
             ]) {
-                assert.ok(colText.includes(field), `Expected ${field} in file SELECT list`);
+                expect(colText.includes(field), `Expected ${field} in file SELECT list`).toBe(true);
             }
         });
         it('non-owning stores emit NULL for type-specific columns', () => {
             const cols = parseColumns(buildSelectList('text'));
             const numericCol = cols.find((c) => c.includes('number_type'));
-            assert.ok(numericCol, 'number_type column not found');
-            assert.ok(numericCol.includes('NULL::varchar'), `Expected NULL for number_type in text store, got: ${numericCol}`);
+            expect(numericCol, 'number_type column not found').toBeTruthy();
+            expect(numericCol?.includes('NULL::varchar'), `Expected NULL for number_type in text store, got: ${numericCol}`).toBe(true);
         });
     });
     describe('base columns', () => {
@@ -132,7 +131,7 @@ describe('storage-store-manifest', () => {
                 const cols = parseColumns(buildSelectList(storeType));
                 for (const baseName of baseNames) {
                     const col = cols.find((c) => c.trim() === baseName);
-                    assert.ok(col, `Expected bare column '${baseName}' in ${storeType} SELECT list`);
+                    expect(col, `Expected bare column '${baseName}' in ${storeType} SELECT list`).toBeTruthy();
                 }
             }
         });

package/dist/modules/storage/tests/storage-versioning.test.js

@@ -5,8 +5,7 @@
  *
  * Copyright (c) Infonomic Company Limited
  */
-import assert from 'node:assert';
-import { after, before, describe, it } from 'node:test';
+import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 import { setupTestDB, teardownTestDB } from '../../../lib/test-helper.js';
 // Test database setup
 let commandBuilders;
@@ -211,7 +210,7 @@ const complexProductDocument = {
 // Global test variables
 let testCollection = {};
 describe('03 Document Creation and Versioning', () => {
-    before(async () => {
+    beforeAll(async () => {
         // Connect to test database
         const testDB = setupTestDB([VersionsCollectionConfig]);
         commandBuilders = testDB.commandBuilders;
@@ -225,7 +224,7 @@ describe('03 Document Creation and Versioning', () => {
         testCollection = { id: collection.id, name: collection.path };
         console.log('Test collection created:', testCollection);
     });
-    after(async () => {
+    afterAll(async () => {
         // Clean up test collection (cascades to documents)
         try {
             await commandBuilders.collections.delete(testCollection.id);
@@ -253,7 +252,7 @@ describe('03 Document Creation and Versioning', () => {
                 status: 'draft',
             });
             console.log('Document created:', result);
-            assert.notStrictEqual(result.document.document_id, null, 'Document creation failed');
+            expect(result.document.document_id, 'Document creation failed').not.toBe(null);
         });
         it('should create a document and document version with the same path', async () => {
             const timestamp = Date.now();
@@ -271,7 +270,7 @@ describe('03 Document Creation and Versioning', () => {
                 status: 'draft',
             });
             console.log('firstVersion created:', firstVersion);
-            assert.notStrictEqual(firstVersion.document.document_id, null, 'Document creation failed');
+            expect(firstVersion.document.document_id, 'Document creation failed').not.toBe(null);
             const secondVersion = await commandBuilders.documents.createDocumentVersion({
                 documentId: firstVersion.document.document_id,
                 collectionId: testCollection.id,
@@ -300,7 +299,7 @@ describe('03 Document Creation and Versioning', () => {
                 locale: 'all',
                 status: 'draft',
             });
-            assert.notStrictEqual(firstVersion.document.document_id, null, 'Document creation failed');
+            expect(firstVersion.document.document_id, 'Document creation failed').not.toBe(null);
             const secondVersion = await commandBuilders.documents.createDocumentVersion({
                 documentId: firstVersion.document.document_id,
                 collectionId: testCollection.id,
@@ -312,7 +311,7 @@ describe('03 Document Creation and Versioning', () => {
                 locale: 'all',
                 status: 'draft',
             });
-            assert.notStrictEqual(secondVersion.document.document_id, null, 'Document creation failed');
+            expect(secondVersion.document.document_id, 'Document creation failed').not.toBe(null);
             const thirdVersion = await commandBuilders.documents.createDocumentVersion({
                 documentId: firstVersion.document.document_id,
                 collectionId: testCollection.id,
@@ -324,7 +323,7 @@ describe('03 Document Creation and Versioning', () => {
                 locale: 'all',
                 status: 'draft',
             });
-            assert.notStrictEqual(thirdVersion.document.document_id, null, 'Document creation failed');
+            expect(thirdVersion.document.document_id, 'Document creation failed').not.toBe(null);
             const versionHistory = await queryBuilders.documents.getDocumentHistory({
                 collection_id: testCollection.id,
                 document_id: firstVersion.document.document_id,

package/package.json

@@ -2,7 +2,7 @@
   "name": "@byline/db-postgres",
   "private": false,
   "license": "MPL-2.0",
-  "version": "1.12.1",
+  "version": "1.12.2",
   "engines": {
     "node": ">=20.9.0"
   },
@@ -41,6 +41,11 @@
       "import": "./dist/database/schema/index.js",
       "require": "./dist/database/schema/index.js"
     },
+    "./testing": {
+      "types": "./dist/lib/test-db.d.ts",
+      "import": "./dist/lib/test-db.js",
+      "require": "./dist/lib/test-db.js"
+    },
     "./package.json": "./package.json"
   },
   "files": [
@@ -52,9 +57,9 @@
     "pg": "^8.20.0",
     "uuid": "^14.0.0",
     "zod": "^4.4.3",
-    "@byline/admin": "1.12.1",
-    "@byline/auth": "1.12.1",
-    "@byline/core": "1.12.1"
+    "@byline/auth": "1.12.2",
+    "@byline/core": "1.12.2",
+    "@byline/admin": "1.12.2"
   },
   "devDependencies": {
     "@biomejs/biome": "2.4.15",
@@ -62,10 +67,12 @@
     "@types/pg": "^8.20.0",
     "chokidar": "^5.0.0",
     "chokidar-cli": "^3.0.0",
+    "dotenv": "^17.4.2",
     "drizzle-kit": "^0.31.10",
     "tsc-alias": "^1.8.17",
     "tsx": "^4.21.0",
-    "typescript": "6.0.3"
+    "typescript": "6.0.3",
+    "vitest": "^4.1.5"
   },
   "publishConfig": {
     "access": "public",
@@ -83,8 +90,9 @@
     "drizzle:push": "tsc -p tsconfig.json && drizzle-kit push --config=drizzle.config.ts",
     "drizzle:drop": "drizzle-kit drop --config=drizzle.config.ts",
     "drizzle:seed": "tsx --no-warnings --env-file=.env ./src/database/seeds/index.ts",
-    "test": "LOG_LEVEL=off tsx --env-file=.env --test --test-concurrency=1 $(find . -name \"*.test.ts\")",
-    "test:one": "LOG_LEVEL=off tsx --no-warnings --env-file=.env --test",
+    "test": "echo 'db-postgres has no unit tests; run pnpm test:integration for the DB-backed suite.'",
+    "test:integration": "LOG_LEVEL=off vitest run --mode=integration",
+    "test:watch": "LOG_LEVEL=off vitest --mode=integration",
     "typecheck": "tsc --noEmit"
   }
 }