Partial Event Building: Update configuration and add unit tests
Changes introducing extensive unit testing of partial event building configuration in TriggerMenuMT (ATR-22242).
- Replace derived config classes in
TrigPartialEventBuildingConfig
with config functions returning a tool instance (required for the unit test from point 3 below to work correctly). - Extend
menu_config_tests.py
with a new consistency check of partial event building configuration. - Add a unit test for the
EventBuildingSequenceSetup
module testing the configuration of PEB writer tools. - Fix stream misconfiguration uncovered by the test added in point 2. The "CSCPEB" chain was streamed to a full-event-building stream "CSC". That stream should not be full-event-building.
Merge request reports
Activity
added Trigger TriggerMenu master review-pending-level-1 labels
CI Result FAILURE (hash 111abf0b)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 24820] CI Result SUCCESS (hash 111abf0b)Athena AthSimulation AthGeneration AnalysisBase externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 24864]added review-approved label and removed review-pending-level-1 label
mentioned in commit fb5e9635
added sweep:ignore label