Refactor message Ack/Nack using events

[rkm]

Proposed Changes

This uses event callbacks to allow decoupling of Consumers (including subclasses) and the RabbitMQ IModel interface. We will now be able to simplify the test suite and reduce the total execution time. The next step will be to remove all usage of IModel outside of the RabbitMQBroker class.

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• New Feature (non-breaking change which adds functionality)
• Breaking Change (fix or feature that would cause existing functionality to not work as expected)
• Documentation-Only Update (if none of the other choices apply)
  • In this case, ensure that the message of the head commit from the source branch is prefixed with [skip ci]

Checklist

By opening this PR, I confirm that I have:

• Reviewed the contributing guidelines for this repository
• Ensured that the PR branch is in sync with the target branch (i.e. it is automatically merge-able)
• Updated any relevant API documentation
• Created or updated any tests if relevant
• Created a news file
  • NOTE: This must include any changes to any of the following files: default.yaml, any of the RabbitMQ server configurations, GlobalOptions.cs
• Listed myself in the CONTRIBUTORS file 🚀
• Requested a review by the @SMI/Reviewers team

Issues

[codecov]

Codecov Report

Attention: Patch coverage is 90.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 65.40%. Comparing base (36299ba) to head (2dc533b).
Report is 13 commits behind head on main.

Files with missing lines	Patch %	Lines
src/SmiServices/Common/Messaging/Consumer.cs	60.00%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2068      +/-   ##
==========================================
- Coverage   65.48%   65.40%   -0.08%     
==========================================
  Files         181      182       +1     
  Lines        6530     6536       +6     
  Branches      963      966       +3     
==========================================
- Hits         4276     4275       -1     
- Misses       1963     1967       +4     
- Partials      291      294       +3     

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

[jas88]

Thanks, good stuff - are you working on that next step this week? I'm going to try to get some divorce stuff out of the way this week before we officially start back next Monday.

[rkm]

I'll spend a bit more time on it today, but probably won't be able to finish it off.