feat: DTOSS-10479 add nems poison container and logic#1440

Merged

SamAinsworth-NHS merged 29 commits intomainfrom

feat/DTOSS-10479-Add-Nems-Poison-Container

Aug 13, 2025

Commits on Aug 7, 2025

feat: added infrastructure for nems-poison container in blob storage
SamAinsworth-NHS
committed
feat: if Nems PDS Update processing fails, copy file to nems-poison container
SamAinsworth-NHS
committed

Commits on Aug 12, 2025

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container
SamAinsworth-NHS
committed
fix: update ServiceBusConnectionString to ServiceBusConnectionString_client_internal in PdsProcessorTests
SamAinsworth-NHS
committed
feat: add support for handling failed NEMS updates by moving files to a poison container
SamAinsworth-NHS
committed
Merge branch 'feat/DTOSS-10479-Add-Nems-Poison-Container' of github.com:NHSDigital/dtos-cohort-manager into feat/DTOSS-10479-Add-Nems-Poison-Container
SamAinsworth-NHS
committed
fix: update ProcessNemsUpdateTests to ensure proper logging for poison file operations
SamAinsworth-NHS
committed
test: enhance ProcessNemsUpdateTests to verify poison file handling for 404 responses
SamAinsworth-NHS
committed
fix: removed blobstream logic from ProcessNemsUpdate as no longer needed
SamAinsworth-NHS
committed
fix: streamline CopyToPoisonContainer method by using config for storage connection string
SamAinsworth-NHS
committed
Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container
SamAinsworth-NHS
authored
feat: changing config name from fileExceptions to NemsPoisonContainer - to separate from caas function fully
SamAinsworth-NHS
committed
refactor: remove NemsMeshPoisonContainer from config and related tests in NemsMeshRetrieval function
SamAinsworth-NHS
committed
fix: remove unnecessary whitespace in NemsMeshRetrievalTests for cleaner code
SamAinsworth-NHS
committed
Merge remote-tracking branch 'origin/main' into feat/DTOSS-10479-Add-Nems-Poison-Container
SamAinsworth-NHS
committed
fix: remove unnecessary whitespace in NemsMeshRetrievalTests for cleaner code
SamAinsworth-NHS
committed
feat: add timestamp option to CopyFileToPoisonAsync overload for improved file management
SamAinsworth-NHS
committed
feat: add unit tests for BlobStorageHelper including timestamp generation and file copying functionality
SamAinsworth-NHS
committed
feat: add unit tests for UploadFileToBlobStorage and GetFileFromBlobStorage methods in BlobStorageHelper to improve code coverage
SamAinsworth-NHS
committed
feat: add BlobStorageHelperTests project to solution for code coverage reporting
SamAinsworth-NHS
committed
Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container
SamAinsworth-NHS
authored
fix: update BlobStorageHelperTests project references and package versions
SamAinsworth-NHS
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: DTOSS-10479 add nems poison container and logic#1440

feat: DTOSS-10479 add nems poison container and logic#1440
SamAinsworth-NHS merged 29 commits intomainfrom
feat/DTOSS-10479-Add-Nems-Poison-Container

Commits on Aug 7, 2025

feat: added infrastructure for nems-poison container in blob storage

feat: if Nems PDS Update processing fails, copy file to nems-poison container

Commits on Aug 12, 2025

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container

fix: update ServiceBusConnectionString to ServiceBusConnectionString_client_internal in PdsProcessorTests

feat: add support for handling failed NEMS updates by moving files to a poison container

Merge branch 'feat/DTOSS-10479-Add-Nems-Poison-Container' of github.com:NHSDigital/dtos-cohort-manager into feat/DTOSS-10479-Add-Nems-Poison-Container

fix: update ProcessNemsUpdateTests to ensure proper logging for poison file operations

test: enhance ProcessNemsUpdateTests to verify poison file handling for 404 responses

fix: removed blobstream logic from ProcessNemsUpdate as no longer needed

fix: streamline CopyToPoisonContainer method by using config for storage connection string

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container

feat: changing config name from fileExceptions to NemsPoisonContainer - to separate from caas function fully

refactor: remove NemsMeshPoisonContainer from config and related tests in NemsMeshRetrieval function

fix: remove unnecessary whitespace in NemsMeshRetrievalTests for cleaner code

Merge remote-tracking branch 'origin/main' into feat/DTOSS-10479-Add-Nems-Poison-Container

fix: remove unnecessary whitespace in NemsMeshRetrievalTests for cleaner code

feat: add timestamp option to CopyFileToPoisonAsync overload for improved file management

feat: add unit tests for BlobStorageHelper including timestamp generation and file copying functionality

feat: add unit tests for UploadFileToBlobStorage and GetFileFromBlobStorage methods in BlobStorageHelper to improve code coverage

feat: add BlobStorageHelperTests project to solution for code coverage reporting

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container

fix: update BlobStorageHelperTests project references and package versions

Commits on Aug 13, 2025

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container

fix: avoid using throws in logic - moving files to poison container is now explicit

refactor: simplify CopyFileToPoisonAsync by delegating to overloaded method

fix: correcting unit test assert post changes

fix: corrected assert for error log in Run_FailsToRetrievePdsRecord_LogsError unit test

Merge branch 'main' into feat/DTOSS-10479-Add-Nems-Poison-Container