@proteinjs/db 1.54.1 → 1.54.3

package/test/reusable/ColumnTypesTests.ts

@@ -1,4 +1,6 @@
-import { Db, DbDriver, Record, Table, DefaultTransactionContextFactory } from '@proteinjs/db';
+import { Db, DbDriver, Record, Table, DefaultTransactionContextFactory, QueryBuilderFactory } from '@proteinjs/db';
+import { StatementFactory } from '@proteinjs/db-query';
+import { StatementConfigFactory } from '../../src/StatementConfigFactory';
 import { DbTestEnvironment } from '../util/DbTestEnvironment';
 import { columnTypesTestTables, TestRecord } from '../util/tables/columnTypesTestTables';
 
@@ -78,5 +80,49 @@ export const columnTypeTests = (
 
       await expect(db.insert(testTable, testRecord)).rejects.toThrow();
     });
+
+    test('A legacy cell the column cannot deserialize never takes the row with it', async () => {
+      // A value written past the serializer — a bare word in a column that stores JSON — the way
+      // a hand-run statement or an older writer leaves it: the update statement the db layer
+      // itself emits, handed the cells as stored text. The row still serves: that field is
+      // omitted, every other field lands, and a query over the table resolves.
+      const testTable: Table<TestRecord> = columnTypesTestTables.Test;
+      const legacy = await db.insert(testTable, {
+        stringColumn: 'legacy',
+        objectColumn: { theme: 'dark' },
+        arrayColumn: ['a', 'b'],
+      });
+      const intact = await db.insert(testTable, {
+        stringColumn: 'intact',
+        objectColumn: { theme: 'light' },
+        arrayColumn: ['c'],
+      });
+      const statementConfigFactory = new StatementConfigFactory(driver.getDbName());
+      const legacyRow = new QueryBuilderFactory()
+        .getQueryBuilder(testTable)
+        .condition({ field: 'id', operator: '=', value: legacy.id });
+      await driver.runDml((config) =>
+        new StatementFactory<TestRecord>().update(
+          testTable.name,
+          { object_column: 'legacyword', array_column: 'legacyword' } as Partial<TestRecord>,
+          legacyRow,
+          statementConfigFactory.getStatementConfig(config)
+        )
+      );
+
+      const fetched = await db.get(testTable, { id: legacy.id });
+      expect(fetched).toMatchObject({ id: legacy.id, stringColumn: 'legacy' });
+      expect(fetched).not.toHaveProperty('objectColumn');
+      expect(fetched).not.toHaveProperty('arrayColumn');
+
+      const all = await db.query(testTable, {});
+      expect(all.map((record) => record.id).sort()).toEqual([legacy.id, intact.id].sort());
+      const other = all.find((record) => record.id === intact.id);
+      expect(other).toMatchObject({ stringColumn: 'intact', objectColumn: { theme: 'light' }, arrayColumn: ['c'] });
+
+      // Clean up
+      await db.delete(testTable, { id: legacy.id });
+      await db.delete(testTable, { id: intact.id });
+    });
   };
 };