Skip to content

Unit testing for Project Team Members router #1915

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: development
Choose a base branch
from
Open

Unit testing for Project Team Members router #1915

wants to merge 1 commit into from

Conversation

jng34
Copy link
Member

@jng34 jng34 commented May 6, 2025

Fixes #1892

What changes did you make and why did you make them ?

  • wrote unit tests for projectTeamMembers.router
  • edited the logic in the GET API route /project/:id/:userId in projectTeamMembers.router.js that incorrectly threw the HTTP code 400 when it should have been 200, which affected the unit test for that route

Screenshots of Passed Tests

image

@JackHaeg JackHaeg requested a review from dannyprikaz May 15, 2025 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dannyprikaz dannyprikaz Awaiting requested review from dannyprikaz

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create New Unit Tests for ./backend/routers/projectTeamMembers.router.js
1 participant
@jng34