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

Increase tests cases coverage and introduce e2e tests #1424

Open
darkowic opened this issue Jul 22, 2024 · 2 comments
Open

Increase tests cases coverage and introduce e2e tests #1424

darkowic opened this issue Jul 22, 2024 · 2 comments

Comments

@darkowic
Copy link

Test automation is crucial for efficient and reliable delivery of a library. We've noticed that the test coverage of the oidc-client is pretty poor — only ~29% of statements are covered. We see that PRs must not necessarily must be provided with appropriate tests before they get merged and this decreases the credibility of the library. Also, there are no e2e tests and the maintainer runs the tests manually for more advanced cases.

I want to propose coming up with a plan on how to improve that matter. Have you ever thought about how to do this better?

@jafin
Copy link
Contributor

jafin commented Jul 22, 2024

PR's are always welcome. ❤️

@guillaume-chervet
Copy link
Contributor

Hi @darkowic ,
It is really a good point. I am lacking of time.
For now i never publish a version whithout testing it and i consume always the new version in my current web application.
Help and PRs would be awesome.

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

No branches or pull requests

3 participants