Skip to content

Commit 7b6ff6e

Browse files
committed
additional tests
1 parent 0992d75 commit 7b6ff6e

File tree

2 files changed

+31
-1
lines changed

2 files changed

+31
-1
lines changed

packages/application/src/patterns/decorators/signature-guard.decorator.test.ts

Lines changed: 24 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import { afterEach, describe, expect, it, vi } from 'vitest';
22
import type { Response } from 'express';
3-
import { SignatureGuard, SignatureType } from './signature-guard.decorator';
3+
import { messageFactoryByType, SignatureGuard, SignatureType } from './signature-guard.decorator';
44

55
const mocks = vi.hoisted(() => ({
66
mockVerifyLedgerPoP: vi.fn(),
@@ -101,3 +101,26 @@ describe('SignatureGuard', () => {
101101
expect(mocks.mockResponse.res.status).toHaveBeenCalledWith(400);
102102
});
103103
});
104+
105+
describe('SignatureGuard messageFactoryByType', () => {
106+
afterEach(() => {
107+
vi.clearAllMocks();
108+
});
109+
110+
const props = {
111+
result: 'Approved',
112+
id: '123',
113+
finalDataCap: 1024,
114+
allocatorType: 'RKH',
115+
};
116+
117+
it.each`
118+
signatureType | expectedMessage
119+
${SignatureType.RefreshReview} | ${'Governance refresh Approved 123 1024 RKH'}
120+
${SignatureType.ApproveGovernanceReview} | ${'Governance Approved 123 1024 RKH'}
121+
${SignatureType.KycOverride} | ${'KYC Override for 123'}
122+
${SignatureType.KycRevoke} | ${'KYC Revoke for 123'}
123+
`('returns the correct message for $signatureType', ({ signatureType, expectedMessage }) => {
124+
expect(messageFactoryByType[signatureType](props)).toBe(expectedMessage);
125+
});
126+
});

packages/application/src/patterns/decorators/signature-guard.decorator.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ interface MessageFactoryProps {
1313

1414
export enum SignatureType {
1515
RefreshReview = 'refreshReview',
16+
ApproveGovernanceReview = 'approveGovernanceReview',
1617
KycOverride = 'kycOverride',
1718
KycRevoke = 'kycRevoke',
1819
}
@@ -23,6 +24,12 @@ export const messageFactoryByType = {
2324
id,
2425
finalDataCap,
2526
allocatorType,
27+
}: MessageFactoryProps) => `Governance refresh ${result} ${id} ${finalDataCap} ${allocatorType}`,
28+
[SignatureType.ApproveGovernanceReview]: ({
29+
result,
30+
id,
31+
finalDataCap,
32+
allocatorType,
2633
}: MessageFactoryProps) => `Governance ${result} ${id} ${finalDataCap} ${allocatorType}`,
2734
[SignatureType.KycOverride]: ({ id }: MessageFactoryProps) => `KYC Override for ${id}`,
2835
[SignatureType.KycRevoke]: ({ id }: MessageFactoryProps) => `KYC Revoke for ${id}`,

0 commit comments

Comments
 (0)