patch 9.1.0474: CI: Test_ColonEight() fails on github runners Commit: https://github.com/vim/vim/commit/d03882baac712de586880775386f4c1e2cf6eade Author: Christian Brabandt <c...@256bit.org> Date: Mon Jun 10 21:06:55 2024 +0200
patch 9.1.0474: CI: Test_ColonEight() fails on github runners Problem: CI: Test_ColonEight() fails on github runners (Ken Takata) Solution: Run the test for files on the C: drive, where dos shortnames are still enabled, refactor the tests to use a single setup function for the preparation fixes: #14954 closes: #14958 Signed-off-by: Christian Brabandt <c...@256bit.org> diff --git a/src/testdir/test_shortpathname.vim b/src/testdir/test_shortpathname.vim index 59361ca28..0c41692a0 100644 --- a/src/testdir/test_shortpathname.vim +++ b/src/testdir/test_shortpathname.vim @@ -16,18 +16,14 @@ func TestIt(file, bits, expected) endif endfunc -func Test_ColonEight() - let save_dir = getcwd() - - " This could change for CygWin to //cygdrive/c . - let dir1 = 'c:/x.x.y' +func s:SetupDir(dir) let trycount = 5 while 1 - if !filereadable(dir1) && !isdirectory(dir1) + if !filereadable(a:dir) && !isdirectory(a:dir) break endif if trycount == 1 - call assert_report("Fatal: '" . dir1 . "' exists, cannot run this test") + call assert_report("Fatal: '" . a:dir . "' exists, cannot run this test") return endif " When tests run in parallel the directory may exist, wait a bit until it @@ -35,6 +31,15 @@ func Test_ColonEight() sleep 5 let trycount -= 1 endwhile +endfunc + + +func Test_ColonEight() + let save_dir = getcwd() + + " This could change for CygWin to //cygdrive/c . + let dir1 = 'c:/x.x.y' + call s:SetupDir(dir1) let file1 = dir1 . '/zz.y.txt' let nofile1 = dir1 . '/z.y.txt' @@ -78,8 +83,8 @@ func Test_ColonEight() endfunc func Test_ColonEight_MultiByte() - let g:test_is_flaky = 1 - let dir = 'Xtest' + let dir = 'c:/Xtest_C8MB' + call s:SetupDir(dir) let file = dir . '/日本語のファイル.txt' diff --git a/src/version.c b/src/version.c index 78668eaad..fc4490d6c 100644 --- a/src/version.c +++ b/src/version.c @@ -704,6 +704,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 474, /**/ 473, /**/ -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/E1sGkTx-004lCK-Fo%40256bit.org.