forked from VimPlug/jedi
Fix an issue with choosing the right lines in get_line_code. Refs #948.
This commit is contained in:
@@ -394,7 +394,7 @@ class BaseDefinition(object):
|
|||||||
lines = parser_cache[self._evaluator.grammar._hashed][path].lines
|
lines = parser_cache[self._evaluator.grammar._hashed][path].lines
|
||||||
|
|
||||||
line_nr = self._name.start_pos[0]
|
line_nr = self._name.start_pos[0]
|
||||||
start_line_nr = line_nr - before
|
start_line_nr = max(line_nr - before, 0)
|
||||||
return ''.join(lines[start_line_nr:line_nr + after + 1])
|
return ''.join(lines[start_line_nr:line_nr + after + 1])
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -171,6 +171,9 @@ def test_get_line_code():
|
|||||||
assert get_line_code(source, line=2) == line + '\n'
|
assert get_line_code(source, line=2) == line + '\n'
|
||||||
assert get_line_code(source, line=2, after=1) == line + '\nother_line'
|
assert get_line_code(source, line=2, after=1) == line + '\nother_line'
|
||||||
assert get_line_code(source, line=2, after=1, before=1) == source
|
assert get_line_code(source, line=2, after=1, before=1) == source
|
||||||
|
# Should just be the whole thing, since there are no more lines on both
|
||||||
|
# sides.
|
||||||
|
assert get_line_code(source, line=2, after=3, before=3) == source
|
||||||
|
|
||||||
|
|
||||||
def test_goto_assignments_follow_imports():
|
def test_goto_assignments_follow_imports():
|
||||||
|
|||||||
Reference in New Issue
Block a user