TrigEDMConfig: fix L1_eTauBDTRoI Aux item and improve unit tests
The unit tests were useless for many cases because errors were not being checked and the test return code was often 0 despite error.
- add missing
Aux.
toL1_eTauBDTRoI
- return failure code when printing an error
- use
noerror.sh
- add a check to ensure Aux follows non-Aux container (if this is violated we are writing data that cannot be deserialized because our Deserialiser has an implicit assumption on the order of the containers: Interface, Aux, dynAux)
Merge request reports
Activity
Filter activity
This merge request affects 1 package:
- Trigger/TriggerCommon/TrigEDMConfig
This merge request affects 3 files:
- Trigger/TriggerCommon/TrigEDMConfig/CMakeLists.txt
- Trigger/TriggerCommon/TrigEDMConfig/python/TriggerEDMRun3.py
- Trigger/TriggerCommon/TrigEDMConfig/python/testEDMRun3.py
added 23.0 Trigger TriggerEDM review-pending-level-1 labels
CI Result SUCCESS (hash dcc63a93)Athena externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 75963]added review-approved label and removed review-pending-level-1 label
added review-approved-point1 label
added review-approved-tier0 label
mentioned in commit 9d3255ca
mentioned in merge request !64944 (merged)
Please register or sign in to reply