ci: enable running on main instance

fix flakey tests

Author: ibhagwan

tests/actions_spec.lua

@@ -102,6 +102,9 @@ T["actions"]["vimcmd"] = new_set({
   },
 }, {
   function(action)
+    -- store mini.test float so we can return to it when
+    -- testing in main instance
+    local win = vim.api.nvim_get_current_win()
     local ctx = function()
       return {
         buf = child.api.nvim_get_current_buf(),
@@ -149,6 +152,12 @@ T["actions"]["vimcmd"] = new_set({
       __expect_lines = true,
       __after_open = function()
         child.wait_until(function() return child.lua_get([[_G._fzf_load_called]]) == true end)
+        -- Wait for previewer to load the LICENSE file entry and title to render
+        child.wait_until(function()
+          local entry = child.lua_get([[FzfLua.utils.fzf_winobj()._previewer.last_entry]])
+          return entry and entry:match("LICENSE") ~= nil
+        end)
+        vim.uv.sleep(100)  -- Allow async title update to render
         if helpers.IS_WIN() then vim.uv.sleep(250) end
       end,
       no_esc = true,
@@ -160,6 +169,9 @@ T["actions"]["vimcmd"] = new_set({
       eq({ "LICENSE", 3 }, { _ctx.name, child.fn.line(".") })
     end
     eq("live_grep", exec_lua([[return _G._fzf_info.cmd]]))
+    -- close tab and return to mini.test float
+    vim.api.nvim_buf_delete(0, { force = true })
+    vim.api.nvim_set_current_win(win)
   end
 })
 

tests/api_spec.lua

@@ -108,6 +108,9 @@ T["api"]["fzf_live"]["table"] = function(multiprocess)
     {
       __expect_lines = true,
       __postprocess_wait = true,
+      __after_open = function()
+        child.wait_until(function() return child.lua_get([[_G._fzf_load_called]]) == true end)
+      end,
       query = 100,
       keymap = { fzf = { load = "pos(-1)" } },
       multiprocess = multiprocess == true,

tests/path_spec.lua

@@ -429,6 +429,7 @@ describe("Testing path module", function()
         stripped = "/tmp/test.txt:42",
         terminal = false
       })
+      vim.api.nvim_buf_delete(buf, { force = true })
     end)
 
     it("man://", function()
@@ -448,6 +449,7 @@ describe("Testing path module", function()
         stripped = "man://ls(1):42:7:",
         terminal = false
       })
+      vim.api.nvim_buf_delete(buf, { force = true })
     end)
 
     it("force_uri", function()
@@ -467,8 +469,9 @@ describe("Testing path module", function()
 
     it("parse loaded path", function()
       helpers.SKIP_IF_WIN()
+      local win = vim.api.nvim_get_current_win()
       local r = {}
-      vim.cmd [[bwipe]]
+      vim.cmd.new()
       local name = "/tmp/foo:bar.txt"
       vim.cmd.edit(name)
       vim.cmd([[sil write]])
@@ -480,6 +483,8 @@ describe("Testing path module", function()
         bufnr = vim.fn.bufnr(),
         bufname = name,
       })
+      vim.api.nvim_buf_delete(0, { force = true })
+      vim.api.nvim_set_current_win(win)
     end)
   end)
 end)

tests/utils_spec.lua

@@ -93,16 +93,22 @@ describe("Testing utils module", function()
   end)
 
   it("wo", function()
+    -- store mini.test float so we can return to it when
+    -- testing in main instance
+    local win = vim.api.nvim_get_current_win()
+    vim.cmd.new()
     utils.wo.nonexist = "this is nop"
     eq(nil, utils.wo.nonexist)
 
     vim.wo[0][0].nu = true -- setlocal
     vim.wo[0].rnu = true   -- setglobal
     eq(vim.wo[0][0].nu, vim.wo.nu)
     eq(vim.wo[0][0].rnu, vim.wo.rnu)
+    vim.api.nvim_buf_delete(0, { force = true })
     vim.cmd.new()
     eq(vim.wo[0][0].nu, vim.wo.nu)
     eq(vim.wo[0][0].rnu, vim.wo.rnu)
+    vim.api.nvim_buf_delete(0, { force = true })
 
     -- same behavior
     utils.wo[0][0].nu = true -- setlocal
@@ -112,6 +118,8 @@ describe("Testing utils module", function()
     vim.cmd.new()
     eq(utils.wo[0][0].nu, utils.wo.nu)
     eq(utils.wo[0][0].rnu, utils.wo.rnu)
+    vim.api.nvim_buf_delete(0, { force = true })
+    vim.api.nvim_set_current_win(win)
   end)
 
   it("strsplit", function()

tests/win_spec.lua

@@ -111,7 +111,10 @@ T["win"]["hide"]["can resume after close CTX win (#1936)"] = function()
   -- can :wqa when there're hide job #1817
   pcall(child.cmd, [[wqa]])
   -- child.is_running() didn't work as expected
-  eq(vim.fn.jobwait({ assert(child.job).id }, 1000)[1], 0)
+  local res = vim.fn.jobwait({ assert(child.job).id }, 1000)[1]
+  -- 0 = normal exit, -3 = already exited/invalid (acceptable race)
+  local valid_res = { [0] = true, [-3] = true }
+  eq(valid_res[res] or res, true) -- HACK: show the res code in the assert
 end
 
 T["win"]["hide"]["actions on multi-select but zero-match #1961"] = function()
@@ -323,7 +326,7 @@ T["win"]["relative=cursor"] = function()
     previewer = function() return require("fzf-lua.test.previewer").builtin end,
     __after_open = function()
       child.wait_until(function() return child.lua_get([[_G._fzf_load_called]]) == true end)
-      if helpers.IS_WIN() then vim.uv.sleep(250) end
+      if not helpers.IS_LINUX() then vim.uv.sleep(250) end
     end,
   }
   helpers.FzfLua.fzf_exec(child, { "foo", "bar", "baz" }, opts)