add test related to post mentor id

[lulu-cao]

Description

Add test to verify the id field in POST /mentors request body will be ignored in the backend.

Related Issue / PR

Issue #486
PR #540 (comment)

Change Type

• Bug Fix
• New Feature
• Code Refactor
• Documentation
• Test
• Other

Screenshots

Pull request checklist

Please check if your PR fulfills the following requirements:

• I checked and followed the contributor guide
• I have tested my changes locally.

[lulu-cao]

@womencodingcommunity Requested test has been added. Thanks for reviewing :)

[womencodingcommunity]

Good contribution — plugging the gap identified in PR #540 is exactly the right follow-up, and the @DisplayName follows the Given-When-Then convention correctly.

One change needed before merge:

[WARNING] The final assertion jsonPath("$.id").value(not(999)) is too loose — it passes for any value that is not 999, including null. Swap it for an exact match on the DB-assigned id (see inline comment). The mock already pins savedMentor to id = 1L, so jsonPath("$.id", is(1)) is both more precise and consistent with the existing shouldCreateMentorAndReturnCreated test.

Two minor optional follow-ups (no blocker):

• when(mentorshipService.create(any())) → any(Mentor.class) for consistency with the rest of the file
• Comment // returns DB-assigned id (e.g. 1) — the e.g. is slightly misleading since the factory always returns 1L; // id = 1L, DB-assigned is clearer

[dricazenck]

Suggested change

	.andExpect(jsonPath("$.id").value(not(999)));
	.andExpect(jsonPath("$.id", is(1)))

[lulu-cao]

Updated 81c83a5

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud