Remove test mode flag in backend

lyuanww · lyuanww · commit 671337918044 · 2025-03-03T12:46:19.000+08:00
diff --git a/docs/dg/cli.md b/docs/dg/cli.md
@@ -11,21 +11,6 @@ syntax reference under Appendix section of User Guide.
 
 <!-- ------------------------------------------------------------------------------------------------------ -->
 
-### `--test-mode`
-
-**`--test-mode`**: Enables test mode behavior.
-
-<box type="info" seamless>
-
-* Used in `ConfigSystemTest`.
-* Can be used for behaviors specific to test code.
-  * E.g. `--fresh-cloning`. Fresh cloning is always `false` when running RepoSense normally, and is only used in
-  system tests.
-
-</box>
-
-<!-- ------------------------------------------------------------------------------------------------------ -->
-
 ### `--fresh-cloning`
 
 **`--fresh-cloning`**: Clones the repo again if it has been cloned before.
@@ -36,6 +21,5 @@ syntax reference under Appendix section of User Guide.
 * Some test cases performs shallow cloning while some does not. Fresh cloning ensures that the test cases that does
 not perform shallow cloning will clone the repo again if the previous test case uses shallow cloning, ensuring
 correctness of the analysis.
-* Requires `--test-mode` flag to be enabled.
 
 </box>
diff --git a/src/main/java/reposense/model/CliArguments.java b/src/main/java/reposense/model/CliArguments.java
@@ -37,7 +37,6 @@ public class CliArguments {
     private boolean isFindingPreviousAuthorsPerformed;
     private boolean isAuthorshipAnalyzed;
     private double originalityThreshold;
-    private boolean isTestMode = ArgsParser.DEFAULT_IS_TEST_MODE;
     private boolean isFreshClonePerformed = ArgsParser.DEFAULT_SHOULD_FRESH_CLONE;
 
     private List<String> locations;
@@ -122,10 +121,6 @@ public boolean isFindingPreviousAuthorsPerformed() {
         return isFindingPreviousAuthorsPerformed;
     }
 
-    public boolean isTestMode() {
-        return isTestMode;
-    }
-
     public boolean isFreshClonePerformed() {
         return isFreshClonePerformed;
     }
@@ -207,7 +202,6 @@ public boolean equals(Object other) {
                 && Objects.equals(this.zoneId, otherCliArguments.zoneId)
                 && this.isFindingPreviousAuthorsPerformed == otherCliArguments.isFindingPreviousAuthorsPerformed
                 && this.isFileSizeLimitIgnored == otherCliArguments.isFileSizeLimitIgnored
-                && this.isTestMode == otherCliArguments.isTestMode
                 && this.isFreshClonePerformed == otherCliArguments.isFreshClonePerformed
                 && Objects.equals(this.locations, otherCliArguments.locations)
                 && this.isViewModeOnly == otherCliArguments.isViewModeOnly
@@ -391,16 +385,6 @@ public Builder isFindingPreviousAuthorsPerformed(boolean isFindingPreviousAuthor
             return this;
         }
 
-        /**
-         * Adds the {@code isTestMode} to CliArguments.
-         *
-         * @param isTestMode Is test mode.
-         */
-        public Builder isTestMode(boolean isTestMode) {
-            this.cliArguments.isTestMode = isTestMode;
-            return this;
-        }
-
         /**
          * Adds the {@code isFreshClonePerformed} to CliArguments.
          *
diff --git a/src/main/java/reposense/parser/ArgsParser.java b/src/main/java/reposense/parser/ArgsParser.java
@@ -18,7 +18,6 @@
 import net.sourceforge.argparse4j.impl.Arguments;
 import net.sourceforge.argparse4j.impl.action.HelpArgumentAction;
 import net.sourceforge.argparse4j.impl.action.VersionArgumentAction;
-import net.sourceforge.argparse4j.inf.ArgumentGroup;
 import net.sourceforge.argparse4j.inf.ArgumentParser;
 import net.sourceforge.argparse4j.inf.ArgumentParserException;
 import net.sourceforge.argparse4j.inf.FeatureControl;
@@ -52,7 +51,6 @@ public class ArgsParser {
     public static final String DEFAULT_REPORT_NAME = "reposense-report";
     public static final int DEFAULT_NUM_CLONING_THREADS = 4;
     public static final int DEFAULT_NUM_ANALYSIS_THREADS = Runtime.getRuntime().availableProcessors();
-    public static final boolean DEFAULT_IS_TEST_MODE = false;
     public static final boolean DEFAULT_SHOULD_FRESH_CLONE = false;
     public static final double DEFAULT_ORIGINALITY_THRESHOLD = 0.51;
 
@@ -75,7 +73,6 @@ public class ArgsParser {
     public static final String[] FIND_PREVIOUS_AUTHORS_FLAGS = new String[] {"--find-previous-authors", "-F"};
     public static final String[] CLONING_THREADS_FLAG = new String[] {"--cloning-threads"};
     public static final String[] ANALYSIS_THREADS_FLAG = new String[] {"--analysis-threads"};
-    public static final String[] TEST_MODE_FLAG = new String[] {"--test-mode"};
     public static final String[] FRESH_CLONING_FLAG = new String[] {"--fresh-cloning"};
     public static final String[] ANALYZE_AUTHORSHIP_FLAGS = new String[] {"--analyze-authorship", "-A"};
     public static final String[] ORIGINALITY_THRESHOLD_FLAGS = new String[] {"--originality-threshold", "-ot"};
@@ -86,7 +83,6 @@ public class ArgsParser {
     private static final String PROGRAM_DESCRIPTION =
             "RepoSense is a contribution analysis tool for Git repositories.";
     private static final String MESSAGE_HEADER_MUTEX = "mutual exclusive arguments";
-    private static final String MESSAGE_HEADER_TESTING = "test mode arguments";
     private static final String MESSAGE_HAVE_SINCE_DATE_UNTIL_DATE_AND_PERIOD =
             "\"Since Date\", \"Until Date\", and \"Period\" cannot be applied together.";
     private static final String MESSAGE_USING_DEFAULT_CONFIG_PATH =
@@ -121,9 +117,6 @@ private static ArgumentParser getArgumentParser() {
                 .addMutuallyExclusiveGroup(MESSAGE_HEADER_MUTEX)
                 .required(false);
 
-        ArgumentGroup argumentGroup = parser
-                .addArgumentGroup(MESSAGE_HEADER_TESTING);
-
         // Boolean flags
         parser.addArgument(HELP_FLAGS)
                 .help("Show help message.")
@@ -272,13 +265,7 @@ private static ArgumentParser getArgumentParser() {
                 .setDefault(DEFAULT_NUM_ANALYSIS_THREADS)
                 .help(FeatureControl.SUPPRESS);
 
-        // Testing flags
-        argumentGroup.addArgument(TEST_MODE_FLAG)
-                .dest(TEST_MODE_FLAG[0])
-                .action(Arguments.storeTrue())
-                .help("Enables testing mode.");
-
-        argumentGroup.addArgument(FRESH_CLONING_FLAG)
+        parser.addArgument(FRESH_CLONING_FLAG)
                 .dest(FRESH_CLONING_FLAG[0])
                 .action(Arguments.storeTrue())
                 .help("Enables fresh cloning. Requires testing mode to be enabled.");
@@ -317,11 +304,11 @@ public static CliArguments parse(String[] args) throws HelpScreenException, Pars
         boolean shouldIncludeLastModifiedDate = results.get(LAST_MODIFIED_DATE_FLAGS[0]);
         boolean shouldPerformShallowCloning = results.get(SHALLOW_CLONING_FLAGS[0]);
         boolean shouldFindPreviousAuthors = results.get(FIND_PREVIOUS_AUTHORS_FLAGS[0]);
-        boolean isTestMode = results.get(TEST_MODE_FLAG[0]);
         boolean isAuthorshipAnalyzed = results.get(ANALYZE_AUTHORSHIP_FLAGS[0]);
         double originalityThreshold = results.get(ORIGINALITY_THRESHOLD_FLAGS[0]);
         int numCloningThreads = results.get(CLONING_THREADS_FLAG[0]);
         int numAnalysisThreads = results.get(ANALYSIS_THREADS_FLAG[0]);
+        boolean shouldPerformFreshCloning = results.get(FRESH_CLONING_FLAG[0]);
 
         CliArguments.Builder cliArgumentsBuilder = new CliArguments.Builder()
                 .configFolderPath(configFolderPath)
@@ -338,9 +325,9 @@ public static CliArguments parse(String[] args) throws HelpScreenException, Pars
                 .isFindingPreviousAuthorsPerformed(shouldFindPreviousAuthors)
                 .numCloningThreads(numCloningThreads)
                 .numAnalysisThreads(numAnalysisThreads)
-                .isTestMode(isTestMode)
                 .isAuthorshipAnalyzed(isAuthorshipAnalyzed)
-                .originalityThreshold(originalityThreshold);
+                .originalityThreshold(originalityThreshold)
+                .isFreshClonePerformed(shouldPerformFreshCloning);
 
         LogsManager.setLogFolderLocation(outputFolderPath);
 
@@ -364,11 +351,6 @@ public static CliArguments parse(String[] args) throws HelpScreenException, Pars
         }
         cliArgumentsBuilder.isAutomaticallyLaunching(isAutomaticallyLaunching);
 
-        boolean shouldPerformFreshCloning = isTestMode
-                ? results.get(FRESH_CLONING_FLAG[0])
-                : DEFAULT_SHOULD_FRESH_CLONE;
-        cliArgumentsBuilder.isFreshClonePerformed(shouldPerformFreshCloning);
-
         return cliArgumentsBuilder.build();
     }
 
diff --git a/src/systemtest/java/reposense/ConfigSystemTest.java b/src/systemtest/java/reposense/ConfigSystemTest.java
@@ -136,7 +136,6 @@ public void test30DaysFromUntilDateWithFindPreviousAuthors() {
      * <br>Formats: {@link ConfigSystemTest#TESTING_FILE_FORMATS TESTING_FILE_FORMATS}
      * <br>Timezone: {@link ConfigSystemTest#TEST_TIME_ZONE TEST_TIME_ZONE}
      * <br>Output Folder Path: {@link ConfigSystemTest#OUTPUT_DIRECTORY OUTPUT_DIRECTORY}
-     * <br>Test Mode: {@code Enabled}
      */
     private InputBuilder initInputBuilder() {
         Path configFolder = loadResource(getClass(), "ConfigSystemTest");
@@ -145,7 +144,6 @@ private InputBuilder initInputBuilder() {
         return new InputBuilder().addConfig(configFolder)
                 .addFormats(formats)
                 .addTimezone(TEST_TIME_ZONE)
-                .addTestMode()
                 .addOutput(OUTPUT_DIRECTORY);
     }
 
diff --git a/src/test/java/reposense/util/InputBuilder.java b/src/test/java/reposense/util/InputBuilder.java
@@ -223,15 +223,6 @@ public InputBuilder addFindPreviousAuthors() {
         return this;
     }
 
-    /**
-     * Adds the flag to enable test mode.
-     * This method should only be called once in one build.
-     */
-    public InputBuilder addTestMode() {
-        input.append(ArgsParser.TEST_MODE_FLAG[0] + WHITESPACE);
-        return this;
-    }
-
     /**
      * Adds the flag to include modified date in lines.
      * This method should only be called once in one build.
