Issue#86 PUT /user/personal_background api endpoint and tests

[mtreacy002]

Description

Allow user to update their personal background data

Fixes #86

Type of Change:

• Code

Code/Quality Assurance Only

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

How Has This Been Tested?

• Login as a user that has no perrsonal background data and get personal details (GET /user/personal_details) so the user_id is saved to cookie.
• send PUT /user/personal_background request and on successful create, see response 201 Created as per screenshot below

• now test update by modifying the personal background. On successful update see response 200 OK as per screenshot below

• with invalid new personal background data payload

Checklist:

• My PR follows the style guidelines of this project
• I have performed a self-review of my own code or materials

Code/Quality Assurance Only

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

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . Here's the PR for PUT /user/personal_background api endpoint and tests. please provide feedback whenever you're available. Thanks

[ramitsawhney27]

Looks good. I like the very detailed docstrings, makes the review and understanding much easier for me!

[mtreacy002]

Looks good. I like the very detailed docstrings, makes the review and understanding much easier for me!

thank you @ramitsawhney27 for your kind feedback.

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . I've modified the background enum logic on backend as discussed in the last BIT weekly meeting. Please re-review once you have time. Thanks

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . I've just force pushed changes to add other fields for physical and mental ability as well as socio-economic status

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . I'vee resolved the mergee conflicts and applied bug fixes discussed on zulip post and issue #94 (temporary solution for local environment only).

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . I've refactor this PR to use only one API endpoint PUT /user/personal_background for both create and update user personal background information. Please review when you have time.

[foongminwong]

The changes made in this PR were tested locally. Following are the results:

1. Code review - Done

2. All possible responses were tested as below:

   • Test PR 87 (Create non-existing personal background)
     Screenshot/gif/url:

     

     Expected Result: As a user, I should be able to create personal background using PUT /user/personal_background
     Actual Result:

   • Test PR 87 (Update existing personal background)
     Screenshot/gif/url:

     

     Expected Result: As a user, I should be able to update existing personal background using PUT /user/personal_background
     Actual Result: Same as expected

3. OS Version: Windows 10

4. Tested JSON:

{
  "gender": "Prefer not to say",
  "age": "Prefer not to say",
  "ethnicity": "Prefer not to say",
  "sexual_orientation": "Prefer not to say",
  "religion": "Prefer not to say",
  "physical_ability": "Prefer not to say",
  "mental_ability": "Prefer not to say",
  "socio_economic": "Prefer not to say",
  "highest_education": "Prefer not to say",
  "years_of_experience": "Prefer not to say",
  "gender_other": "Prefer not to say",
  "ethnicity_other": "Prefer not to say",
  "sexual_orientation_other": "Prefer not to say",
  "religion_other": "Prefer not to say",
  "physical_ability_other": "Prefer not to say",
  "mental_ability_other": "Prefer not to say",
  "socio_economic_other": "Prefer not to say",
  "highest_education_other": "Prefer not to say",
  "is_public": true
}

[mtreacy002]

Update @anitab-org/bridgeintech-maintainers . I've just resolved the merge conflict on this PR. Please re-approve and only merge this PR after the PR #85 which is this PR's blocker is merged to develop. Thanks