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
Add a test for the UI API #2040
Conversation
I expect this test to fail on all MacOS X builds that aren't BUILDONLY. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also to backport test to 1.1.0 if you want.
Yes, I do, just forgot to lable properly... |
The best way to test the UI interface is currently by using an openssl command that uses password_callback. The only one that does this is 'genrsa'. Since password_callback uses a UI method derived from UI_OpenSSL(), it ensures that one gets tested well enough as well.
Actually, no... this wasn't a good enough test. I didn't notice that when
Me, I'm tempted to go for the first option. It's our live code, might as well exercise it, ... and it's easier. |
So I decided to change |
@richsalz, this requires a re-approval on your part... or someone else in the team. |
+0.3% coverage? Cool :-) |
The Travis failures is exactly what I expected, and show that his test does what it's supposed to do. |
+1 |
The best way to test the UI interface is currently by using an openssl command that uses password_callback. The only one that does this is 'genrsa'. Since password_callback uses a UI method derived from UI_OpenSSL(), it ensures that one gets tested well enough as well. Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from #2040)
Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from #2040)
The best way to test the UI interface is currently by using an openssl command that uses password_callback. The only one that does this is 'genrsa'. Since password_callback uses a UI method derived from UI_OpenSSL(), it ensures that one gets tested well enough as well. Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from #2040) (cherry picked from commit 17ac8ea)
Merged |
Checklist
Description of change
The best way to test the UI interface is currently by using an openssl
command that uses password_callback. The only one that does this is
'genrsa'.
Since password_callback uses a UI method derived from UI_OpenSSL(), it
ensures that one gets tested well enough as well.