From 5a8ba7526532f8bd6dde93764cccc5277d568349 Mon Sep 17 00:00:00 2001 From: Horacio Sanson Date: Tue, 21 May 2019 13:33:47 +0900 Subject: Fix eclipse.jdt.ls workspace data path. We were setting the -data parameter to the project root but this caused the language server to fail initialization and synch of gradle dependencies. As consequence ALE failed to work fully on gradle projects. This fix sets the workspace to the parent folder of the project root. Normally this corresponds to the correct Eclipse workspace path. When this is not the case, this fix also allows users to explicitly set the absolute path to the workspace via configuration variable. --- .../test_eclipselsp_command_callback.vader | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) (limited to 'test/command_callback/test_eclipselsp_command_callback.vader') diff --git a/test/command_callback/test_eclipselsp_command_callback.vader b/test/command_callback/test_eclipselsp_command_callback.vader index 2d2b4613..f25ed5fc 100644 --- a/test/command_callback/test_eclipselsp_command_callback.vader +++ b/test/command_callback/test_eclipselsp_command_callback.vader @@ -61,11 +61,11 @@ Execute(The eclipselsp callback should return the correct default value): \ '-noverify', \ '-Xmx1G', \ '-jar', - \ '', + \ ale#Escape(''), \ '-configuration', - \ b:cfg, + \ ale#Escape(b:cfg), \ '-data', - \ '' + \ ale#Escape(ale#path#Simplify('')) \] AssertLinter 'java', join(cmd, ' ') @@ -79,11 +79,11 @@ Execute(The eclipselsp callback should allow custom executable): \ '-noverify', \ '-Xmx1G', \ '-jar', - \ '', + \ ale#Escape(''), \ '-configuration', - \ b:cfg, + \ ale#Escape(b:cfg), \ '-data', - \ '' + \ ale#Escape(ale#path#Simplify('')) \] AssertLinter '/bin/foobar', join(cmd, ' ') @@ -97,10 +97,10 @@ Execute(The eclipselsp callback should allow custom configuration path): \ '-noverify', \ '-Xmx1G', \ '-jar', - \ '', + \ ale#Escape(''), \ '-configuration', - \ ale#path#Simplify('/home/config'), + \ ale#Escape(ale#path#Simplify('/home/config')), \ '-data', - \ '' + \ ale#Escape(ale#path#Simplify('')) \] AssertLinter 'java', join(cmd, ' ') -- cgit v1.2.3