PR: Compute Projects switcher results in a worker to avoid freezes

spyder/app/tests/test_mainwindow.py

@@ -2535,9 +2535,14 @@ def example_def_2():
 
 
 @flaky(max_runs=3)
-def test_switcher_project_files(main_window, qtbot, tmpdir):
+def test_switcher_project_files(main_window, pytestconfig, qtbot, tmp_path):
     """Test the number of items in the switcher when a project is active."""
-    # Wait until the window is fully up
+    # Disable pytest stdin capture to make calls to fzf work. Idea taken from:
+    # https://github.com/pytest-dev/pytest/issues/2189#issuecomment-449512764
+    capmanager = pytestconfig.pluginmanager.getplugin('capturemanager')
+    capmanager.suspend_global_capture(in_=True)
+
+    # Wait until the console is fully up
     shell = main_window.ipyconsole.get_current_shellwidget()
     qtbot.waitUntil(
         lambda: shell.spyder_kernel_ready and shell._prompt_html is not None,
@@ -2550,47 +2555,75 @@ def test_switcher_project_files(main_window, qtbot, tmpdir):
     editorstack = main_window.editor.get_current_editorstack()
 
     # Create a temp project directory
-    project_dir = to_text_string(tmpdir.mkdir('test'))
+    project_dir = tmp_path / 'test-projects-switcher'
+    project_dir.mkdir()
 
     # Create project
     with qtbot.waitSignal(projects.sig_project_loaded):
-        projects.create_project(project_dir)
+        projects.create_project(str(project_dir))
 
-    # Create four empty files in the project dir
-    for i in range(3):
-        main_window.editor.new("test_file"+str(i)+".py")
+    # Create some empty files in the project dir
+    n_files_project = 3
+    for i in range(n_files_project):
+        fpath = project_dir / f"test_file{i}.py"
+        fpath.touch()
 
+    # Check that the switcher has been populated in Projects
+    qtbot.waitUntil(
+        lambda: projects.get_widget()._default_switcher_paths != [],
+        timeout=1000
+    )
+
+    # Assert that the number of items in the switcher is correct
     switcher.open_switcher()
-    n_files_project = len(projects.get_project_filenames())
     n_files_open = editorstack.get_stack_count()
+    assert switcher.count() == n_files_open + n_files_project
+    switcher.on_close()
 
-    # Assert that the number of items in the switcher is correct
-    assert switcher_widget.model.rowCount() == n_files_open + n_files_project
+    # Assert only two items have visible sections
+    switcher.open_switcher()
+
+    sections = []
+    for row in range(switcher.count()):
+        item = switcher_widget.model.item(row)
+        if item._section_visible:
+            sections.append(item.get_section())
+
+    assert set(sections) == {"Editor", "Project"}
     switcher.on_close()
 
-    # Close all files opened in editorstack
-    main_window.editor.close_all_files()
+    # Assert searching text in the switcher works as expected
+    switcher.open_switcher()
+    switcher.set_search_text('0')
+    qtbot.wait(500)
+    assert switcher.count() == 1
+    switcher.on_close()
 
+    # Remove project file and check the switcher is updated
+    n_files_project -= 1
+    os.remove(osp.join(str(project_dir), 'test_file1.py'))
     switcher.open_switcher()
-    n_files_project = len(projects.get_project_filenames())
-    n_files_open = editorstack.get_stack_count()
-    assert switcher_widget.model.rowCount() == n_files_open + n_files_project
+    qtbot.wait(500)
+    assert switcher.count() == n_files_open + n_files_project
     switcher.on_close()
 
     # Select file in the project explorer
     idx = projects.get_widget().treewidget.get_index(
-        osp.join(project_dir, 'test_file0.py'))
+        osp.join(str(project_dir), 'test_file0.py')
+    )
     projects.get_widget().treewidget.setCurrentIndex(idx)
 
     # Press Enter there
     qtbot.keyClick(projects.get_widget().treewidget, Qt.Key_Enter)
 
     switcher.open_switcher()
-    n_files_project = len(projects.get_project_filenames())
     n_files_open = editorstack.get_stack_count()
-    assert switcher_widget.model.rowCount() == n_files_open + n_files_project
+    assert switcher.count() == n_files_open + n_files_project - 1
     switcher.on_close()
 
+    # Resume capturing
+    capmanager.resume_global_capture()
+
 
 @flaky(max_runs=3)
 @pytest.mark.skipif(sys.platform == 'darwin',

spyder/app/utils.py

@@ -275,8 +275,20 @@ def create_application():
     # The try/except is necessary to run the main window tests on their own.
     try:
         app.set_font()
-    except AttributeError:
-        pass
+    except AttributeError as error:
+        if running_under_pytest():
+            # Set font options to avoid a ton of Qt warnings when running tests
+            app_family = app.font().family()
+            app_size = app.font().pointSize()
+            CONF.set('appearance', 'app_font/family', app_family)
+            CONF.set('appearance', 'app_font/size', app_size)
+
+            from spyder.config.fonts import MEDIUM, MONOSPACE
+            CONF.set('appearance', 'monospace_app_font/family', MONOSPACE[0])
+            CONF.set('appearance', 'monospace_app_font/size', MEDIUM)
+        else:
+            # Raise in case the error is valid
+            raise error
 
     # Required for correct icon on GNOME/Wayland:
     if hasattr(app, 'setDesktopFileName'):

spyder/plugins/editor/utils/switcher_manager.py

@@ -78,16 +78,9 @@ def create_editor_switcher(self):
             _('Start typing the name of an open file'))
 
         editorstack = self._editorstack()
-
-        # Since editor open files are inserted at position 0, the
-        # list needs to be reversed so they're shown in order.
         editor_list = editorstack.data.copy()
-        editor_list.reverse()
-
-        paths = [data.filename.lower()
-                 for data in editor_list]
-        save_statuses = [data.newly_created
-                         for data in editor_list]
+        paths = [data.filename for data in editor_list]
+        save_statuses = [data.newly_created for data in editor_list]
         short_paths = shorten_paths(paths, save_statuses)
 
         for idx, data in enumerate(editor_list):
@@ -99,15 +92,18 @@ def create_editor_switcher(self):
             if len(paths[idx]) > 75:
                 path = short_paths[idx]
             else:
-                path = osp.dirname(data.filename.lower())
+                path = osp.dirname(data.filename)
             last_item = (idx + 1 == len(editor_list))
-            self._switcher.add_item(title=title,
-                                    description=path,
-                                    icon=icon,
-                                    section=self._section,
-                                    data=data,
-                                    last_item=last_item)
-        self._switcher.set_current_row(0)
+
+            self._switcher.add_item(
+                title=title,
+                description=path,
+                icon=icon,
+                section=self._section,
+                data=data,
+                last_item=last_item,
+                score=0  # To make these items appear above those from Projects
+            )
 
     def create_line_switcher(self):
         """Populate switcher with line info."""

spyder/plugins/projects/plugin.py

@@ -216,7 +216,7 @@ def on_switcher_available(self):
         self._switcher.sig_item_selected.connect(
             self._handle_switcher_selection)
         self._switcher.sig_search_text_available.connect(
-            self._handle_switcher_results)
+            self._handle_switcher_search)
 
     @on_plugin_teardown(plugin=Plugins.Editor)
     def on_editor_teardown(self):
@@ -281,7 +281,7 @@ def on_switcher_teardown(self):
         self._switcher.sig_item_selected.disconnect(
             self._handle_switcher_selection)
         self._switcher.sig_search_text_available.disconnect(
-            self._handle_switcher_results)
+            self._handle_switcher_search)
         self._switcher = None
 
     def on_close(self, cancelable=False):
@@ -508,17 +508,11 @@ def _handle_switcher_modes(self, mode):
         mode: str
             The selected mode (open files "", symbol "@" or line ":").
         """
-        items = self.get_widget().handle_switcher_modes()
-        for (title, description, icon, section, path, is_last_item) in items:
-            self._switcher.add_item(
-                title=title,
-                description=description,
-                icon=icon,
-                section=section,
-                data=path,
-                last_item=is_last_item
-            )
-        self._switcher.set_current_row(0)
+        # Don't compute anything if we're not in files mode
+        if mode != "":
+            return
+
+        self.get_widget().display_default_switcher_items()
 
     def _handle_switcher_selection(self, item, mode, search_text):
         """
@@ -540,20 +534,19 @@ def _handle_switcher_selection(self, item, mode, search_text):
         self.get_widget().handle_switcher_selection(item, mode, search_text)
         self._switcher.hide()
 
-    def _handle_switcher_results(self, search_text, items_data):
+    def _handle_switcher_search(self, search_text):
         """
         Handle user typing in switcher to filter results.
 
-        Load switcher results when a search text is typed for projects.
         Parameters
         ----------
         text: str
             The current search text in the switcher dialog box.
-        items_data: list
-            List of items shown in the switcher.
         """
-        items = self.get_widget().handle_switcher_results(search_text,
-                                                          items_data)
+        self.get_widget().handle_switcher_search(search_text)
+
+    def _display_items_in_switcher(self, items, setup=True):
+        """Display a list of items in the switcher."""
         for (title, description, icon, section, path, is_last_item) in items:
             self._switcher.add_item(
                 title=title,
@@ -562,5 +555,8 @@ def _handle_switcher_results(self, search_text, items_data):
                 section=section,
                 data=path,
                 last_item=is_last_item,
-                score=100
+                score=1e10  # To make the editor results appear first
             )
+
+        if setup:
+            self._switcher.setup()

spyder/plugins/projects/tests/test_plugin.py

@@ -351,7 +351,7 @@ def test_project_explorer_tree_root(projects, tmpdir, qtbot):
     # Open the projects.
     for ppath in [ppath1, ppath2]:
         projects.open_project(path=ppath)
-        projects.get_widget()._update_explorer(None)
+        projects.get_widget()._setup_project(ppath)
 
         # Check that the root path of the project explorer tree widget is
         # set correctly.