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.
This commit is contained in:
KernelDeimos
2026-01-13 16:27:08 -05:00
committed by Eric Dubé
parent 768a35e9b5
commit d08a828226
@@ -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']));
});