Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: get user testimonials #389

Closed
wants to merge 3 commits into from

Conversation

Laban254
Copy link

@Laban254 Laban254 commented Mar 1, 2025

Description

This PR introduces an API endpoint to retrieve all testimonials associated with a specific user.

Key Changes:

  • Added GET /api/v1/testimonials/user/{user_id} endpoint.
  • Validated user_id parameter to ensure it exists and is correctly formatted.
  • Wrote unit tests.

Related Issue

#372

How Has This Been Tested?

  • Manually tested the endpoint using Postman with valid and invalid inputs.
  • Ran unit tests covering:
    • Successful retrieval of testimonials.
    • Requests with an invalid or non-existing user_id.
    • Users with no testimonials.
  • Ensured all existing tests pass.

**Screenshots **

Success
success

Invalid User ID
invalid user id

Unavaillable User
Unavaillable user

Types of Changes

  • New feature (non-breaking change that adds functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • Breaking change (fix or feature that affects existing functionality)

Checklist

  • My code follows the project’s coding style.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests pass.

@Laban254 Laban254 changed the title Feat/get user testimonials feat: get user testimonials Mar 1, 2025
Copy link
Contributor

@NwokoyeChigozie NwokoyeChigozie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix conflicts

@Laban254
Copy link
Author

Laban254 commented Mar 1, 2025

updated pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants