Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Removing `not` from a parenthesized multi-line expression drops the parentheses (InvalidGeneratedSyntaxException, 3.8.0)

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#602 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python
Lĩnh vực
testing

Hướng nghiên cứu

Lỗi nằm ở operator_remove_unary_ops trong src/mutmut/mutation/mutators.py, hàm này trả về toán hạng của biểu thức not hoặc ~ và làm mất lpar/rpar trên nút UnaryOperation. Hãy tái hiện bằng đoạn mã ngắn mutate_file_contents trong issue và kiểm tra ast.parse có lỗi với mutant hay không. Hoàn thành khi toán hạng giữ nguyên dấu ngoặc ngoài, mutant phân tích cú pháp được, và ví dụ trong issue tạo ra các mutant hợp lệ.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

🤖 Written by Claude (AI assistant). The reproduction below was run on a clean environment.

Summary

operator_remove_unary_ops replaces a not / ~ expression with its operand. In libcst, the parentheses that wrap the whole unary expression belong to the UnaryOperation node (lpar / rpar), so they are dropped together with the operator. When the expression spans several lines (which is how black and ruff format a long boolean return), the mutant is invalid Python and mutmut run aborts with InvalidGeneratedSyntaxException, so no mutant of the run gets tested.

Reproduction (mutmut 3.8.0, Python 3.12)

mylib/core.py:

def is_not_before(a, b):
    return (
        not a
        < b
    )

tests/test_core.py:

from mylib.core import is_not_before


def test_is_not_before():
    assert is_not_before(2, 1) is True
    assert is_not_before(1, 2) is False

pyproject.toml:

[tool.mutmut]
source_paths = ["mylib/"]
pytest_add_cli_args_test_selection = ["tests/"]

mutmut run:

mutmut.__main__.InvalidGeneratedSyntaxException: Mutmut generated invalid python syntax for Mutmut generated invalid python syntax for mutants/mylib/core.py. If the original file has valid python syntax, please file an issue with a minimal reproducible example file.. If the original file has valid python syntax, please file an issue with a minimal reproducible example file.

It also reproduces without running any test:

import ast
from mutmut.mutation.file_mutation import mutate_file_contents

mf = mutate_file_contents("mylib/core.py", open("mylib/core.py").read())
ast.parse(mf.code)  # SyntaxError: unexpected indent

The generated mutant:

def x_is_not_before__mutmut_1(a, b):
    return a
        < b

Cause

src/mutmut/mutation/mutators.py (same on main):

def operator_remove_unary_ops(node: cst.UnaryOperation) -> Iterable[cst.BaseExpression]:
    if isinstance(node.operator, cst.Not | cst.BitInvert):
        yield node.expression

In (not a\n < b), the parentheses are node.lpar / node.rpar, and node.expression is the Comparison without them.

Suggested fix

Carry the parentheses over to the operand:

def operator_remove_unary_ops(node: cst.UnaryOperation) -> Iterable[cst.BaseExpression]:
    if isinstance(node.operator, cst.Not | cst.BitInvert):
        yield node.expression.with_changes(
            lpar=[*node.lpar, *node.expression.lpar],
            rpar=[*node.expression.rpar, *node.rpar],
        )

With this change, the example above generates 2 valid mutants (1 killed, 1 survived), and mutmut show mylib.core.x_is_not_before__mutmut_1 gives:

 def is_not_before(a, b):
     return (
-        not a
+        a
         < b
     )

Side note: the exception message is printed twice ("Mutmut generated invalid python syntax for Mutmut generated invalid python syntax for ...").

Ngôn ngữ chính
Python
Star
1.5k
Fork
181
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của boxed/mutmut

Tất cả issue của boxed/mutmut

Issue tương tự

Thêm issue về Python

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.