You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by sh...@apache.org on 2021/01/23 02:29:19 UTC
[hadoop] branch branch-2.10 updated: HDFS-15632.
AbstractContractDeleteTest should set recursive peremeter to true for
recursive test cases. Contributed by Anton Kutuzov.
This is an automated email from the ASF dual-hosted git repository.
shv pushed a commit to branch branch-2.10
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-2.10 by this push:
new 0ed997a HDFS-15632. AbstractContractDeleteTest should set recursive peremeter to true for recursive test cases. Contributed by Anton Kutuzov.
0ed997a is described below
commit 0ed997abababef26ac06623e1a663d806fdbb17f
Author: Anton Kutuzov <an...@yandex.ru>
AuthorDate: Wed Jan 20 18:38:02 2021 +0300
HDFS-15632. AbstractContractDeleteTest should set recursive peremeter to true for recursive test cases. Contributed by Anton Kutuzov.
(cherry picked from commit 91d4ba57c5b85379303ac8fb2a1a03ba10b07d4e)
---
.../org/apache/hadoop/fs/contract/AbstractContractDeleteTest.java | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractContractDeleteTest.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractContractDeleteTest.java
index 328c8e1..08df1d4 100644
--- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractContractDeleteTest.java
+++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/contract/AbstractContractDeleteTest.java
@@ -49,18 +49,17 @@ public abstract class AbstractContractDeleteTest extends
Path path = path("testDeleteNonexistentPathRecursive");
assertPathDoesNotExist("leftover", path);
ContractTestUtils.rejectRootOperation(path);
- assertFalse("Returned true attempting to delete"
+ assertFalse("Returned true attempting to recursively delete"
+ " a nonexistent path " + path,
- getFileSystem().delete(path, false));
+ getFileSystem().delete(path, true));
}
-
@Test
public void testDeleteNonexistentPathNonRecursive() throws Throwable {
Path path = path("testDeleteNonexistentPathNonRecursive");
assertPathDoesNotExist("leftover", path);
ContractTestUtils.rejectRootOperation(path);
- assertFalse("Returned true attempting to recursively delete"
+ assertFalse("Returned true attempting to non recursively delete"
+ " a nonexistent path " + path,
getFileSystem().delete(path, false));
}
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org