commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Benedikt Ritter (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (VALIDATOR-418) UrlValidatorTest: testIsValid() does not run all tests
Date Tue, 14 Feb 2017 07:24:41 GMT

    [ https://issues.apache.org/jira/browse/VALIDATOR-418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15865257#comment-15865257
] 

Benedikt Ritter commented on VALIDATOR-418:
-------------------------------------------

{code}
~/w/a/c/commons-validator > svn ci -m "VALIDATOR-418: UrlValidatorTest: testIsValid() does
not run all tests. Thanks to Robert McGuigan."
Sending        RELEASE-NOTES.txt
Sending        src/changes/changes.xml
Sending        src/test/java/org/apache/commons/validator/routines/UrlValidatorTest.java
Transmitting file data ...done
Committing transaction...
Committed revision 1782919.
{code}

Thank you!

> UrlValidatorTest: testIsValid() does not run all tests
> ------------------------------------------------------
>
>                 Key: VALIDATOR-418
>                 URL: https://issues.apache.org/jira/browse/VALIDATOR-418
>             Project: Commons Validator
>          Issue Type: Bug
>            Reporter: Robert McGuigan
>            Priority: Minor
>         Attachments: Validator-418-test.patch
>
>
> The method testIsValid() does not iterate through all possible combinations of scheme,
authority, etc.  Namely, the last element in testUrlScheme is skipped.  This is due to a bug
in incrementTestPartsIndex, which returns false (indicating the maximum set of indices has
been reached) prematurely.
> Incidentally, the last test scheme currently present is the empty scheme "".  This test
case has been removed in the provided patch because the url string parser appears to assume
that the scheme is nonempty.  (e.g. if the scheme is empty, the scheme field will often be
populated with the authority data).



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message