apex-mutation-testing 1.7.0 → 1.7.1

package/README.md

@@ -22,7 +22,7 @@ sf apex mutation test run --apex-class MyClass --test-class MyClassTest
 
 ## What is it mutation testing ?
 
-Mutation testing is a software testing technique that evaluates the quality of your test suite by introducing small changes (mutations) to your code and checking if your tests can detect these changes. It helps identify weaknesses in your test coverage by measuring how effectively your tests can catch intentional bugs. cf [wikipedia](https://en.wikipedia.org/wiki/Mutation_testing)
+Mutation testing is a software testing technique that evaluates the quality of your test suite by introducing small changes (mutations) to your code and checking if your tests can detect these changes. It helps identify weaknesses in your test coverage by measuring how effectively your tests can catch intentional bugs. cf [wikipedia](https://en.wikipedia.org/wiki/Mutation_testing) 
 
 The apex-mutation-testing plugin implements this technique for Salesforce Apex code by:
 
@@ -127,7 +127,7 @@ The config file supports the following attributes:
 | `threshold`           | `number`   | Minimum mutation score (0–100) required for the command to succeed                                                |
 | `skipPatterns`        | `string[]` | RE2 regex patterns to skip lines from mutation (e.g., `System\\.debug`)                                           |
 | `lines`               | `string[]` | Line ranges to restrict mutation to (e.g., `1-10`, `42`)                                                          |
-| `mutationGrouping`    | `boolean`  | Pack mutations whose covering tests are disjoint into a single deploy + test run; runs the full pipeline (test-induced clique lower bound → DSATUR → exact backtracking coloring). Off by default.                |
+| `mutationGrouping`    | `boolean`  | Pack mutations with disjoint tests into one deploy + run (clique → DSATUR → exact coloring). Off by default.      |
 
 **Mutual exclusivity:** You cannot specify both `include` and `exclude` within the same group.
 For example, setting both `mutators.include` and `mutators.exclude` will result in an error.
@@ -412,7 +412,7 @@ EXAMPLES
     $ sf apex mutation test run --apex-class MyClass --test-class MyClassTest --dry-run
 ```
 
-_See code: [src/commands/apex/mutation/test/run.ts](https://github.com/scolladon/apex-mutation-testing/blob/v1.7.0/src/commands/apex/mutation/test/run.ts)_
+_See code: [src/commands/apex/mutation/test/run.ts](https://github.com/scolladon/apex-mutation-testing/blob/v1.7.1/src/commands/apex/mutation/test/run.ts)_
 <!-- commandsstop -->
 <!-- markdownlint-enable MD040 -->
 

package/lib/mutator/unaryOperatorInsertionMutator.d.ts

@@ -4,5 +4,5 @@ import { BaseListener } from './baseListener.js';
 export declare class UnaryOperatorInsertionMutator extends BaseListener {
     constructor(typeRegistry?: TypeRegistry);
     enterPrimaryExpression(ctx: ParserRuleContext): void;
-    private isLiteral;
+    private isNonAssignableReceiver;
 }

package/lib/mutator/unaryOperatorInsertionMutator.js

@@ -1,24 +1,21 @@
-import { ParserRuleContext } from 'antlr4ts';
+import { ArrayExpressionContext, DotExpressionContext, IdPrimaryContext, } from 'apex-parser';
 import { BaseListener } from './baseListener.js';
 export class UnaryOperatorInsertionMutator extends BaseListener {
     constructor(typeRegistry) {
         super(typeRegistry);
     }
     enterPrimaryExpression(ctx) {
+        if (this.isNonAssignableReceiver(ctx)) {
+            return;
+        }
         if (ctx.childCount !== 1) {
             return;
         }
         const primary = ctx.getChild(0);
-        if (!(primary instanceof ParserRuleContext)) {
+        if (!(primary instanceof IdPrimaryContext)) {
             return;
         }
         const text = primary.text;
-        if (this.isLiteral(text)) {
-            return;
-        }
-        if (text === 'this' || text === 'super') {
-            return;
-        }
         if (this.typeRegistry) {
             const methodName = this.getEnclosingMethodName(ctx);
             if (methodName && !this.typeRegistry.isNumericOperand(methodName, text)) {
@@ -39,15 +36,13 @@ export class UnaryOperatorInsertionMutator extends BaseListener {
             this.createMutation(ctx.start, ctx.stop, text, `--${text}`);
         }
     }
-    isLiteral(text) {
-        if (/^\d/.test(text))
-            return true;
-        if (text.startsWith("'"))
-            return true;
-        const lower = text.toLowerCase();
-        if (lower === 'true' || lower === 'false' || lower === 'null')
-            return true;
-        return false;
+    // The receiver of a dot (`a.b`) or array (`a[i]`) access is not an assignable
+    // target: `a++.b` / `a++[i]` are invalid. The index of `a[i]` is assignable, so
+    // only the receiver (first child) of an ArrayExpression is rejected.
+    isNonAssignableReceiver(ctx) {
+        const parent = ctx.parent;
+        return (parent instanceof DotExpressionContext ||
+            (parent instanceof ArrayExpressionContext && parent.expression(0) === ctx));
     }
 }
 //# sourceMappingURL=unaryOperatorInsertionMutator.js.map

package/lib/mutator/unaryOperatorInsertionMutator.js.map

@@ -1 +1 @@
-{"version":3,"file":"unaryOperatorInsertionMutator.js","sourceRoot":"","sources":["../../src/mutator/unaryOperatorInsertionMutator.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,UAAU,CAAA;AAE5C,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAA;AAEhD,MAAM,OAAO,6BAA8B,SAAQ,YAAY;IAC7D,YAAY,YAA2B;QACrC,KAAK,CAAC,YAAY,CAAC,CAAA;IACrB,CAAC;IAED,sBAAsB,CAAC,GAAsB;QAC3C,IAAI,GAAG,CAAC,UAAU,KAAK,CAAC,EAAE,CAAC;YACzB,OAAM;QACR,CAAC;QAED,MAAM,OAAO,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAA;QAC/B,IAAI,CAAC,CAAC,OAAO,YAAY,iBAAiB,CAAC,EAAE,CAAC;YAC5C,OAAM;QACR,CAAC;QAED,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,CAAA;QAEzB,IAAI,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,EAAE,CAAC;YACzB,OAAM;QACR,CAAC;QAED,IAAI,IAAI,KAAK,MAAM,IAAI,IAAI,KAAK,OAAO,EAAE,CAAC;YACxC,OAAM;QACR,CAAC;QAED,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;YACtB,MAAM,UAAU,GAAG,IAAI,CAAC,sBAAsB,CAAC,GAAG,CAAC,CAAA;YACnD,IAAI,UAAU,IAAI,CAAC,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,UAAU,EAAE,IAAI,CAAC,EAAE,CAAC;gBACxE,OAAM;YACR,CAAC;QACH,CAAC;QAED,IAAI,GAAG,CAAC,KAAK,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;YAC1B,gFAAgF;YAChF,qEAAqE;YACrE,MAAM,QAAQ,GAAG,IAAI,CAAC,uBAAuB,CAAC,GAAG,CAAC,CAAA;YAClD,IAAI,CAAC,QAAQ,EAAE,CAAC;gBACd,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,IAAI,IAAI,CAAC,CAAA;YAC7D,CAAC;YACD,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,CAAA;YAC3D,IAAI,CAAC,QAAQ,EAAE,CAAC;gBACd,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,IAAI,IAAI,CAAC,CAAA;YAC7D,CAAC;YACD,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,CAAA;QAC7D,CAAC;IACH,CAAC;IAEO,SAAS,CAAC,IAAY;QAC5B,IAAI,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC;YAAE,OAAO,IAAI,CAAA;QACjC,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC;YAAE,OAAO,IAAI,CAAA;QACrC,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,EAAE,CAAA;QAChC,IAAI,KAAK,KAAK,MAAM,IAAI,KAAK,KAAK,OAAO,IAAI,KAAK,KAAK,MAAM;YAAE,OAAO,IAAI,CAAA;QAC1E,OAAO,KAAK,CAAA;IACd,CAAC;CACF"}
+{"version":3,"file":"unaryOperatorInsertionMutator.js","sourceRoot":"","sources":["../../src/mutator/unaryOperatorInsertionMutator.ts"],"names":[],"mappings":"AACA,OAAO,EACL,sBAAsB,EACtB,oBAAoB,EACpB,gBAAgB,GACjB,MAAM,aAAa,CAAA;AAEpB,OAAO,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAA;AAEhD,MAAM,OAAO,6BAA8B,SAAQ,YAAY;IAC7D,YAAY,YAA2B;QACrC,KAAK,CAAC,YAAY,CAAC,CAAA;IACrB,CAAC;IAED,sBAAsB,CAAC,GAAsB;QAC3C,IAAI,IAAI,CAAC,uBAAuB,CAAC,GAAG,CAAC,EAAE,CAAC;YACtC,OAAM;QACR,CAAC;QAED,IAAI,GAAG,CAAC,UAAU,KAAK,CAAC,EAAE,CAAC;YACzB,OAAM;QACR,CAAC;QAED,MAAM,OAAO,GAAG,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAA;QAC/B,IAAI,CAAC,CAAC,OAAO,YAAY,gBAAgB,CAAC,EAAE,CAAC;YAC3C,OAAM;QACR,CAAC;QAED,MAAM,IAAI,GAAG,OAAO,CAAC,IAAI,CAAA;QAEzB,IAAI,IAAI,CAAC,YAAY,EAAE,CAAC;YACtB,MAAM,UAAU,GAAG,IAAI,CAAC,sBAAsB,CAAC,GAAG,CAAC,CAAA;YACnD,IAAI,UAAU,IAAI,CAAC,IAAI,CAAC,YAAY,CAAC,gBAAgB,CAAC,UAAU,EAAE,IAAI,CAAC,EAAE,CAAC;gBACxE,OAAM;YACR,CAAC;QACH,CAAC;QAED,IAAI,GAAG,CAAC,KAAK,IAAI,GAAG,CAAC,IAAI,EAAE,CAAC;YAC1B,gFAAgF;YAChF,qEAAqE;YACrE,MAAM,QAAQ,GAAG,IAAI,CAAC,uBAAuB,CAAC,GAAG,CAAC,CAAA;YAClD,IAAI,CAAC,QAAQ,EAAE,CAAC;gBACd,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,IAAI,IAAI,CAAC,CAAA;YAC7D,CAAC;YACD,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,CAAA;YAC3D,IAAI,CAAC,QAAQ,EAAE,CAAC;gBACd,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,GAAG,IAAI,IAAI,CAAC,CAAA;YAC7D,CAAC;YACD,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC,KAAK,EAAE,GAAG,CAAC,IAAI,EAAE,IAAI,EAAE,KAAK,IAAI,EAAE,CAAC,CAAA;QAC7D,CAAC;IACH,CAAC;IAED,8EAA8E;IAC9E,gFAAgF;IAChF,qEAAqE;IAC7D,uBAAuB,CAAC,GAAsB;QACpD,MAAM,MAAM,GAAG,GAAG,CAAC,MAAM,CAAA;QACzB,OAAO,CACL,MAAM,YAAY,oBAAoB;YACtC,CAAC,MAAM,YAAY,sBAAsB,IAAI,MAAM,CAAC,UAAU,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAC3E,CAAA;IACH,CAAC;CACF"}