Applying the Inline Method refactoring does not add the required import.

Offen
  • Stimmt etwas nicht?
#743 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 5 Tagen

@Narek-Khachikyan arbeitet bereits daran.

Seit 08.11.2026.

  • #942 von @Narek-Khachikyan — offen

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
devtools

Rechercherichtung

Reproduce with the two files in the report: src/main.py defines Base.split with the default argument sys.maxsize, and src/test.py calls it, so inlining should copy that default into test.py. Start from rope's Inline Method refactoring and check how it collects imports for names used in the inlined body. Done when applying the refactoring to Base.split leaves import sys at the top of test.py, and a test covers the default-argument case.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug inline-refactor

Steps to reproduce the behavior:

  1. Code before refactoring:

structure:
project
| - src
| -- __init__.py
| -- main.py
| -- test.py

main.py

import sys


class Base:
    def __init__(self, text):
        self.raw = text

    def split(self, sep=None, maxsplit=sys.maxsize):
        return self.raw.split(sep, maxsplit)[0]

test.py

from unittest import TestCase
from src.split_main import Base

class Test(TestCase):
    def test_split(self):
        blob = Base('Beautiful is better')
        self.assertEqual(blob.split(), ['Beautiful'])

  1. Apply the Inline Method to Base.split().

  2. Expected code after refactoring:

structure:
project
| - src
| -- __init__.py
| -- main.py
| -- test.py

main.py

class Base:
    def __init__(self, text):
        self.raw = text

test.py

import sys
from unittest import TestCase
from src.split_main import Base

class Test(TestCase):
    def test_split(self):
        blob = Base('Beautiful is better')
        self.assertEqual(blob.raw.split(None, sys.maxsize)[0], ['Beautiful'])
  1. The "import sys" is not automatically added.
Vorherrschende Sprache
Python
Sterne
2.2k
Forks
197
Ø Merge
2 T. 17 Std.
Gemergte PRs (30 T.)
11

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus python-rope/rope

Alle Issues in python-rope/rope

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.