Skip to content

Commit 0f6b861

Browse files
committed
adding repo tests
1 parent b4323b8 commit 0f6b861

File tree

6 files changed

+16
-20
lines changed

6 files changed

+16
-20
lines changed

infrastructure/terraform/modules/backend-api/module_create_routing_config_lambda.tf

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,13 @@ module "create_routing_config_lambda" {
2222
log_retention_in_days = var.log_retention_in_days
2323

2424
iam_policy_document = {
25-
body = data.aws_iam_policy_document.get_routing_config_lambda_policy.json
25+
body = data.aws_iam_policy_document.create_routing_config_lambda_policy.json
2626
}
2727

2828
lambda_env_vars = local.backend_lambda_environment_variables
2929
function_s3_bucket = var.function_s3_bucket
3030
function_code_base_path = local.lambdas_dir
31-
function_code_dir = "backend-api/dist/created-routing-config"
31+
function_code_dir = "backend-api/dist/create-routing-config"
3232

3333
send_to_firehose = var.send_to_firehose
3434
log_destination_arn = var.log_destination_arn

infrastructure/terraform/modules/backend-api/spec.tmpl.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -759,7 +759,7 @@
759759
],
760760
"type": "string"
761761
},
762-
"UploadLetterProperties": {
762+
"UploadLetterProperties": {
763763
"allOf": [
764764
{
765765
"$ref": "#/components/schemas/BaseLetterTemplateProperties"

lambdas/backend-api/src/__tests__/templates/api/create-routing-config.test.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,12 @@
11
import type { APIGatewayProxyEvent, Context } from 'aws-lambda';
22
import { mock } from 'jest-mock-extended';
33
import {
4-
TemplateDto,
5-
CreateUpdateTemplate,
64
CreateUpdateRoutingConfig,
75
RoutingConfig,
86
} from 'nhs-notify-backend-client';
9-
import type { Logger } from 'nhs-notify-web-template-management-utils/logger';
107
import { createHandler } from '@backend-api/templates/api/create-routing-config';
118
import type { RoutingConfigClient } from '@backend-api/templates/app/routing-config-client';
129

13-
jest.mock('nhs-notify-web-template-management-utils/logger', () => ({
14-
logger: mock<Logger>({
15-
child: jest.fn().mockReturnThis(),
16-
}),
17-
}));
18-
1910
function setup() {
2011
const routingConfigClient = mock<RoutingConfigClient>();
2112
const mocks = { routingConfigClient };

lambdas/backend-api/src/__tests__/templates/infra/routing-config-repository.test.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ describe('RoutingConfigRepository', () => {
4040
TableName: TABLE_NAME,
4141
Key: {
4242
id: 'b9b6d56b-421e-462f-9ce5-3012e3fdb27f',
43-
owner: 'nhs-notify-client-id',
43+
owner: 'CLIENT#nhs-notify-client-id',
4444
},
4545
});
4646
});
@@ -96,4 +96,10 @@ describe('RoutingConfigRepository', () => {
9696
});
9797
});
9898
});
99+
100+
// describe('create', () => {
101+
// test('should ', async () => {
102+
103+
// });
104+
// });
99105
});

lambdas/backend-client/src/schemas/routing-config.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,6 @@ export const $RoutingConfig = schemaFor<RoutingConfig>()(
119119
id: z.uuidv4(),
120120
status: z.enum(ROUTING_CONFIG_STATUS_LIST),
121121
createdAt: z.string(),
122-
createdBy: z.string(),
123122
updatedAt: z.string(),
124-
updatedBy: z.string(),
125123
})
126124
);

lambdas/backend-client/src/types/generated/types.gen.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,11 @@ export type Channel = 'EMAIL' | 'LETTER' | 'NHSAPP' | 'SMS';
7070
export type ChannelType = 'primary' | 'secondary';
7171

7272
export type ClientConfiguration = {
73+
/**
74+
* @deprecated
75+
*/
7376
campaignId?: string;
77+
campaignIds?: Array<string>;
7478
features: ClientFeatures;
7579
};
7680

@@ -224,11 +228,8 @@ export type TemplateSuccessList = {
224228

225229
export type TemplateType = 'NHS_APP' | 'EMAIL' | 'SMS' | 'LETTER';
226230

227-
export type UploadLetterProperties = {
228-
files?: LetterFiles;
229-
language: Language;
230-
letterType: LetterType;
231-
templateType: 'LETTER';
231+
export type UploadLetterProperties = BaseLetterTemplateProperties & {
232+
campaignId: string;
232233
};
233234

234235
export type VersionedFileDetails = {

0 commit comments

Comments
 (0)