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
Fix #3656 Improve camel-quarkus-paho-mqtt5 test coverage #3709
Fix #3656 Improve camel-quarkus-paho-mqtt5 test coverage #3709
Conversation
fe04663
to
7d25a4d
Compare
|
||
private void removeKeyStore(String keystore) { | ||
try { | ||
Files.delete(Paths.get(keystore)); |
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.
Should we unset the SSL system properties here?
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.
good catch! I will unset these properties.
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, I guess we want this to always happen. E.g even if the call to the ProducerTemplate
throws an exception.
...t5/src/test/java/org/apache/camel/quarkus/component/paho/mqtt5/it/PahoMqtt5TestResource.java
Show resolved
Hide resolved
385912c
to
b100281
Compare
No description provided.