none positions shouldn't be an option, (0, 0) is the 'invalid' or 'generated' position for objects that are always found

jedi/evaluate.py

@@ -949,8 +949,12 @@ def get_defined_names_for_position(scope, position=None, start_scope=None):
         return names
     names_new = []
     for n in names:
-        if n.start_pos < position:
+     try:
+        if n.start_pos[0] is not None and n.start_pos < position:
             names_new.append(n)
+     except:
+        print(n, position, n.parent())
+        raise
     return names_new
 
 

jedi/imports.py

@@ -80,8 +80,8 @@ class ImportPath(parsing.Base):
         """
         i = self.import_stmt
         # This is not an existing Import statement. Therefore, set position to
-        # None.
-        zero = (None, None)
+        # 0 (0 is not a valid line number).
+        zero = (0, 0)
         n = parsing.Name(i.namespace.names[1:], zero, zero, self.import_stmt)
         new = parsing.Import(zero, zero, n)
         new.parent = weakref.ref(parent)