tests: cleanup

Author: AntoineGS

lua/copilot/logger/init.lua

@@ -51,7 +51,14 @@ end
 ---@param ... any
 local function notify_log(log_level, msg, ...)
   local log_msg = format_log(log_level, msg, ...)
-  vim.notify(log_msg, log_level)
+
+  if vim.in_fast_event then
+    vim.schedule(function()
+      vim.notify(log_msg, log_level)
+    end)
+  else
+    vim.notify(log_msg, log_level)
+  end
 end
 
 ---@param log_level integer -- one of the vim.log.levels

tests/scripts/minimal_init.lua

@@ -5,7 +5,7 @@ vim.cmd([[let &rtp.=','.getcwd()]])
 if #vim.api.nvim_list_uis() == 0 then
   -- Add 'mini.nvim' to 'runtimepath' to be able to use 'mini.test'
   -- Assumed that 'mini.nvim' is stored in 'deps/mini.nvim'
-  vim.cmd("set rtp+=tests/deps/mini.nvim")
+  vim.cmd("set rtp+=deps/mini.nvim")
 
   -- Set up 'mini.test'
   require("mini.test").setup()

tests/test_base_to_organize.lua

@@ -1,23 +1,16 @@
 local eq = MiniTest.expect.equality
--- local neq = MiniTest.expect.no_equality
--- local u = require("tests.utils")
-local expect_error = MiniTest.expect.error
 local child = MiniTest.new_child_neovim()
+-- local env = require("tests.env")
 
 local T = MiniTest.new_set({
   hooks = {
     pre_case = function()
-      -- Restart child process with custom 'init.lua' script
       child.restart({ "-u", "tests/scripts/minimal_init.lua" })
       child.lua([[M = require('copilot')]])
-
       child.lua([[c = require('copilot.command')]])
-      -- child.lua([[c.enable()]])
-      -- child.lua([[c.attach({ force = true })]])
-      -- child.lua([[conf = require('copilot.config')]])
       child.lua([[s = require('copilot.status')]])
+      -- child.fn.setenv("GITHUB_COPILOT_TOKEN", env.COPILOT_TOKEN)
     end,
-    -- Stop once all test cases are finished
     post_once = child.stop,
   },
 })
@@ -42,11 +35,4 @@ T["lua()"]["setup called, copilot.setup_done is true"] = function()
   eq(child.lua("return M.setup_done"), true)
 end
 
-T["lua()"]["Copilot status, not initialized, returns error"] = function()
-  run_setup()
-  expect_error(function()
-    child.cmd(":Copilot status")
-  end, ".*not initialized.*")
-end
-
 return T

tests/test_client.lua

@@ -1,7 +1,6 @@
--- local eq = MiniTest.expect.equality
--- local neq = MiniTest.expect.no_equality
 local child = MiniTest.new_child_neovim()
 local u = require("tests.utils")
+-- local env = require("tests.env")
 
 local T = MiniTest.new_set({
   hooks = {
@@ -11,18 +10,16 @@ local T = MiniTest.new_set({
       end
     end,
     pre_case = function()
-      -- Restart child process with custom 'init.lua' script
       child.restart({ "-u", "tests/scripts/minimal_init.lua" })
       child.lua([[M = require('copilot')]])
       child.lua([[c = require('copilot.client')]])
-      -- child.lua([[conf = require('copilot.config')]])
       child.lua([[s = require('copilot.status')]])
       child.lua([[cmd = require('copilot.command')]])
       child.lua([[a = require('copilot.api')]])
       child.lua("logger = require('copilot.logger')")
       -- child.lua([[require("osv").launch({ port = 8086 })]])
+      -- child.fn.setenv("GITHUB_COPILOT_TOKEN", env.COPILOT_TOKEN)
     end,
-    -- Stop once all test cases are finished
     post_once = child.stop,
   },
 })

tests/test_panel.lua

@@ -1,8 +1,6 @@
 local eq = MiniTest.expect.equality
--- local neq = MiniTest.expect.no_equality
--- local reference_screenshot = MiniTest.expect.reference_screenshot
 local child = MiniTest.new_child_neovim()
--- local u = require("tests.utils")
+-- local env = require("tests.env")
 
 local T = MiniTest.new_set({
   hooks = {
@@ -18,6 +16,7 @@ local T = MiniTest.new_set({
       child.lua("cmd = require('copilot.command')")
       child.lua("p = require('copilot.panel')")
       -- child.lua([[require("osv").launch({ port = 8086 })]])
+      -- child.fn.setenv("GITHUB_COPILOT_TOKEN", env.COPILOT_TOKEN)
     end,
     post_once = child.stop,
   },

tests/test_suggestion.lua

@@ -1,8 +1,6 @@
--- local eq = MiniTest.expect.equality
--- local neq = MiniTest.expect.no_equality
 local reference_screenshot = MiniTest.expect.reference_screenshot
 local child = MiniTest.new_child_neovim()
--- local u = require("tests.utils")
+-- local env = require("tests.env")
 
 local T = MiniTest.new_set({
   hooks = {
@@ -17,6 +15,7 @@ local T = MiniTest.new_set({
       child.lua("M = require('copilot')")
       child.lua("cmd = require('copilot.command')")
       -- child.lua([[require("osv").launch({ port = 8086 })]])
+      -- child.fn.setenv("GITHUB_COPILOT_TOKEN", env.COPILOT_TOKEN)
     end,
     post_once = child.stop,
   },