Mutator configuration

bundles/org.pitest.pitclipse.core/src/org/pitest/pitclipse/core/Mutators.java

@@ -0,0 +1,117 @@
+/*******************************************************************************
+ * Copyright 2012-2021 Phil Glover and contributors
+ *  
+ * Licensed under the Apache License, Version 2.0 (the "License"); you may not
+ * use this file except in compliance with the License.  You may obtain a copy
+ * of the License at
+ *  
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *  
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+ * License for the specific language governing permissions and limitations under
+ * the License.
+ ******************************************************************************/
+
+package org.pitest.pitclipse.core;
+
+import java.util.ArrayList;
+
+import org.pitest.mutationtest.engine.gregor.config.Mutator;
+
+/**
+ * Enum which holds information about the mutators of pit.<br>
+ * The name of new values <b>must</b> be the exact String, which is used by PIT
+ * in the class {@link Mutator}.
+ */
+@SuppressWarnings("checkstyle:LineLength")
+public enum Mutators {
+    OLD_DEFAULTS("Old defaults", "&Old default Mutators"),
+    DEFAULTS("Defaults", "&Default Mutators"),
+    STRONGER("Stronger defaults", "&Stronger Mutators"),
+    ALL("All", "&All Mutators"),
+    CONDITIONALS_BOUNDARY("Conditionals Boundary", "Replaces the relational operators <, <=, >, >=", true),
+    INCREMENTS("Increments", "Mutates increments, decrements and assignment increments and decrements of local variables (stack variables). Replaces increments with decrements and vice versa", true),
+    INVERT_NEGS("Invert Negatives", "Inverts negation of integer and floating point numbers", true),
+    MATH("Math", "Replaces binary arithmetic operations for either integer or floating-point arithmetic with another operation", true),
+    NEGATE_CONDITIONALS("Negate Conditionals", "Mutates all conditionals found", true),
+    RETURN_VALS("Return Values", "Mutates the return values of method calls. Depending on the return type of the method another mutation is used"),
+    VOID_METHOD_CALLS("Void Method Call", "Removes method calls to void methods", true),
+    CONSTRUCTOR_CALLS("Constructor Call", "Replaces constructor calls with null values"),
+    EMPTY_RETURNS("Empty Returns", "Replaces return values with an 'empty' value", true),
+    FALSE_RETURNS("False Returns", "Replaces primitive and boxed boolean return values with false", true),
+    TRUE_RETURNS("True Returns", "Replaces primitive and boxed boolean return values with true", true),
+    INLINE_CONSTS("Inline Constant", "Mutates inline constants. An inline constant is a literal value assigned to a non-final variable"),
+    NULL_RETURNS("Null Returns", "Replaces return values with null. Method that can be mutated by the EMPTY_RETURNS mutator or that are directly annotated with NotNull are not mutated", true),
+    NON_VOID_METHOD_CALLS("Non Void Method Call", "Removes method calls to non void methods. Their return value is replaced by the Java Default Value for that specific type"),
+    PRIMITIVE_RETURNS("Primite Returns", "Replaces int, short, long, char, float and double return values with 0", true),
+    REMOVE_CONDITIONALS("Remove Conditionals", "Removes all conditionals statements such that the guarded statements always execute"),
+    REMOVE_INCREMENTS("Remove Increments", "Removes local variable increments"),
+    EXPERIMENTAL_ARGUMENT_PROPAGATION("Experimentation Argument Propagation", "Replaces method call with one of its parameters of matching type"),
+    EXPERIMENTAL_BIG_INTEGER("Experimental Big Integer", "Swaps big integer methods"),
+    EXPERIMENTAL_NAKED_RECEIVER("Experimental Naked Receiver", "Replaces method call with a naked receiver"),
+    EXPERIMENTAL_MEMBER_VARIABLE("Experimental Member Variable", "Removes assignments to member variables. Can even remove assignments to final members. The members will be initialized with their Java Default Value"),
+    EXPERIMENTAL_SWITCH("Experimental Switch", "Finds the first label within a switch statement that differs from the default label. Mutates the switch statement by replacing the default label (wherever it is used) with this label. All the other labels are replaced by the default one"),
+    ABS("Negation", "Replaces any use of a numeric variable (local variable, field, array cell) with its negation"),
+    AOR("Arithmetic Operator Replacement", "Like the Math mutator, replaces binary arithmetic operations for either integer or floating-point arithmetic with another operation"),
+    AOD("Arithmetic Operator Deletion", "Replaces an arithmetic operation with one of its members"),
+    CRCR("Constant Replacement", "Like the Inline Constant mutator, mutates inline constant"),
+    OBBN("Bitwise Operator", "Mutates bitwise and (&) and or (|)"),
+    ROR("Relational Operator Replacement", "Replaces a relational operator with another one"),
+    UOI("Unary Operator Insertion", "Inserts a unary operator (increment or decrement) to a variable call. Affects local variables, array variables, fields and parameters");
+
+    /**
+     * Descriptor which is used to display the name of this mutator in a table
+     */
+    private final String descriptor;
+    /**
+     * Description of this mutator
+     */
+    private final String description;
+    /**
+     * true, if this mutator is contained in the DEFAULTS group of PIT
+     */
+    private final boolean activeByDefault;
+
+    private Mutators(String descriptor, String description) {
+        this(descriptor, description, false);
+    }
+
+    private Mutators(String descriptor, String description, boolean activeByDefault) {
+        this.descriptor = descriptor;
+        this.description = description;
+        this.activeByDefault = activeByDefault;
+    }
+
+    public String getDescriptor() {
+        return descriptor;
+    }
+
+    public String getDescription() {
+        return description;
+    }
+
+    public boolean isActiveByDefault() {
+        return activeByDefault;
+    }
+
+    public static ArrayList<Mutators> getMainGroup() {
+        final ArrayList<Mutators> mainGroup = new ArrayList<Mutators>();
+        mainGroup.add(DEFAULTS);
+        mainGroup.add(STRONGER);
+        mainGroup.add(ALL);
+        mainGroup.add(OLD_DEFAULTS);
+        return mainGroup;
+    }
+
+    public static ArrayList<String> getDefaultMutators() {
+        ArrayList<String> defaultMutators = new ArrayList<String>();
+        for (Mutators m : values()) {
+            if (m.isActiveByDefault()) {
+                defaultMutators.add(m.name());
+            }
+        }
+        return defaultMutators;
+    }
+}

bundles/org.pitest.pitclipse.core/src/org/pitest/pitclipse/core/PitCoreActivator.java

@@ -17,12 +17,12 @@
 package org.pitest.pitclipse.core;
 
 import static org.eclipse.core.runtime.FileLocator.getBundleFile;
-import static org.pitest.pitclipse.core.preferences.PitPreferences.AVOID_CALLS_TO;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.AVOID_CALLS;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.EXCLUDED_CLASSES;
-import static org.pitest.pitclipse.core.preferences.PitPreferences.EXCLUDED_METHODS;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.EXCLUDE_METHODS;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.INCREMENTAL_ANALYSIS;
-import static org.pitest.pitclipse.core.preferences.PitPreferences.PIT_EXECUTION_MODE;
-import static org.pitest.pitclipse.core.preferences.PitPreferences.PIT_MUTATORS;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.EXECUTION_SCOPE;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.MUTATORS;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.RUN_IN_PARALLEL;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.TIMEOUT;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.TIMEOUT_FACTOR;
@@ -267,13 +267,13 @@ public File getHistoryFile() {
 
     public PitConfiguration getConfiguration() {
         IPreferenceStore preferenceStore = getPreferenceStore();
-        String executionMode = preferenceStore.getString(PIT_EXECUTION_MODE);
-        String mutators = preferenceStore.getString(PIT_MUTATORS);
+        String executionMode = preferenceStore.getString(EXECUTION_SCOPE);
+        String mutators = preferenceStore.getString(MUTATORS);
         boolean parallelRun = preferenceStore.getBoolean(RUN_IN_PARALLEL);
         boolean incrementalAnalysis = preferenceStore.getBoolean(INCREMENTAL_ANALYSIS);
         String excludedClasses = preferenceStore.getString(EXCLUDED_CLASSES);
-        String excludedMethods = preferenceStore.getString(EXCLUDED_METHODS);
-        String avoidCallsTo = preferenceStore.getString(AVOID_CALLS_TO);
+        String excludedMethods = preferenceStore.getString(EXCLUDE_METHODS);
+        String avoidCallsTo = preferenceStore.getString(AVOID_CALLS);
         String timeout = preferenceStore.getString(TIMEOUT);
         String timeoutFactor = preferenceStore.getString(TIMEOUT_FACTOR);
         PitConfiguration.Builder builder = PitConfiguration.builder().withParallelExecution(parallelRun)
@@ -292,8 +292,8 @@ public PitConfiguration getConfiguration() {
                 break;
             }
         }
-        for (PitMutators mutatorMode : PitMutators.values()) {
-            if (mutatorMode.getId().equals(mutators)) {
+        for (Mutators mutatorMode : Mutators.values()) {
+            if (mutatorMode.name().equals(mutators)) {
                 builder.withMutators(mutatorMode.toString());
                 break;
             }
@@ -302,10 +302,14 @@ public PitConfiguration getConfiguration() {
     }
 
     public void setExecutionMode(PitExecutionMode pitExecutionMode) {
-        getPreferenceStore().setValue(PIT_EXECUTION_MODE, pitExecutionMode.getId());
+        getPreferenceStore().setValue(EXECUTION_SCOPE, pitExecutionMode.getId());
     }
 
-    public void setMutators(PitMutators mutators) {
-        getPreferenceStore().setValue(PIT_MUTATORS, mutators.getId());
+    public void setMutators(Mutators mutators) {
+        getPreferenceStore().setValue(MUTATORS, mutators.name());
+    }
+
+    public String getDefaultMutators() {
+        return getPreferenceStore().getString(MUTATORS);
     }
 }

bundles/org.pitest.pitclipse.core/src/org/pitest/pitclipse/core/preferences/PitPreferences.java

@@ -20,25 +20,39 @@
  * Constants used to deal with Pitclipse's preferences.
  */
 public final class PitPreferences {
+    public static final String MUTATORS_LABEL = "Mutators";
+    public static final String MUTATORS = "pitMutators";
+
+    public static final String INDIVIDUAL_MUTATORS = "pitIndividualMutators";
 
-    public static final String PIT_MUTATORS = "pitMutators";
-
-    public static final String USE_INCREMENTAL_ANALYSIS = "Use &incremental analysis";
-    public static final String EXCLUDE_CLASSES_FROM_PIT = "E&xcluded classes (e.g.*IntTest)";
-    public static final String MUTATION_TESTS_RUN_IN_PARALLEL = "Mutation tests run in para&llel";
-    public static final String EXCLUDE_METHODS_FROM_PIT = "Excluded &methods (e.g.*toString*)";
-    public static final String AVOID_CALLS_FROM_PIT = "&Avoid calls to";
-    public static final String PIT_TIMEOUT = "Pit Ti&meout";
-    public static final String PIT_TIMEOUT_FACTOR = "Timeout &Factor";
-    public static final String EXCLUDED_METHODS = "excludedMethods";
-    public static final String AVOID_CALLS_TO = "avoidCallsTo";
-    public static final String EXCLUDED_CLASSES = "excludedClasses";
+    public static final String INCREMENTAL_ANALYSIS_LABEL = "Use &incremental analysis";
     public static final String INCREMENTAL_ANALYSIS = "incrementalAnalysis";
-    public static final String PIT_EXECUTION_MODE = "pitExecutionMode";
+
+    public static final String EXCLUDE_CLASSES_LABEL = "E&xcluded classes (e.g.*IntTest)";
+    public static final String EXCLUDED_CLASSES = "excludedClasses";
+
+    public static final String RUN_IN_PARALLEL_LABEL = "Mutation tests run in para&llel";
     public static final String RUN_IN_PARALLEL = "runInParallel";
+
+    public static final String EXCLUDE_METHODS_LABEL = "Excluded &methods (e.g.*toString*)";
+    public static final String EXCLUDE_METHODS = "excludedMethods";
+
+    public static final String AVOID_CALLS_LABEL = "&Avoid calls to";
+    public static final String AVOID_CALLS = "avoidCallsTo";
+
+    public static final String TIMEOUT_LABEL = "Pit Ti&meout";
     public static final String TIMEOUT = "pitTimeout";
+
+    public static final String TIMEOUT_FACTOR_LABEL = "Timeout &Factor";
     public static final String TIMEOUT_FACTOR = "pitTimeoutFactor";
 
+
+    public static final String EXECUTION_SCOPE_LABEL = "Pit execution scope";
+    public static final String EXECUTION_SCOPE = "pitExecutionMode";
+
+    public static final String PREFERENCE_DESCRIPTION_LABEL = "Pitclipse Preferences";
+    public static final String MUTATORS_DESCRIPTION_LABEL = "Mutator Preferences";
+
     private PitPreferences() {
         // utility class should not be instantiated
     }

bundles/org.pitest.pitclipse.core/src/org/pitest/pitclipse/core/preferences/PreferenceInitializer.java

@@ -21,10 +21,10 @@
 import org.pitest.pitclipse.core.PitCoreActivator;
 import org.pitest.pitclipse.runner.config.PitConfiguration;
 
-import static org.pitest.pitclipse.core.preferences.PitPreferences.AVOID_CALLS_TO;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.AVOID_CALLS;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.EXCLUDED_CLASSES;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.INCREMENTAL_ANALYSIS;
-import static org.pitest.pitclipse.core.preferences.PitPreferences.PIT_EXECUTION_MODE;
+import static org.pitest.pitclipse.core.preferences.PitPreferences.EXECUTION_SCOPE;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.RUN_IN_PARALLEL;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.TIMEOUT;
 import static org.pitest.pitclipse.core.preferences.PitPreferences.TIMEOUT_FACTOR;
@@ -47,10 +47,10 @@ public class PreferenceInitializer extends AbstractPreferenceInitializer {
     @Override
     public void initializeDefaultPreferences() {
         IPreferenceStore store = PitCoreActivator.getDefault().getPreferenceStore();
-        store.setDefault(PIT_EXECUTION_MODE, "containingProject");
+        store.setDefault(EXECUTION_SCOPE, "containingProject");
         store.setDefault(RUN_IN_PARALLEL, true);
         store.setDefault(INCREMENTAL_ANALYSIS, false);
-        store.setDefault(AVOID_CALLS_TO, DEFAULT_AVOID_CALLS_TO_LIST);
+        store.setDefault(AVOID_CALLS, DEFAULT_AVOID_CALLS_TO_LIST);
         store.setDefault(DEFAULT_MUTATORS, "defaultMutators");
         store.setDefault(TIMEOUT, DEFAULT_TIMEOUT);
         store.setDefault(TIMEOUT_FACTOR, DEFAULT_TIMEOUT_FACTOR.toString());

bundles/org.pitest.pitclipse.launch.ui/META-INF/MANIFEST.MF

@@ -16,6 +16,7 @@ Require-Bundle: org.eclipse.core.runtime;bundle-version="[3.11.1,4.0.0)",
  org.pitest.pitclipse.launch,
  org.eclipse.jface;bundle-version="[3.11.1,4.0.0)",
  org.eclipse.ui.workbench;bundle-version="[3.107.1,4.0.0)",
- org.eclipse.jdt.debug.ui;bundle-version="[3.7.101,4.0.0)"
+ org.eclipse.jdt.debug.ui;bundle-version="[3.7.101,4.0.0)",
+ org.pitest;bundle-version="1.6.7"
 Import-Package: com.google.common.collect;version="21.0.0"
 Export-Package: org.pitest.pitclipse.launch.ui