Skip to content

Conversation

@scheidtdav
Copy link
Collaborator

Type of Change

  • Dependency upgrade
  • Bug fix (non-breaking change)
  • Breaking change
    • e.g. a fixed bug or new feature that may break something else
  • New feature
  • Code quality improvements
    • e.g. refactoring, documentation, tests, tooling, ...

Implementation

All occurences of hard coded credentials for test users in the unit tests have been replaced by a method that implements randomized credentials.
This is using crypto (mostly randomBytes but also randomUUID) to generate the credentials. The rest is still handled by the tests themselves.

I was facing collisions sometimes and was running out of names to name my test users. This solution helps with my lack of fantasy :-)

Checklist

  • I gave this pull request a meaningful title
  • My pull request is targeting the dev branch
  • I have added documentation to my code
  • I have deleted code that I have commented out

Additional Information

A lot of the files show a bunch of whitespace changes due to not all files being formatted according to the prettier rules of the project. This was kept to eventually have all files follow the prettier rules in place.

  • This PR closes #

@jona159 jona159 merged commit fbd6688 into dev Dec 10, 2025
9 of 10 checks passed
@jona159 jona159 deleted the feat/generate-credentials-for-tests branch December 10, 2025 09:41
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.

3 participants