This is an automated email from the ASF dual-hosted git repository.

adebreceni pushed a commit to branch minifi-api-reduced
in repository https://gitbox.apache.org/repos/asf/nifi-minifi-cpp.git


The following commit(s) were added to refs/heads/minifi-api-reduced by this 
push:
     new 74eed81e1 Windows proc update
74eed81e1 is described below

commit 74eed81e105da7346e4df70cc0cd75371a0f59f4
Author: Adam Debreceni <adebrec...@apache.org>
AuthorDate: Tue Jun 24 15:14:38 2025 +0200

    Windows proc update
---
 extensions/smb/tests/FetchSmbTests.cpp        | 5 +++--
 extensions/smb/tests/ListAndFetchSmbTests.cpp | 8 ++++----
 extensions/smb/tests/ListSmbTests.cpp         | 3 ++-
 extensions/smb/tests/PutSmbTests.cpp          | 7 ++++---
 4 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/extensions/smb/tests/FetchSmbTests.cpp 
b/extensions/smb/tests/FetchSmbTests.cpp
index 0fe708167..921b92341 100644
--- a/extensions/smb/tests/FetchSmbTests.cpp
+++ b/extensions/smb/tests/FetchSmbTests.cpp
@@ -24,13 +24,14 @@
 #include "unit/SingleProcessorTestController.h"
 #include "utils/OsUtils.h"
 #include "core/Resource.h"
+#include "unit/ProcessorUtils.h"
 
 namespace org::apache::nifi::minifi::extensions::smb::test {
 
 REGISTER_RESOURCE(MockSmbConnectionControllerService, ControllerService);
 
 TEST_CASE("FetchSmb invalid network path") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<FetchSmb>("FetchSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<FetchSmb>("FetchSmb")};
   const auto fetch_smb = controller.getProcessor();
   auto smb_connection_node = 
controller.plan->addController("MockSmbConnectionControllerService", 
"smb_connection_controller_service");
   REQUIRE(controller.plan->setProperty(smb_connection_node, 
SmbConnectionControllerService::Hostname, 
utils::OsUtils::getHostName().value_or("localhost")));
@@ -43,7 +44,7 @@ TEST_CASE("FetchSmb invalid network path") {
 }
 
 TEST_CASE("FetchSmb tests") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<FetchSmb>("FetchSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<FetchSmb>("FetchSmb")};
   const auto fetch_smb = controller.getProcessor();
 
   auto smb_connection_node = 
controller.plan->addController("MockSmbConnectionControllerService", 
"smb_connection_controller_service");
diff --git a/extensions/smb/tests/ListAndFetchSmbTests.cpp 
b/extensions/smb/tests/ListAndFetchSmbTests.cpp
index 0c2abed97..bdbe86550 100644
--- a/extensions/smb/tests/ListAndFetchSmbTests.cpp
+++ b/extensions/smb/tests/ListAndFetchSmbTests.cpp
@@ -33,10 +33,10 @@ using minifi::processors::ReadFromFlowFileTestProcessor;
 TEST_CASE("ListSmb and FetchSmb work together") {
   TestController controller;
   auto plan = controller.createPlan();
-  auto list_smb = dynamic_cast<ListSmb*>(plan->addProcessor("ListSmb", 
"list_smb"));
-  auto fetch_smb = dynamic_cast<FetchSmb*>(plan->addProcessor("FetchSmb", 
"fetch_smb"));
-  auto read_from_success_relationship = 
dynamic_cast<ReadFromFlowFileTestProcessor*>(plan->addProcessor("ReadFromFlowFileTestProcessor",
 "read_from_success_relationship"));
-  auto read_from_failure_relationship = 
dynamic_cast<ReadFromFlowFileTestProcessor*>(plan->addProcessor("ReadFromFlowFileTestProcessor",
 "read_from_failure_relationship"));
+  auto list_smb = plan->addProcessor<ListSmb>("list_smb");
+  auto fetch_smb = plan->addProcessor<ListSmb>("fetch_smb");
+  auto read_from_success_relationship = 
plan->addProcessor<ReadFromFlowFileTestProcessor>("read_from_success_relationship");
+  auto read_from_failure_relationship = 
plan->addProcessor<ReadFromFlowFileTestProcessor>("read_from_failure_relationship");
 
   plan->addConnection(list_smb, ListSmb::Success, fetch_smb);
 
diff --git a/extensions/smb/tests/ListSmbTests.cpp 
b/extensions/smb/tests/ListSmbTests.cpp
index 15120d029..d7b197831 100644
--- a/extensions/smb/tests/ListSmbTests.cpp
+++ b/extensions/smb/tests/ListSmbTests.cpp
@@ -25,13 +25,14 @@
 #include "range/v3/algorithm/count_if.hpp"
 #include "range/v3/algorithm/find_if.hpp"
 #include "core/Resource.h"
+#include "unit/ProcessorUtils.h"
 
 namespace org::apache::nifi::minifi::extensions::smb::test {
 
 REGISTER_RESOURCE(MockSmbConnectionControllerService, ControllerService);
 
 TEST_CASE("ListSmb invalid network path") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<ListSmb>("ListSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<ListSmb>("ListSmb")};
   const auto list_smb = controller.getProcessor();
   auto smb_connection_node = 
controller.plan->addController("MockSmbConnectionControllerService", 
"smb_connection_controller_service");
   REQUIRE(controller.plan->setProperty(smb_connection_node, 
SmbConnectionControllerService::Hostname, 
utils::OsUtils::getHostName().value_or("localhost")));
diff --git a/extensions/smb/tests/PutSmbTests.cpp 
b/extensions/smb/tests/PutSmbTests.cpp
index a60f99e43..1f86d83e2 100644
--- a/extensions/smb/tests/PutSmbTests.cpp
+++ b/extensions/smb/tests/PutSmbTests.cpp
@@ -23,6 +23,7 @@
 #include "unit/SingleProcessorTestController.h"
 #include "range/v3/algorithm/count_if.hpp"
 #include "core/Resource.h"
+#include "unit/ProcessorUtils.h"
 
 namespace org::apache::nifi::minifi::extensions::smb::test {
 
@@ -35,7 +36,7 @@ std::string checkFileContent(const std::filesystem::path& 
path) {
 }
 
 TEST_CASE("PutSmb invalid network path") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<PutSmb>("PutSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<PutSmb>("PutSmb")};
   const auto put_smb = controller.getProcessor();
   auto smb_connection_node = 
controller.plan->addController("MockSmbConnectionControllerService", 
"smb_connection_controller_service");
   REQUIRE(controller.plan->setProperty(smb_connection_node, 
SmbConnectionControllerService::Hostname, 
utils::OsUtils::getHostName().value_or("localhost")));
@@ -48,7 +49,7 @@ TEST_CASE("PutSmb invalid network path") {
 }
 
 TEST_CASE("PutSmb conflict resolution test") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<PutSmb>("PutSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<PutSmb>("PutSmb")};
   const auto put_smb = controller.getProcessor();
 
   auto temp_directory = controller.createTempDirectory();
@@ -133,7 +134,7 @@ TEST_CASE("PutSmb conflict resolution test") {
 }
 
 TEST_CASE("PutSmb create missing dirs test") {
-  minifi::test::SingleProcessorTestController 
controller{std::make_unique<PutSmb>("PutSmb")};
+  minifi::test::SingleProcessorTestController 
controller{minifi::test::utils::make_processor<PutSmb>("PutSmb")};
   const auto put_smb = controller.getProcessor();
 
   auto temp_directory = controller.createTempDirectory();

Reply via email to