You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Steve Loughran (JIRA)" <ji...@apache.org> on 2017/12/07 11:45:00 UTC
[jira] [Commented] (HADOOP-15097)
AbstractContractDeleteTest::testDeleteNonEmptyDirRecursive with misleading
path
[ https://issues.apache.org/jira/browse/HADOOP-15097?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16281743#comment-16281743 ]
Steve Loughran commented on HADOOP-15097:
-----------------------------------------
Well spotted; no doubt a copy and paste error.
its important to have unique pathnames so that eventually consistent object stores don't have tests interfering with each other.
Patch welcome, ideally tested against any object stores to which you have credentials
> AbstractContractDeleteTest::testDeleteNonEmptyDirRecursive with misleading path
> -------------------------------------------------------------------------------
>
> Key: HADOOP-15097
> URL: https://issues.apache.org/jira/browse/HADOOP-15097
> Project: Hadoop Common
> Issue Type: Bug
> Components: fs, test
> Affects Versions: 3.0.0-beta1
> Reporter: zhoutai.zt
> Priority: Minor
>
> @Test
> public void testDeleteNonEmptyDirRecursive() throws Throwable {
> Path path = path("{color:red}testDeleteNonEmptyDirNonRecursive{color}");
> mkdirs(path);
> Path file = new Path(path, "childfile");
> ContractTestUtils.writeTextFile(getFileSystem(), file, "goodbye, world",
> true);
> assertDeleted(path, true);
> assertPathDoesNotExist("not deleted", file);
> }
> change testDeleteNonEmptyDirNonRecursive to testDeleteNonEmptyDirRecursive
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org