mirror of
https://github.com/neovim/neovim.git
synced 2025-09-06 03:18:16 +00:00
test: exepath() returns correct path with cmd.exe, powershell #21928
test(exepath): test if exepath returns correct path with multiple Windows shells This test covers the changes from #21175 where exepath() is set to prefer file extensions in powershell.exe aswell as in cmd.exe. In both shells, the file with a valid extension should be returned instead of the extensionless file.
This commit is contained in:
@@ -5,21 +5,21 @@ local command = helpers.command
|
|||||||
local exc_exec = helpers.exc_exec
|
local exc_exec = helpers.exc_exec
|
||||||
local matches = helpers.matches
|
local matches = helpers.matches
|
||||||
local is_os = helpers.is_os
|
local is_os = helpers.is_os
|
||||||
|
local set_shell_powershell = helpers.set_shell_powershell
|
||||||
|
local eval = helpers.eval
|
||||||
|
|
||||||
|
local find_dummies = function(ext_pat)
|
||||||
|
local tmp_path = eval('$PATH')
|
||||||
|
command('let $PATH = fnamemodify("./test/functional/fixtures/bin", ":p")')
|
||||||
|
matches('null' .. ext_pat, call('exepath', 'null'))
|
||||||
|
matches('true' .. ext_pat, call('exepath', 'true'))
|
||||||
|
matches('false' .. ext_pat, call('exepath', 'false'))
|
||||||
|
command("let $PATH = '"..tmp_path.."'")
|
||||||
|
end
|
||||||
|
|
||||||
describe('exepath()', function()
|
describe('exepath()', function()
|
||||||
before_each(clear)
|
before_each(clear)
|
||||||
|
|
||||||
it('returns 1 for commands in $PATH', function()
|
|
||||||
local exe = is_os('win') and 'ping' or 'ls'
|
|
||||||
local ext_pat = is_os('win') and '%.EXE$' or '$'
|
|
||||||
matches(exe .. ext_pat, call('exepath', exe))
|
|
||||||
command('let $PATH = fnamemodify("./test/functional/fixtures/bin", ":p")')
|
|
||||||
ext_pat = is_os('win') and '%.CMD$' or '$'
|
|
||||||
matches('null' .. ext_pat, call('exepath', 'null'))
|
|
||||||
matches('true' .. ext_pat, call('exepath', 'true'))
|
|
||||||
matches('false' .. ext_pat, call('exepath', 'false'))
|
|
||||||
end)
|
|
||||||
|
|
||||||
it('fails for invalid values', function()
|
it('fails for invalid values', function()
|
||||||
for _, input in ipairs({'v:null', 'v:true', 'v:false', '{}', '[]'}) do
|
for _, input in ipairs({'v:null', 'v:true', 'v:false', '{}', '[]'}) do
|
||||||
eq('Vim(call):E1174: String required for argument 1', exc_exec('call exepath('..input..')'))
|
eq('Vim(call):E1174: String required for argument 1', exc_exec('call exepath('..input..')'))
|
||||||
@@ -32,11 +32,32 @@ describe('exepath()', function()
|
|||||||
end)
|
end)
|
||||||
|
|
||||||
if is_os('win') then
|
if is_os('win') then
|
||||||
|
it('returns 1 for commands in $PATH (Windows)', function()
|
||||||
|
local exe = 'ping'
|
||||||
|
matches(exe .. '%.EXE$', call('exepath', exe))
|
||||||
|
end)
|
||||||
|
|
||||||
it('append extension if omitted', function()
|
it('append extension if omitted', function()
|
||||||
local filename = 'cmd'
|
local filename = 'cmd'
|
||||||
local pathext = '.exe'
|
local pathext = '.exe'
|
||||||
clear({env={PATHEXT=pathext}})
|
clear({env={PATHEXT=pathext}})
|
||||||
eq(call('exepath', filename..pathext), call('exepath', filename))
|
eq(call('exepath', filename..pathext), call('exepath', filename))
|
||||||
end)
|
end)
|
||||||
|
|
||||||
|
it('returns file WITH extension if files both with and without extension exist in $PATH', function()
|
||||||
|
local ext_pat = '%.CMD$'
|
||||||
|
find_dummies(ext_pat)
|
||||||
|
set_shell_powershell()
|
||||||
|
find_dummies(ext_pat)
|
||||||
|
end)
|
||||||
|
else
|
||||||
|
it('returns 1 for commands in $PATH (not Windows)', function()
|
||||||
|
local exe = 'ls'
|
||||||
|
matches(exe .. '$', call('exepath', exe))
|
||||||
|
end)
|
||||||
|
|
||||||
|
it('returns file WITHOUT extension if files both with and without extension exist in $PATH', function()
|
||||||
|
find_dummies('$')
|
||||||
|
end)
|
||||||
end
|
end
|
||||||
end)
|
end)
|
||||||
|
Reference in New Issue
Block a user