Author: eli
Date: Mon Aug 15 21:47:00 2011
New Revision: 1158025

URL: http://svn.apache.org/viewvc?rev=1158025&view=rev
Log:
HDFS-2233. Add WebUI tests with URI reserved chars. Contributed by Eli Collins

Modified:
    hadoop/common/trunk/hdfs/CHANGES.txt
    hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java

Modified: hadoop/common/trunk/hdfs/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hdfs/CHANGES.txt?rev=1158025&r1=1158024&r2=1158025&view=diff
==============================================================================
--- hadoop/common/trunk/hdfs/CHANGES.txt (original)
+++ hadoop/common/trunk/hdfs/CHANGES.txt Mon Aug 15 21:47:00 2011
@@ -663,6 +663,8 @@ Trunk (unreleased changes)
     HDFS-2237. Change UnderReplicatedBlocks from public to package private.
     (szetszwo)
 
+    HDFS-2233. Add WebUI tests with URI reserved chars. (eli)
+
   OPTIMIZATIONS
 
     HDFS-1458. Improve checkpoint performance by avoiding unnecessary image

Modified: hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java
URL: http://svn.apache.org/viewvc/hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java?rev=1158025&r1=1158024&r2=1158025&view=diff
==============================================================================
--- hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java (original)
+++ hadoop/common/trunk/hdfs/src/test/hdfs/org/apache/hadoop/hdfs/server/datanode/TestDatanodeJsp.java Mon Aug 15 21:47:00 2011
@@ -82,31 +82,22 @@ public class TestDatanodeJsp {
     try {
       cluster = new MiniDFSCluster.Builder(CONF).build();
       cluster.waitActive();
-      
-      testViewingFile(cluster, "/test-file", false);
-      testViewingFile(cluster, "/tmp/test-file", false);
-      testViewingFile(cluster, "/tmp/test-file%with goofy&characters", false);
-      
-      testViewingFile(cluster, "/test-file", true);
-      testViewingFile(cluster, "/tmp/test-file", true);
-      testViewingFile(cluster, "/tmp/test-file%with goofy&characters", true);
-
-      testViewingFile(cluster, "/foo bar", true);
-      testViewingFile(cluster, "/foo+bar", true);
-      testViewingFile(cluster, "/foo;bar", true);
-      testViewingFile(cluster, "/foo=bar", true);
-      testViewingFile(cluster, "/foo,bar", true);
-      testViewingFile(cluster, "/foo?bar", true);
-      testViewingFile(cluster, "/foo\">bar", true);
-      
-      testViewingFile(cluster, "/foo bar", false);
-      // See HDFS-2233
-      //testViewingFile(cluster, "/foo+bar", false);
-      //testViewingFile(cluster, "/foo;bar", false);
-      testViewingFile(cluster, "/foo=bar", false);
-      testViewingFile(cluster, "/foo,bar", false);
-      testViewingFile(cluster, "/foo?bar", false);
-      testViewingFile(cluster, "/foo\">bar", false);
+      String paths[] = {
+        "/test-file",
+        "/tmp/test-file",
+        "/tmp/test-file%with goofy&characters",
+        "/foo bar/foo bar",
+        "/foo+bar/foo+bar",
+        "/foo;bar/foo;bar",
+        "/foo=bar/foo=bar",
+        "/foo,bar/foo,bar",
+        "/foo?bar/foo?bar",
+        "/foo\">bar/foo\">bar"
+      };
+      for (String p : paths) {
+        testViewingFile(cluster, p, false);
+        testViewingFile(cluster, p, true);
+      }
     } finally {
       if (cluster != null) {
         cluster.shutdown();