[HOTE-1100] feat: result-status-lambda

[TSeelig]

Description

HOTE-1100 - Reactive Results: Enabler 1 - Create Result Status Lambda
Introduces new lambda to update result status. Currently only accepts result available.

Context

Implements 1 of 3 lambdas required for epic: Reactive result after clinician contact - Result Available

Type of changes

• Refactoring (non-breaking change)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would change existing functionality)
• Bug fix (non-breaking change which fixes an issue)

Checklist

• I am familiar with the contributing guidelines
• I have followed the code style of the project
• I have added tests to cover my changes
• I have updated the documentation accordingly
• This PR is a result of pair or mob programming

---

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

• I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

[github-actions]

Lambdas Coverage Report

Lines	Statements	Branches	Functions
	98.48% (1559/1583)	92.62% (402/434)	96.77% (240/248)

[github-actions]

UI Coverage Report

Lines	Statements	Branches	Functions
	96.16% (5669/5895)	88.22% (682/773)	88.06% (214/243)

[Copilot]

Pull request overview

Adds a new result-status-lambda intended to accept a FHIR Task and record a “result available” status update for an order.

Changes:

• Introduces result-status-lambda handler, request validation schema, and unit tests.
• Adds ResultService DB helper for inserting result_status records.
• Extends OrderService with a query to fetch the patient UID for an order (for ownership checks).

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
lambdas/src/result-status-lambda/models.ts	Adds Zod schema for validating incoming FHIR Task payloads for result status updates.
lambdas/src/result-status-lambda/init.ts	Wires up commons + Postgres client + ResultService/OrderService dependencies.
lambdas/src/result-status-lambda/init.test.ts	Adds init wiring/unit tests for environment construction and singleton behaviour.
lambdas/src/result-status-lambda/index.ts	Implements the result status endpoint handler (parse/validate, authorise, write status).
lambdas/src/result-status-lambda/index.test.ts	Adds handler unit tests covering parsing, validation, auth, DB error paths, and success.
lambdas/src/lib/db/result-db.ts	Adds DB helper to insert into result_status.
lambdas/src/lib/db/result-db.test.ts	Unit tests for ResultService.updateResultStatus.
lambdas/src/lib/db/order-db.ts	Adds OrderPatientReference type and retrievePatientIdFromOrder query method.
lambdas/src/lib/db/order-db.test.ts	Unit tests for the new retrievePatientIdFromOrder method.

Comments suppressed due to low confidence (1)

lambdas/src/result-status-lambda/init.test.ts:151

• Leaving //TODO notes in committed tests (and missing the common init-module "rejection retry" test seen in other lambdas) makes the suite incomplete; add the rejection-retry coverage (e.g., assert init can be called again after buildEnvironment() throws) and remove the TODOs.

  describe("singleton behavior", () => {
    it("should return the same Environment instance on multiple calls to init", () => {
      const env1 = init();
      const env2 = init();

      expect(env1).toBe(env2);
    });
  });
});

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 5 comments.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

lambdas/src/lib/db/order-db.ts:103

• updateOrderStatusAndResultStatus has correlationId available but the error log omits it; this makes failed status updates harder to trace across services/log aggregation.

      console.error("order-db", "Failed to update order and result status", {
        error,
        orderUid,
      });
      throw error;

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.

[TSeelig]

Raised NHSDigital/hometest-mgmt-terraform#99 for hometest-mgmt-terraform update

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.

[Copilot]

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 7 comments.

[Copilot]

Avoid console.log in Playwright tests; use the shared Logger utility (tests/utils/Logger.ts) or remove the log line to keep test output clean and consistent.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
94.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud