diff options
author | Yee Cheng Chin <ychin.git@gmail.com> | 2024-02-14 20:34:58 +0100 |
---|---|---|
committer | Christian Brabandt <cb@256bit.org> | 2024-02-14 20:34:58 +0100 |
commit | 49f2ba6d41d3c6142deaa4a50b0b16e03969a904 (patch) | |
tree | b0b83e539025b287aab24fa15bf8cab9b0b1861a /src/testdir | |
parent | 34e4a05d02a016fe230495be8f6c60ddd56f9567 (diff) |
patch 9.1.0107: CI: Fix MacOS-14 testsv9.1.0107
Problem: CI: Fix MacOS-14 tests (after 9.1.0070)
Solution: Re-enable sound tests by granting Mic access,
disable Test_diff_screen because of buggy MacOS diff
(non GNU version), re-enable Test_term_gettitle()
(Yee Cheng Chin)
macos-14 runner was turned on in #13943, but it had to turn off a few
tests in order for CI to run. Re-enable them and fix the underlying
issues.
* `Test_diff_screen`: The test failure is due to a bug in Apple's diff
utility. Apple introduced a new diff tool based on FreeBSD in macOS 13
and it has buggy behaviors when using unified diff (`-U0`) and the
diff is on the first line of the file. Simply disable this test for
now if we detect Apple diff (instead of the old GNU diff). Can
re-enable this in the future if Apple fixes the issue.
* `Test_play_event` / `Test_play_silent`: GitHub Actions currently has
an issue with playing sound in CI in macos-14 runners. It for some
reason triggers a microphone permission dialog popup which blocks the
CI action (see https://github.com/actions/runner-images/issues/9330).
To fix this, add a temporary step in macos-14 to manually allow
microphone permissions in the runner.
* `Test_term_gettitle`: I could not reproduce the failure, so I just
turned it on and it seems to run just fine. Maybe it's a timing issue
and whatnot but either way that should be fixed when we can reproduce
the issue.
closes: #14032
Signed-off-by: Yee Cheng Chin <ychin.git@gmail.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
Diffstat (limited to 'src/testdir')
-rw-r--r-- | src/testdir/test_diffmode.vim | 4 | ||||
-rw-r--r-- | src/testdir/test_sound.vim | 7 | ||||
-rw-r--r-- | src/testdir/test_terminal2.vim | 4 |
3 files changed, 2 insertions, 13 deletions
diff --git a/src/testdir/test_diffmode.vim b/src/testdir/test_diffmode.vim index bae94f6a87..b44879f71e 100644 --- a/src/testdir/test_diffmode.vim +++ b/src/testdir/test_diffmode.vim @@ -871,8 +871,8 @@ func VerifyInternal(buf, dumpfile, extra) endfunc func Test_diff_screen() - if has('osx') && !empty($CI) && system('uname -m') =~# 'arm64' - throw 'Skipped: FIXME: This test fails on M1 Mac on GitHub Actions' + if has('osxdarwin') && system('diff --version') =~ '^Apple diff' + throw 'Skipped: unified diff does not work properly on this macOS version' endif let g:test_is_flaky = 1 diff --git a/src/testdir/test_sound.vim b/src/testdir/test_sound.vim index 020d458912..e97ac6198b 100644 --- a/src/testdir/test_sound.vim +++ b/src/testdir/test_sound.vim @@ -15,9 +15,6 @@ func Test_play_event() if has('win32') throw 'Skipped: Playing event with callback is not supported on Windows' endif - if has('osx') && !empty($CI) && system('uname -m') =~# 'arm64' - throw 'Skipped: FIXME: Running this test on M1 Mac hangs on GitHub Actions' - endif let g:playcallback_count = 0 let g:id = 0 let event_name = 'bell' @@ -38,10 +35,6 @@ func Test_play_event() endfunc func Test_play_silent() - if has('osx') && !empty($CI) && system('uname -m') =~# 'arm64' - throw 'Skipped: FIXME: Running this test on M1 Mac hangs on GitHub Actions' - endif - let fname = fnamemodify('silent.wav', '%p') let g:playcallback_count = 0 diff --git a/src/testdir/test_terminal2.vim b/src/testdir/test_terminal2.vim index c7d1a9a7bd..9798c51940 100644 --- a/src/testdir/test_terminal2.vim +++ b/src/testdir/test_terminal2.vim @@ -536,10 +536,6 @@ endfunc " Test for term_gettitle() func Test_term_gettitle() - if has('osx') && !empty($CI) && system('uname -m') =~# 'arm64' - throw 'Skipped: FIXME: Title got on M1 Mac is broken on GitHub Actions' - endif - " term_gettitle() returns an empty string for a non-terminal buffer " and for a non-existing buffer. call assert_equal('', bufnr('%')->term_gettitle()) |