Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion src/mutmut/mutation/file_mutation.py
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,12 @@ def _skip_node_and_children(self, node: cst.CSTNode) -> bool:
if isinstance(decorator, cst.Name) and decorator.value in ("staticmethod", "classmethod"):
return False
return True
if isinstance(node, cst.ClassDef) and len(node.decorators):

# decorators are executed at definition time, so mutating them can raise
# exceptions on import. A decorated class is still recursed into, because
# its decorator is kept on the original class and only its methods get
# trampolines, so reasons 1) and 3) above do not apply to it.
if isinstance(node, cst.Decorator):
return True

return False
Expand Down
18 changes: 18 additions & 0 deletions tests/mutation/test_mutation.py
Original file line number Diff line number Diff line change
Expand Up @@ -954,6 +954,24 @@ def x(self):
assert not mutants


@pytest.mark.parametrize("decorator", ["@dataclass", "@dataclass(frozen=True)"])
def test_mutate_methods_of_decorated_classes(decorator):
# The decorator of a class is not copied into the trampoline, so
# decorating a class must not stop its methods from being mutated
source = f"""
{decorator}
class Foo:
x: int
y: int

def sum(self):
return self.x + self.y
""".strip()

mutants = mutants_for_source(source)
assert mutants == [source.replace("self.x + self.y", "self.x - self.y")]


def test_do_not_mutate_pattern_single_line(patch_config):
source = 'logger.info("hello")'
patch_config("do_not_mutate_patterns", [r"logger\.\w+\("])
Expand Down
Loading