[vim/vim] Test_incsearch_substitute_dump() consistently fails when vim configured with --enable-gui=none --disable-arabic (#4569)

9 views
Skip to first unread message

Dominique Pellé

unread,
Jun 20, 2019, 5:05:18 PM6/20/19
to vim/vim, Subscribed

Test_incsearch_substitute_dump() consistently fails when vim is configured with --enable-gui=none --disable-arabic.

Steps ro reproduce

$ ./configure --with-features=huge --enable-gui=none --disable-arabic 
$ make -j8
$ cd test_dir
$ make test_search
...snip...
Executed 45 tests in  11.654872 seconds
1 FAILED:
Found errors in Test_incsearch_substitute_dump():
Run 1:
function RunTheTest[40]..Test_incsearch_substitute_dump[40]..VerifyScreenDump line 55: See dump file difference: call term_dumpdiff("testdir/failed/Test_incsearch_substitute_03.dump", "testdir/dumps/Test_incsearch_substitute_03.dump"); difference in line 9: "|:|.|,|.|+|2|s|/> @43|4|,|1| @10|T|o|p| "
function RunTheTest[40]..Test_incsearch_substitute_dump[91]..VerifyScreenDump line 55: See dump file difference: call term_dumpdiff("testdir/failed/Test_incsearch_substitute_12.dump", "testdir/dumps/Test_incsearch_substitute_12.dump"); difference in line 9: "| +0&&@3> @65"
Run 2:
function RunTheTest[40]..Test_incsearch_substitute_dump[40]..VerifyScreenDump line 55: See dump file difference: call term_dumpdiff("testdir/failed/Test_incsearch_substitute_03.dump", "testdir/dumps/Test_incsearch_substitute_03.dump"); difference in line 9: "|:|.|,|.|+|2|s|/> @43|4|,|1| @10|T|o|p| "
function RunTheTest[40]..Test_incsearch_substitute_dump[91]..VerifyScreenDump line 55: See dump file difference: call term_dumpdiff("testdir/failed/Test_incsearch_substitute_12.dump", "testdir/dumps/Test_incsearch_substitute_12.dump"); difference in line 9: "| +0&&@3> @65"
Flaky test failed too often, giving up
Makefile:75: recipe for target 'tes

Note that the configure options --enable-gui=none --disable-arabic matter to reproduce the bug.
The same test pass without disabling arabic.

Environment

  • Vim-8.1.1875
  • xubuntu-18.04.2 x86_64


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub

Bram Moolenaar

unread,
Jun 20, 2019, 8:31:26 PM6/20/19
to vim/vim, Subscribed

Closed #4569 via 693f7dc.

Bram Moolenaar

unread,
Jun 20, 2019, 8:31:33 PM6/20/19
to vim...@googlegroups.com, Dominique Pellé

> `Test_incsearch_substitute_dump()` consistently fails when vim is
> configured with `--enable-gui=3Dnone --disable-arabic`.

Hmm, it's the arabic feature that clears the command line after
redrawing for 'incsearch'. It should always remove the ruler.
Somehow that should already happen, but it doesn't.


--
If you're sending someone Styrofoam, what do you pack it in?

/// Bram Moolenaar -- Br...@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 ///
Reply all
Reply to author
Forward
0 new messages