# HG changeset patch # User Christian Brabandt # Date 1504098006 -7200 # Node ID 67bc88591ededc9300b402c4eda82e2b20dcd834 # Parent 399765ed6ff91b0c5faeb0f3514701815e26d047 patch 8.0.1017: test for MS-Windows $HOME always passes commit https://github.com/vim/vim/commit/dde60341115effc9789742c2f4565d184e98b6b5 Author: Bram Moolenaar Date: Wed Aug 30 14:55:42 2017 +0200 patch 8.0.1017: test for MS-Windows $HOME always passes Problem: Test for MS-Windows $HOME always passes. Solution: Rename the test function. Make the test pass. diff --git a/src/testdir/test_windows_home.vim b/src/testdir/test_windows_home.vim --- a/src/testdir/test_windows_home.vim +++ b/src/testdir/test_windows_home.vim @@ -38,76 +38,73 @@ func CheckHomeIsInSubprocessEnvironment( endfunc func CheckHome(exp, ...) - "call assert_equal(a:exp, $HOME) - "call assert_equal(a:exp, expand('~', ':p')) + call assert_equal(a:exp, $HOME) + call assert_equal(a:exp, expand('~', ':p')) if !a:0 call CheckHomeIsMissingFromSubprocessEnvironment() else - call CheckHomeIsInSubprocessEnvironment(a:exp) + call CheckHomeIsInSubprocessEnvironment(a:1) endif endfunc -func TestWindowsHome() +func Test_WindowsHome() command! -nargs=* SaveEnv call save_env() command! -nargs=* RestoreEnv call restore_env() command! -nargs=* UnletEnv call unlet_env() + set noshellslash - SaveEnv $HOME $USERPROFILE $HOMEDRIVE $HOMEPATH + let save_home = $HOME + SaveEnv $USERPROFILE $HOMEDRIVE $HOMEPATH try - RestoreEnv - UnletEnv $HOME $USERPROFILE $HOMEPATH - let $HOMEDRIVE = 'C:' - call CheckHome('C:\') - - RestoreEnv - UnletEnv $HOME $USERPROFILE + " Normal behavior: use $HOMEDRIVE and $HOMEPATH, ignore $USERPROFILE + let $USERPROFILE = 'unused' let $HOMEDRIVE = 'C:' let $HOMEPATH = '\foobar' + let $HOME = '' " Force recomputing "homedir" call CheckHome('C:\foobar') - RestoreEnv - UnletEnv $HOME $HOMEDRIVE $HOMEPATH - let $USERPROFILE = 'C:\foo' - call CheckHome('C:\foo') + " Same, but with $HOMEPATH not set + UnletEnv $HOMEPATH + let $HOME = '' " Force recomputing "homedir" + call CheckHome('C:\') - RestoreEnv - UnletEnv $HOME + " Use $USERPROFILE if $HOMEPATH and $HOMEDRIVE are empty + UnletEnv $HOMEDRIVE $HOMEPATH let $USERPROFILE = 'C:\foo' - let $HOMEDRIVE = 'C:' - let $HOMEPATH = '\baz' + let $HOME = '' " Force recomputing "homedir" call CheckHome('C:\foo') - RestoreEnv + " If $HOME is set the others don't matter let $HOME = 'C:\bar' - let $USERPROFILE = 'C:\foo' - let $HOMEDRIVE = 'C:' - let $HOMEPATH = '\baz' - call CheckHome('C:\bar', 1) + let $USERPROFILE = 'unused' + let $HOMEDRIVE = 'unused' + let $HOMEPATH = 'unused' + call CheckHome('C:\bar', 'C:\bar') - RestoreEnv - let $HOME = '%USERPROFILE%\bar' + " If $HOME contains %USERPROFILE% it is expanded let $USERPROFILE = 'C:\foo' - let $HOMEDRIVE = 'C:' - let $HOMEPATH = '\baz' - call CheckHome('%USERPROFILE%\bar', 1) + let $HOME = '%USERPROFILE%\bar' + let $HOMEDRIVE = 'unused' + let $HOMEPATH = 'unused' + call CheckHome('C:\foo\bar', '%USERPROFILE%\bar') - RestoreEnv - let $HOME = '%USERPROFILE' + " Invalid $HOME is kept let $USERPROFILE = 'C:\foo' - let $HOMEDRIVE = 'C:' - let $HOMEPATH = '\baz' - call CheckHome('%USERPROFILE', 1) + let $HOME = '%USERPROFILE' + let $HOMEDRIVE = 'unused' + let $HOMEPATH = 'unused' + call CheckHome('%USERPROFILE', '%USERPROFILE') - RestoreEnv + " %USERPROFILE% not at start of $HOME is not expanded + let $USERPROFILE = 'unused' let $HOME = 'C:\%USERPROFILE%' - let $USERPROFILE = 'C:\foo' - let $HOMEDRIVE = 'C:' - let $HOMEPATH = '\baz' - call CheckHome('C:\%USERPROFILE%', 1) + let $HOMEDRIVE = 'unused' + let $HOMEPATH = 'unused' + call CheckHome('C:\%USERPROFILE%', 'C:\%USERPROFILE%') if has('channel') RestoreEnv - UnletEnv $HOME + let $HOME = save_home let env = '' let job = job_start('cmd /c set', {'out_cb': {ch,x->[env,execute('let env=x')]}}) sleep 1 diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -770,6 +770,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 1017, +/**/ 1016, /**/ 1015,