Skip to content

Commit 0fa44a5

Browse files
add: mock methods for batch API operations in tests
- Add getWalletOwnersBatch mock method to SubscriptionClient test mocks - Add shouldSendBatch mock method to SubscriptionClient test mocks - Ensure test files compile with new batch method signatures - Maintain test coverage for refactored batch processing functionality 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 4e22786 commit 0fa44a5

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

apps/dispatcher/src/services/triggers/new-proposal-trigger.service.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,11 @@ describe('NewProposalTriggerHandler', () => {
4848
mockSubscriptionClient = {
4949
getDaoSubscribers: jest.fn(),
5050
shouldSend: jest.fn(),
51+
shouldSendBatch: jest.fn(),
5152
markAsSent: jest.fn(),
52-
getWalletOwners: jest.fn()
53+
getWalletOwners: jest.fn(),
54+
getWalletOwnersBatch: jest.fn(),
55+
getFollowedAddresses: jest.fn()
5356
} as jest.Mocked<ISubscriptionClient>;
5457

5558
mockNotificationClient = {

apps/dispatcher/src/services/triggers/proposal-finished-trigger.service.test.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,11 @@ describe('ProposalFinishedTriggerHandler', () => {
4141
mockSubscriptionClient = {
4242
getDaoSubscribers: jest.fn(),
4343
shouldSend: jest.fn(),
44+
shouldSendBatch: jest.fn(),
4445
markAsSent: jest.fn(),
45-
getWalletOwners: jest.fn()
46+
getWalletOwners: jest.fn(),
47+
getWalletOwnersBatch: jest.fn(),
48+
getFollowedAddresses: jest.fn()
4649
} as jest.Mocked<ISubscriptionClient>;
4750

4851
mockNotificationClient = {

0 commit comments

Comments
 (0)