HADOOP-15497. TestTrash should use proper test path to avoid failing on Windows. Contributed by Anbang Hu.

This commit is contained in:
Inigo Goiri 2018-05-29 09:11:08 -07:00
parent 31ab960f4f
commit 3c75f8e493

View File

@ -49,9 +49,11 @@
*/ */
public class TestTrash { public class TestTrash {
private final static Path TEST_DIR = new Path(GenericTestUtils.getTempPath( private final static File BASE_PATH = new File(GenericTestUtils.getTempPath(
"testTrash")); "testTrash"));
private final static Path TEST_DIR = new Path(BASE_PATH.getAbsolutePath());
@Before @Before
public void setUp() throws IOException { public void setUp() throws IOException {
// ensure each test initiates a FileSystem instance, // ensure each test initiates a FileSystem instance,
@ -682,7 +684,7 @@ public void tearDown() throws IOException {
static class TestLFS extends LocalFileSystem { static class TestLFS extends LocalFileSystem {
Path home; Path home;
TestLFS() { TestLFS() {
this(new Path(TEST_DIR, "user/test")); this(TEST_DIR);
} }
TestLFS(final Path home) { TestLFS(final Path home) {
super(new RawLocalFileSystem() { super(new RawLocalFileSystem() {
@ -809,8 +811,8 @@ public static void verifyMoveEmptyDirToTrash(FileSystem fs,
*/ */
public static void verifyTrashPermission(FileSystem fs, Configuration conf) public static void verifyTrashPermission(FileSystem fs, Configuration conf)
throws IOException { throws IOException {
Path caseRoot = new Path( Path caseRoot = new Path(BASE_PATH.getPath(),
GenericTestUtils.getTempPath("testTrashPermission")); "testTrashPermission");
try (FileSystem fileSystem = fs){ try (FileSystem fileSystem = fs){
Trash trash = new Trash(fileSystem, conf); Trash trash = new Trash(fileSystem, conf);
FileSystemTestWrapper wrapper = FileSystemTestWrapper wrapper =