remove test

treeverse · Oct 4, 2023 · eeae69b · eeae69b
1 parent 44265b2
commit eeae69b
Showing 1 changed file with 0 additions and 28 deletions.
diff --git a/clients/hadoopfs/src/test/java/io/lakefs/LakeFSFileSystemServerTest.java b/clients/hadoopfs/src/test/java/io/lakefs/LakeFSFileSystemServerTest.java
@@ -701,34 +701,6 @@ public void testRename_existingDirToExistingNonEmptyDirName() throws IOException
         Assert.assertFalse(renamed);
     }
 
-    // /**
-    //  * Check that a file is renamed when working against a lakeFS version
-    //  * where CopyObject API doesn't exist
-    //  */
-    // TODO(johnnyaug): Do we still need this test?
-    //
-    // @Test
-    // public void testRename_fallbackStageAPI() throws ApiException, IOException {
-    //     Path src = new Path("lakefs://repo/main/existing-dir1/existing.src");
-    //     ObjectLocation srcObjLoc = fs.pathToObjectLocation(src);
-    //     mockExistingFilePath(srcObjLoc);
-
-    //     Path fileInDstDir = new Path("lakefs://repo/main/existing-dir2/existing.src");
-    //     ObjectLocation fileObjLoc = fs.pathToObjectLocation(fileInDstDir);
-    //     Path dst = new Path("lakefs://repo/main/existing-dir2");
-    //     ObjectLocation dstObjLoc = fs.pathToObjectLocation(dst);
-
-    //     mockExistingDirPath(dstObjLoc, ImmutableList.of(fileObjLoc));
-    //     mockDirectoryMarker(fs.pathToObjectLocation(src.getParent()));
-    //     mockMissingCopyAPI();
-
-    //     boolean renamed = fs.rename(src, dst);
-    //     Assert.assertTrue(renamed);
-    //     Path expectedDstPath = new Path("lakefs://repo/main/existing-dir2/existing.src");
-    //     Assert.assertTrue(dstPathLinkedToSrcPhysicalAddress(srcObjLoc, fs.pathToObjectLocation(expectedDstPath)));
-    //     verifyObjDeletion(srcObjLoc);
-    // }
-
     @Test
     public void testRename_srcAndDstOnDifferentBranch() throws IOException {
         Path src = new Path("lakefs://repo/branch/existing.src");