#2054 - Added status reason to Twilio delivery status

[k-macmillan]

Description

Saw many messages end up in a permanent failure or technical failure status with no status reason. This should not happen. Adjusted Twilio's delivery status to correctly fill that information in.

There was a bug with updating status_reason to None when a notification is put in the delivered status. if incoming_status_reason is not None led to it ignoring the updated status_reason. The test mocked it out and the mock has been removed.

issue #2054

How Has This Been Tested?

Updated tests. Regression passes.

Checklist

• [x] I have assigned myself to this PR
• [x] PR has an appropriate title: #9999 - What the thing does
• [x] PR has a detailed description, including links to specific documentation
• [x] I have added the appropriate labels to the PR.
• [x] I did not remove any parts of the template, such as checkboxes even if they are not used
• [x] My code follows the style guidelines of this project
• [x] I have performed a self-review of my own code
• [x] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to any documentation
• [x] My changes generate no new warnings
• [x] I have added tests that prove my fix is effective or that my feature works. Testing guidelines
• [x] I have ensured the latest main is merged into my branch and all checks are green prior to review
• [x] New and existing unit tests pass locally with my changes
• [ ] Any dependent changes have been merged and published in downstream modules
• [ ] The ticket was moved into the DEV test column when I began testing this change