VimPlug/jedi-fork
1
0
Fork 0
forked from VimPlug/jedi
Code Pull Requests Activity

Fix an issue where a default value was wrongly used

Browse Source
This commit is contained in:
Dave Halter
2018-01-31 00:11:30 +01:00
parent 2b9429be38
commit 3ae0560f1c
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
jedi/evaluate/compiled/access.py
View File

@@ -316,7 +316,7 @@ class DirectObjectAccess(object):
# -> just set it to None # -> just set it to None
if default is _sentinel: if default is _sentinel:
raise raise
return default return self._create_access(default)
def get_safe_value(self): def get_safe_value(self):
if type(self._obj) in (bool, bytes, float, int, str, unicode, slice): if type(self._obj) in (bool, bytes, float, int, str, unicode, slice):

3
test/test_api/test_classes.py
View File

@@ -290,6 +290,9 @@ def test_type(Script):
else: else:
assert c.type in ('function', 'instance') assert c.type in ('function', 'instance')
for c in Script('list.').completions():
assert c.type
# Github issue #397, type should never raise an error. # Github issue #397, type should never raise an error.
for c in Script('import os; os.path.').completions(): for c in Script('import os; os.path.').completions():
assert c.type assert c.type