-
Notifications
You must be signed in to change notification settings - Fork 70
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
Create automated tests #12
Comments
Hey @szuecs, do you still want this? If so, I'd label "Help Wanted" and offer it to the Go community. |
@LappleApple thank you for the hint :) |
I'm interested in writing some tests if you'd like to assign it to me. |
Thanks! I'll get started on that today. |
@szuecs - I'm looking for clarification. At first, I thought the goal was to add test coverage via unit/integration tests. After reading the comment regarding dex from 2016, I started looking into the technology but I'm not entirely clear on the goal here. Did you envision us spinning up a dex container and mock out IdPs and then running examples using this library against the dex federator? Sorry for the confusion, and thanks for any further info! |
Hey @szuecs - would it be ok if I just add unit tests to fulfill this ticket? |
Use https://github.com/coreos/dex to have an automated test setup with travis-ci as Brandon Philips pointed out. This will help to make it useful for non Zalando organizations, because it shows a usage that is not dependent to Zalando.
The text was updated successfully, but these errors were encountered: