Feat/refactor-email-service

[andram11]

Description

refactor of the email service in order to:

• create one single sendEmail function which uses variables for each type of template
• not run in test mode
• added unit tests for email service

Issue link

Fixes # (https://app.clickup.com/t/86b2343x3)

Type of change

• Bug fix (non-breaking change which fixes an issue)
• Feature updates / changes
• Tests
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Unit tests were created

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules
• I have updated the change log

[github-actions]

@timDeHof, @cherylli
One business day has passed since the review started. Give priority to reviews as much as possible.

[cherylli]

@andram11 could you please resolve merge conflicts when you have a chance? Thanks!

[timDeHof]

Looks great! All the tests passed! I added some suggestions to improve the tests of the email service that could help improve the overall test organization and reduce code duplication.

[timDeHof]

Looks great

[cherylli]

Looks good, exactly what I wanted. Looked at the code and ran all tests ✅