[#2103] Refactor parser package for greater organisation of classes

src/main/java/reposense/RepoSense.java

@@ -15,9 +15,9 @@
 import reposense.model.ReportConfiguration;
 import reposense.model.RunConfigurationDecider;
 import reposense.parser.ArgsParser;
-import reposense.parser.InvalidCsvException;
-import reposense.parser.InvalidHeaderException;
-import reposense.parser.ParseException;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
+import reposense.parser.exceptions.ParseException;
 import reposense.report.ReportGenerator;
 import reposense.system.LogsManager;
 import reposense.system.ReportServer;

src/main/java/reposense/model/CliRunConfiguration.java

@@ -5,8 +5,8 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
-import reposense.parser.InvalidLocationException;
-import reposense.parser.ParseException;
+import reposense.parser.exceptions.InvalidLocationException;
+import reposense.parser.exceptions.ParseException;
 import reposense.system.LogsManager;
 
 /**

src/main/java/reposense/model/ConfigRunConfiguration.java

@@ -9,9 +9,9 @@
 
 import reposense.parser.AuthorConfigCsvParser;
 import reposense.parser.GroupConfigCsvParser;
-import reposense.parser.InvalidCsvException;
-import reposense.parser.InvalidHeaderException;
 import reposense.parser.RepoConfigCsvParser;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
 import reposense.system.LogsManager;
 
 /**

src/main/java/reposense/model/RepoLocation.java

@@ -12,7 +12,7 @@
 import java.util.regex.Pattern;
 
 import reposense.git.GitRemote;
-import reposense.parser.InvalidLocationException;
+import reposense.parser.exceptions.InvalidLocationException;
 import reposense.report.ErrorSummary;
 import reposense.util.StringsUtil;
 import reposense.util.SystemUtil;

src/main/java/reposense/model/RunConfiguration.java

@@ -3,9 +3,9 @@
 import java.io.IOException;
 import java.util.List;
 
-import reposense.parser.InvalidCsvException;
-import reposense.parser.InvalidHeaderException;
-import reposense.parser.ParseException;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
+import reposense.parser.exceptions.ParseException;
 
 /**
  * Interface to get configurations for current RepoSense run.

src/main/java/reposense/parser/ArgsParser.java

@@ -28,6 +28,18 @@
 import reposense.model.CliArguments;
 import reposense.model.FileType;
 import reposense.model.ReportConfiguration;
+import reposense.parser.exceptions.ParseException;
+import reposense.parser.types.AlphanumericArgumentType;
+import reposense.parser.types.AnalysisThreadsArgumentType;
+import reposense.parser.types.AssetsFolderArgumentType;
+import reposense.parser.types.CloningThreadsArgumentType;
+import reposense.parser.types.ConfigFolderArgumentType;
+import reposense.parser.types.OutputFolderArgumentType;
+import reposense.parser.types.PeriodArgumentType;
+import reposense.parser.types.ReportFolderArgumentType;
+import reposense.parser.types.SinceDateArgumentType;
+import reposense.parser.types.UntilDateArgumentType;
+import reposense.parser.types.ZoneIdArgumentType;
 import reposense.system.LogsManager;
 import reposense.util.TimeUtil;
 

src/main/java/reposense/parser/AuthorConfigCsvParser.java

@@ -10,6 +10,8 @@
 import reposense.model.Author;
 import reposense.model.AuthorConfiguration;
 import reposense.model.RepoLocation;
+import reposense.parser.exceptions.InvalidLocationException;
+import reposense.parser.exceptions.ParseException;
 
 /**
  * Container for the values parsed from {@code author-config.csv} file.

src/main/java/reposense/parser/CsvParser.java

@@ -24,6 +24,9 @@
 import org.apache.commons.csv.CSVFormat;
 import org.apache.commons.csv.CSVRecord;
 
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
+import reposense.parser.exceptions.ParseException;
 import reposense.system.LogsManager;
 
 /**

src/main/java/reposense/parser/GroupConfigCsvParser.java

@@ -9,6 +9,7 @@
 import reposense.model.FileType;
 import reposense.model.GroupConfiguration;
 import reposense.model.RepoLocation;
+import reposense.parser.exceptions.InvalidLocationException;
 
 /**
  * Container for the values parsed from {@code group-config.csv} file.

src/main/java/reposense/parser/RepoConfigCsvParser.java

@@ -10,6 +10,7 @@
 import reposense.model.FileType;
 import reposense.model.RepoConfiguration;
 import reposense.model.RepoLocation;
+import reposense.parser.exceptions.InvalidLocationException;
 import reposense.util.FileUtil;
 import reposense.util.StringsUtil;
 

src/main/java/reposense/parser/InvalidCsvException.java → src/main/java/reposense/parser/exceptions/InvalidCsvException.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.exceptions;
 
 /**
  * Signals that there is a major error in a CSV file (e.g. wrong number of columns, zero valid records).

src/main/java/reposense/parser/InvalidHeaderException.java → src/main/java/reposense/parser/exceptions/InvalidHeaderException.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.exceptions;
 
 /**
  * Signals that there is a problem in the header of csv config file.

src/main/java/reposense/parser/InvalidLocationException.java → src/main/java/reposense/parser/exceptions/InvalidLocationException.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.exceptions;
 
 /**
  * Signals that location cannot be represented by {@code URL} or {@code Path}.

src/main/java/reposense/parser/ParseException.java → src/main/java/reposense/parser/exceptions/ParseException.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.exceptions;
 
 /**
  * Signals that there is an exception when parsing a string.

src/main/java/reposense/parser/AlphanumericArgumentType.java → src/main/java/reposense/parser/types/AlphanumericArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.util.regex.Pattern;
 

src/main/java/reposense/parser/AnalysisThreadsArgumentType.java → src/main/java/reposense/parser/types/AnalysisThreadsArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import net.sourceforge.argparse4j.inf.Argument;
 import net.sourceforge.argparse4j.inf.ArgumentParser;

src/main/java/reposense/parser/AssetsFolderArgumentType.java → src/main/java/reposense/parser/types/AssetsFolderArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.nio.file.Path;
 import java.nio.file.Paths;

src/main/java/reposense/parser/CloningThreadsArgumentType.java → src/main/java/reposense/parser/types/CloningThreadsArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import net.sourceforge.argparse4j.inf.Argument;
 import net.sourceforge.argparse4j.inf.ArgumentParser;

src/main/java/reposense/parser/ConfigFolderArgumentType.java → src/main/java/reposense/parser/types/ConfigFolderArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.nio.file.Files;
 import java.nio.file.Path;
@@ -9,6 +9,7 @@
 import net.sourceforge.argparse4j.inf.ArgumentParser;
 import net.sourceforge.argparse4j.inf.ArgumentParserException;
 import net.sourceforge.argparse4j.inf.ArgumentType;
+import reposense.parser.RepoConfigCsvParser;
 
 /**
  * Checks the argument of {@code --config} flag.

src/main/java/reposense/parser/DateArgumentType.java → src/main/java/reposense/parser/types/DateArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.time.LocalDateTime;
 import java.util.Optional;

src/main/java/reposense/parser/OutputFolderArgumentType.java → src/main/java/reposense/parser/types/OutputFolderArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.nio.file.Path;
 import java.nio.file.Paths;
@@ -8,6 +8,7 @@
 import net.sourceforge.argparse4j.inf.ArgumentParser;
 import net.sourceforge.argparse4j.inf.ArgumentParserException;
 import net.sourceforge.argparse4j.inf.ArgumentType;
+import reposense.parser.ArgsParser;
 
 /**
  * Checks the argument of {@code --output} flag.

src/main/java/reposense/parser/PeriodArgumentType.java → src/main/java/reposense/parser/types/PeriodArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.util.Optional;
 import java.util.regex.Pattern;
@@ -7,6 +7,7 @@
 import net.sourceforge.argparse4j.inf.ArgumentParser;
 import net.sourceforge.argparse4j.inf.ArgumentParserException;
 import net.sourceforge.argparse4j.inf.ArgumentType;
+import reposense.parser.exceptions.ParseException;
 
 /**
  * Verifies and parses a string-formatted period to an integer.

src/main/java/reposense/parser/ReportFolderArgumentType.java → src/main/java/reposense/parser/types/ReportFolderArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.nio.file.Path;
 import java.nio.file.Paths;

src/main/java/reposense/parser/SinceDateArgumentType.java → src/main/java/reposense/parser/types/SinceDateArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.time.Instant;
 import java.time.LocalDateTime;

src/main/java/reposense/parser/UntilDateArgumentType.java → src/main/java/reposense/parser/types/UntilDateArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.time.LocalDateTime;
 import java.util.Optional;

src/main/java/reposense/parser/ZoneIdArgumentType.java → src/main/java/reposense/parser/types/ZoneIdArgumentType.java

@@ -1,4 +1,4 @@
-package reposense.parser;
+package reposense.parser.types;
 
 import java.time.DateTimeException;
 import java.time.ZoneId;

src/main/java/reposense/util/TimeUtil.java

@@ -9,7 +9,7 @@
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
-import reposense.parser.SinceDateArgumentType;
+import reposense.parser.types.SinceDateArgumentType;
 import reposense.system.LogsManager;
 
 /**

src/systemtest/java/reposense/ConfigSystemTest.java

@@ -15,7 +15,7 @@
 import org.junit.jupiter.api.parallel.ExecutionMode;
 
 import reposense.model.SupportedDomainUrlMap;
-import reposense.parser.SinceDateArgumentType;
+import reposense.parser.types.SinceDateArgumentType;
 import reposense.report.ErrorSummary;
 import reposense.util.FileUtil;
 import reposense.util.InputBuilder;

src/systemtest/java/reposense/LocalRepoSystemTest.java

@@ -17,7 +17,7 @@
 import reposense.model.RepoConfiguration;
 import reposense.model.RepoLocation;
 import reposense.model.SupportedDomainUrlMap;
-import reposense.parser.SinceDateArgumentType;
+import reposense.parser.types.SinceDateArgumentType;
 import reposense.report.ErrorSummary;
 import reposense.util.FileUtil;
 import reposense.util.InputBuilder;

src/test/java/reposense/model/RepoLocationTest.java

@@ -8,7 +8,7 @@
 import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.Test;
 
-import reposense.parser.InvalidLocationException;
+import reposense.parser.exceptions.InvalidLocationException;
 import reposense.util.AssertUtil;
 import reposense.util.SystemUtil;
 

src/test/java/reposense/parser/ArgsParserTest.java

@@ -26,6 +26,8 @@
 import reposense.model.FileTypeTest;
 import reposense.model.RepoConfiguration;
 import reposense.model.RunConfigurationDecider;
+import reposense.parser.exceptions.ParseException;
+import reposense.parser.types.SinceDateArgumentType;
 import reposense.util.FileUtil;
 import reposense.util.InputBuilder;
 import reposense.util.TestUtil;

src/test/java/reposense/parser/AuthorConfigParserTest.java

@@ -17,6 +17,8 @@
 import reposense.model.Author;
 import reposense.model.AuthorConfiguration;
 import reposense.model.RepoLocation;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
 
 public class AuthorConfigParserTest {
  private static final Path AUTHOR_CONFIG_EMPTY_LOCATION_FILE = loadResource(AuthorConfigParserTest.class,

src/test/java/reposense/parser/GroupConfigParserTest.java

@@ -12,6 +12,8 @@
 
 import reposense.model.FileType;
 import reposense.model.GroupConfiguration;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
 
 public class GroupConfigParserTest {
  private static final Path GROUP_CONFIG_MULTI_LOCATION_FILE = loadResource(GroupConfigParserTest.class,

src/test/java/reposense/parser/RepoConfigParserTest.java

@@ -19,6 +19,8 @@
 import reposense.model.FileType;
 import reposense.model.RepoConfiguration;
 import reposense.model.RepoLocation;
+import reposense.parser.exceptions.InvalidCsvException;
+import reposense.parser.exceptions.InvalidHeaderException;
 import reposense.util.InputBuilder;
 import reposense.util.TestUtil;
 

src/test/java/reposense/report/ErrorSummaryTest.java

@@ -4,7 +4,7 @@
 import org.junit.jupiter.api.Test;
 
 import reposense.model.RepoLocation;
-import reposense.parser.InvalidLocationException;
+import reposense.parser.exceptions.InvalidLocationException;
 
 public class ErrorSummaryTest {
  @Test