prevent venv* folders to be parsed by default

[Tazoeur]

Hello,

The require('neotest').summary.toggle() launches the search for tests, and I see that my venv* folders are parsed by neotest. That's a behaviour I'd like to prevent.

I've tried to tweak it with the is_test_file function, but I lost the original behaviour which was doing a great job.

I feel like this folder exclusion could be a useful default, but maybe having a configuration option like exclude_pattern = { "venv*", "dist"} would be better.

I am sorry if there is something trivial that I am missing; I am still in the early stages of building a proper development workflow, and I am not yet confident in my Neovim/Lua skills.

[juliancoffee]

@Tazoeur
I was bug hunting for my setup, and in the meantime, I found a few things:

1. It seems that neotest-python does filter the venv folder, but only that venv.
2. You can tweak it with is_test_file, yes, although I wish there was a sensible way to extend the defaults.

Fyi, here's a default version.

function M.is_test_file(file_path)
  if not vim.endswith(file_path, ".py") then
    return false
  end
  local elems = vim.split(file_path, Path.path.sep)
  local file_name = elems[#elems]
  return vim.startswith(file_name, "test_") or vim.endswith(file_name, "_test.py")
end

It doesn't filter out venv, because it's filter_dir job (which neotest adapters expose to neotest itself), although neotest-python doesn't have a way to configure it, which is unfortunate :(

For a second, I thought that I could just specify it in adapters table, but no :(

Anyway, here's my is_test_file.

is_test_file = function(file_path)
    local Path = require('plenary.path')
    local neotest_default = require('neotest-python.base')

    -- first filter stupid stuff
    local elems = vim.split(file_path, Path.path.sep)
    if vim.tbl_contains(elems, '__pycache__') then
        return false
    end

    -- then try the default matcher
    if neotest_default.is_test_file(file_path) then
        return true
    end

    -- if all else fails, try custom logic
    local file_name = elems[#elems]
    return vim.startswith(file_name, 'tests')
end,

I'm not exactly sure if calling the neotest's internals is wise, so you can just duplicate their code instead, but I'm with you, we do need a way to extend is_test_file (or filter_dir) that doesn't involve just plain overrides.