Commit82094c1

committed

Fix __signature__ support if object has a __file__

1 parent26fc2b5 commit82094c1Copy full SHA for 82094c1

File tree

+15

-7

lines changed

+15

-7

lines changed

Lines changed: 7 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -289,8 +289,14 @@ def getfuncprops(func: str, f: Callable) -> Optional[FuncProps]:`
`289`	`289`	`returnNone`
`290`	`290`	`try:`
`291`	`291`	`argspec=_get_argspec_from_signature(f)`
	`292`	`+try:`
	`293`	`+argspec=_fix_default_values(f,argspec)`
	`294`	`+exceptKeyErrorasex:`
	`295`	`+# Parsing of the source failed. If f has a __signature__, we trust it.`
	`296`	`+ifnothasattr(f,"__signature__"):`
	`297`	`+raiseex`
`292`	`298`	`fprops=FuncProps(`
`293`		`-func,_fix_default_values(f,argspec),is_bound_method`
	`299`	`+func,argspec,is_bound_method`
`294`	`300`	`)`
`295`	`301`	`except (TypeError,KeyError,ValueError):`
`296`	`302`	`argspec_pydoc=_getpydocspec(f)`

Lines changed: 8 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`importcollections`
`2`	`2`	`importinspect`
	`3`	`+importos`
`3`	`4`	`importsocket`
`4`	`5`	`importsys`
`5`	`6`	`importtempfile`
`@@ -523,13 +524,14 @@ def __init__(self, args, *kwargs):`
`523`	`524`	`inspect.Parameter("pinetree", inspect.Parameter.KEYWORD_ONLY),`
`524`	`525`	`])`
`525`	`526`	`"""`
`526`		`-forlineincode.split("\n"):`
`527`		`-print(line[8:])`
`528`		`-self.repl.push(line[8:])`
	`527`	`+code= [x[8:]forxincode.split("\n")]`
	`528`	`+forlineincode:`
	`529`	`+self.repl.push(line)`
`529`	`530`
`530`		`-self.assertTrue(self.repl.complete())`
`531`		`-self.assertTrue(hasattr(self.repl.matches_iter,"matches"))`
`532`		`-self.assertEqual(self.repl.matches_iter.matches, ["apple2=","apple="])`
	`531`	`+withmock.patch("bpython.inspection.inspect.getsourcelines",return_value=(code,None)):`
	`532`	`+self.assertTrue(self.repl.complete())`
	`533`	`+self.assertTrue(hasattr(self.repl.matches_iter,"matches"))`
	`534`	`+self.assertEqual(self.repl.matches_iter.matches, ["apple2=","apple="])`
`533`	`535`
`534`	`536`
`535`	`537`	`if__name__=="__main__":`

Comments

(0)