Skip to content

VED-856: Improve Code Coverage in Codebase#1319

Merged
Akol125 merged 11 commits intomasterfrom
VED-856-Improve-Coverage
Mar 24, 2026
Merged

VED-856: Improve Code Coverage in Codebase#1319
Akol125 merged 11 commits intomasterfrom
VED-856-Improve-Coverage

Conversation

@Akol125
Copy link
Copy Markdown
Contributor

@Akol125 Akol125 commented Mar 22, 2026 •
edited
Loading

Summary

  • Routine Change
  • Improve Test coverage across files in the codebase

Add any other relevant notes or explanations here. Remove this line if you have nothing to add.

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
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 22, 2026

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

VED-856

@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 23, 2026 12:10 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 23, 2026 12:11 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 23, 2026 12:12 — with GitHub Actions Inactive
Thomas-Boyle
Thomas-Boyle previously approved these changes Mar 23, 2026
View reviewed changes
avshetty1980
avshetty1980 reviewed Mar 24, 2026
View reviewed changes
Comment thread lambdas/backend/tests/test_get_status_handler.py Outdated
avshetty1980
avshetty1980 reviewed Mar 24, 2026
View reviewed changes
Comment thread lambdas/backend/tests/test_local_lambda.py Outdated
avshetty1980
avshetty1980 reviewed Mar 24, 2026
View reviewed changes
Comment thread lambdas/backend/tests/test_timer.py
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 24, 2026

Quality Gate Passed 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

avshetty1980
avshetty1980 approved these changes Mar 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@avshetty1980 avshetty1980 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, looks good

@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 24, 2026 11:35 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 24, 2026 11:35 — with GitHub Actions Inactive
@Akol125 Akol125 temporarily deployed to internal-dev-sandbox March 24, 2026 11:36 — with GitHub Actions Inactive
@Akol125 Akol125 enabled auto-merge (squash) March 24, 2026 11:36
@Akol125 Akol125 merged commit ec75255 into master Mar 24, 2026
15 of 17 checks passed
@Akol125 Akol125 deleted the VED-856-Improve-Coverage branch March 24, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@avshetty1980 avshetty1980 avshetty1980 approved these changes

@Thomas-Boyle Thomas-Boyle Thomas-Boyle left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Akol125 @avshetty1980 @Thomas-Boyle @motola