Skip to content

Commit 98ce6fd

Browse files
fix: handle optional properties in getExistingExtensions and add default response for getExistingTaxonomies test
1 parent 293fdba commit 98ce6fd

File tree

2 files changed

+12
-4
lines changed

2 files changed

+12
-4
lines changed

api/src/services/contentMapper.service.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1590,14 +1590,14 @@ const getExistingExtensions = async ({existingStackId, token_payload}: any) => {
15901590
const headers: Record<string, string> = { api_key: existingStackId };
15911591
if (token_payload?.is_sso) {
15921592
const accessToken = await getAccessToken(
1593-
token_payload.region,
1594-
token_payload.user_id,
1593+
token_payload?.region,
1594+
token_payload?.user_id,
15951595
);
15961596
headers.authorization = `Bearer ${accessToken}`;
15971597
} else {
15981598
headers.authtoken = await getAuthtoken(
1599-
token_payload.region,
1600-
token_payload.user_id,
1599+
token_payload?.region,
1600+
token_payload?.user_id,
16011601
);
16021602
}
16031603

api/tests/unit/controllers/projects.contentMapper.controller.test.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,14 @@ describe('projects.contentMapper.controller', () => {
8383
expect(res.status).toHaveBeenCalledWith(200);
8484
});
8585

86+
it('getExistingTaxonomies should default to 200 when status is omitted', async () => {
87+
mockContentMapperService.getExistingTaxonomies.mockResolvedValue({ taxonomies: [] });
88+
89+
await contentMapperController.getExistingTaxonomies(req, res);
90+
91+
expect(res.status).toHaveBeenCalledWith(200);
92+
});
93+
8694
it('getSingleContentTypes should return 201', async () => {
8795
mockContentMapperService.getSingleContentTypes.mockResolvedValue({ title: 'Blog' });
8896

0 commit comments

Comments
 (0)