alphabetise

Author: alexnuttall

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

@@ -25,10 +25,10 @@ module "lambda_event_publisher" {
   }
 
   lambda_env_vars = {
-    SNS_TOPIC_ARN             = coalesce(var.sns_topic_arn, aws_sns_topic.main.arn)
+    EVENT_SOURCE              = "//notify.nhs.uk/${var.component}/${var.group}/${var.environment}"
     ROUTING_CONFIG_TABLE_NAME = aws_dynamodb_table.routing_configuration.name
+    SNS_TOPIC_ARN             = coalesce(var.sns_topic_arn, aws_sns_topic.main.arn)
     TEMPLATES_TABLE_NAME      = aws_dynamodb_table.templates.name
-    EVENT_SOURCE              = "//notify.nhs.uk/${var.component}/${var.group}/${var.environment}"
   }
 
   function_s3_bucket      = var.function_s3_bucket

tests/test-team/template-mgmt-event-tests/routing-config.event.spec.ts

@@ -45,9 +45,7 @@ test.describe('Event publishing - Routing Config', () => {
 
     expect(submittedResponse.status()).toBe(200);
 
-    // Note: not ideal - but we are expecting 0 events and there can be a delay
-    // in events arriving. We should wait for a moment
-    // 5 seconds seems to largest delay when testing locally
+    // 5s is longest observed delivery delay
     await setTimeout(5000);
 
     // This would throw if a routing config event was present,