From ad0c9bd5987d483aec891a1ce76534397fbe98ab Mon Sep 17 00:00:00 2001 From: Clare Macrae Date: Fri, 8 Dec 2023 19:59:33 +0000 Subject: [PATCH] comment: Add a TODO to Postponer.test.ts --- tests/Scripting/Postponer.test.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/Scripting/Postponer.test.ts b/tests/Scripting/Postponer.test.ts index 4fbc3856c2..16fc7a7045 100644 --- a/tests/Scripting/Postponer.test.ts +++ b/tests/Scripting/Postponer.test.ts @@ -171,6 +171,8 @@ describe('postpone - UI text', () => { it('should include date type and new date in context menu labels', () => { const task = new TaskBuilder().dueDate(today).build(); + // TODO This text is misleading if the date is already in the future. + // In that case, it should still be 'Postpone'??? expect(postponeMenuItemTitle(task, 1, 'day')).toEqual('Due in a day, on Mon 4th Dec'); expect(postponeMenuItemTitle(task, 2, 'days')).toEqual('Due in 2 days, on Tue 5th Dec'); });