🧪 mutator plugin tests

.vscode/settings.json

@@ -11,6 +11,7 @@
     "python.testing.pytestArgs": [
         "-vv",
         "-s",
+        // "--sort-mode=ordered",
         "tests"
     ],
     "python.analysis.extraPaths": [
@@ -45,4 +46,5 @@
     ],
     "scss.format.preserveNewLines": false,
     "scss.format.newlineBetweenSelectors": false,
+    "terminal.integrated.cwd": "C:\\DEVPY"
 }

notes.ipynb

@@ -28,6 +28,17 @@
     "Decorator: instead of replacing func, replace only the decorator part"
    ]
   },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "import ast\n",
+    "\n",
+    "print(ast.dump(ast.parse(\"a = f'abc{a}def'\"),indent=4))"
+   ]
+  },
   {
    "cell_type": "code",
    "execution_count": null,
@@ -52,7 +63,7 @@
    "name": "python",
    "nbconvert_exporter": "python",
    "pygments_lexer": "ipython3",
-   "version": "3.12.0"
+   "version": "3.12.3"
   }
  },
  "nbformat": 4,

pyproject.toml

@@ -58,7 +58,7 @@ where = ["src"]
 [tool.pytest.ini_options]
 pythonpath = ["src", "tests"]
 testpaths = ["tests"]
-addopts = "-vv --cov-reset --cov-report=html:cov-html --cov-report=term"
+addopts = "-vv --cov-reset --cov-report=html:cov-html --cov-report=term --cov-context=test"
 sort_mode = "random"
 sort_bucket = "session"
 filterwarnings = [
@@ -77,6 +77,7 @@ command_line = "pytest -x --assert=plain -o pythonpath='{PYTHONPATH}' --sort-mod
 
 [tool.coverage.run]
 branch = true
+include = ["src/**"]
 
 [tool.coverage.report]
 skip_empty = true
@@ -98,9 +99,9 @@ mypy_path = "src,tests"
 
 [tool.ruff]
 target-version = "py39"
-show-source = true
-select = ["ALL"]
-ignore = [
+output-format = "full"
+lint.select = ["ALL"]
+lint.ignore = [
     "COM812",  # missing-trailing-comma
     "D203",    # 1 blank line required before class docstring
     "D213",    # multi-line-summary-second-line
@@ -133,12 +134,12 @@ exclude = [
     "dist",
     "venv",
 ]
-task-tags = ["nomut", "TODO", "FIXME", "XXX"]
+lint.task-tags = ["nomut", "TODO", "FIXME", "XXX"]
 
 [tool.ruff.lint.isort]
 known-first-party = ["poodle"]
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint.per-file-ignores]
 "src/*/__init__.py" = [
     "F401", # imported but unused
     "F403", # import * used

requirements.txt

@@ -14,5 +14,6 @@ pysass
 pytest
 pytest-sort>=1.3.0
 pytest-cov
+coverage[toml]
 
 mock-decorator

src/poodle/__init__.py

@@ -11,9 +11,9 @@
     Mutant,
     MutantTrial,
     MutantTrialResult,
+    RunResult,
     TestingResults,
     TestingSummary,
-    RunResult,
 )
 from poodle.common.echo_wrapper import EchoWrapper
 from poodle.common.exceptions import (

src/poodle/common/exceptions.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+
 class PoodleTestingFailedError(Exception):
     """Poodle testing failed."""
 

src/poodle/common/hook_spec.py

@@ -8,7 +8,7 @@
 import pluggy
 
 from .config import PoodleConfigData
-from .data import FileMutation, Mutant, TestingResults, RunResult
+from .data import FileMutation, Mutant, RunResult, TestingResults
 from .echo_wrapper import EchoWrapper
 from .option_collector import PoodleOptionCollector
 

src/poodle/core.py

@@ -52,14 +52,14 @@ def main_process(config_data: PoodleConfigData, secho: EchoWrapper) -> None:
 
 
 poodle_header_str = r"""
-|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|
-    ____                  ____         ''',
-   / __ \____  ____  ____/ / /__    o_)O \)____)"
-  / /_/ / __ \/ __ \/ __  / / _ \    \_        )
- / ____/ /_/ / /_/ / /_/ / /  __/      '',,,,,,
-/_/    \____/\____/\__,_/_/\___/         ||  ||
-Mutation Tester Version {version:<15} "--'"--'
-|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|
+|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|\/|
+    ____                  ____         ''',          ''',
+   / __ \____  ____  ____/ / /__    o_)O \)____)" o_)O \)____)"
+  / /_/ / __ \/ __ \/ __  / / _ \    \_        )   \_        )
+ / ____/ /_/ / /_/ / /_/ / /  __/      '',,,,,,      '',,,,,,
+/_/    \____/\____/\__,_/_/\___/         ||  ||        ||  ||
+Mutation Tester Version {version:<15} "--'"--'      "--'"--'
+|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|/\|
 
 """
 

src/poodle/mutators/mutator_compare.py

@@ -62,8 +62,9 @@ def create_mutations(self, parsed_ast: ast.Module, config: PoodleConfigData) ->
 
     def visit_Compare(self, node: ast.Compare) -> None:
         """Identify replacement Operations and create Mutants."""
-
         for idx, op in enumerate(node.ops):
+            if type(op) not in self.type_map:
+                continue
             for new_op in self.type_map[type(op)]:
                 mut = deepcopy(node)
                 mut.ops[idx] = new_op()

src/poodle/mutators/mutator_lambda.py

@@ -38,7 +38,7 @@ def visit_Lambda(self, node: ast.Lambda) -> None:
         """Replace body of Lambda with None or empty string."""
         if isinstance(node.body, ast.Constant) and node.body.value is None:
             mut = deepcopy(node)
-            mut.body = ast.Constant("")
+            mut.body = ast.Constant(" ")
             self.mutants.append(self.create_file_mutation(mut, ast.unparse(mut)))
         else:
             mut = deepcopy(node)

src/poodle/mutators/mutator_operation.py

@@ -7,7 +7,7 @@
 
 import pluggy
 
-from poodle import FileMutation, MutatorBase, PoodleConfigData, EchoWrapper
+from poodle import EchoWrapper, FileMutation, MutatorBase, PoodleConfigData
 
 hookimpl = pluggy.HookimplMarker("poodle")
 
@@ -66,7 +66,7 @@ class OperationMutator(ast.NodeVisitor, MutatorBase):
             ast.RShift: [ast.LShift],
             ast.BitOr: [ast.BitAnd],
             ast.BitXor: [ast.BitOr, ast.BitAnd],
-            ast.BitAnd: [ast.BitOr],
+            ast.BitAnd: [ast.BitOr, ast.BitXor],
         },
         "max": {
             ast.Add: [ast.Sub, ast.Mult, ast.Div, ast.FloorDiv, ast.Mod, ast.Pow],
@@ -84,10 +84,10 @@ class OperationMutator(ast.NodeVisitor, MutatorBase):
         },
     }
 
-    @hookimpl()
+    @hookimpl(specname="configure")
     def configure(self, config: PoodleConfigData, secho: EchoWrapper) -> None:
         mutator_opts = config.merge_dict_from_config("mutator_opts", {})
-        level = mutator_opts.get("operator_level", "std")
+        level = str(mutator_opts.get("operator_level", "std")).lower()
 
         if level not in self.type_map_levels:
             secho(f"WARN: Invalid value operator_opts.operator_level={level}.  Using Default value 'std'", fg="yellow")

src/poodle/mutators/mutator_subscript.py

@@ -28,7 +28,7 @@ def create_mutations(self, parsed_ast: ast.Module, config: PoodleConfigData) ->
         """Visit all nodes and return created mutants."""
         if not self.is_enabled(config):
             return []
-        
+
         self.mutants = []
         self.visit(parsed_ast)
         return self.mutants
@@ -37,5 +37,5 @@ def visit_Subscript(self, node: ast.Subscript) -> None:
         """Replace Call to retrieve from Dict or Array with None."""
         if self.is_annotation(node):
             return
-        
+
         self.mutants.append(self.create_file_mutation(node, "None"))

src/poodle/reporters/__init__.py

@@ -2,4 +2,4 @@
 
 from . import reporter_html, reporter_json, reporter_sysout
 
-reporter_plugins = [reporter_html, reporter_json, reporter_sysout]
+reporter_plugins = [reporter_html, reporter_json, reporter_sysout]

tests/common/test_config_base.py

@@ -423,7 +423,7 @@ def test_get_option_from_config_no_cmd(self):
         config.poodle_config = mock.MagicMock()
         config.poodle_config.test_option = "pc_data"
         config.config_file_data = {"test_option": "file_data"}
-        assert config.get_option_from_config("test_option","option") == ("pc_data", "poodle_config.py")
+        assert config.get_option_from_config("test_option", "option") == ("pc_data", "poodle_config.py")
 
     def test_get_option_from_config_pc(self):
         config = config_base.PoodleConfigBase()

tests/common/test_config_util.py

@@ -1,16 +1,17 @@
 from __future__ import annotations
 
+import importlib
+import sys
 from pathlib import Path
 from unittest import mock
 
 import pytest
 
 from poodle.common import config_util
-import sys
-import importlib
 
 m_name = config_util.__name__
 
+
 @pytest.fixture(autouse=True)
 def _reset() -> None:
     importlib.reload(config_util)