Skip to content

Fix/Test cases coverage#261

Open
Diya-0816 wants to merge 5 commits intokaw393939:mainfrom
Diya-0816:fix/test_cases_coverage
Open

Fix/Test cases coverage#261
Diya-0816 wants to merge 5 commits intokaw393939:mainfrom
Diya-0816:fix/test_cases_coverage

Conversation

@Diya-0816
Copy link

This PR enhances test coverage and validation for user-related operations by introducing new test cases and enforcing stricter input checks.

Changes Implemented:

  • Added test cases for:
    • Username validation (min length, allowed characters, duplicate check)
    • Password validation (basic format checks if applicable)
    • Profile field edge cases (empty update payloads)
  • Enforced that profile update requests must include at least one non-null field using a @root_validator.
  • Ensured invalid update requests return 422 Unprocessable Entity.
  • Confirmed duplicate usernames return a 409 Conflict response.
  • Ensured overall test suite passes and improves coverage threshold.

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