Add missing tests in Mentorship Relation

[diananova]

Description

Add missing tests for Mentorship relation. Fixes #990

New Tests

Mentioned in the issue:

• [x] Same id for mentor and mentee (MENTOR_ID_SAME_AS_MENTEE_ID)
• [x] Incorrect End Time (END_TIME_BEFORE_PRESENT)
• [x] Incorrect Mentor time (ex. MENTOR_TIME_GREATER_THAN_MAX_TIME)
• [x] Mentor not available (MENTOR_NOT_AVAILABLE_TO_MENTOR)
• [x] Mentee not available (MENTEE_NOT_AVAIL_TO_BE_MENTORED)
• [x] Reject uninvolved relation request (CANT_REJECT_UNINVOLVED_RELATION_REQUEST)

Other Tests suggestions :

• [x] Reject request AUTHORISATION_TOKEN_IS_MISSING
• [x] Reject request TOKEN_HAS_EXPIRED

Type of Change:

• Code
• Quality Assurance

Code/Quality Assurance Only

• New feature (non-breaking change which adds functionality pre-approved by mentors)

How Has This Been Tested?

python -m unittest tests/mentorship_relation/test_dao_creation.py

Checklist:

• [x] My PR follows the style guidelines of this project
• [x] I have performed a self-review of my own code or materials
• [x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas

Code/Quality Assurance Only

• [x] My changes generate no new warnings
• [x] I have added tests that prove my fix is effective or that my feature works
• [x] New and existing unit tests pass locally with my changes

[codecov[bot]]

Codecov Report

Merging #1136 (abe18e5) into develop (4193589) will increase coverage by 0.33%. The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1136      +/-   ##
===========================================
+ Coverage    92.91%   93.25%   +0.33%     
===========================================
  Files           38       38              
  Lines         2062     2062              
===========================================
+ Hits          1916     1923       +7     
+ Misses         146      139       -7     

Impacted Files	Coverage Δ
app/api/dao/mentorship_relation.py	100.00% <0.00%> (+3.95%)	:arrow_up:

[diananova]

Ready for review!

[epicadk]

Thanks looks good : )

[isabelcosta]

will merge since tests are passing and this is not a production code change