added tests, fixed one bug

Author: Claudenw

src/main/java/org/codehaus/plexus/util/MatchPattern.java

@@ -65,7 +65,7 @@ private MatchPattern( String source, String separator )
      * @return the source string without Ant or Regex pattern markers.
      */
     public String getSource() {
-        return source;
+        return regexPattern == null ? source : regexPattern;
     }
 
     public boolean matchPath( String str, boolean isCaseSensitive )

src/test/java/org/codehaus/plexus/util/MatchPatternTest.java

@@ -16,6 +16,8 @@
  * limitations under the License.
  */
 
+import static org.junit.Assert.assertArrayEquals;
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
@@ -30,6 +32,20 @@
  */
 public class MatchPatternTest
 {
+    /**
+     * <p>testGetSource</p>
+     */
+    @Test
+    public void testGetSource()
+    {
+        MatchPattern mp = MatchPattern.fromString( "ABC*" );
+        assertEquals("ABC*", mp.getSource());
+        mp = MatchPattern.fromString( "%ant[some/ABC*]" );
+        assertEquals("some/ABC*", mp.getSource());
+        mp = MatchPattern.fromString( "%regex[[ABC].*]" );
+        assertEquals("[ABC].*", mp.getSource());
+    }
+
     /**
      * <p>testMatchPath.</p>
      *
@@ -63,4 +79,20 @@ public void testMatchPatternStart()
         assertFalse( mp.matchPatternStart( "XXXX", false ) );
     }
 
+    /**
+     * <p>testTokenizePathToString.</p>
+     */
+    @Test
+    public void testTokenizePathToString()
+    {
+        String[] expected = {"hello", "world"};
+        String[] actual = MatchPattern.tokenizePathToString("hello/world", "/");
+        assertArrayEquals(expected, actual);
+
+        actual = MatchPattern.tokenizePathToString("/hello/world", "/");
+        assertArrayEquals(expected, actual);
+
+        actual = MatchPattern.tokenizePathToString("/hello/world/", "/");
+        assertArrayEquals(expected, actual);
+    }
 }

src/test/java/org/codehaus/plexus/util/MatchPatternsTest.java

@@ -16,11 +16,15 @@
  * limitations under the License.
  */
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
 import org.junit.Test;
 
+import java.util.Arrays;
+import java.util.List;
+
 /**
  * <p>MatchPatternsTest class.</p>
  *
@@ -30,6 +34,18 @@
  */
 public class MatchPatternsTest
 {
+    /**
+     * <p>testGetSource</p>
+     */
+    @Test
+    public void testGetSources()
+    {
+        List<String> expected = Arrays.asList("ABC**", "some/ABC*", "[ABC].*");
+        MatchPatterns from = MatchPatterns.from( "ABC**", "%ant[some/ABC*]", "%regex[[ABC].*]" );
+        List<String> actual = from.getSources();
+        assertEquals(expected, actual);
+    }
+
     /**
      * <p>testMatches.</p>
      *

src/test/java/org/codehaus/plexus/util/SelectorUtilsTest.java

@@ -16,6 +16,7 @@
  * limitations under the License.
  */
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
 
@@ -31,6 +32,41 @@
  */
 public class SelectorUtilsTest
 {
+    /**
+     * <p>testExtractPattern.</p>
+     */
+    @Test
+    public void testExtractPattern() {
+        assertEquals("[A-Z].*", SelectorUtils.extractPattern("%regex[[A-Z].*]", "/"));
+        assertEquals("ABC*", SelectorUtils.extractPattern("%ant[ABC*]", "/"));
+        assertEquals("some/ABC*", SelectorUtils.extractPattern("%ant[some/ABC*]", "/"));
+        assertEquals("some\\ABC*", SelectorUtils.extractPattern("%ant[some\\ABC*]", "\\"));
+        assertEquals("some/ABC*", SelectorUtils.extractPattern("%ant[some\\ABC*]", "/"));
+        assertEquals("some\\ABC*", SelectorUtils.extractPattern("%ant[some/ABC*]", "\\"));
+    }
+
+    /**
+     * <p>testIsAntPrefixedPattern.</p>
+     */
+    @Test
+    public void testIsAntPrefixedPattern() {
+        assertFalse(SelectorUtils.isAntPrefixedPattern("%ant[A]")); // single char not allowed
+        assertTrue(SelectorUtils.isAntPrefixedPattern("%ant[AB]"));
+        assertFalse(SelectorUtils.isAntPrefixedPattern("%ant[]"));
+        assertFalse(SelectorUtils.isAntPrefixedPattern("*"));
+    }
+
+    /**
+     * <p>testIsRegexPrefixedPattern.</p>
+     */
+    @Test
+    public void testIsRegexPrefixedPattern() {
+        assertFalse(SelectorUtils.isRegexPrefixedPattern("%regex[A]")); // single char not allowed
+        assertTrue(SelectorUtils.isRegexPrefixedPattern("%regex[.*]"));
+        assertFalse(SelectorUtils.isRegexPrefixedPattern("%regex[]"));
+        assertFalse(SelectorUtils.isRegexPrefixedPattern("*"));
+    }
+
     /**
      * <p>testMatchPath_DefaultFileSeparator.</p>
      */