adamdebreceni commented on a change in pull request #920:
URL: https://github.com/apache/nifi-minifi-cpp/pull/920#discussion_r508484578



##########
File path: extensions/standard-processors/tests/unit/TailFileTests.cpp
##########
@@ -1129,8 +1129,11 @@ TEST_CASE("TailFile finds and finishes the renamed file 
and continues with the n
   utils::Identifier tail_file_uuid = 
utils::IdGenerator::getIdGenerator()->generate();
   const core::Relationship success_relationship{"success", "everything is 
fine"};
 
+  // use persistent state storage that defaults to rocksDB, not volatile
+  const auto configuration = std::make_shared<minifi::Configure>();

Review comment:
       is this necessary? `TestController::createPlan` default constructs a 
`Configure` object if `nullptr` is provided, and by default a 
`RocksDbPersistableKeyValueStoreService` is constructed




----------------------------------------------------------------
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


Reply via email to