MdtCondDBData: Pass the MdtIdHelper

Hi,

the MdtCondDBData has a contra-intuitive interface, in particular, the isGood method to check whether the identifier is good or not. That's just checking if the identifier can be found in one of the various data-member vectors. The main reason for the short-coming is that the class cannot transform between tubeId, chamberId and multiLayerId due to the missing MdtIdHelper. That data member is now added during the construction phase of the class

Adding: @nkoehler, @cheidegg

This merge request affects 2 packages:

• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg
• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondData

Affected files list will not be printed in this case

Adding @rosati ,@wleight ,@jojungge as watchers

added MuonSpectrometer master review-pending-level-1 labels

CI Result FAILURE (hash ad9196ca)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30105]

Tests failed due to infrastructure problems. Let's give it another go

Jenkins please retry a build

This merge request affects 2 packages:

• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondAlg
• MuonSpectrometer/MuonConditions/MuonCondGeneral/MuonCondData

Affected files list will not be printed in this case

Adding @rosati ,@wleight ,@jojungge as watchers

CI Result SUCCESS (hash ad9196ca)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 30121]

Changes look fine, tests pass. L1 approving.

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

mentioned in commit e2a7c662

merged

added sweep:ignore label