You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Benedikt Ritter (JIRA)" <ji...@apache.org> on 2017/02/14 07:25:41 UTC

[jira] [Resolved] (VALIDATOR-418) UrlValidatorTest: testIsValid() does not run all tests

     [ https://issues.apache.org/jira/browse/VALIDATOR-418?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benedikt Ritter resolved VALIDATOR-418.
---------------------------------------
       Resolution: Fixed
         Assignee: Benedikt Ritter
    Fix Version/s: 1.6

> 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
>            Assignee: Benedikt Ritter
>            Priority: Minor
>             Fix For: 1.6
>
>         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)