Updated the iceberg_real_hsi_test.py to include many of the elements … - #83
Merged
Merged
Conversation
…that are part of the --integtest-verbosity changes.
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…that are part of the --integtest-verbosity changes.
These changes are intended for the
fddaq-v5.7.0software release.Description
In discussions of possible improvements to our regression/integration tests, it was suggested to provide several different levels of verbosity. These changes are part of implementing those, and this PR is best tested and merged with DUNE-DAQ/integrationtest#152. The
integrationtestPR has some details about the changes.The
iceberg_real_hsi_testis not currently fully operational, but these changes were an attempt to keep it somewhat up-to-date nonetheless.There were a few changes that were made in addition to the ones needed for verbosity levels. Those include the following:
nanorcname was replaced with the more genericdunercnamebasic_checkscommon function for verifying successful running of the integrationtest fixtures is now being usedtest_dunerc_successfunction to better match other test filesPlease see the suggested instructions for testing these changes in DUNE-DAQ/integrationtest#152.
Type of change
Testing checklist
dunedaq_integtest_bundle.sh)