Skip to content

Commit 8536bbe

Browse files
committed
Add and fix tests
1 parent 17d5b42 commit 8536bbe

File tree

2 files changed

+29
-0
lines changed

2 files changed

+29
-0
lines changed

alcs-frontend/src/app/features/application/application.component.spec.ts

+8
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import { ApplicationSubmissionService } from '../../services/application/applica
1515
import { ApplicationSubmissionStatusService } from '../../services/application/application-submission-status/application-submission-status.service';
1616

1717
import { ApplicationComponent } from './application.component';
18+
import { ApplicationDecisionConditionCardService } from '../../services/application/decision/application-decision-v2/application-decision-condition/application-decision-condition-card/application-decision-condition-card.service';
1819

1920
describe('ApplicationComponent', () => {
2021
let component: ApplicationComponent;
@@ -25,6 +26,7 @@ describe('ApplicationComponent', () => {
2526
let mockReviewService: DeepMocked<ApplicationReviewService>;
2627
let mockAppSubmissionService: DeepMocked<ApplicationSubmissionService>;
2728
let mockAppStatusService: DeepMocked<ApplicationSubmissionStatusService>;
29+
let mockApplicationDecisionConditionCardService: DeepMocked<ApplicationDecisionConditionCardService>;
2830

2931
beforeEach(async () => {
3032
mockAppDetailService = createMock();
@@ -40,6 +42,8 @@ describe('ApplicationComponent', () => {
4042
mockAppSubmissionService = createMock();
4143
mockAppStatusService = createMock();
4244

45+
mockApplicationDecisionConditionCardService = createMock();
46+
4347
await TestBed.configureTestingModule({
4448
providers: [
4549
{
@@ -70,6 +74,10 @@ describe('ApplicationComponent', () => {
7074
provide: ApplicationSubmissionStatusService,
7175
useValue: mockAppStatusService,
7276
},
77+
{
78+
provide: ApplicationDecisionConditionCardService,
79+
useValue: mockApplicationDecisionConditionCardService,
80+
},
7381
{
7482
provide: ActivatedRoute,
7583
useValue: {

services/apps/alcs/src/alcs/application-decision/application-decision-condition/application-decision-condition-card/application-decision-condition-card.controller.spec.ts

+21
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,20 @@ import { classes } from 'automapper-classes';
1616
import { ClsService } from 'nestjs-cls';
1717
import { mockKeyCloakProviders } from '../../../../../test/mocks/mockTypes';
1818
import { ApplicationDecisionProfile } from '../../../../common/automapper/application-decision-v2.automapper.profile';
19+
import { ApplicationDecisionV2Service } from '../../application-decision-v2/application-decision/application-decision-v2.service';
1920

2021
describe('ApplicationDecisionConditionCardController', () => {
2122
let controller: ApplicationDecisionConditionCardController;
2223
let mockService: DeepMocked<ApplicationDecisionConditionCardService>;
2324
let mockModificationService: DeepMocked<ApplicationModificationService>;
2425
let mockReconsiderationService: DeepMocked<ApplicationReconsiderationService>;
26+
let mockApplicationDecisionService: DeepMocked<ApplicationDecisionV2Service>;
2527

2628
beforeEach(async () => {
2729
mockService = createMock();
2830
mockModificationService = createMock();
2931
mockReconsiderationService = createMock();
32+
mockApplicationDecisionService = createMock();
3033

3134
const module: TestingModule = await Test.createTestingModule({
3235
imports: [
@@ -48,6 +51,10 @@ describe('ApplicationDecisionConditionCardController', () => {
4851
provide: ApplicationReconsiderationService,
4952
useValue: mockReconsiderationService,
5053
},
54+
{
55+
provide: ApplicationDecisionV2Service,
56+
useValue: mockApplicationDecisionService,
57+
},
5158
{
5259
provide: ClsService,
5360
useValue: {},
@@ -116,11 +123,25 @@ describe('ApplicationDecisionConditionCardController', () => {
116123
mockService.getByBoardCard.mockResolvedValue(conditionCard);
117124
mockReconsiderationService.getByApplicationDecisionUuid.mockResolvedValue([]);
118125
mockModificationService.getByApplicationDecisionUuid.mockResolvedValue([]);
126+
mockApplicationDecisionService.getDecisionOrder.mockResolvedValue(1);
119127

120128
const result = await controller.getByCardUuid(uuid);
121129

122130
expect(mockService.getByBoardCard).toHaveBeenCalledWith(uuid);
123131
expect(result).toBeInstanceOf(ApplicationDecisionConditionCardBoardDto);
124132
expect(result.fileNumber).toEqual('file-number');
125133
});
134+
135+
it('should return condition cards by application file number', async () => {
136+
const fileNumber = 'example-file-number';
137+
const conditionCard = new ApplicationDecisionConditionCard();
138+
conditionCard.decision = { uuid: 'decision-uuid' } as any;
139+
mockApplicationDecisionService.getForDecisionConditionCardsByFileNumber.mockResolvedValue([conditionCard]);
140+
141+
const result = await controller.getByApplicationFileNumber(fileNumber);
142+
143+
expect(mockApplicationDecisionService.getForDecisionConditionCardsByFileNumber).toHaveBeenCalledWith(fileNumber);
144+
expect(result).toBeInstanceOf(Array);
145+
expect(result[0]).toBeInstanceOf(ApplicationDecisionConditionCardDto);
146+
});
126147
});

0 commit comments

Comments
 (0)