Fix: [AEA-5977] - Fix log level for Splunk report dependency (#2465)

## Summary

- Routine Change

### Details

Adjust dependent log line for PSU with Pharmacy log reporting.

---------

Signed-off-by: Connor Avery <[email protected]>

Author: connoravo-nhs

packages/common/utilities/src/index.ts

@@ -1,8 +1,11 @@
+// Log messages which are used for reporting purposes.
+// Before editing, consider reviewing in-use splunk reports.
 export {testPrescriptionsConfig, TestPrescriptionsConfig, getTestPrescriptions} from "./testConfig.js"
 export const LOG_MESSAGES = {
   PSU0001: "Transitioning item status.",
   PSU0002: "Notify request",
-  PSU0003: "Updated notification state"
+  PSU0003: "Updated notification state",
+  PSU0004: "Building data item for task."
 } as const
 
 export {initiatedSSMProvider} from "./ssmUtil.js"

packages/updatePrescriptionStatus/src/updatePrescriptionStatus.ts

@@ -32,7 +32,7 @@ import {
   testPrescription1Intercept,
   testPrescription2Intercept
 } from "./utils/testPrescriptionIntercept"
-import {getTestPrescriptions, initiatedSSMProvider} from "@psu-common/utilities"
+import {getTestPrescriptions, initiatedSSMProvider, LOG_MESSAGES} from "@psu-common/utilities"
 
 export const LAMBDA_TIMEOUT_MS = 9500
 // this is length of time from now when records in dynamodb will automatically be expired
@@ -364,7 +364,8 @@ export function buildDataItems(
 
   for (const requestEntry of requestEntries) {
     const task = requestEntry.resource as Task
-    logger.debug("Building data item for task.", {task: task, id: task.id})
+    // The following log line is utilised for PSU "with Pharmacy" reporting, required at info level for Prod running
+    logger.info(LOG_MESSAGES.PSU0004, {task: task, id: task.id})
 
     const repeatNo = task.input?.[0]?.valueInteger
 

packages/updatePrescriptionStatus/tests/testHandler.test.ts

@@ -36,6 +36,7 @@ import {
   timeoutResponse
 } from "../src/utils/responses"
 import {QueryCommand, TransactionCanceledException, TransactWriteItemsCommand} from "@aws-sdk/client-dynamodb"
+import {LOG_MESSAGES} from "@psu-common/utilities"
 
 const {mockSend: dynamoDBMockSend} = mockDynamoDBClient()
 
@@ -55,7 +56,8 @@ const mockInitiatedSSMProvider = {
 
 jest.unstable_mockModule("@psu-common/utilities", async () => ({
   getTestPrescriptions: getTestPrescriptions,
-  initiatedSSMProvider: mockInitiatedSSMProvider
+  initiatedSSMProvider: mockInitiatedSSMProvider,
+  LOG_MESSAGES: LOG_MESSAGES
 }))
 
 const {handler, logger} = await import("../src/updatePrescriptionStatus")

packages/updatePrescriptionStatus/tests/testPrescriptionIntercept.test.ts

@@ -18,6 +18,7 @@ import {
   getTestPrescriptions
 } from "./utils/testUtils"
 import {GetItemCommand, TransactionCanceledException, TransactWriteItemsCommand} from "@aws-sdk/client-dynamodb"
+import {LOG_MESSAGES} from "@psu-common/utilities"
 
 export const mockGetParametersByName = jest.fn(async () => {
   return {}
@@ -29,7 +30,8 @@ const mockInitiatedSSMProvider = {
 
 jest.unstable_mockModule("@psu-common/utilities", async () => ({
   initiatedSSMProvider: mockInitiatedSSMProvider,
-  getTestPrescriptions: getTestPrescriptions // Use the mocked version defined in testUtils.ts
+  getTestPrescriptions: getTestPrescriptions, // Use the mocked version defined in testUtils.ts
+  LOG_MESSAGES: LOG_MESSAGES
 }))
 
 const {mockSend} = mockDynamoDBClient()

packages/updatePrescriptionStatus/tests/testUpdatePrescriptionStatus.test.ts

@@ -20,6 +20,8 @@ import {APIGatewayProxyEvent} from "aws-lambda"
 
 import * as content from "../src/validation/content"
 import {TransactionCanceledException} from "@aws-sdk/client-dynamodb"
+import {LOG_MESSAGES} from "@psu-common/utilities"
+
 const mockValidateEntry = mockInternalDependency("../../src/validation/content", content, "validateEntry")
 
 const mockGetParametersByName = jest.fn(async () => Promise.resolve(
@@ -32,7 +34,8 @@ const mockInitiatedSSMProvider = {
 
 jest.unstable_mockModule("@psu-common/utilities", async () => ({
   getTestPrescriptions: getTestPrescriptions,
-  initiatedSSMProvider: mockInitiatedSSMProvider
+  initiatedSSMProvider: mockInitiatedSSMProvider,
+  LOG_MESSAGES: LOG_MESSAGES
 }))
 
 const {castEventBody, getXRequestID, validateEntries, handleTransactionCancelledException, buildDataItems, TTL_DELTA} =

packages/updatePrescriptionStatus/tests/validation/testRequestValidationViaHandler.test.ts

@@ -21,6 +21,7 @@ import {ONE_DAY_IN_MS} from "../../src/validation/content"
 import requestSingleItem from "../../../specification/examples/request-dispatched.json"
 import requestMultipleItems from "../../../specification/examples/request-multiple-items.json"
 import {accepted, badRequest, bundleWrap} from "../../src/utils/responses"
+import {LOG_MESSAGES} from "@psu-common/utilities"
 
 const mockGetParametersByName = jest.fn(async () => Promise.resolve(
   {[process.env.ENABLE_NOTIFICATIONS_PARAM!]: "false"}
@@ -32,7 +33,8 @@ const mockInitiatedSSMProvider = {
 
 jest.unstable_mockModule("@psu-common/utilities", async () => ({
   getTestPrescriptions: getTestPrescriptions,
-  initiatedSSMProvider: mockInitiatedSSMProvider
+  initiatedSSMProvider: mockInitiatedSSMProvider,
+  LOG_MESSAGES: LOG_MESSAGES
 }))
 
 const {handler} = await import("../../src/updatePrescriptionStatus")