Repository: hadoop Updated Branches: refs/heads/branch-2 340cd5f1b -> 36451f2d5 refs/heads/branch-2.9 98b086cec -> 60feb43b7 refs/heads/branch-3.0 9889e55e5 -> 95a4665ad refs/heads/branch-3.1 80641508c -> d7aa93b2f refs/heads/trunk b167d6076 -> 427fd027a
HADOOP-15308. TestConfiguration fails on Windows because of paths. Contributed by Xiao Liang. Project: http://git-wip-us.apache.org/repos/asf/hadoop/repo Commit: http://git-wip-us.apache.org/repos/asf/hadoop/commit/427fd027 Tree: http://git-wip-us.apache.org/repos/asf/hadoop/tree/427fd027 Diff: http://git-wip-us.apache.org/repos/asf/hadoop/diff/427fd027 Branch: refs/heads/trunk Commit: 427fd027a353f665846f43dfd73faf7561fedc07 Parents: b167d60 Author: Inigo Goiri <inigo...@apache.org> Authored: Tue Mar 13 20:01:07 2018 -0700 Committer: Inigo Goiri <inigo...@apache.org> Committed: Tue Mar 13 20:01:07 2018 -0700 ---------------------------------------------------------------------- .../org/apache/hadoop/conf/TestConfiguration.java | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hadoop/blob/427fd027/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java ---------------------------------------------------------------------- diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java index c9dd7cc..f1d68cd 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/conf/TestConfiguration.java @@ -61,7 +61,6 @@ import static org.apache.hadoop.conf.StorageUnit.MB; import static org.apache.hadoop.conf.StorageUnit.TB; import static org.hamcrest.core.Is.is; import static org.junit.Assert.*; -import static org.junit.Assert.assertArrayEquals; import org.apache.commons.lang.StringUtils; import org.apache.hadoop.conf.Configuration.IntegerRanges; @@ -93,8 +92,8 @@ public class TestConfiguration { final static String CONFIG_CORE = new File("./core-site.xml") .getAbsolutePath(); final static String CONFIG_FOR_ENUM = new File("./test-config-enum-TestConfiguration.xml").getAbsolutePath(); - final static String CONFIG_FOR_URI = "file://" - + new File("./test-config-uri-TestConfiguration.xml").getAbsolutePath(); + final static String CONFIG_FOR_URI = new File( + "./test-config-uri-TestConfiguration.xml").toURI().toString(); private static final String CONFIG_MULTI_BYTE = new File( "./test-config-multi-byte-TestConfiguration.xml").getAbsolutePath(); @@ -877,7 +876,8 @@ public class TestConfiguration { out.close(); out=new BufferedWriter(new FileWriter(CONFIG)); writeHeader(); - declareSystemEntity("configuration", "d", CONFIG2); + declareSystemEntity("configuration", "d", + new Path(CONFIG2).toUri().toString()); writeConfiguration(); appendProperty("a", "b"); appendProperty("c", "&d;"); @@ -1749,7 +1749,7 @@ public class TestConfiguration { assertEquals("test.key2", jp1.getKey()); assertEquals("value2", jp1.getValue()); assertEquals(true, jp1.isFinal); - assertEquals(fileResource.toUri().getPath(), jp1.getResource()); + assertEquals(fileResource.toString(), jp1.getResource()); // test xml format outWriter = new StringWriter(); @@ -1760,7 +1760,7 @@ public class TestConfiguration { assertEquals(1, actualConf1.size()); assertEquals("value2", actualConf1.get("test.key2")); assertTrue(actualConf1.getFinalParameters().contains("test.key2")); - assertEquals(fileResource.toUri().getPath(), + assertEquals(fileResource.toString(), actualConf1.getPropertySources("test.key2")[0]); // case 2: dump an non existing property @@ -2271,7 +2271,8 @@ public class TestConfiguration { final File tmpDir = GenericTestUtils.getRandomizedTestDir(); tmpDir.mkdirs(); final String ourUrl = new URI(LocalJavaKeyStoreProvider.SCHEME_NAME, - "file", new File(tmpDir, "test.jks").toString(), null).toString(); + "file", new File(tmpDir, "test.jks").toURI().getPath(), + null).toString(); conf = new Configuration(false); conf.set(CredentialProviderFactory.CREDENTIAL_PROVIDER_PATH, ourUrl); @@ -2299,7 +2300,8 @@ public class TestConfiguration { final File tmpDir = GenericTestUtils.getRandomizedTestDir(); tmpDir.mkdirs(); final String ourUrl = new URI(LocalJavaKeyStoreProvider.SCHEME_NAME, - "file", new File(tmpDir, "test.jks").toString(), null).toString(); + "file", new File(tmpDir, "test.jks").toURI().getPath(), + null).toString(); conf = new Configuration(false); conf.set(CredentialProviderFactory.CREDENTIAL_PROVIDER_PATH, ourUrl); --------------------------------------------------------------------- To unsubscribe, e-mail: common-commits-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-commits-h...@hadoop.apache.org