patch 9.1.0134: CI: Test_term_gettitle() is flaky on MacOS 14

Commit: 
https://github.com/vim/vim/commit/6a46c19b2507bce632b5e67ef31e47e60a95ed10
Author: Christian Brabandt <c...@256bit.org>
Date:   Sat Feb 24 15:56:34 2024 +0100

    patch 9.1.0134: CI: Test_term_gettitle() is flaky on MacOS 14
    
    Problem:  Test_term_gettitle() is flaky on MacOS 14 and Github runners
    Solution: Skip the test on Github CI
    
    closes: #14085
    
    It fails with this: '^\[No Name\] - VIM\d*$' does not match 'e] - VIM'
    It is not clear why term_gettitle() only get's the last part of the
    expected title (perhaps there is a Carriage return in there or the
    terminal window is too small?)
    
    So let's just skip this test for now.
    
    Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/src/testdir/test_terminal2.vim b/src/testdir/test_terminal2.vim
index 9798c5194..05b228ed7 100644
--- a/src/testdir/test_terminal2.vim
+++ b/src/testdir/test_terminal2.vim
@@ -535,6 +535,7 @@ func Test_term_getcursor()
 endfunc
 
 " Test for term_gettitle()
+" Known to be flaky on Mac-OS X and the GH runners
 func Test_term_gettitle()
   " term_gettitle() returns an empty string for a non-terminal buffer
   " and for a non-existing buffer.
@@ -544,6 +545,13 @@ func Test_term_gettitle()
   if !has('title') || empty(&t_ts)
     throw "Skipped: can't get/set title"
   endif
+  if has('osx') && !empty($CI) && system('uname -m') =~# 'arm64'
+    " This test often fails with the following error message on Github runners
+    " MacOS-14
+    " '^\[No Name\] - VIM\d*$' does not match 'e] - VIM'
+    " Why? Is the terminal that runs Vim too small?
+    throw 'Skipped: FIXME: Running this test on M1 Mac fails on GitHub Actions'
+  endif
 
   let term = term_start([GetVimProg(), '--clean', '-c', 'set noswapfile', 
'-c', 'set title'])
   call TermWait(term)
diff --git a/src/version.c b/src/version.c
index 1405854d8..9ad2015c5 100644
--- a/src/version.c
+++ b/src/version.c
@@ -704,6 +704,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    134,
 /**/
     133,
 /**/

-- 
-- 
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/E1rdtjy-00GY3s-DA%40256bit.org.

Reply via email to