Author: suresh Date: Sun Dec 16 23:22:22 2012 New Revision: 1422713 URL: http://svn.apache.org/viewvc?rev=1422713&view=rev Log: HDFS-4316. branch-trunk-win contains test code accidentally added during work on fixing tests on Windows. Contributed by Chris Nauroth.
Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/CHANGES.branch-trunk-win.txt hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsCreateMkdir.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsPermission.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSymlink.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestHDFSFileContextMainOperations.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemAtHdfsRoot.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemHdfs.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsAtHdfsRoot.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsDefaultValue.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsFileStatusHdfs.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsHdfs.java hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/CHANGES.branch-trunk-win.txt URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/CHANGES.branch-trunk-win.txt?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/CHANGES.branch-trunk-win.txt (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/CHANGES.branch-trunk-win.txt Sun Dec 16 23:22:22 2012 @@ -6,3 +6,5 @@ branch-trunk-win changes - unreleased HDFS-4163. HDFS distribution build fails on Windows. (Chris Nauroth via suresh) + HDFS-4316. branch-trunk-win contains test code accidentally added during + work on fixing tests on Windows. (Chris Nauroth via suresh) Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsCreateMkdir.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsCreateMkdir.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsCreateMkdir.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsCreateMkdir.java Sun Dec 16 23:22:22 2012 @@ -37,11 +37,7 @@ public class TestFcHdfsCreateMkdir exten private static MiniDFSCluster cluster; private static Path defaultWorkingDirectory; - - public TestFcHdfsCreateMkdir() { - super(new FileContextTestHelper(true)); - } - + @BeforeClass public static void clusterSetupAtBegining() throws IOException, LoginException, URISyntaxException { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsPermission.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsPermission.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsPermission.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsPermission.java Sun Dec 16 23:22:22 2012 @@ -56,10 +56,6 @@ public class TestFcHdfsPermission extend cluster.shutdown(); } - public TestFcHdfsPermission() { - super(new FileContextTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSetUMask.java Sun Dec 16 23:22:22 2012 @@ -78,10 +78,7 @@ public class TestFcHdfsSetUMask { private static final FsPermission WIDE_OPEN_TEST_UMASK = FsPermission .createImmutable((short) (0777 ^ 0777)); - - private final FileContextTestHelper fileContextTestHelper = - new FileContextTestHelper(true); - + @BeforeClass public static void clusterSetupAtBegining() throws IOException, LoginException, URISyntaxException { @@ -230,7 +227,7 @@ public class TestFcHdfsSetUMask { FsPermission expectedPerms) throws IOException { Path f = getTestRootPath(fc,"foo"); fc.setUMask(umask); - fileContextTestHelper.createFile(fc, f); + createFile(fc, f); Assert.assertTrue(isFile(fc, f)); Assert.assertEquals("permissions on file are wrong", expectedPerms , fc.getFileStatus(f).getPermission()); @@ -244,19 +241,12 @@ public class TestFcHdfsSetUMask { Path fParent = getTestRootPath(fc, "NonExisting"); Assert.assertFalse(exists(fc, fParent)); fc.setUMask(umask); - fileContextTestHelper.createFile(fc, f); + createFile(fc, f); Assert.assertTrue(isFile(fc, f)); Assert.assertEquals("permissions on file are wrong", expectedFilePerms, fc.getFileStatus(f).getPermission()); Assert.assertEquals("permissions on parent directory are wrong", expectedDirPerms, fc.getFileStatus(fParent).getPermission()); } - - private Path getTestRootPath(FileContext fc) { - return fileContextTestHelper.getTestRootPath(fc); - } - - private Path getTestRootPath(FileContext fc, String pathString) { - return fileContextTestHelper.getTestRootPath(fc, pathString); - } + } Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSymlink.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSymlink.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSymlink.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestFcHdfsSymlink.java Sun Dec 16 23:22:22 2012 @@ -17,6 +17,7 @@ */ package org.apache.hadoop.fs; +import static org.apache.hadoop.fs.FileContextTestHelper.getAbsoluteTestRootDir; import static org.junit.Assert.assertEquals; import static org.junit.Assert.fail; @@ -100,17 +101,11 @@ public class TestFcHdfsSymlink extends F cluster.shutdown(); } - public TestFcHdfsSymlink() { - super(new FileContextTestHelper(true)); - } - @Test /** Access a file using a link that spans Hdfs to LocalFs */ public void testLinkAcrossFileSystems() throws IOException { - Path localDir = new Path("file", null, getAbsoluteTestRootDir(fc) + - "/test"); - Path localFile = new Path("file", null, getAbsoluteTestRootDir(fc) + - "/test/file"); + Path localDir = new Path("file://"+getAbsoluteTestRootDir(fc)+"/test"); + Path localFile = new Path("file://"+getAbsoluteTestRootDir(fc)+"/test/file"); Path link = new Path(testBaseDir1(), "linkToFile"); FileContext localFc = FileContext.getLocalFSFileContext(); localFc.delete(localDir, true); @@ -126,8 +121,7 @@ public class TestFcHdfsSymlink extends F @Test /** Test renaming a file across two file systems using a link */ public void testRenameAcrossFileSystemsViaLink() throws IOException { - Path localDir = new Path("file", null, getAbsoluteTestRootDir(fc) + - "/test"); + Path localDir = new Path("file://"+getAbsoluteTestRootDir(fc)+"/test"); Path hdfsFile = new Path(testBaseDir1(), "file"); Path link = new Path(testBaseDir1(), "link"); Path hdfsFileNew = new Path(testBaseDir1(), "fileNew"); Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestHDFSFileContextMainOperations.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestHDFSFileContextMainOperations.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestHDFSFileContextMainOperations.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/TestHDFSFileContextMainOperations.java Sun Dec 16 23:22:22 2012 @@ -19,6 +19,7 @@ package org.apache.hadoop.fs; import static org.apache.hadoop.fs.FileContextTestHelper.exists; +import static org.apache.hadoop.fs.FileContextTestHelper.getTestRootPath; import java.io.IOException; import java.net.URISyntaxException; @@ -77,10 +78,6 @@ public class TestHDFSFileContextMainOper cluster.shutdown(); } - public TestHDFSFileContextMainOperations() { - super(new FileContextTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemAtHdfsRoot.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemAtHdfsRoot.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemAtHdfsRoot.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemAtHdfsRoot.java Sun Dec 16 23:22:22 2012 @@ -66,10 +66,6 @@ public class TestViewFileSystemAtHdfsRoo } } - public TestViewFileSystemAtHdfsRoot() { - super(new FileSystemTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemHdfs.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemHdfs.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemHdfs.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFileSystemHdfs.java Sun Dec 16 23:22:22 2012 @@ -81,17 +81,13 @@ public class TestViewFileSystemHdfs exte cluster.shutdown(); } - public TestViewFileSystemHdfs() { - super(new FileSystemTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { // create the test root on local_fs fsTarget = fHdfs; fsTarget2 = fHdfs2; - targetTestRoot2 = getAbsoluteTestRootPath(fsTarget2); + targetTestRoot2 = FileSystemTestHelper.getAbsoluteTestRootPath(fsTarget2); super.setUp(); } Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsAtHdfsRoot.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsAtHdfsRoot.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsAtHdfsRoot.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsAtHdfsRoot.java Sun Dec 16 23:22:22 2012 @@ -63,10 +63,6 @@ public class TestViewFsAtHdfsRoot extend cluster.shutdown(); } - public TestViewFsAtHdfsRoot() { - super(new FileContextTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsDefaultValue.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsDefaultValue.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsDefaultValue.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsDefaultValue.java Sun Dec 16 23:22:22 2012 @@ -81,7 +81,7 @@ public class TestViewFsDefaultValue { cluster = new MiniDFSCluster.Builder(CONF).numDataNodes(DFS_REPLICATION_DEFAULT + 1).build(); cluster.waitClusterUp(); fHdfs = cluster.getFileSystem(); - new FileSystemTestHelper(true).createFile(fHdfs, testFileName); + FileSystemTestHelper.createFile(fHdfs, testFileName); Configuration conf = ViewFileSystemTestSetup.createConfig(); ConfigUtil.addLink(conf, "/tmp", new URI(fHdfs.getUri().toString() + "/tmp")); Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsFileStatusHdfs.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsFileStatusHdfs.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsFileStatusHdfs.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsFileStatusHdfs.java Sun Dec 16 23:22:22 2012 @@ -58,9 +58,6 @@ public class TestViewFsFileStatusHdfs { private static FileSystem fHdfs; private static FileSystem vfs; - private final FileSystemTestHelper fileSystemTestHelper = - new FileSystemTestHelper(true); - @BeforeClass public static void clusterSetupAtBegining() throws IOException, LoginException, URISyntaxException { @@ -82,7 +79,7 @@ public class TestViewFsFileStatusHdfs { @Test public void testFileStatusSerialziation() throws IOException, URISyntaxException { - long len = fileSystemTestHelper.createFile(fHdfs, testfilename); + long len = FileSystemTestHelper.createFile(fHdfs, testfilename); FileStatus stat = vfs.getFileStatus(new Path(testfilename)); assertEquals(len, stat.getLen()); // check serialization/deserialization @@ -98,9 +95,9 @@ public class TestViewFsFileStatusHdfs { @Test public void testGetFileChecksum() throws IOException, URISyntaxException { // Create two different files in HDFS - fileSystemTestHelper.createFile(fHdfs, someFile); - FileSystemTestHelper.createFile(fHdfs, - fileSystemTestHelper.getTestRootPath(fHdfs, someFile + "other"), 1, 512); + FileSystemTestHelper.createFile(fHdfs, someFile); + FileSystemTestHelper.createFile(fHdfs, FileSystemTestHelper + .getTestRootPath(fHdfs, someFile + "other"), 1, 512); // Get checksum through ViewFS FileChecksum viewFSCheckSum = vfs.getFileChecksum( new Path("/vfstmp/someFileForTestGetFileChecksum")); Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsHdfs.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsHdfs.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsHdfs.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/fs/viewfs/TestViewFsHdfs.java Sun Dec 16 23:22:22 2012 @@ -62,10 +62,6 @@ public class TestViewFsHdfs extends View cluster.shutdown(); } - public TestViewFsHdfs() { - super(new FileContextTestHelper(true)); - } - @Override @Before public void setUp() throws Exception { Modified: hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java URL: http://svn.apache.org/viewvc/hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java?rev=1422713&r1=1422712&r2=1422713&view=diff ============================================================================== --- hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java (original) +++ hadoop/common/branches/branch-trunk-win/hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/web/TestFSMainOperationsWebHdfs.java Sun Dec 16 23:22:22 2012 @@ -18,6 +18,7 @@ package org.apache.hadoop.hdfs.web; import static org.apache.hadoop.fs.FileSystemTestHelper.exists; +import static org.apache.hadoop.fs.FileSystemTestHelper.getTestRootPath; import java.io.IOException; import java.net.URI;