VED-842: Exception Handlers

[JamesW1-NHS]

Summary

• Routine Change

This has resolved itself into a bug fix and a pair of unit tests.

Bug fix in delta_backend - failure reports were being written to SQS twice.

Unit tests added in delta_backend and recordforwarder - to check that exceptions are raised (or not) and logged when there is an unexpected error. The source code itself has not been changed in this respect.

Note to testers: Functionality change in delta_backend only. No change to API or to other lambdas.

Reviews Required

• Dev
• Test
• Tech Author
• Product Owner

Review Checklist

ℹ️ This section is to be filled in by the reviewer.

• I have reviewed the changes in this PR and they fill all of the acceptance criteria of the ticket.
• If there were infrastructure, operational, or build changes, I have made sure there is sufficient evidence that the changes will work.
• If there were changes that are outside of the regular release processes e.g. account infrastructure to setup, manual setup for external API integrations, secrets to set, then I have checked that the developer has flagged this to the Tech Lead as release steps.
• I have checked that no Personal Identifiable Data (PID) is logged as part of the changes.

[github-actions]

This branch is working on a ticket in the NHS England VED JIRA Project. Here's a handy link to the ticket:

VED-842

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud