Added support for several different verbosity levels in our integration tests#152
Open
bieryAtFnal wants to merge 15 commits intodevelopfrom
Open
Added support for several different verbosity levels in our integration tests#152bieryAtFnal wants to merge 15 commits intodevelopfrom
bieryAtFnal wants to merge 15 commits intodevelopfrom
Conversation
added 13 commits
March 26, 2026 16:50
…rol for another message type.
….py so that they reliably get closed.
… control output when a specified string is found in the RC output.
…le confiugrations per pytest file (integtest)
…g is disabled, to prevent spurious complaints in that situation.
…o make it easier to deal with DRUNC message colorization)
This was referenced Apr 13, 2026
added 2 commits
April 14, 2026 08:41
…_checks.py so that it appears at a slightly lower level.
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.
The changes are intended for the
fddaq-v5.7.0release.Description
In discussions of possible improvements to our regression/integration tests, it was suggested to provide several different levels of verbosity. As described in a presentation at the 31-Mar-2026 SWI&T meeting, we currently have only two options available: one with very limited output and one with the full output. The proposal is to move to a model in which 6 different verbosity levels are supported (where the sixth one enables
druncdebug messages).Suggested instructions for testing the changes are provided below. Please note that the new default verbosity level is 3, so there will be less console output unless the full amount is specifically requested.
In addition to the verbosity-level changes, there are changes included in this PR that are not directly related, but are beneficial. These include:
run_dunercfixture, as before, and there is also now a check for warning messages that may have been caught during the setup of the tests. These latter messages were being silently ignored previously.druncoutput when it is encountered (This is only useful when only part of thedruncoutput is being displayed, of course)Here are the command-line options that control the verbosity level and the triggering of the full
druncoutput:dunedaq_integtest_bundle.sh:--verbosity <level>: requested level of console messages, in range 1-6, where 1 is least, 6 is DRUNC debug--trigger-full-rc-output <phrase that will trigger the full printout of run control messages>: the phrase can be a Python regex, which can be useful in handling colorized textpytestdirectly:--integtest-verbosity <level>- controls the volume of messages that are printed out by the integration test infrastructure--dunerc-fullprint-watch-string <string>- A phrase that, if found in run control messages, will trigger the printout of all RC messagesThese changes are correlated with ones in the following repositories:
Here are suggested steps for testing:
Type of change
Testing checklist
dunedaq_integtest_bundle.sh)