Skip to content

Bugfix for CA TrigInDetValidation tests using specific preexec_trig settings

Elliot Watton requested to merge ewatton/athena:ewatton_preexec_trig_bugfix into main

This MR enables the tests in the TrigInDetValidation package using CA for the RDOtoRDOTrigger step to use custom preexec_trig settings defined in the test file that are correctly propagated to the Reco_tf command. This was noticed when ITk-specific settings set in the test files were not being propagated to the Reco_tf command used.

This will impact the ITk tests introduced with !68090 (merged) such that the error with the missing supercell container should not appear (discussed in ATR-28606). However, this error should be rectified once new RDOs are available with the supercell container information. This MR will also impact the lowpt tau test using CA introduced with !68486 (merged) such that the correct flags are being setup as defined in the test file.

This change is also included with the non-CA tests, however, this should not impact anything as none of the non-CA tests use a preexec_trig in their test files (instead use preinclude and postinclude files). Also, the non-CA tests will hopefully be removed soon once all the others have been migrated to CA.

Tagging @baines, @tamartin, @smh

Merge request reports