test: remove inspect test; set NODE_PATH in nodejs_spec.lua

provider#node#can_inspect will fail on some systems because it is common
to have old node versions in OS (any Linux OS that has LTS releases)
and CI (Travis, Appveyor).

NODE_PATH can be trivially set with VimL.
Build scripts don't have to set it for the nodejs tests to work.
NODE_PATH is optional to begin with and is used only as a workaround
for the neovim node.js host.
This commit is contained in:
Jan Edmund Lazo 2017-12-16 16:29:33 -05:00 committed by Justin M. Keyes
parent a1adfdc7d5
commit 5b692124cc
3 changed files with 1 additions and 12 deletions

View File

@ -39,9 +39,6 @@ where.exe neovim-ruby-host.bat || goto :error
cmd /c npm.cmd install -g neovim || goto :error
where.exe neovim-node-host.cmd || goto :error
for /f %%F in ('cmd /c npm root -g') do (
set NODE_PATH=%%F
)
mkdir .deps
cd .deps

View File

@ -23,7 +23,6 @@ if test "$CLANG_SANITIZER" != "TSAN" ; then
# Additional threads are only created when the builtin UI starts, which
# doesn't happen in the unit/functional tests
run_test run_unittests
export NODE_PATH="$(npm root -g)"
run_test run_functionaltests
fi
run_test run_oldtests

View File

@ -4,8 +4,6 @@ local missing_provider = helpers.missing_provider
local command = helpers.command
local write_file = helpers.write_file
local eval = helpers.eval
local sleep = helpers.sleep
local funcs = helpers.funcs
local retry = helpers.retry
do
@ -20,12 +18,7 @@ end
before_each(function()
clear()
end)
describe('nodejs', function()
it('can inspect', function()
eq(1, funcs['provider#node#can_inspect']())
end)
command([[let $NODE_PATH = get(split(system('npm root -g'), "\n"), 0, '')]])
end)
describe('nodejs host', function()