Code coverage covers too much by default and runs in the wrong directory

[maruel]

First, the code coverage integration is really nice. Thanks Ray for doing this, it is really appreciated!

Problem

The cwd argument is not set for jobstart()?

At the moment, code coverage :GoCoverage runs with "./..." by default, unless "-p" is provided. Ref:

go.nvim/lua/go/coverage.lua

Line 387 in 6368756

argsstr = '.' .. utils.sep() .. '...'

I suspect this is done as a workaround for the lack of cwd argument.

Generally speaking, running unit tests in subpackages is quite inefficient when one only wants the data for the current file(s) so the default should be flipped and default instead to "." with the cwd flag correctly set. These lines are unnecessary IMHO:

go.nvim/lua/go/coverage.lua

Lines 377 to 385 in 6368756

	log('extend pkg')
	table.remove(args, 1)
	local pkg = require('go.package').pkg_from_path(nil, vim.api.nvim_get_current_buf())
	if vfn.empty(pkg) == 1 then
	vim.notify('No package found in current directory.')
	argsstr = ''
	else
	argsstr = pkg[1]
	end

Repro

• With a go package with foo/bar.go, foo/bar_test.go, baz/baz.go
• cd baz
• nvim ../foo/bar.go
• :GoCoverage

It currently fails to show any coverage.