A test for completely builtin names

IPython/core/completer.py

@@ -2014,7 +2014,7 b' class IPCompleter(Completer):'
 
         # Start with a clean slate of completions
         matches = []
-        custom_res = self.dispatch_custom_completer(text)
+        
         # FIXME: we should extend our api to return a dict with completions for
         # different types of objects.  The rlcomplete() method could then
         # simply collapse the dict into a list for readline, but we'd have
@@ -2025,13 +2025,7 b' class IPCompleter(Completer):'
                 full_text = line_buffer
             completions = self._jedi_matches(
                 cursor_pos, cursor_line, full_text)
-        if custom_res is not None:
-            # did custom completers produce something?
-            matches = [(m, 'custom') for m in custom_res]
-        else:
-            # Extend the list of completions with the results of each
-            # matcher, so we return results to the user from all
-            # namespaces.
+                
             if self.merge_completions:
                 matches = []
                 for matcher in self.matchers:
@@ -2048,6 +2042,7 b' class IPCompleter(Completer):'
                                for m in matcher(text)]
                     if matches:
                         break
+                    
         seen = set()
         filtered_matches = set()
         for m in matches:
@@ -2056,10 +2051,13 b' class IPCompleter(Completer):'
                 filtered_matches.add(m)
                 seen.add(t)
 
-        _filtered_matches = sorted(
-            set(filtered_matches), key=lambda x: completions_sorting_key(x[0]))\
-            [:MATCHES_LIMIT]
+        _filtered_matches = sorted(filtered_matches, key=lambda x: completions_sorting_key(x[0]))
+
+        custom_res = [(m, 'custom') for m in self.dispatch_custom_completer(text) or []]
+        
+        _filtered_matches = custom_res or _filtered_matches
 
+        _filtered_matches = _filtered_matches[:MATCHES_LIMIT]
         _matches = [m[0] for m in _filtered_matches]
         origins = [m[1] for m in _filtered_matches]
 

IPython/core/tests/test_completer.py

@@ -175,6 +175,20 b' class TestCompleter(unittest.TestCase):'
 
         ip.complete("x.")
 
+    def test_custom_completion_ordering(self):
+        """Test that errors from custom attribute completers are silenced."""
+        ip = get_ipython()
+
+        _, matches = ip.complete('in')
+        assert matches.index('input') < matches.index('int')
+
+        def complete_example(a):
+            return ['example2', 'example1']
+
+        ip.Completer.custom_completers.add_re('ex*', complete_example)
+        _, matches = ip.complete('ex')
+        assert matches.index('example2') < matches.index('example1')
+
     def test_unicode_completions(self):
         ip = get_ipython()
         # Some strings that trigger different types of completion.  Check them both