NIFI-1267: TestListFile attempts to ensure that files that are not readable are skipped but the assumptions made are not valid if running as root; removed this check, since it is not reliable
Signed-off-by: joewitt <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/nifi/repo Commit: http://git-wip-us.apache.org/repos/asf/nifi/commit/0c7617b6 Tree: http://git-wip-us.apache.org/repos/asf/nifi/tree/0c7617b6 Diff: http://git-wip-us.apache.org/repos/asf/nifi/diff/0c7617b6 Branch: refs/heads/master Commit: 0c7617b686ac6cf0904fed1ab8afa1ba2ea6fa83 Parents: 9524f63 Author: Mark Payne <[email protected]> Authored: Mon Dec 7 17:41:13 2015 -0500 Committer: joewitt <[email protected]> Committed: Mon Dec 7 20:42:07 2015 -0500 ---------------------------------------------------------------------- .../apache/nifi/processors/standard/TestListFile.java | 11 ----------- 1 file changed, 11 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/nifi/blob/0c7617b6/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListFile.java ---------------------------------------------------------------------- diff --git a/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListFile.java b/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListFile.java index 9fd7a3d..7111686 100644 --- a/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListFile.java +++ b/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListFile.java @@ -473,17 +473,6 @@ public class TestListFile { runner.assertAllFlowFilesTransferred(ListFile.REL_SUCCESS); final List<MockFlowFile> successFiles1 = runner.getFlowFilesForRelationship(ListFile.REL_SUCCESS); assertEquals(3, successFiles1.size()); - - // make file2 unreadable and test (setReadable() does not work on Windows) - if (!System.getProperty("os.name").toLowerCase().startsWith("windows")) { - assertTrue(file2.setReadable(false)); - runner.clearTransferState(); - runner.setProperty(ListFile.RECURSE, "false"); - runner.run(); - runner.assertAllFlowFilesTransferred(ListFile.REL_SUCCESS); - final List<MockFlowFile> successFiles2 = runner.getFlowFilesForRelationship(ListFile.REL_SUCCESS); - assertEquals(2, successFiles2.size()); - } } @Test
