Skip to content

Azure storage service tests#1260

Open
Jonopono123 wants to merge 10 commits intomainfrom
azureStorageServiceTests
Open

Azure storage service tests#1260
Jonopono123 wants to merge 10 commits intomainfrom
azureStorageServiceTests

Conversation

@Jonopono123
Copy link
Copy Markdown
Contributor

What

Test coverage for AzureStorageService class

Why

Since its creation the AzureStorageService class has not had any test coverage, and it's in our best interest to add this.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Internal change (non-breaking change with no effect on the functionality affecting end users)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • I have performed a self-review of my code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@Jonopono123 Jonopono123 requested a review from a team as a code owner April 29, 2026 14:17
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Images built and published to ECR using a Build Id of PR-1025-d143234

# Conflicts:
#	gp2gp-translator/src/test/java/uk/nhs/adaptors/pss/translator/storage/AzureStorageServiceTest.java
@Jonopono123 Jonopono123 force-pushed the azureStorageServiceTests branch 2 times, most recently from 775a5dc to e498778 Compare May 6, 2026 10:48
@Jonopono123 Jonopono123 force-pushed the azureStorageServiceTests branch from e498778 to fae30e8 Compare May 6, 2026 10:55
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Looks good. All 1 mutations in this change were killed.

class surviving killed
💯uk.nhs.adaptors.pss.translator.storage.AzureStorageService 0 1

See https://pitest.org

@MartinWheelerMT
Copy link
Copy Markdown
Collaborator

This has been addressed in a separate PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants