You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2016/05/02 18:46:59 UTC

svn commit: r1742007 - /commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java

Author: sebb
Date: Mon May  2 16:46:44 2016
New Revision: 1742007

URL: http://svn.apache.org/viewvc?rev=1742007&view=rev
Log:
Must also test return code

Modified:
    commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java

Modified: commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java
URL: http://svn.apache.org/viewvc/commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java?rev=1742007&r1=1742006&r2=1742007&view=diff
==============================================================================
--- commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java (original)
+++ commons/proper/validator/trunk/src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java Mon May  2 16:46:44 2016
@@ -278,13 +278,13 @@ protected void setUp() {
     public void testValidator391OK() {
         String[] schemes = {"file"};
         UrlValidator urlValidator = new UrlValidator(schemes);
-        urlValidator.isValid("file:///C:/path/to/dir/");
+        assertTrue(urlValidator.isValid("file:///C:/path/to/dir/"));
     }
     
     public void testValidator391FAILS() {
         String[] schemes = {"file"};
         UrlValidator urlValidator = new UrlValidator(schemes);
-        urlValidator.isValid("file:/C:/path/to/dir/");
+        assertTrue(urlValidator.isValid("file:/C:/path/to/dir/"));
     }
     
     public void testValidator309() {