Skip to content

Commit 97d0ea0

Browse files
committed
test(analytics): scope the routing case's warn assertion to the degradation it is about
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012zTkyNHJ7TkuN2oXtP5x37
1 parent 961cdd9 commit 97d0ea0

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

packages/services/service-analytics/src/__tests__/raw-sql-object-routing.test.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,13 @@ describe('executeRawSql auto-bridge routes by object (#5033)', () => {
254254
{ stage: 'won', deal_count: 2 },
255255
{ stage: 'lost', deal_count: 1 },
256256
]);
257-
expect(warn).not.toHaveBeenCalled();
257+
// Scoped to the degradation this case is about, exactly as its sibling
258+
// above spells it — ⛔ not a blanket "no warning at all". The analytics
259+
// plugin now also reports at init when no `security` service is registered
260+
// to answer the OBJECT-LEVEL read grant, and this fixture deliberately
261+
// registers none; a blanket assertion would read that deliberate report as
262+
// a routing regression.
263+
expect(warn.mock.calls.map(String).join('\n')).not.toMatch(/is unavailable/);
258264
});
259265
});
260266

0 commit comments

Comments
 (0)