summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBram Moolenaar <Bram@vim.org>2018-03-25 20:31:32 +0200
committerBram Moolenaar <Bram@vim.org>2018-03-25 20:31:32 +0200
commitcf67a509e93167f14c892301e13de14636cedc61 (patch)
tree3c0b05d4e719a83e6321d6e5eb14aa057ef547dc
parent2de50f87622cd4e631fd17845e16a94ed5992b80 (diff)
patch 8.0.1644: terminal API tests still failv8.0.1644
Problem: Terminal API tests still fail. Solution: Explicitly set 'title' in the terminal job. (Ozaki Kiichi, closes #2750)
-rw-r--r--src/testdir/screendump.vim16
-rw-r--r--src/testdir/test_terminal.vim9
-rw-r--r--src/version.c2
3 files changed, 13 insertions, 14 deletions
diff --git a/src/testdir/screendump.vim b/src/testdir/screendump.vim
index 60c0ccb012..87a5823cc2 100644
--- a/src/testdir/screendump.vim
+++ b/src/testdir/screendump.vim
@@ -24,7 +24,9 @@ source shared.vim
" By default uses a size of 20 lines and 75 columns.
" Returns the buffer number of the terminal.
"
-" Options is a dictionary (not used yet).
+" Options is a dictionary, these items are recognized:
+" "rows" - height of the terminal window (max. 20)
+" "cols" - width of the terminal window (max. 78)
func RunVimInTerminal(arguments, options)
" If Vim doesn't exit a swap file remains, causing other tests to fail.
" Remove it here.
@@ -47,17 +49,15 @@ func RunVimInTerminal(arguments, options)
set t_Co=256 background=light
hi Normal ctermfg=NONE ctermbg=NONE
- " Make the window 20 lines high, unless told otherwise.
- let rows = 20
- if has_key(a:options, 'rows')
- let rows = a:options['rows']
- endif
+ " Make the window 20 lines high and 75 columns, unless told otherwise.
+ let rows = get(a:options, 'rows', 20)
+ let cols = get(a:options, 'cols', 75)
let cmd = GetVimCommandClean()
" Add -v to have gvim run in the terminal (if possible)
let cmd .= ' -v ' . a:arguments
- let buf = term_start(cmd, {'curwin': 1, 'term_rows': rows, 'term_cols': 75})
- call assert_equal([rows, 75], term_getsize(buf))
+ let buf = term_start(cmd, {'curwin': 1, 'term_rows': rows, 'term_cols': cols})
+ call assert_equal([rows, cols], term_getsize(buf))
return buf
endfunc
diff --git a/src/testdir/test_terminal.vim b/src/testdir/test_terminal.vim
index 2589b26cc8..f981fcc950 100644
--- a/src/testdir/test_terminal.vim
+++ b/src/testdir/test_terminal.vim
@@ -1029,10 +1029,10 @@ func Test_terminal_api_drop_newwin()
return
endif
call assert_equal(1, winnr('$'))
- set title
" Use the title termcap entries to output the escape sequence.
call writefile([
+ \ 'set title',
\ 'exe "set t_ts=\<Esc>]51; t_fs=\x07"',
\ 'let &titlestring = ''["drop","Xtextfile"]''',
\ 'redraw',
@@ -1046,7 +1046,6 @@ func Test_terminal_api_drop_newwin()
call StopVimInTerminal(buf)
call delete('Xscript')
bwipe Xtextfile
- set title&
endfunc
func Test_terminal_api_drop_oldwin()
@@ -1058,10 +1057,10 @@ func Test_terminal_api_drop_oldwin()
let textfile_winid = win_getid()
call assert_equal(2, winnr('$'))
call win_gotoid(firstwinid)
- set title
" Use the title termcap entries to output the escape sequence.
call writefile([
+ \ 'set title',
\ 'exe "set t_ts=\<Esc>]51; t_fs=\x07"',
\ 'let &titlestring = ''["drop","Xtextfile"]''',
\ 'redraw',
@@ -1074,7 +1073,6 @@ func Test_terminal_api_drop_oldwin()
call StopVimInTerminal(buf)
call delete('Xscript')
bwipe Xtextfile
- set title&
endfunc
func TryThis(bufnum, arg)
@@ -1086,10 +1084,10 @@ func Test_terminal_api_call()
if !CanRunVimInTerminal()
return
endif
- set title
" Use the title termcap entries to output the escape sequence.
call writefile([
+ \ 'set title',
\ 'exe "set t_ts=\<Esc>]51; t_fs=\x07"',
\ 'let &titlestring = ''["call","TryThis",["hello",123]]''',
\ 'redraw',
@@ -1104,5 +1102,4 @@ func Test_terminal_api_call()
call delete('Xscript')
unlet g:called_bufnum
unlet g:called_arg
- set title&
endfunc
diff --git a/src/version.c b/src/version.c
index acca91ad84..1a8c10ea47 100644
--- a/src/version.c
+++ b/src/version.c
@@ -767,6 +767,8 @@ static char *(features[]) =
static int included_patches[] =
{ /* Add new patch number below this line */
/**/
+ 1644,
+/**/
1643,
/**/
1642,