0
0
mirror of https://github.com/vim/vim.git synced 2025-09-26 04:04:07 -04:00

patch 8.1.1181: tests for mouse clicks are a bit flaky

Problem:    Tests for mouse clicks are a bit flaky when run in an interactive
            terminal.
Solution:   Use "xterm2" instead of "xterm" for 'ttymouse' to avoid spurious
            drag events.
This commit is contained in:
Bram Moolenaar
2019-04-17 17:08:27 +02:00
parent 113bf0672b
commit 2b00b9b0f3
2 changed files with 20 additions and 18 deletions

View File

@@ -7,7 +7,7 @@ endif
" Helper function to emit a terminal escape code. " Helper function to emit a terminal escape code.
func TerminalEscapeCode(code_xterm, code_sgr, row, col, m) func TerminalEscapeCode(code_xterm, code_sgr, row, col, m)
if &ttymouse ==# 'xterm' if &ttymouse ==# 'xterm2'
" need to use byte encoding here. " need to use byte encoding here.
let str = list2str([a:code_xterm, a:col + 0x20, a:row + 0x20]) let str = list2str([a:code_xterm, a:col + 0x20, a:row + 0x20])
if has('iconv') if has('iconv')
@@ -50,7 +50,7 @@ func Test_xterm_mouse_click()
set mouse=a term=xterm set mouse=a term=xterm
call setline(1, ['line 1', 'line 2', 'line 3 is a bit longer']) call setline(1, ['line 1', 'line 2', 'line 3 is a bit longer'])
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
go go
@@ -76,7 +76,7 @@ func Test_xterm_mouse_wheel()
set mouse=a term=xterm set mouse=a term=xterm
call setline(1, range(1, 100)) call setline(1, range(1, 100))
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
go go
@@ -112,7 +112,7 @@ func Test_xterm_mouse_drag_window_separator()
let save_ttymouse = &ttymouse let save_ttymouse = &ttymouse
set mouse=a term=xterm set mouse=a term=xterm
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
@@ -122,8 +122,8 @@ func Test_xterm_mouse_drag_window_separator()
let row = rowseparator let row = rowseparator
let col = 1 let col = 1
" When 'ttymouse' is 'xterm', row/col bigger than 223 are not supported. " When 'ttymouse' is 'xterm2', row/col bigger than 223 are not supported.
if ttymouse_val !=# 'xterm' || row <= 223 if ttymouse_val !=# 'xterm2' || row <= 223
call MouseLeftClick(row, col) call MouseLeftClick(row, col)
let row -= 1 let row -= 1
call MouseLeftDrag(row, col) call MouseLeftDrag(row, col)
@@ -142,8 +142,8 @@ func Test_xterm_mouse_drag_window_separator()
let row = 1 let row = 1
let col = colseparator let col = colseparator
" When 'ttymouse' is 'xterm', row/col bigger than 223 are not supported. " When 'ttymouse' is 'xterm2', row/col bigger than 223 are not supported.
if ttymouse_val !=# 'xterm' || col <= 223 if ttymouse_val !=# 'xterm2' || col <= 223
call MouseLeftClick(row, col) call MouseLeftClick(row, col)
let col -= 1 let col -= 1
call MouseLeftDrag(row, col) call MouseLeftDrag(row, col)
@@ -169,7 +169,7 @@ func Test_xterm_mouse_drag_statusline()
let save_laststatus = &laststatus let save_laststatus = &laststatus
set mouse=a term=xterm laststatus=2 set mouse=a term=xterm laststatus=2
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
@@ -178,8 +178,8 @@ func Test_xterm_mouse_drag_statusline()
let row = rowstatusline let row = rowstatusline
let col = 1 let col = 1
if ttymouse_val ==# 'xterm' && row > 223 if ttymouse_val ==# 'xterm2' && row > 223
" When 'ttymouse' is 'xterm', row/col bigger than 223 are not supported. " When 'ttymouse' is 'xterm2', row/col bigger than 223 are not supported.
continue continue
endif endif
@@ -210,7 +210,7 @@ func Test_xterm_mouse_click_tab()
set mouse=a term=xterm set mouse=a term=xterm
let row = 1 let row = 1
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
e Xfoo e Xfoo
@@ -258,9 +258,9 @@ func Test_xterm_mouse_click_X_to_close_tab()
let row = 1 let row = 1
let col = &columns let col = &columns
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
if ttymouse_val ==# 'xterm' && col > 223 if ttymouse_val ==# 'xterm2' && col > 223
" When 'ttymouse' is 'xterm', row/col bigger than 223 are not supported. " When 'ttymouse' is 'xterm2', row/col bigger than 223 are not supported.
continue continue
endif endif
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
@@ -304,7 +304,7 @@ func Test_xterm_mouse_drag_to_move_tab()
set mouse=a term=xterm mousetime=1 set mouse=a term=xterm mousetime=1
let row = 1 let row = 1
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
e Xtab1 e Xtab1
@@ -353,7 +353,7 @@ func Test_xterm_mouse_double_click_to_create_tab()
let row = 1 let row = 1
let col = 10 let col = 10
for ttymouse_val in ['xterm', 'sgr'] for ttymouse_val in ['xterm2', 'sgr']
let msg = 'ttymouse=' .. ttymouse_val let msg = 'ttymouse=' .. ttymouse_val
exe 'set ttymouse=' . ttymouse_val exe 'set ttymouse=' . ttymouse_val
e Xtab1 e Xtab1
@@ -402,7 +402,7 @@ func Test_xterm_mouse_click_in_fold_columns()
let save_term = &term let save_term = &term
let save_ttymouse = &ttymouse let save_ttymouse = &ttymouse
let save_foldcolumn = &foldcolumn let save_foldcolumn = &foldcolumn
set mouse=a term=xterm foldcolumn=3 set mouse=a term=xterm foldcolumn=3 ttymouse=xterm2
" Create 2 nested folds. " Create 2 nested folds.
call setline(1, range(1, 7)) call setline(1, range(1, 7))

View File

@@ -771,6 +771,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
1181,
/**/ /**/
1180, 1180,
/**/ /**/