On 29 September 2010 12:28,  <nia...@apache.org> wrote:
> Author: niallp
> Date: Wed Sep 29 11:28:11 2010
> New Revision: 1002582
>
> URL: http://svn.apache.org/viewvc?rev=1002582&view=rev
> Log:
> IO-246 FilenameUtils - wildcardMatch gives incorrect results - thanks to wa
>
> Modified:
>    commons/proper/io/trunk/src/java/org/apache/commons/io/FilenameUtils.java
>    
> commons/proper/io/trunk/src/test/org/apache/commons/io/FilenameUtilsWildcardTestCase.java
...

> --- 
> commons/proper/io/trunk/src/test/org/apache/commons/io/FilenameUtilsWildcardTestCase.java
>  (original)
> +++ 
> commons/proper/io/trunk/src/test/org/apache/commons/io/FilenameUtilsWildcardTestCase.java
>  Wed Sep 29 11:28:11 2010
> @@ -182,6 +182,30 @@ public class FilenameUtilsWildcardTestCa
>         assertMatch("log.log.abc.log.abc.d", "*log?abc?d", true);
>     }
>
> +    /**
> +     * See https://issues.apache.org/jira/browse/IO-246
> +     */
> +    public void test_IO_246() {
> +
> +        // Tests for "*?"
> +        assertMatch("aaa", "*?", false);

Why assert that this fails?

Unless "*?" is documented as being disallowed, I don't think it should
be specifically disallowed by the test.

After all, "?" can follow "*" e.g. "*.?"

I think we should just document that "*?" does not currently work.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to