Skip to content

Speed up CI test suite on non-Linux platforms#420

Merged
blink1073 merged 2 commits intoCalysto:mainfrom
blink1073:speed-up-ci-tests
Mar 24, 2026
Merged

Speed up CI test suite on non-Linux platforms#420
blink1073 merged 2 commits intoCalysto:mainfrom
blink1073:speed-up-ci-tests

Conversation

@blink1073
Copy link
Copy Markdown
Contributor

@blink1073 blink1073 commented Mar 24, 2026

References

N/A

Description

The macOS CI cover job takes ~4min vs ~1min on Ubuntu. This PR skips slow tests that depend on ipcluster or have slow platform-specific initialization on non-Linux CI, and reduces unnecessary sleep/timeout durations in tests.

Changes

  • Skip test_parallel_magic and test_px_error_reports_exception_not_noisy_traceback on non-Linux (they require ipcluster and take ~71s on macOS vs <1s on Ubuntu)
  • Skip test_matplotlib_magic_sets_backend on non-Linux (~12s on macOS vs ~2s on Ubuntu)
  • Reduce test_expect_timeout pexpect timeout from 5s to 1s
  • Reduce sleep 1 to sleep 0.1 in CR-handling tests in test_replwrap.py and test_process_metakernel.py

Backwards-incompatible changes

None

Testing

Existing CI — these tests continue to run on Linux.

AI usage

  • Some or all of the content of this PR was generated by AI.
  • The human author has carefully reviewed this PR and run this code.
  • AI tools and models used: Claude Code (claude-opus-4-6)

- Skip slow ipcluster-dependent parallel magic tests on non-Linux CI
- Skip slow matplotlib backend test on non-Linux CI
- Reduce pexpect timeout test from 5s to 1s
- Reduce bash sleep in CR-handling tests from 1s to 0.1s
@blink1073 blink1073 enabled auto-merge (squash) March 24, 2026 02:46
@blink1073 blink1073 disabled auto-merge March 24, 2026 02:50
@blink1073 blink1073 merged commit 84f58d7 into Calysto:main Mar 24, 2026
7 checks passed
@blink1073 blink1073 deleted the speed-up-ci-tests branch March 24, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant