Skip to content

ParallelTestRunner.jl#16

Merged
lkdvos merged 3 commits intomainfrom
testrunner
Apr 30, 2026
Merged

ParallelTestRunner.jl#16
lkdvos merged 3 commits intomainfrom
testrunner

Conversation

@lkdvos
Copy link
Copy Markdown
Collaborator

@lkdvos lkdvos commented Apr 30, 2026

This is a minor change to alter the tests to use the ParallelTestRunner.jl interface.

To make this work, I also had to update the printing machinery slightly, since string(x) does not now about the context, so it would print types as if they are shown in the Main module, which doesn't necessarily have AlgorithmsInterface loaded.
This became visible here since ParallelTestRunner starts new processes to really isolate the tests, therefore you can't really control what is visible in Main (and @eval Main using AlgorithmsInterface feels very wrong 😉 )

Let me know if the explanation is clear enough, I'm happy to include a small readme file in the tests as well if you think that could be helpful

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.57%. Comparing base (750916e) to head (2fdbe81).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   99.57%   99.57%           
=======================================
  Files           6        6           
  Lines         235      235           
=======================================
  Hits          234      234           
  Misses          1        1           

☔ 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.

@lkdvos
Copy link
Copy Markdown
Collaborator Author

lkdvos commented Apr 30, 2026

Re the changelog, given that we haven't released anything yet I don't think it is that helpful to update this, so if you are okay with it I would just keep this for now.

@lkdvos lkdvos requested a review from kellertuer April 30, 2026 15:05
Copy link
Copy Markdown
Member

@kellertuer kellertuer left a comment

Choose a reason for hiding this comment

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

Sure, the Changelog can be ignored here. It is really just to later have in mind that at least one itemise point per PR would be nice.

@lkdvos lkdvos merged commit f87561f into main Apr 30, 2026
15 of 16 checks passed
@lkdvos lkdvos deleted the testrunner branch April 30, 2026 15:35
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.

2 participants