The Inline Method refactoring removed an import

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

Maintainer antworten meist innerhalb von 7 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
Ein halber Tag
Anfängerfreundlichkeit
55/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
tooling

Rechercherichtung

Start with the reproduction: apply Inline Method to StringlikeMixin.find in mixins.py, then compare the test_mixins.py imports before and after. The bug is that from nose.tools import * disappears, so the inlined assert_equal call breaks. Find where rope's inline refactoring rewrites imports in the edited module, and check whether it drops names that the rewritten code still uses. Done means test_mixins.py keeps the nose import after inlining and the test runs.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

The Inline Method refactoring removed an import from the test file, causing an error when the test file is run.

Steps to reproduce the behavior:

  1. Code before refactoring:
-- mixins.py
from __future__ import unicode_literals

class StringlikeMixin(object):
    def __init__(self, text):
        self._text = text

    def _strkey(self):
        return self._text

    def find(self, sub, start=0, end=None):
        if end is None:
            end = len(self._strkey())
        return self._strkey().find(sub, start, end)

-- test_mixins.py
from __future__ import unicode_literals, absolute_import
from nose.tools import *
from mixins import StringlikeMixin


def test_find():
    text = 'Beautiful is better than ugly.'
    blob = StringlikeMixin(text)

    assert_equal(
        blob.find('better', 5, len(blob._strkey())),
        text.find('better', 5, len(text))
    )

  1. Apply the inline method to StringlikeMixin.find

  2. Code after refactoring:

-- mixins.py
from __future__ import unicode_literals

class StringlikeMixin(object):
    def __init__(self, text):
        self._text = text

    def _strkey(self):
        return self._text


-- test_mixins.py
from __future__ import unicode_literals, absolute_import

from mixins import StringlikeMixin

def test_find():
    text = 'Beautiful is better than ugly.'
    blob = StringlikeMixin(text)

    assert_equal(
        blob._strkey().find('better', 5, len(blob._strkey())),
        text.find('better', 5, len(text))
    )
Vorherrschende Sprache
Python
Sterne
2.2k
Forks
197
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
5

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.