Skip to content

[rootest] refine root/io tests, part 2 #19699

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

[rootest] refine root/io tests, part 2 #19699

wants to merge 7 commits into from

Conversation

linev
Copy link
Member

@linev linev commented Aug 20, 2025

  1. Use cmake for io/dirdelete test
  2. Link executable with dictionary lib in io/evolution/enums test
  3. Use more distinct fixtures names in several io tests

@linev linev requested a review from pcanal August 20, 2025 13:44
@linev linev self-assigned this Aug 20, 2025
@linev linev requested review from bellenot and dpiparo as code owners August 20, 2025 13:44
@github-actions GitHub Actions
Copy link

Test Results

    21 files      21 suites   3d 11h 36m 43s ⏱️
 3 560 tests  3 559 ✅ 0 💤 1 ❌
73 007 runs  73 006 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit fb32387.

@linev linev changed the title [rootest] refine io tests, part 2 [rootest] refine root/io tests, part 2 Aug 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pcanal pcanal Awaiting requested review from pcanal

@bellenot bellenot Awaiting requested review from bellenot bellenot is a code owner

@dpiparo dpiparo Awaiting requested review from dpiparo dpiparo is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@linev linev

Labels
affects:master in:Testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@linev