msharee9 commented on issue #734: MINIFICPP-1157 Implement lightweight C2 
heartbeat.
URL: https://github.com/apache/nifi-minifi-cpp/pull/734#issuecomment-590521920
 
 
   > @msharee9 The new functionality should be tested.
   > Something like `C2JstackTest` for requesting a DESCRIBE manifest can be 
added (although some fixes are required, because `C2JstackTest` does nothing at 
the moment, because C2 is misconfigured in it, and it asserts nothing, so it 
passes anyway... the C2 configuration and assertion approach can be used from 
`C2VerifyHeartbeatAndStop`).
   > The lightweight heartbeat can be tested by extending/parameterizing 
`C2VerifyHeartbeatAndStop`.
   
   @bakaid Refactored existing C2Jstack test and added new tests for light 
weight heartbeat and describe manifest

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to