Skip to content

fix: test_ep_postprocessor tests on Github runner - #971

Merged
mhoeijm merged 9 commits into
mainfrom
refactor/ep-postprocessor-tests
Mar 27, 2025
Merged

mhoeijm merged 9 commits into
mainfrom
refactor/ep-postprocessor-tests

Conversation

@mhoeijm

@mhoeijm mhoeijm commented Mar 27, 2025 •

Copy link
Copy Markdown
Contributor

Github runner has no graphics, so any call to pyvista.Plotter.render will throw an error. Mocking the methods that rely on calls to render().

@mhoeijm mhoeijm changed the title fix: test on gh runner fix: test_ep_postprocessor tests on gh runner Mar 27, 2025
@mhoeijm
mhoeijm requested a review from jorgepiloto March 27, 2025 09:33
@mhoeijm
mhoeijm marked this pull request as ready for review March 27, 2025 09:33
@mhoeijm mhoeijm changed the title fix: test_ep_postprocessor tests on gh runner fix: test_ep_postprocessor tests on Github runner Mar 27, 2025
Comment thread tests/heart/postprocessor/test_ep_postprocessor.py Outdated
@mhoeijm
mhoeijm requested a review from jorgepiloto March 27, 2025 10:53
@mhoeijm
mhoeijm merged commit 00e1484 into main Mar 27, 2025
@mhoeijm
mhoeijm deleted the refactor/ep-postprocessor-tests branch March 27, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants