Skip to content

Added the verbosity_helper to the log_file_checks call in listrev_test… - #127

Open
bieryAtFnal wants to merge 1 commit into
developfrom
kbiery/logfile_check_verbosity
Open

Added the verbosity_helper to the log_file_checks call in listrev_test…#127
bieryAtFnal wants to merge 1 commit into
developfrom
kbiery/logfile_check_verbosity

Conversation

@bieryAtFnal

Copy link
Copy Markdown
Collaborator

…so that it respects the verbosity level that a user requests.

Description

Without this, logfile test summaries get printed out even when the user has requested a low verbosity level.

To test this, run dunedaq_integtest_bundle.sh --verb 2 -r listrev without using this branch and notice that there are messages displayed on the console like the following:

.🚧 Note: problems found in 52 lines in /tmp/pytest-of-biery/pytest-16059/run2/log_biery_drunc_console_output.txt were ignored based on 18 phrase(s). 🚧

With this branch, those messages are suppressed, as they should be at verbosity level 2.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing checklist

  • Full set of integration tests pass (dunedaq_integtest_bundle.sh)

…t so that it respects the verbosity level that a user requests.
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