From d08a828226dbedb0bc34bd16ac2ee487511cfa8f Mon Sep 17 00:00:00 2001 From: KernelDeimos <7225168+KernelDeimos@users.noreply.github.com> Date: Tue, 13 Jan 2026 16:27:08 -0500 Subject: [PATCH] test(data-access): avoid SQL error in `app:es` It turned out this error was not coming from `app`, but it was coming from `app:es` due to a pre-existing issue. Since these tests are meant to compare the resulting behavior and `app` can handle this case while `app:es` cannot, the best course of action is to avoid invoking the issue and only test the intended behavior. --- .../src/modules/data-access/AppService.comp.test.js | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/backend/src/modules/data-access/AppService.comp.test.js b/src/backend/src/modules/data-access/AppService.comp.test.js index 2b03370bf..7be4e8333 100644 --- a/src/backend/src/modules/data-access/AppService.comp.test.js +++ b/src/backend/src/modules/data-access/AppService.comp.test.js @@ -316,14 +316,7 @@ describe('AppService Regression Prevention Tests', () => { }, }); - // Update it - const updated = await crudQ.update.call(service, { - object: { uid: created.uid }, - id: { name: 'update-test-app' }, - options: {}, - }); - - // Verify update worked - the object fields should be merged + // Update title and description await crudQ.update.call(service, { object: { uid: created.uid, @@ -381,16 +374,18 @@ describe('AppService Regression Prevention Tests', () => { }, }); - // Update with filetype associations + // Update with filetype associations (include title to avoid empty SET clause) await crudQ.update.call(service, { object: { uid: created.uid, + title: 'Filetype App Updated', filetype_associations: ['txt', 'md', 'json'], }, id: { name: 'filetype-app' }, }); const read = await crudQ.read.call(service, { uid: created.uid }); + expect(read.title).toBe('Filetype App Updated'); expect(read.filetype_associations).toEqual( expect.arrayContaining(['txt', 'md', 'json'])); });