Skip to content

Fail slow tests#7392

Draft
user27182 wants to merge 4 commits intomainfrom
testing/fail_slow
Draft

Fail slow tests#7392
user27182 wants to merge 4 commits intomainfrom
testing/fail_slow

Conversation

@user27182
Copy link
Copy Markdown
Contributor

Overview

Use https://github.com/jwodder/pytest-fail-slow to fail slow tests

@user27182 user27182 marked this pull request as draft April 9, 2025 04:31
@pyvista-bot pyvista-bot added the maintenance Low-impact maintenance activity label Apr 9, 2025
@user27182
Copy link
Copy Markdown
Contributor Author

This plugin is very helpful. Already found some low-hanging fruit to speed up CI:

tiny pages tests are taking a long time on macOS:

___________________ test_tinypages[False-PYVISTA_PLOT_SKIP] ____________________
Test passed but took too long to run: Duration 46.58355129099982s > 1.0s
_______________ test_tinypages[False-PYVISTA_PLOT_SKIP_OPTIONAL] _______________
Test passed but took too long to run: Duration 46.18138849999991s > 1.0s
____________________ test_tinypages[True-PYVISTA_PLOT_SKIP] ____________________
Test passed but took too long to run: Duration 46.061539541s > 1.0s
_______________ test_tinypages[True-PYVISTA_PLOT_SKIP_OPTIONAL] ________________
Test passed but took too long to run: Duration 46.224338708000005s > 1.0s

plus others

______________________ test_volume_rendering_rectilinear _______________________
Test passed but took too long to run: Duration 43.41575133300012s > 1.0s
___________________________________ test_pbr ___________________________________
Test passed but took too long to run: Duration 142.26619474999984s > 1.0s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Low-impact maintenance activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants