[ #1213] Updated tests for notify scripts to use the step runner format.

[richardgaunt]

Partial implementation of #1213

Checklist before requesting a review

• I have formatted the subject to include ticket number as [#123] Verb in past tense with dot at the end.
• I have added a link to the issue tracker
• I have provided information in Changed section about WHY something was done if this was not a normal implementation
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• I have run new and existing relevant tests locally with my changes, and they passed
• I have provided screenshots, where applicable

Changed

1. Updated most tests to step_runner - note multi-line strings don't work with this helper.

Screenshots

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.50%. Comparing base (55674fd) to head (d1067d6).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #1221   +/-   ##
========================================
  Coverage    72.50%   72.50%           
========================================
  Files           39       39           
  Lines         2855     2855           
========================================
  Hits          2070     2070           
  Misses         785      785           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[AlexSkrypnyk]

@richardgaunt
Could you please revert the changes to tests that are not using mocks. We need to use STEPS only with tests that user the mocking system.

Sorry, I should have been more clear about the requirements.

[AlexSkrypnyk]

@richardgaunt
Could you please revert this. I should have been more clear: we need to only convert tests that are using mocks. This one does not use mocks so it is easier to maintain it as normal asserts. sorry