Skip to content

[rdf] Fix dataframe_snapshot tests on Windows #19379

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.

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

Merged
merged 1 commit into from
Jul 16, 2025

Conversation

bellenot
Copy link
Member

dataframe_snapshot.cxx: Fix the following compilation error:

dataframe_snapshot.cxx(1000,34): error C2326: 'RDFSnapshotMore_CompositeTypeWithNameClash_Test::TestBody::FileGuard::~FileGuard(void)': function cannot access 'fName'

and the following run-time error:

467: [ RUN      ] RDFSnapshotMore.Lazy
467: C:\root-dev\git\master\tree\dataframe\test\dataframe_snapshot.cxx(860): error: Value of: gSystem->AccessPathName(fname1)
467:   Actual: false
467: Expected: true

dataframe_snapshot_ntuple.cxx: Fix the following error:

469: [ RUN      ] RDFSnapshotRNTupleTest.InnerFields
469: ROOT::TestSupport::ForbidDiagnostics::handler(): Diagnostic in 'Snapshot':
469: Column electron.pt will be saved as electron_pt
469: ROOT::TestSupport::ForbidDiagnostics::handler(): Diagnostic in 'Snapshot':
469: Column jets.electrons will be saved as jets_electrons
469: C:\root-dev\git\master\core\testsupport\src\TestSupport.cxx(81): error: Failed
469: Received unexpected diagnostic of severity 5000 at 'TFile::TFile' reading 'could not delete C:\root-dev\build\x64\relwithdebinfo\tree\dataframe\test\RDFSnapshotRNTuple_inner_fields.root (errno: 13) Permission denied'.
469: Suppress those using ROOT/TestSupport.hxx
469: unknown file: error: C++ exception with description "Snapshot: could not create output file RDFSnapshotRNTuple_inner_fields.root" thrown in the test body.
469: [  FAILED  ] RDFSnapshotRNTupleTest.InnerFields (110 ms)

`dataframe_snapshot.cxx`: Fix the following compilation error:
```
dataframe_snapshot.cxx(1000,34): error C2326: 'RDFSnapshotMore_CompositeTypeWithNameClash_Test::TestBody::FileGuard::~FileGuard(void)': function cannot access 'fName'
```
and the following run-time error:
```
467: [ RUN      ] RDFSnapshotMore.Lazy
467: C:\root-dev\git\master\tree\dataframe\test\dataframe_snapshot.cxx(860): error: Value of: gSystem->AccessPathName(fname1)
467:   Actual: false
467: Expected: true
```
`dataframe_snapshot_ntuple.cxx`: Fix the following error:
```
469: [ RUN      ] RDFSnapshotRNTupleTest.InnerFields
469: ROOT::TestSupport::ForbidDiagnostics::handler(): Diagnostic in 'Snapshot':
469: Column electron.pt will be saved as electron_pt
469: ROOT::TestSupport::ForbidDiagnostics::handler(): Diagnostic in 'Snapshot':
469: Column jets.electrons will be saved as jets_electrons
469: C:\root-dev\git\master\core\testsupport\src\TestSupport.cxx(81): error: Failed
469: Received unexpected diagnostic of severity 5000 at 'TFile::TFile' reading 'could not delete C:\root-dev\build\x64\relwithdebinfo\tree\dataframe\test\RDFSnapshotRNTuple_inner_fields.root (errno: 13) Permission denied'.
469: Suppress those using ROOT/TestSupport.hxx
469: unknown file: error: C++ exception with description "Snapshot: could not create output file RDFSnapshotRNTuple_inner_fields.root" thrown in the test body.
469: [  FAILED  ] RDFSnapshotRNTupleTest.InnerFields (110 ms)
```
@bellenot bellenot self-assigned this Jul 15, 2025
Copy link

github-actions bot commented Jul 15, 2025

Test Results

    21 files      21 suites   3d 12h 19m 38s ⏱️
 3 203 tests  3 203 ✅ 0 💤 0 ❌
65 612 runs  65 612 ✅ 0 💤 0 ❌

Results for commit ea8f06c.

♻️ This comment has been updated with latest results.

@bellenot bellenot merged commit 85aadc5 into root-project:master Jul 16, 2025
47 of 48 checks passed
@bellenot bellenot deleted the fix-dataframe_snapshot branch July 16, 2025 10:38
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