Cleanup of MuonIdHelpers, fix mm/stgc/csc warnings (ATLASRECTS-4494)

Hi,

this MR is a cleanup of MuonIdHelpers and AtlasDetectorIDHelper. By passing the MessageSvc from the muon IdHelpers to AtlasDetectorID, also the warnings for non-present mm/stgc/csc depending on the chosen ATLAS layouts are gone. Fixes ATLASRECTS-4494. Adding full-unit-tests since it will likely change log files if warnings drop out.

Best, Nico

added full-unit-tests label

WARNING: big files (>100K) are found in the changeset

192K in file MuonSpectrometer/MuonIdHelpers/share/muon_id_test.ref

This merge request affects 2 packages:

• DetectorDescription/AtlasDetDescr
• MuonSpectrometer/MuonIdHelpers

Affected files list will not be printed in this case

Adding @rosati ,@wleight ,@nkoehler ,@rbianchi as watchers

added Geometry MuonSpectrometer master review-pending-level-1 labels

CI Result FAILURE (hash 0c626881)

	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 25122]

added 1 commit

• 574b3322 - update unit test references

Compare with previous version

WARNING: big files (>100K) are found in the changeset

192K in file MuonSpectrometer/MuonIdHelpers/share/muon_id_test.ref

This merge request affects 5 packages:

• DetectorDescription/AtlasDetDescr
• MuonSpectrometer/MuonCnv/MuonEventAthenaPool
• MuonSpectrometer/MuonCnv/MuonEventTPCnv
• MuonSpectrometer/MuonIdHelpers
• Trigger/TrigSteer/DecisionHandling

Affected files list will not be printed in this case

Adding @tamartin ,@rosati ,@wleight ,@nkoehler ,@rbianchi as watchers

added Trigger label

CI Result SUCCESS (hash 574b3322)

	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 25135]

Do not see anything problematic here, approving.

Pavol [as L2 MR shifter]

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit f6f27c72

added sweep:ignore label