feat(neotest): pass --color argument to neotest adaper

[Ydot19]

Address issue #554

Open Questions / Remarks

• I am not sure when it is appropriate to check cargoArgs vs cargoExtraArgs. The latter seems deprecated but not sure
• Will move this out of draft after testing the change

[github-actions]

Review Checklist

Does this PR follow the Contribution Guidelines? Following is a partial checklist:

Proper conventional commit scoping:

• For example, fix(lsp): some lsp-related bugfix

• Pull request title has the appropriate conventional commit prefix.

If applicable:

• Tested
  • Tests have been added.
  • Tested manually (Steps to reproduce in PR description).
• Updated documentation.

[mrcjkb]

Suggested change

	local filtered = vim.tbl_filter(function(v)
	return string.match(v, '^--color=')
	end, runnable.args.cargoArgs)
	if not vim.tbl_isempty(filtered) then
	-- add first occurrence of filter argument contain --color
	table.insert(args, filtered[0])
	else
	-- cargo test needs to pass --color=never to the test runner too
	table.insert(args, '--color=never')
	end
	---@type string | nil
	local color_arg = vim.iter(runnable.args.cargoArgs):find(function(arg)
	return string.match(arg, '^--color=') ~= nil
	end)
	if color_arg then
	table.insert(args, color_arg)
	else
	-- cargo test needs to pass --color=never to the test runner too
	table.insert(args, '--color=never')
	end

[mrcjkb]

I am not sure when it is appropriate to check cargoArgs vs cargoExtraArgs. The latter seems deprecated but not sure

Yep, cargoExtraArgs is deprecated. No need to check it.