adamdebreceni commented on a change in pull request #1043:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1043#discussion_r608472239
##########
File path: extensions/http-curl/tests/VerifyInvokeHTTPTest.cpp
##########
@@ -120,6 +126,31 @@ class VerifyInvokeHTTPOKResponse : public VerifyInvokeHTTP
{
}
};
+class VerifyInvokeHTTPOK200Response : public VerifyInvokeHTTP {
+ public:
+ void runAssertions() override {
+ using org::apache::nifi::minifi::utils::verifyLogLinePresenceInPollTime;
+ assert(verifyLogLinePresenceInPollTime(std::chrono::seconds(6),
+ "key:invokehttp.status.code value:200",
+ "response code 200"));
Review comment:
which handler responds with `200`? we redirect to `/`, does the civetweb
have some default handler on `/`?
--
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:
us...@infra.apache.org
