getattr is now also allowed on modules, fixes #116

This commit is contained in:
David Halter
2013-05-03 17:00:07 +04:30
parent 1d45105461
commit 4865505215
2 changed files with 6 additions and 1 deletions

View File

@@ -408,7 +408,7 @@ class Execution(Executable):
return []
for obj in objects:
if not isinstance(obj, (Instance, Class)):
if not isinstance(obj, (Instance, Class, pr.Module)):
debug.warning('getattr called without instance')
continue

View File

@@ -238,6 +238,11 @@ class TestRegression(TestBase):
# jedi-vim #70
s = """def foo("""
assert self.function_definition(s) is None
jedi.set_debug_function(jedi.debug.print_to_stdout)
# jedi-vim #116
s = """import functools; test = getattr(functools, 'partial'); test("""
check(self.function_definition(s), 'partial', 0)
jedi.set_debug_function(None)
@unittest.skip("function_definition at ``f( |)`` does not work")
def test_function_definition_empty_paren_pre_space(self):