Skip to content

Fix password validator uppercase validation - #429

Open
delaewoenamavemee-crypto wants to merge 1 commit into
larymak:mainfrom
delaewoenamavemee-crypto:fix-password-uppercase-validation
Open

delaewoenamavemee-crypto wants to merge 1 commit into
larymak:mainfrom
delaewoenamavemee-crypto:fix-password-uppercase-validation

Conversation

@delaewoenamavemee-crypto

Copy link
Copy Markdown

Description

Fixes the password validator so that a password must contain at least one uppercase letter.

Problem

The password validator was checking for any letter instead of specifically checking for an uppercase letter. This allowed passwords without uppercase characters to be accepted as valid.

Changes Made

  • Updated the password validation check to require an uppercase letter.
  • Added a test to verify that a password without an uppercase letter is rejected.

Testing

The test was run successfully using:

python -m unittest test_password_validator.py

Result: OK

@github-actions

Copy link
Copy Markdown

Hi @delaewoenamavemee-crypto! 👋

Thank you for submitting a pull request! We appreciate your contribution and will review your changes as soon as possible.

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.

1 participant