mutmut 3.0.4__tar.gz → 3.1.0__tar.gz

{mutmut-3.0.4 → mutmut-3.1.0}/HISTORY.rst

@@ -1,6 +1,24 @@
 Changelog
 ---------
 
+3.1.0
+~~~~~
+
+* Correctly handle mutation for generator functions (`yield`).
+
+* Fixed so that `from __future__` lines are always first.
+
+* If no stats are collected exit directly, as that is a breaking error for mutation testing.
+
+* Changed name mangling to make mutants less likely to trigger name-based python magic, like in pytest where functions named `test_*` have special meaning.
+
+
+3.0.5
+~~~~~
+
+* Another attempt to get the pypi package to work
+
+
 3.0.4
 ~~~~~
 

{mutmut-3.0.4 → mutmut-3.1.0}/MANIFEST.in

@@ -2,5 +2,5 @@ include HISTORY.rst
 include LICENSE
 include README.rst
 include requirements.txt
-recursive-include mutmut *.tcss
 exclude tests/*.*
+recursive-include mutmut *.*

{mutmut-3.0.4 → mutmut-3.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: mutmut
-Version: 3.0.4
+Version: 3.1.0
 Summary: mutation testing for Python 3
 Home-page: https://github.com/boxed/mutmut
 Author: Anders Hovmöller

{mutmut-3.0.4 → mutmut-3.1.0}/mutmut/__init__.py

@@ -4,7 +4,7 @@ from parso.python.tree import (
     Number,
 )
 
-__version__ = '3.0.4'
+__version__ = '3.1.0'
 
 
 # We have a global whitelist for constants of the pattern __all__, __version__, etc

{mutmut-3.0.4 → mutmut-3.1.0}/mutmut/__main__.py

@@ -189,14 +189,18 @@ def _mutmut_trampoline(orig, mutants, *args, **kwargs):
     elif mutant_under_test == 'stats':
         from mutmut.__main__ import record_trampoline_hit
         record_trampoline_hit(orig.__module__ + '.' + orig.__name__)
-        return orig(*args, **kwargs)
+        ret orig(*args, **kwargs)
+        return  # for the yield case
     prefix = orig.__module__ + '.' + orig.__name__ + '__mutmut_'
     if not mutant_under_test.startswith(prefix):
-        return orig(*args, **kwargs)
+        ret orig(*args, **kwargs)
+        return  # for the yield case
     mutant_name = mutant_under_test.rpartition('.')[-1]
-    return mutants[mutant_name](*args, **kwargs)
+    ret mutants[mutant_name](*args, **kwargs)
 
 """
+yield_from_trampoline_impl = trampoline_impl.replace('ret ', 'yield from ').replace('_mutmut_trampoline', '_mutmut_yield_from_trampoline')
+trampoline_impl = trampoline_impl.replace('ret ', 'return ')
 
 
 def create_mutants():
@@ -297,7 +301,7 @@ def write_all_mutants_to_file(*, out, source, filename):
     return mutant_names, hash_by_function_name
 
 
-def build_trampoline(orig_name, mutants, class_name=None):
+def build_trampoline(*, orig_name, mutants, class_name, is_generator):
     assert orig_name not in NEVER_MUTATE_FUNCTION_NAMES
 
     mangled_name = mangle_function_name(name=orig_name, class_name=class_name)
@@ -309,11 +313,18 @@ def build_trampoline(orig_name, mutants, class_name=None):
         access_prefix = f'object.__getattribute__(self, "'
         access_suffix = '")'
 
+    if is_generator:
+        return_or_yield_statement = 'yield from'
+        trampoline_name = '_mutmut_yield_from_trampoline'
+    else:
+        return_or_yield_statement = 'return'
+        trampoline_name = '_mutmut_trampoline'
+
     return f"""
 {mutants_dict}
 
 def {orig_name}({'self, ' if class_name is not None else ''}*args, **kwargs):
-    return _mutmut_trampoline({access_prefix}{mangled_name}__mutmut_orig{access_suffix}, {access_prefix}{mangled_name}__mutmut_mutants{access_suffix}, *args, **kwargs) 
+    {return_or_yield_statement} {trampoline_name}({access_prefix}{mangled_name}__mutmut_orig{access_suffix}, {access_prefix}{mangled_name}__mutmut_mutants{access_suffix}, *args, **kwargs) 
 
 {orig_name}.__signature__ = _mutmut_signature({mangled_name}__mutmut_orig)
 {mangled_name}__mutmut_orig.__name__ = '{mangled_name}'
@@ -447,6 +458,23 @@ class FuncContext:
         return False
 
 
+def is_generator(node):
+    assert node.type == 'funcdef'
+
+    def _is_generator(n):
+        if n is not node and n.type in ('funcdef', 'classdef'):
+            return False
+
+        if n.type == 'keyword' and n.value == 'yield':
+            return True
+
+        for c in getattr(n, 'children', []):
+            if _is_generator(c):
+                return True
+        return False
+    return _is_generator(node)
+
+
 def yield_mutants_for_function(node, *, class_name=None, no_mutate_lines):
     assert node.type == 'funcdef'
 
@@ -481,7 +509,7 @@ def yield_mutants_for_function(node, *, class_name=None, no_mutate_lines):
         finally:
             context.stack.pop()
 
-    trampoline = build_trampoline(node.name.value, context.mutants, class_name=class_name)
+    trampoline = build_trampoline(orig_name=node.name.value, mutants=context.mutants, class_name=class_name, is_generator=is_generator(node))
     if class_name is not None:
         trampoline = indent(trampoline, '    ')
     yield 'trampoline', trampoline, None, None
@@ -508,15 +536,38 @@ def yield_mutants_for_class_body(node, no_mutate_lines):
             yield 'filler', child_node.get_code(), None, None
 
 
+def is_from_future_import_node(c):
+    if c.type == 'simple_stmt':
+        if c.children:
+            c2 = c.children[0]
+            if c2.type == 'import_from' and c2.children[1].type == 'name' and c2.children[1].value == '__future__':
+                return True
+    return False
+
+
+def yield_future_imports(node):
+    for c in node.children:
+        if is_from_future_import_node(c):
+            yield 'filler', c.get_code(), None, None
+
+
 def yield_mutants_for_module(node, no_mutate_lines):
+    assert node.type == 'file_input'
+
+    # First yield `from __future__`, then the rest
+    yield from yield_future_imports(node)
+
     yield 'trampoline_impl', trampoline_impl, None, None
+    yield 'trampoline_impl', yield_from_trampoline_impl, None, None
     yield 'filler', '\n', None, None
-    assert node.type == 'file_input'
     for child_node in node.children:
         if child_node.type == 'funcdef':
             yield from yield_mutants_for_function(child_node, no_mutate_lines=no_mutate_lines)
         elif child_node.type == 'classdef':
             yield from yield_mutants_for_class(child_node, no_mutate_lines=no_mutate_lines)
+        elif is_from_future_import_node(child_node):
+            # Don't yield `from __future__` after trampoline
+            pass
         else:
             yield 'filler', child_node.get_code(), None, None
 
@@ -636,10 +687,7 @@ class PytestRunner(TestRunner):
             raise BadTestExecutionCommandsException(params)
         return exit_code
 
-
     def run_stats(self, *, tests):
-        import pytest
-
         class StatsCollector:
             def pytest_runtest_teardown(self, item, nextitem):
                 unused(nextitem)
@@ -719,6 +767,8 @@ def mangle_function_name(*, name, class_name):
     if class_name:
         assert CLASS_NAME_SEPARATOR not in class_name
         prefix = f'x{CLASS_NAME_SEPARATOR}{class_name}{CLASS_NAME_SEPARATOR}'
+    else:
+        prefix = 'x_'
     return f'{prefix}{name}'
 
 
@@ -733,6 +783,9 @@ def orig_function_and_class_names_from_key(mutant_name):
     if CLASS_NAME_SEPARATOR in r:
         class_name = r[r.index(CLASS_NAME_SEPARATOR) + 1: r.rindex(CLASS_NAME_SEPARATOR)]
         r = r[r.rindex(CLASS_NAME_SEPARATOR) + 1:]
+    else:
+        assert r.startswith('x_')
+        r = r[2:]
     return r, class_name
 
 
@@ -937,7 +990,7 @@ def run_stats_collection(runner, tests=None):
 
     if not mutmut.tests_by_mangled_function_name:
         print('failed to collect stats, no active tests found')
-        return
+        exit(1)
 
     save_stats()
 
@@ -1083,7 +1136,14 @@ def run(mutant_names, *, max_children):
     time = datetime.now() - start
     print(f'    done in {round(time.total_seconds()*1000)}ms', )
 
-    sys.path.insert(0, os.path.abspath('mutants'))
+    src_path = (Path('mutants') / 'src')
+    source_path = (Path('mutants') / 'source')
+    if src_path.exists():
+        sys.path.insert(0, str(src_path.absolute()))
+    elif source_path.exists():
+        sys.path.insert(0, str(source_path.absolute))
+    else:
+        sys.path.insert(0, os.path.abspath('mutants'))
 
     # TODO: config/option for runner
     # runner = HammettRunner()

{mutmut-3.0.4 → mutmut-3.1.0}/mutmut.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: mutmut
-Version: 3.0.4
+Version: 3.1.0
 Summary: mutation testing for Python 3
 Home-page: https://github.com/boxed/mutmut
 Author: Anders Hovmöller

{mutmut-3.0.4 → mutmut-3.1.0}/mutmut.egg-info/SOURCES.txt

@@ -8,6 +8,10 @@ setup.cfg
 setup.py
 ./mutmut/__init__.py
 ./mutmut/__main__.py
+./mutmut/result_browser_layout.tcss
+mutmut/.DS_Store
+mutmut/__init__.py
+mutmut/__main__.py
 mutmut/result_browser_layout.tcss
 mutmut.egg-info/PKG-INFO
 mutmut.egg-info/SOURCES.txt
@@ -15,4 +19,8 @@ mutmut.egg-info/dependency_links.txt
 mutmut.egg-info/entry_points.txt
 mutmut.egg-info/not-zip-safe
 mutmut.egg-info/requires.txt
-mutmut.egg-info/top_level.txt
+mutmut.egg-info/top_level.txt
+mutmut/__pycache__/__init__.cpython-311.pyc
+mutmut/__pycache__/__init__.cpython-312.pyc
+mutmut/__pycache__/__main__.cpython-311.pyc
+mutmut/__pycache__/__main__.cpython-312.pyc

{mutmut-3.0.4 → mutmut-3.1.0}/setup.cfg

@@ -1,6 +1,3 @@
-[wheel]
-universal = 1
-
 [tool:pytest]
 testpaths = tests
 addopts = --junitxml=testreport.xml --strict -r fEsxXw

{mutmut-3.0.4 → mutmut-3.1.0}/setup.py

@@ -79,6 +79,9 @@ setup(
     url='https://github.com/boxed/mutmut',
     packages=find_packages('.'),
     package_dir={'': '.'},
+    package_data={
+        'mutmut': ['*.tcss'],
+    },
     include_package_data=True,
     license="BSD",
     zip_safe=False,