Skip to content

Commit

Permalink
patch 8.2.4157: terminal test fails because Windows sets the title
Browse files Browse the repository at this point in the history
Problem:    Terminal test fails because Windows sets the title.
Solution:   Add the "vterm_title" testing override and use it in the test.
            (Ozaki Kiichi, closes #9556)
  • Loading branch information
ichizok authored and brammool committed Jan 20, 2022
1 parent 726f7f9 commit ae1bd87
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 0 deletions.
2 changes: 2 additions & 0 deletions runtime/doc/testing.txt
Original file line number Diff line number Diff line change
Expand Up @@ -208,6 +208,8 @@ test_override({name}, {val}) *test_override()*
ui_delay time in msec to use in ui_delay(); overrules a
wait time of up to 3 seconds for messages
uptime overrules sysinfo.uptime
vterm_title setting the window title by a job running in a
terminal window
ALL clear all overrides ({val} is not used)

"starting" is to be used when a test should behave like
Expand Down
1 change: 1 addition & 0 deletions src/globals.h
Original file line number Diff line number Diff line change
Expand Up @@ -1642,6 +1642,7 @@ EXTERN int nfa_fail_for_testing INIT(= FALSE);
EXTERN int no_query_mouse_for_testing INIT(= FALSE);
EXTERN int ui_delay_for_testing INIT(= 0);
EXTERN int reset_term_props_on_termresponse INIT(= FALSE);
EXTERN int disable_vterm_title_for_testing INIT(= FALSE);
EXTERN long override_sysinfo_uptime INIT(= -1);
EXTERN int override_autoload INIT(= FALSE);

Expand Down
2 changes: 2 additions & 0 deletions src/terminal.c
Original file line number Diff line number Diff line change
Expand Up @@ -3088,6 +3088,8 @@ handle_settermprop(
switch (prop)
{
case VTERM_PROP_TITLE:
if (disable_vterm_title_for_testing)
break;
strval = vim_strnsave((char_u *)value->string.str,
value->string.len);
if (strval == NULL)
Expand Down
2 changes: 2 additions & 0 deletions src/testdir/test_terminal.vim
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ let s:python = PythonProg()
let $PROMPT_COMMAND=''

func Test_terminal_basic()
call test_override('vterm_title', 1)
au TerminalOpen * let b:done = 'yes'
let buf = Run_shell_in_terminal({})

Expand All @@ -37,6 +38,7 @@ func Test_terminal_basic()
call assert_equal("", bufname(buf))

au! TerminalOpen
call test_override('ALL', 0)
unlet g:job
endfunc

Expand Down
2 changes: 2 additions & 0 deletions src/testing.c
Original file line number Diff line number Diff line change
Expand Up @@ -1053,6 +1053,8 @@ f_test_override(typval_T *argvars, typval_T *rettv UNUSED)
ui_delay_for_testing = val;
else if (STRCMP(name, (char_u *)"term_props") == 0)
reset_term_props_on_termresponse = val;
else if (STRCMP(name, (char_u *)"vterm_title") == 0)
disable_vterm_title_for_testing = val;
else if (STRCMP(name, (char_u *)"uptime") == 0)
override_sysinfo_uptime = val;
else if (STRCMP(name, (char_u *)"autoload") == 0)
Expand Down
2 changes: 2 additions & 0 deletions src/version.c
Original file line number Diff line number Diff line change
Expand Up @@ -750,6 +750,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
/**/
4157,
/**/
4156,
/**/
Expand Down

0 comments on commit ae1bd87

Please sign in to comment.