Skip to content

Commit

Permalink
Test without cleanupreporter
Browse files Browse the repository at this point in the history
  • Loading branch information
jeroen committed Aug 7, 2024
1 parent 6fd2bf9 commit 9d3b36b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
12 changes: 9 additions & 3 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ jobs:
config:
- {os: macos-13, r: 'release'}
- {os: macos-14, r: 'release'}
- {os: macos-latest, r: 'release', homebrew: true}
- {os: macos-13, r: 'release', homebrew: true}
- {os: macos-14, r: 'release', homebrew: true}
- {os: windows-latest, r: '4.1'}
- {os: windows-latest, r: '4.2'}
- {os: windows-latest, r: 'release'}
Expand All @@ -36,8 +37,13 @@ jobs:

- uses: r-lib/actions/setup-pandoc@v2

- if: matrix.config.homebrew
run: brew install ffmpeg
- if: runner.os == 'macos'
run: |
if [ "${{matrix.config.homebrew}}" ]; then
brew install ffmpeg
else
brew remove ffmpeg || true
fi
- uses: r-lib/actions/setup-r@v2
with:
Expand Down
2 changes: 1 addition & 1 deletion tests/testthat.R
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
library(testthat)
library(av)

if (ps::ps_is_supported()) {
if (0 && ps::ps_is_supported()) {
# This sometimes has false positives on MacOS https://github.com/r-lib/ps/issues/90
reporter <- ps::CleanupReporter(testthat::CheckReporter)$new(proc_cleanup = FALSE, proc_fail = FALSE, conn_fail = FALSE)
test_check("av", reporter = reporter)
Expand Down

0 comments on commit 9d3b36b

Please sign in to comment.