Skip to content

Commit a2057e0

Browse files
sarthak688claude
andcommitted
fix(subscriptions): use domain-specific error constant in updateCategory test
Matches the parallel updateTopic error test on the same suite. Addresses review comment on PR #517. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 7105f28 commit a2057e0

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/unit/services/notification/subscriptions.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ describe('SubscriptionService Unit Tests', () => {
208208
});
209209

210210
it('should propagate errors', async () => {
211-
mockApiClient.post.mockRejectedValue(createMockError(TEST_CONSTANTS.ERROR_MESSAGE));
211+
mockApiClient.post.mockRejectedValue(createMockError(NOTIFICATION_TEST_CONSTANTS.ERROR_SUBSCRIPTION_INVALID));
212212

213213
await expect(
214214
subscriptionService.updateCategory(NOTIFICATION_TEST_CONSTANTS.TENANT_ID, [
@@ -219,7 +219,7 @@ describe('SubscriptionService Unit Tests', () => {
219219
notificationMode: NotificationMode.InApp,
220220
},
221221
])
222-
).rejects.toThrow(TEST_CONSTANTS.ERROR_MESSAGE);
222+
).rejects.toThrow(NOTIFICATION_TEST_CONSTANTS.ERROR_SUBSCRIPTION_INVALID);
223223
});
224224
});
225225
});

0 commit comments

Comments
 (0)