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

Tracking issue for tests #15

Open
2 of 8 tasks
Erik1000 opened this issue Sep 18, 2021 · 0 comments · May be fixed by #32
Open
2 of 8 tasks

Tracking issue for tests #15

Erik1000 opened this issue Sep 18, 2021 · 0 comments · May be fixed by #32
Labels
enhancement New feature or request help wanted Extra attention is needed
Milestone

Comments

@Erik1000
Copy link
Member

Erik1000 commented Sep 18, 2021

I have currently no time to add good tests. Especially the tests involving crypto need special attention and time.

We would need tests for:

  • user id and user attribute removal
  • key validation (if there are really keys for all operations)
  • Certificate if it handles secret and non-secret material correctly
  • key policy (if really only curve 25519 suite is accepted and if unencrypted secret key material is rejected)
  • challenge signature checks
  • that refresh tokens are really denied after a use (this is probably pain to test, because it needs database I/O)
  • that the check constraint in the database really rejects invalid names (pain to check too because of database I/O)
  • certification pagination test (if the order is as expected, has_next/previous is correct, after/before work as expected)
@Erik1000 Erik1000 added enhancement New feature or request help wanted Extra attention is needed labels Sep 18, 2021
@Erik1000 Erik1000 added this to the 0.2 milestone Sep 18, 2021
@Erik1000 Erik1000 linked a pull request Nov 2, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant