test: more tests, remove windows from test target

Author: joshzcold

.github/workflows/tests.yaml

@@ -33,8 +33,9 @@ jobs:
         include:
           - os: macos-latest
             neovim_version: v0.11.3
-          - os: windows-latest
-            neovim_version: v0.11.3
+          # TODO include windows when I have access to test on a windows machine.
+          # - os: windows-latest
+          #   neovim_version: v0.11.3
     runs-on: ${{ matrix.os }}
 
     steps:

README.md

@@ -289,12 +289,12 @@ return {
 
 - [x] Linux
 
-- [x] MacOS (Mostly should work)
+- [x] MacOS
 
-  - [x] I am detecting python interpreters in homebrew and hatch. Does the community use other methods of installing python?
+  - [x] I am detecting python interpreters in homebrew and hatch and uv. Testing in ci.
 
 - [ ] Windows (Un tested)
-  - [ ] Most likely spots where we are mixing up `/` vs `\` in paths
+  - [ ] Need to test this plugin on a windows machine to verify. I have seen online that neovim users are deciding on WezTerm + WSL to handle support for neovim plugins.
 
 ## Special Thanks
 

tests/_test_template.lua

@@ -14,7 +14,7 @@ local T = new_set({
       -- Restart child process with custom 'init.lua' script
       child.restart({ '-u', 'scripts/minimal_init.lua' })
       -- Load tested plugin
-      child.lua([[M = require('python')]])
+      child.lua([[require('python').setup()]])
     end,
     -- This will be executed one after all tests from this set are finished
     post_once = child.stop,

tests/test_interpreters.lua

@@ -0,0 +1,34 @@
+
+-- Define helper aliases
+local new_set = MiniTest.new_set
+local expect, eq = MiniTest.expect, MiniTest.expect.equality
+
+-- Create (but not start) child Neovim object
+local child = MiniTest.new_child_neovim()
+
+-- Define main test set of this file
+local T = new_set({
+  -- Register hooks
+  hooks = {
+    -- This will be executed before every (even nested) case
+    pre_case = function()
+      -- Restart child process with custom 'init.lua' script
+      child.restart({ '-u', 'scripts/minimal_init.lua' })
+      -- Load tested plugin
+      child.lua([[require('python').setup()]])
+    end,
+    -- This will be executed one after all tests from this set are finished
+    post_once = child.stop,
+  },
+})
+
+T['interpreters'] = MiniTest.new_set()
+
+T['interpreters']['python'] = function()
+  child.lua("inter = require('python.venv.interpreters')")
+  local pythons = child.lua("return inter.python_interpreters()")
+  assert(#pythons > 0)
+end
+
+-- Return test set which will be collected and execute inside `MiniTest.run()`
+return T

tests/test_keymap.lua

@@ -0,0 +1,33 @@
+-- Define helper aliases
+local new_set = MiniTest.new_set
+local expect, eq = MiniTest.expect, MiniTest.expect.equality
+
+-- Create (but not start) child Neovim object
+local child = MiniTest.new_child_neovim()
+
+-- Define main test set of this file
+local T = new_set({
+  -- Register hooks
+  hooks = {
+    -- This will be executed before every (even nested) case
+    pre_case = function()
+      -- Restart child process with custom 'init.lua' script
+      child.restart({ '-u', 'scripts/minimal_init.lua' })
+      -- Load tested plugin
+      child.lua([[require('python').setup()]])
+    end,
+    -- This will be executed one after all tests from this set are finished
+    post_once = child.stop,
+  },
+})
+
+-- Keymaps can be loaded
+T['keymaps'] = function()
+  child.cmd("cd examples/python_projects/uv")
+  child.cmd("e main.py")
+  local keymap = child.cmd_capture("map <leader>pi")
+  assert(string.find(keymap, "python venv install", 1, true))
+end
+
+-- Return test set which will be collected and execute inside `MiniTest.run()`
+return T

tests/test_luasnip.lua

@@ -0,0 +1,31 @@
+-- Define helper aliases
+local new_set = MiniTest.new_set
+local expect, eq = MiniTest.expect, MiniTest.expect.equality
+
+-- Create (but not start) child Neovim object
+local child = MiniTest.new_child_neovim()
+
+-- Define main test set of this file
+local T = new_set({
+  -- Register hooks
+  hooks = {
+    -- This will be executed before every (even nested) case
+    pre_case = function()
+      -- Restart child process with custom 'init.lua' script
+      child.restart({ '-u', 'scripts/minimal_init.lua' })
+      -- Load tested plugin
+      child.lua([[require('python').setup({ python_lua_snippets = true })]])
+    end,
+    -- This will be executed one after all tests from this set are finished
+    post_once = child.stop,
+  },
+})
+
+T['snips will load'] = function()
+  child.lua([[M = require('python.snip')]])
+  child.lua([[M.load_snippets()]])
+  eq(child.lua([[return vim.g._python_nvim_snippets_loaded]]), true)
+end
+
+-- Return test set which will be collected and execute inside `MiniTest.run()`
+return T

tests/test_uv.lua

@@ -0,0 +1,44 @@
+-- Define helper aliases
+local new_set = MiniTest.new_set
+local expect, eq = MiniTest.expect, MiniTest.expect.equality
+
+-- Create (but not start) child Neovim object
+local child = MiniTest.new_child_neovim()
+
+-- Define main test set of this file
+local T = new_set({
+  -- Register hooks
+  hooks = {
+    -- This will be executed before every (even nested) case
+    pre_case = function()
+      -- Restart child process with custom 'init.lua' script
+      child.restart({ '-u', 'scripts/minimal_init.lua' })
+      -- Load tested plugin
+      child.lua([[require('python').setup()]])
+    end,
+    -- This will be executed one after all tests from this set are finished
+    post_once = child.stop,
+  },
+})
+T['uv'] = MiniTest.new_set({
+  hooks = {
+    pre_case = function()
+    end
+  }
+})
+
+T['uv']['command'] = function()
+  child.cmd("cd examples/python_projects/uv")
+  child.cmd("e main.py")
+  child.cmd([[!rm -rf .venv]])
+  child.cmd([[UV sync]])
+  local dir = child.lua("return vim.fn.expand('%:p:h')")
+  local venv_dir = vim.fs.joinpath(dir, ".venv")
+
+  -- give some seconds for uv sync to download package
+  vim.loop.sleep(3000)
+  eq(child.lua(("return vim.fn.isdirectory('%s')"):format(venv_dir)), 1)
+end
+
+-- Return test set which will be collected and execute inside `MiniTest.run()`
+return T

tests/test_venv_create.lua

@@ -37,7 +37,6 @@ end
 
 -- NOTE: requires uv installed on system
 T['create']['uv_sync'] = function()
-  local IS_WINDOWS = vim.uv.os_uname().sysname == 'Windows_NT'
   child.lua("create = require('python.venv.create')")
   child.cmd("cd examples/python_projects/uv")
   child.cmd("!rm -rf .venv")
@@ -48,19 +47,9 @@ T['create']['uv_sync'] = function()
   local dep_path = vim.fn.system(
     [[examples/python_projects/uv/.venv/bin/python -c 'import sys; print(sys.path[-1], end="")']])
 
-  -- Have to do a special test for windows to check for requests being installed in .venv
-  if IS_WINDOWS then
-    print(vim.fn.system([[ dir examples\python_projects\uv\.venv\Lib\site-packages ]]))
-    print(vim.fn.system([[ cd examples\python_projects\uv && uv sync ]]))
-    eq(vim.fn.isdirectory([[examples\python_projects\uv\.venv\Lib\site-packages\requests]]), 1)
-    return
-  end
-
-  print("\ndep_path: " .. dep_path .. "\n")
   assert(string.find(dep_path, "python_projects", 1, true))
 
   local check_dir = vim.fs.joinpath(dep_path, "requests")
-  print("\nchecking: " .. check_dir .. "\n")
   eq(vim.fn.isdirectory(check_dir), 1)
 end