pvillard31 commented on code in PR #11553:
URL: https://github.com/apache/nifi/pull/11553#discussion_r3789346797


##########
nifi-extension-bundles/nifi-mqtt-bundle/nifi-mqtt-processors/src/test/java/org/apache/nifi/processors/mqtt/TestConsumeMQTT.java:
##########
@@ -831,12 +831,12 @@ private void publishMessage(final String payload, final 
int qos) {
     }
 
     private static String addSSLContextService(TestRunner testRunner) throws 
InitializationException {

Review Comment:
   Can we add behavioral tests that exercise both the Paho and HiveMQ TLS 
paths, including createContext(), createTrustManager(), and the optional 
createKeyManager(), instead of only validating the property wiring?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to