To: vim_dev@googlegroups.com Subject: Patch 8.2.2632 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 8.2.2632 Problem: Not all command line arguments are tested. Solution: Add tests for -D and -serverlist. (Dominique Pellé, closes #7992) Files: src/testdir/test_clientserver.vim, src/testdir/test_startup.vim *** ../vim-8.2.2631/src/testdir/test_clientserver.vim 2021-01-11 18:46:55.947382209 +0100 --- src/testdir/test_clientserver.vim 2021-03-20 21:40:53.114132959 +0100 *************** *** 43,48 **** --- 43,56 ---- " When using valgrind it takes much longer. call WaitForAssert({-> assert_match(name, serverlist())}) + if !has('win32') + if RunVim([], [], '--serverlist >Xtest_serverlist') + let lines = readfile('Xtest_serverlist') + call assert_true(index(lines, 'XVIMTEST') >= 0) + endif + call delete('Xtest_serverlist') + endif + eval name->remote_foreground() call remote_send(name, ":let testvar = 'yes'\") *** ../vim-8.2.2631/src/testdir/test_startup.vim 2021-03-17 12:27:20.095476909 +0100 --- src/testdir/test_startup.vim 2021-03-20 21:40:53.114132959 +0100 *************** *** 1121,1126 **** --- 1121,1141 ---- endif endfunc + " Test for the "-D" (debugger) argument + func Test_D_arg() + CheckRunVimInTerminal + + let cmd = GetVimCommandCleanTerm() .. ' -D' + let buf = term_start(cmd, {'term_rows' : 10}) + call WaitForAssert({-> assert_equal("running", term_getstatus(buf))}) + + call WaitForAssert({-> assert_equal('Entering Debug mode. Type "cont" to continue.', + \ term_getline(buf, 7))}) + call WaitForAssert({-> assert_equal('>', term_getline(buf, 10))}) + + call StopVimInTerminal(buf) + endfunc + " Test for too many edit argument errors func Test_too_many_edit_args() " Can't catch the output of gvim. *** ../vim-8.2.2631/src/version.c 2021-03-20 19:55:31.664402760 +0100 --- src/version.c 2021-03-20 21:42:11.085996250 +0100 *************** *** 752,753 **** --- 752,755 ---- { /* Add new patch number below this line */ + /**/ + 2632, /**/ -- hundred-and-one symptoms of being an internet addict: 4. Your eyeglasses have a web site burned in on them. /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org ///