♻️ Slack Service Refactor

[PepperMoJ]

👀 Purpose

To keep our codebase healthy and streamlined, we regularly review and refactor functionality in the operations-engineering repository. This time, is was the slack_service.py.

♻️ What's Changed

• Removed the send_message_to_plaintext_channel function in favour of consistent usage of existing messaging functions in the Slack Service.
• Refactored all slack messages that are sent to use this new approach.
• Added test coverage for add new functions created.

Please see commit history for further and more detailed information.

📝 Notes

Some messages have been modified slightly by removing the 'greeting' and 'goodbye' to make them a little more streamlined, though if we miss this we can certainly add them back (and, as a variable as all messaging code is in the same service).

---

✅ Things to Check (Optional)

• I have run all unit tests, and they pass.
• I have ensured my code follows the project's coding standards.
• I have checked that all new dependencies are up to date and necessary.

[github-actions]

🦙 MegaLinter status: ❌ ERROR

Descriptor	Linter	Files	Fixed	Errors	Elapsed time
✅ PYTHON	flake8	17		0	0.68s
✅ PYTHON	isort	17	15	0	0.45s
❌ PYTHON	pylint	17		7	4.82s
✅ REPOSITORY	gitleaks	yes		no	1.98s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by