adam-markovics commented on a change in pull request #1252: URL: https://github.com/apache/nifi-minifi-cpp/pull/1252#discussion_r806957651
########## File path: extensions/aws/processors/ListS3.h ########## @@ -33,6 +33,7 @@ namespace org { namespace apache { namespace nifi { namespace minifi { + Review comment: Done. ########## File path: CMakeLists.txt ########## @@ -611,6 +612,11 @@ if (ENABLE_KUBERNETES) createExtension(KUBERNETES-EXTENSIONS "KUBERNETES EXTENSIONS" "This enables Kubernetes support" "extensions/kubernetes") endif() +## Add the test-processors extension +if (ENABLE_ALL OR NOT SKIP_TESTS OR ENABLE_TEST_PROCESSORS) + createExtension(TEST-PROCESSORS-EXTENSION "TEST-PROCESSORS EXTENSION" "This enables processors purely for testing purposes" "extensions/test-processors") +endif() Review comment: So you just want the condition to be `if (NOT SKIP_TESTS)` ? `ENABLE_TEST_PROCESSORS` was introduced to avoid building all unit tests in docker integration tests, where `SKIP_TESTS` is true by default. But these processors are still needed for integration tests. ########## File path: extensions/coap/tests/CoapC2VerifyHeartbeat.cpp ########## @@ -92,10 +92,10 @@ class VerifyCoAPServer : public CoapIntegrationBase { } void queryRootProcessGroup(std::shared_ptr<core::ProcessGroup> pg) override { - std::shared_ptr<core::Processor> proc = pg->findProcessorByName("invoke"); + auto proc = pg->findProcessorByName("invoke"); Review comment: Done. ########## File path: extensions/coap/tests/CoapC2VerifyHeartbeat.cpp ########## @@ -92,10 +92,10 @@ class VerifyCoAPServer : public CoapIntegrationBase { } void queryRootProcessGroup(std::shared_ptr<core::ProcessGroup> pg) override { - std::shared_ptr<core::Processor> proc = pg->findProcessorByName("invoke"); + auto proc = pg->findProcessorByName("invoke"); assert(proc != nullptr); - std::shared_ptr<minifi::processors::InvokeHTTP> inv = std::dynamic_pointer_cast<minifi::processors::InvokeHTTP>(proc); + auto inv = dynamic_cast<minifi::processors::InvokeHTTP*>(proc); Review comment: Done. ########## File path: extensions/http-curl/tests/C2NullConfiguration.cpp ########## @@ -61,10 +61,10 @@ class VerifyC2Server : public HTTPIntegrationBase { } void queryRootProcessGroup(std::shared_ptr<core::ProcessGroup> pg) override { - std::shared_ptr<core::Processor> proc = pg->findProcessorByName("invoke"); + auto* const proc = pg->findProcessorByName("invoke"); assert(proc != nullptr); - std::shared_ptr<minifi::processors::InvokeHTTP> inv = std::dynamic_pointer_cast<minifi::processors::InvokeHTTP>(proc); + const auto inv = dynamic_cast<minifi::processors::InvokeHTTP*>(proc); Review comment: Done. -- 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: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org