Elimar Riesebieter wrote: > Hi, > > building v8.2.0460 gives the following test failures an amd64: > > Failures: > From test_excmd.vim: > Found errors in Test_confirm_cmd_cancel(): > Run 1: > function > RunTheTest[40]..Test_confirm_cmd_cancel[18]..StopVimInTerminal[10]..WaitForAssert[2]..<SNR>7_WaitForCommon[11]..<lambda>16 > line 1: Expected 'finished' but got 'running' > Run 2: > function > RunTheTest[40]..Test_confirm_cmd_cancel[18]..StopVimInTerminal[10]..WaitForAssert[2]..<SNR>7_WaitForCommon[11]..<lambda>20 > line 1: Expected 'finished' but got 'running' > Run 3: > function > RunTheTest[40]..Test_confirm_cmd_cancel[18]..StopVimInTerminal[10]..WaitForAssert[2]..<SNR>7_WaitForCommon[11]..<lambda>24 > line 1: Expected 'finished' but got 'running' > Run 4: > function > RunTheTest[40]..Test_confirm_cmd_cancel[18]..StopVimInTerminal[10]..WaitForAssert[2]..<SNR>7_WaitForCommon[11]..<lambda>28 > line 1: Expected 'finished' but got 'running' > Run 5: > function > RunTheTest[40]..Test_confirm_cmd_cancel[18]..StopVimInTerminal[10]..WaitForAssert[2]..<SNR>7_WaitForCommon[11]..<lambda>32 > line 1: Expected 'finished' but got 'running' > Flaky test failed too often, giving up > From test_alot.vim: > Found errors in Test_compiler(): > function RunTheTest[40]..Test_compiler line 23: command did not > fail: clist > function RunTheTest[40]..Test_compiler line 29: Pattern '\\n \\d\\+ > Xfoo.pl:3: Global symbol "$foo" requires explicit package name' does not > match '\n21 Xfoo.pl:3: Global symbol "$foo" requires expl:3: Global symbol > "$foo" requires explicit package name (did you forget to declare "my $foo"?)'
I see the same failure in Test_confirm_cmd_cancel() on one of my 2 machines. The attached patch fixes it for me. Can you also check? All other test pass for me. I don't see the failure in Test_compiler(). What does "/usr/bin/perl --version" say on your machine? Mine says: $ /usr/bin/perl --version This is perl 5, version 26, subversion 1 (v5.26.1) built for x86_64-linux-gnu-thread-multi (with 67 registered patches, see perl -V for more detail) ...snip... Regards Dominique -- -- 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/CAON-T_gZCumyJsOiZ3y1qLRUSrcyCXsZ1O1F5toqeeoLfPFF5w%40mail.gmail.com.
diff --git a/src/testdir/test_excmd.vim b/src/testdir/test_excmd.vim index ec252a660..d5588d670 100644 --- a/src/testdir/test_excmd.vim +++ b/src/testdir/test_excmd.vim @@ -264,6 +264,8 @@ func Test_confirm_cmd_cancel() call WaitForAssert({-> assert_match('^\[Y\]es, (N)o, (C)ancel: *$', \ term_getline(buf, 20))}, 1000) call term_sendkeys(buf, "N") + call WaitForAssert({-> assert_match('^ *0,0-1 All$', + \ term_getline(buf, 20))}, 1000) call StopVimInTerminal(buf) endfunc