You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2009/07/25 00:16:29 UTC

svn commit: r797671 - /commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java

Author: ggregory
Date: Fri Jul 24 22:16:29 2009
New Revision: 797671

URL: http://svn.apache.org/viewvc?rev=797671&view=rev
Log:
Refactor comoon code. URLCodec test coverage now 100%/100%.

Modified:
    commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java

Modified: commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java
URL: http://svn.apache.org/viewvc/commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java?rev=797671&r1=797670&r2=797671&view=diff
==============================================================================
--- commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java (original)
+++ commons/proper/codec/trunk/src/test/org/apache/commons/codec/net/URLCodecTest.java Fri Jul 24 22:16:29 2009
@@ -138,13 +138,21 @@
             fail("DecoderException should have been thrown");
         } catch (DecoderException e) {
             // Expected. Move on
-        }
+        }        
         try {
+            // Bad 1st char after %
             urlCodec.decode("%WW");
             fail("DecoderException should have been thrown");
         } catch (DecoderException e) {
             // Expected. Move on
         }
+        try {
+            // Bad 2nd char after %
+            urlCodec.decode("%0W");
+            fail("DecoderException should have been thrown");
+        } catch (DecoderException e) {
+            // Expected. Move on
+        }
         this.validateState(urlCodec);
     }