Merge pull request #212 from nixonwidjaja/fix-claim

Fix claim bug
AY2324S1-CS2103-F13-2 · Nov 13, 2023 · 496d716 · 496d716
2 parents 0365e10 + cd56924
commit 496d716
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/src/main/java/seedu/address/logic/parser/ParserUtil.java b/src/main/java/seedu/address/logic/parser/ParserUtil.java
@@ -8,6 +8,7 @@
 
 import seedu.address.commons.core.index.Index;
 import seedu.address.commons.util.StringUtil;
+import seedu.address.logic.Messages;
 import seedu.address.logic.parser.exceptions.ParseException;
 import seedu.address.model.person.Address;
 import seedu.address.model.person.Birthday;
@@ -157,6 +158,9 @@ public static Birthday parseDob(String dob) throws ParseException {
     public static Claim parseClaim(String claimAmount) throws ParseException {
         requireNonNull(claimAmount);
         String trimmed = claimAmount.trim();
+        if (trimmed.length() > 14) {
+            throw new ParseException(Messages.TOO_LARGE_A_NUMBER);
+        }
         if (!Claim.comtainsSymbol(trimmed)) {
             throw new ParseException(Claim.NO_SYMBOLS_ERROR);
         } else if (!Claim.isCorrectAmountType(trimmed)) {

diff --git a/src/test/java/seedu/address/logic/parser/ClaimCommandParserTest.java b/src/test/java/seedu/address/logic/parser/ClaimCommandParserTest.java
@@ -14,6 +14,7 @@
 
 import org.junit.jupiter.api.Test;
 
+import seedu.address.logic.Messages;
 import seedu.address.logic.commands.ClaimCommand;
 
 public class ClaimCommandParserTest {
@@ -35,6 +36,12 @@ public void parse_withEmptyIndex_returnsFailure() {
         assertParseFailure(parser, userInput, CLAIM_EMPTY_INDEX);
     }
 
+    @Test
+    public void parse_claimTooLarge_returnsFailure() {
+        String userInput = "1 $/+100000000000000000000000000000000";
+        assertParseFailure(parser, userInput, Messages.TOO_LARGE_A_NUMBER);
+    }
+
     @Test
     public void parse_withEmptyAmount_returnsFailure() {
         String userInput = String.valueOf(INDEX_FIRST_PERSON.getOneBased());