Merge branch 'master' of https://github.com/AY2425S1-CS2103T-W10-4/tp into Feature-Complete-AutoCompletePath

* 'master' of https://github.com/AY2425S1-CS2103T-W10-4/tp:
  Add the requirement of only one file name into the User guide
  Update the archive command and load command

Author: Jianwen0451

docs/UserGuide.md

@@ -172,6 +172,8 @@ Example: `archive pa/mybook.json`
 
 The file name must ends with ".json" and must not contain any slash "/".
 
+There should be only one file name provided.
+
 #### Warning
 
 All the entries in the current address book will be cleared.
@@ -190,6 +192,8 @@ Example: `load pa/mybook.json`
 
 The file name must ends with ".json", must not contain any slash "/" and must point to an existing address book .json file.
 
+There should be only one file name provided.
+
 #### Warning
 Avoid loading non-address book .json files as it may result in unexpected behaviours
 

src/main/java/seedu/address/logic/commands/ArchiveCommand.java

@@ -16,6 +16,7 @@ public class ArchiveCommand extends Command {
     public static final String MESSAGE_USAGE = COMMAND_WORD + " " + PREFIX_PATH + "FileName\n"
             + "The file name should be a path of an json file "
             + "and not contain any slash `/` \n"
+            + "There should be only one path provided.\n"
             + "Example: " + COMMAND_WORD + " " + PREFIX_PATH + "mybook.json";
     public static final String MESSAGE_SUCCESS = "Archive Path changed to: %1$s";
 

src/main/java/seedu/address/logic/commands/LoadCommand.java

@@ -16,6 +16,7 @@ public class LoadCommand extends Command {
     public static final String MESSAGE_USAGE = COMMAND_WORD + " " + PREFIX_PATH + "FileName\n"
             + "The file name should be a path of an json file containing an address book "
             + "and not contain any slash `/` \n"
+            + "There should be only one path provided.\n"
             + "Example: " + COMMAND_WORD + " " + PREFIX_PATH + "mybook.json";
     public static final String MESSAGE_SUCCESS = "Loading file from: %1$s";
 

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

@@ -26,6 +26,7 @@ public ArchiveCommand parse(String args) throws ParseException {
         if (!arePrefixesPresent(argMultimap, PREFIX_PATH)) {
             throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, ArchiveCommand.MESSAGE_USAGE));
         }
+        argMultimap.verifyNoDuplicatePrefixesFor(PREFIX_PATH);
         Path path = ParserUtil.parsePathWithoutCheck(argMultimap.getValue(PREFIX_PATH).get());
         return new ArchiveCommand(path);
     }

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

@@ -26,6 +26,7 @@ public LoadCommand parse(String args) throws ParseException {
         if (!arePrefixesPresent(argMultimap, PREFIX_PATH)) {
             throw new ParseException(String.format(MESSAGE_INVALID_COMMAND_FORMAT, LoadCommand.MESSAGE_USAGE));
         }
+        argMultimap.verifyNoDuplicatePrefixesFor(PREFIX_PATH);
         Path path = ParserUtil.parsePathWithCheck(argMultimap.getValue(PREFIX_PATH).get());
         return new LoadCommand(path);
     }

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

@@ -1,24 +1,29 @@
 package seedu.address.logic.parser;
 
 import static seedu.address.logic.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_PATH;
 import static seedu.address.logic.parser.CommandParserTestUtil.assertParseFailure;
 import static seedu.address.logic.parser.CommandParserTestUtil.assertParseSuccess;
 
 import java.nio.file.Paths;
 
 import org.junit.jupiter.api.Test;
 
+import seedu.address.logic.Messages;
 import seedu.address.logic.commands.ArchiveCommand;
 
 public class ArchiveCommandParserTest {
     private static final String INPUT_MISSING_PREFIX = "archive mybook.json";
+    private static final String INPUT_MULTIPLE_PATH = "archive pa/mybook.json pa/mybook2.json";
     private static final String VALID_INPUT = "archive pa/TestValidInput.json";
     private static final ArchiveCommandParser PARSER = new ArchiveCommandParser();
 
     @Test
     void invalid_input_throwException() {
         assertParseFailure(PARSER, INPUT_MISSING_PREFIX,
                 String.format(MESSAGE_INVALID_COMMAND_FORMAT, ArchiveCommand.MESSAGE_USAGE));
+        assertParseFailure(PARSER, INPUT_MULTIPLE_PATH,
+                String.format(Messages.getErrorMessageForDuplicatePrefixes(PREFIX_PATH)));
     }
 
     @Test

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

@@ -1,6 +1,7 @@
 package seedu.address.logic.parser;
 
 import static seedu.address.logic.Messages.MESSAGE_INVALID_COMMAND_FORMAT;
+import static seedu.address.logic.parser.CliSyntax.PREFIX_PATH;
 import static seedu.address.logic.parser.CommandParserTestUtil.assertParseFailure;
 import static seedu.address.logic.parser.CommandParserTestUtil.assertParseSuccess;
 
@@ -10,18 +11,22 @@
 
 import org.junit.jupiter.api.Test;
 
+import seedu.address.logic.Messages;
 import seedu.address.logic.commands.LoadCommand;
 
 
 public class LoadCommandParserTest {
     private static final String INPUT_MISSING_PREFIX = "load mybook.json";
+    private static final String INPUT_MULTIPLE_PATH = "load pa/mybook.json, pa/mybook2.json";
     private static final String VALID_INPUT = "load pa/TestValidInput.json";
     private static final LoadCommandParser PARSER = new LoadCommandParser();
 
     @Test
     void invalid_input_throwException() {
         assertParseFailure(PARSER, INPUT_MISSING_PREFIX,
                 String.format(MESSAGE_INVALID_COMMAND_FORMAT, LoadCommand.MESSAGE_USAGE));
+        assertParseFailure(PARSER, INPUT_MULTIPLE_PATH,
+                String.format(Messages.getErrorMessageForDuplicatePrefixes(PREFIX_PATH)));
     }
 
     @Test