Merge pull request #219 from reynardian/find-bugfixes-pe-d

Update find command to allow partial matching of phone number with less than 3 digits

Author: Annazx

src/main/java/seedu/address/logic/parser/FindCommandParser.java

@@ -125,7 +125,9 @@ private void validateValueByPrefix(Prefix prefix, String value) throws ParseExce
         } else if (prefix.equals(PREFIX_ADDRESS)) {
             ParserUtil.parseAddress(value);
         } else if (prefix.equals(PREFIX_PARENT_PHONE)) {
-            ParserUtil.parsePhone(value);
+            if (!value.matches("\\d+")) {
+                throw new ParseException("Phone numbers should only contain digits.");
+            }
         } else if (prefix.equals(PREFIX_PARENT_EMAIL)) {
             ParserUtil.parseEmail(value);
         } else if (prefix.equals(PREFIX_TAG)) {

src/test/java/seedu/address/logic/parser/FindCommandParserTest.java

@@ -50,6 +50,12 @@ public void parse_validArgs_returnsFindCommand() {
         duplicatePrefixMap.put(PREFIX_NAME, Arrays.asList("Alice", "Bob"));
         assertParseSuccess(parser, " n/Alice n/Bob", new FindCommand(
                 new NameContainsKeywordsPredicate(duplicatePrefixMap)));
+
+        // 3. Short Phone Number
+        Map<Prefix, List<String>> phoneMap = new HashMap<>();
+        phoneMap.put(PREFIX_PARENT_PHONE, Arrays.asList("91"));
+        assertParseSuccess(parser, " pc/91", new FindCommand(
+                new NameContainsKeywordsPredicate(phoneMap)));
     }
 
     @Test
@@ -98,7 +104,7 @@ public void parse_invalidPrefix_throwsParseException() {
     public void parse_invalidValue_throwsParseException() {
         // 1. Invalid format for specific types
         assertParseFailure(parser, " a/notAnAge", Age.MESSAGE_CONSTRAINTS);
-        assertParseFailure(parser, " pc/notAPhone", seedu.address.model.person.Phone.MESSAGE_CONSTRAINTS);
+        assertParseFailure(parser, " pc/notAPhone", "Phone numbers should only contain digits.");
 
         // 2. Empty values (after the prefix)
         assertParseFailure(parser, " n/ ", Name.MESSAGE_CONSTRAINTS);