lordgamez commented on a change in pull request #1205:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1205#discussion_r737244286



##########
File path: extensions/sftp/tests/ListSFTPTests.cpp
##########
@@ -223,6 +223,7 @@ TEST_CASE_METHOD(ListSFTPTestsFixture, "ListSFTP list 
non-readable dir", "[ListS
   REQUIRE(false == LogTestController::getInstance().contains("from ListSFTP to 
relationship success"));
   REQUIRE(LogTestController::getInstance().contains("Failed to open remote 
directory \"nifi_test\", error: LIBSSH2_FX_PERMISSION_DENIED"));
   REQUIRE(LogTestController::getInstance().contains("There are no files to 
list. Yielding."));
+  REQUIRE(0 == chmod((src_dir + "/vfs/nifi_test").c_str(), 0755));

Review comment:
       As  a directory it needed to be executable for the removal to access its 
subcontents.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to