Skip to content

Commit 6bc570f

Browse files
CopilotPM-Labs
authored andcommitted
test: cover dueReminder and document valid reminder values
1 parent 568a85b commit 6bc570f

2 files changed

Lines changed: 26 additions & 3 deletions

File tree

‎src/index.ts‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,9 @@ class TrelloServer {
198198
.int()
199199
.nullable()
200200
.optional()
201-
.describe('Due date reminder in minutes before due date (0 for due time reminder)'),
201+
.describe(
202+
'Due date reminder in minutes before due date (e.g., null to remove reminder, 0 at due time, 1440 one day before)'
203+
),
202204
start: z
203205
.string()
204206
.optional()
@@ -241,7 +243,9 @@ class TrelloServer {
241243
.int()
242244
.nullable()
243245
.optional()
244-
.describe('New due date reminder in minutes before due date (0 for due time reminder)'),
246+
.describe(
247+
'New due date reminder in minutes before due date (e.g., null to remove reminder, 0 at due time, 1440 one day before)'
248+
),
245249
start: z
246250
.string()
247251
.optional()

‎tests/unit/trello-client.test.ts‎

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,26 @@ describe('TrelloClient', () => {
210210
start: undefined,
211211
dueComplete: true,
212212
idLabels: undefined,
213-
pos: undefined,
213+
});
214+
});
215+
216+
it('should pass numeric due reminder value', async () => {
217+
mockAxiosInstance.put.mockResolvedValue({ data: { id: 'c1' } });
218+
219+
const client = createClient();
220+
await client.updateCard(undefined, {
221+
cardId: 'c1',
222+
dueReminder: 60,
223+
});
224+
225+
expect(mockAxiosInstance.put).toHaveBeenCalledWith('/cards/c1', {
226+
name: undefined,
227+
desc: undefined,
228+
due: undefined,
229+
dueReminder: 60,
230+
start: undefined,
231+
dueComplete: undefined,
232+
idLabels: undefined,
214233
});
215234
});
216235
});

0 commit comments

Comments
 (0)