fix: avoid running discovery twice

Previously found adapters shouldn't have their discovery run twice on startup. See #13
nvim-neotest · Oct 6, 2022 · e2d1378 · e2d1378
1 parent e7a9cf3
commit e2d1378
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/lua/neotest/client/init.lua b/lua/neotest/client/init.lua
@@ -454,11 +454,11 @@ function NeotestClient:_update_adapters(dir)
     if not found[adapter_id] then
       self._adapters[adapter_id] = adapter
       found[adapter_id] = true
-    end
-    if config.projects[root].discovery.enabled then
-      self:_update_positions(root, { adapter = adapter_id })
-    else
-      self:_update_open_buf_positions(adapter_id)
+      if config.projects[root].discovery.enabled then
+        self:_update_positions(root, { adapter = adapter_id })
+      else
+        self:_update_open_buf_positions(adapter_id)
+      end
     end
   end
   local root = lib.files.is_dir(dir) and dir or async.fn.getcwd()
@@ -467,12 +467,11 @@ function NeotestClient:_update_adapters(dir)
     if not found[adapter_id] then
       self._adapters[adapter_id] = adapter
       found[adapter_id] = true
-    end
-
-    if config.projects[root].discovery.enabled then
-      self:_update_positions(root, { adapter = adapter_id })
-    else
-      self:_update_open_buf_positions(adapter_id)
+      if config.projects[root].discovery.enabled then
+        self:_update_positions(root, { adapter = adapter_id })
+      else
+        self:_update_open_buf_positions(adapter_id)
+      end
     end
   end
 end