Skip to content

Update integration tests#414

Merged
eflumerf merged 22 commits into
developfrom
eflumerf/IntegrationTestUpdates
Sep 17, 2025
Merged

Update integration tests#414
eflumerf merged 22 commits into
developfrom
eflumerf/IntegrationTestUpdates

Conversation

@eflumerf
Copy link
Copy Markdown
Member

Applies updates that try to fix disabled_output_test, insufficient_disk_space_test, and large_trigger_record_test.

insufficient_disk_space_test still requires root setup, and I was unable to get large_trigger_record_test to pass on my laptop VM (though indications were that this was mostly a performance-limited issue).

@eflumerf eflumerf requested a review from bieryAtFnal May 19, 2025 19:53
@eflumerf eflumerf self-assigned this May 19, 2025
@eflumerf
Copy link
Copy Markdown
Member Author

eflumerf commented Sep 4, 2025

Updated with changes in #440, that change-set should be merged first.

eflumerf and others added 15 commits September 8, 2025 11:57
…successfully (expected number of files = 4 [two TRs in each of two runs]; tweaks in expected fragment sizes to allow for N and N+1 WIB frames; printout of correct or incorrect numbers of files)
A few modifications in large_trigger_record_test.py to get it to run successfully
…tweaked an expected TC fragment size in that test.
…ty_factor to simulate a nearly-full disk instead of requiring the user to create a temporary disk with limited space.
… to add ones that have been recently fixed, and to have the shorter ones run first.
…py to try to reliably get 2 data files produced.
Several additional contributions to help get all dfmodules integtests working
Copy link
Copy Markdown
Collaborator

@bieryAtFnal bieryAtFnal left a comment

Choose a reason for hiding this comment

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

LGTM

@eflumerf eflumerf merged commit 08d2636 into develop Sep 17, 2025
2 checks passed
@eflumerf eflumerf deleted the eflumerf/IntegrationTestUpdates branch September 17, 2025 13:06
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