chore: Add CursorLimit constraint

pulsate-dev · Dec 16, 2024 · a33ac55 · a33ac55
1 parent 6198728
commit a33ac55
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/pkg/notification/adaptor/repository/dummy/notification.ts b/pkg/notification/adaptor/repository/dummy/notification.ts
@@ -5,6 +5,7 @@ import type {
 } from '../../../model/notification.js';
 import {
   NOTIFICATION_DEFAULT_LIMIT,
+  NOTIFICATION_MAX_LIMIT,
   type NotificationFilter,
   type NotificationRepository,
 } from '../../../model/repository/notification.js';
@@ -48,6 +49,10 @@ export class InMemoryNotificationRepository implements NotificationRepository {
     const limit = Option.isSome(filter.limit)
       ? Option.unwrap(filter.limit)
       : NOTIFICATION_DEFAULT_LIMIT;
+    if (limit > NOTIFICATION_MAX_LIMIT) {
+      // ToDo: Define NotificationCursorLimitOutOfRangeError
+      return Result.err(new Error('Limit exceeds the maximum value'));
+    }
 
     if (Option.isNone(filter.cursor)) {
       return Result.ok(res.slice(0, limit));