Fix progress reporting with autoimport plugin (#530)

last-partizan · web-flow · commit 4f6d7cb5b125 · 2024-03-12T20:57:48.000-05:00
diff --git a/pylsp/plugins/rope_autoimport.py b/pylsp/plugins/rope_autoimport.py
@@ -37,7 +37,7 @@ def reload_cache(
         config: Config,
         workspace: Workspace,
         files: Optional[List[Document]] = None,
-        single_thread: Optional[bool] = False,
+        single_thread: Optional[bool] = True,
     ):
         if self.is_blocked():
             return
diff --git a/test/plugins/test_autoimport.py b/test/plugins/test_autoimport.py
@@ -313,7 +313,7 @@ def test_autoimport_code_actions_and_completions_for_notebook_document(
     )
     assert rope_autoimport_settings.get("completions", {}).get("enabled", False) is True
     assert rope_autoimport_settings.get("memory", False) is True
-    wait_for_condition(lambda: not cache.thread.is_alive())
+    wait_for_condition(lambda: not cache.is_blocked())
 
     # 1.
     quick_fixes = server.code_actions("cell_1_uri", {}, make_context("os", 0, 0, 2))

Original file line number	Diff line number	Diff line change
`@@ -313,7 +313,7 @@ def test_autoimport_code_actions_and_completions_for_notebook_document(`
`313`	`313`	`)`
`314`	`314`	`assert rope_autoimport_settings.get("completions", {}).get("enabled", False) is True`
`315`	`315`	`assert rope_autoimport_settings.get("memory", False) is True`
`316`		`- wait_for_condition(lambda: not cache.thread.is_alive())`
	`316`	`+ wait_for_condition(lambda: not cache.is_blocked())`
`317`	`317`
`318`	`318`	`# 1.`
`319`	`319`	`quick_fixes = server.code_actions("cell_1_uri", {}, make_context("os", 0, 0, 2))`