From 1656ddcae29b0f75556742d263fd208d75fc2177 Mon Sep 17 00:00:00 2001 From: Maurice Date: Fri, 19 Jun 2026 18:09:04 +0200 Subject: [PATCH] test(days): bump over-long-time assertion to the new 250 limit (#1252) Follow-up to raising the day-note time cap to 250: the unit test still sent 151 chars expecting a 400, which now passes validation and fell through to an unmocked service call. --- server/tests/unit/nest/days.controller.test.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/server/tests/unit/nest/days.controller.test.ts b/server/tests/unit/nest/days.controller.test.ts index 7edda513..9df69314 100644 --- a/server/tests/unit/nest/days.controller.test.ts +++ b/server/tests/unit/nest/days.controller.test.ts @@ -127,8 +127,8 @@ describe('DayNotesController (parity with the legacy /api/.../days/:dayId/notes }); it('400 on an over-long time', () => { - expect(thrown(() => new DayNotesController(notesSvc()).create(user, '5', '3', { text: 'ok', time: 'y'.repeat(151) }))).toEqual({ - status: 400, body: { error: 'time must be 150 characters or less' }, + expect(thrown(() => new DayNotesController(notesSvc()).create(user, '5', '3', { text: 'ok', time: 'y'.repeat(251) }))).toEqual({ + status: 400, body: { error: 'time must be 250 characters or less' }, }); });