Skip to content

Conversation

@antoniojbt
Copy link
Owner

What was changed and why

  • Added skip guard for multicore-dependent plot tests so they don't run when multicore futures aren't supported or CRAN core limits are active.

Tests added

  • tests/testthat/test-plot-parallel.R

Backward compatibility

  • No breaking changes.

Code coverage change

  • No change expected.

https://chatgpt.com/codex/tasks/task_e_68b205a2452c83268793cca961df0c82

Copilot AI review requested due to automatic review settings August 29, 2025 20:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds proper test environment handling for parallel plotting functionality by skipping multicore-dependent tests when the environment doesn't support multicore processing or when CRAN imposes core limitations.

  • Introduces conditional test skipping for parallel plotting tests
  • Prevents false test failures on systems without multicore support
  • Adds placeholder test structure for future parallel plotting functionality

@codecov
Copy link

codecov bot commented Aug 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.97%. Comparing base (e412fcb) to head (98150cf).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #143   +/-   ##
=======================================
  Coverage   83.97%   83.97%           
=======================================
  Files          64       64           
  Lines        1423     1423           
=======================================
  Hits         1195     1195           
  Misses        228      228           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

…gement-in-epi_utils_multicore

Codex/adjust future plan management in epi utils multicore
@antoniojbt antoniojbt merged commit 2d50ee8 into master Aug 29, 2025
3 of 4 checks passed
@antoniojbt antoniojbt deleted the codex/add-guard-to-test-plot-parallel.r branch August 29, 2025 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants