API-45666 Move VANotify job, save harder, and log more

[cilestin]

Summary

• Move VANotify to after saving the POA record in PoaVBMSUpdater
• Should let the record save successfully before trying to send email, hopefully allowing the status to move into UPDATED as expected even when the email fails.

Related issue(s)

API-45666

Testing done

• New code is covered by unit tests

What areas of the site does it impact?

Background jobs & process order

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution
• Documentation has been updated (link to documentation)
• No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
• Feature/bug has a monitor built into Datadog (if applicable)
• If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
• I added a screenshot of the developed feature

Requested Feedback

[rockwellwindsor-va]

Happy path is confirmed to work as expected. Recieved the email as expected, statues as expected.

• When the VA Notify accepted job fails I also got the statues that I believe we are going for here so this looks good to me

@cilestin Once the Rubocop linting stuff gets updated I can stamp this.