paulononato/backstage
1
0
Fork 0
Code Issues Pull Requests Actions 45 Packages Projects Releases Wiki Activity

catalog-backend: fix syncSearchRows test to expect first-wins semantics

Browse Source 
Update the 'keeps one row when original_value casing differs' test to
expect original_value: 'V' (first occurrence) instead of 'v' (last
occurrence), matching the first-wins dedup semantics that were aligned
with buildEntitySearch in a prior commit.

Signed-off-by: Fredrik Adelöw <freben@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
Fredrik Adelöw
2026-05-11 18:42:24 +02:00
parent 62c67c752b
commit 9fa8b118c2
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/catalog-backend/src/database/operations/stitcher/syncSearchRows.test.ts
+3 -2
View File
@@ -193,14 +193,15 @@ describe.each(databases.eachSupportedId())('syncSearchRows, %p', databaseId => {
it('keeps one row when original_value casing differs for same key+value', async () => {
// Two entries with the same lowercased (key, value) but different
// original_value casing are deduplicated — the UNIQUE constraint on
// (entity_id, key, value) allows only one. The last occurrence wins.
// (entity_id, key, value) allows only one. The first occurrence wins,
// matching the first-wins semantics of buildEntitySearch.
await syncSearchRows(knex, 'e1', [row('a', 'v', 'V')]);
await syncSearchRows(knex, 'e1', [row('a', 'v', 'V'), row('a', 'v', 'v')]);
const rows = await getSearchRows();
expect(rows).toHaveLength(1);
expect(rows[0]).toEqual(
expect.objectContaining({ key: 'a', value: 'v', original_value: 'v' }),
expect.objectContaining({ key: 'a', value: 'v', original_value: 'V' }),
);
});