main-unit_test-Identifier

Further expand unit tests for Identifier package classes, adding: HWIdentifier, Identifiable, IdHelper tests. Small adjustments made to existing tests.

This merge request affects 1 package:

• DetectorDescription/Identifier

This merge request affects 8 files:

• DetectorDescription/Identifier/CMakeLists.txt
• DetectorDescription/Identifier/Identifier/IdHelper.h
• DetectorDescription/Identifier/test/HWIdentifier_test.cxx
• DetectorDescription/Identifier/test/IdHelper_test.cxx
• DetectorDescription/Identifier/test/Identifiable_test.cxx
• DetectorDescription/Identifier/test/Identifier32_test.cxx
• DetectorDescription/Identifier/test/IdentifierHash_test.cxx
• DetectorDescription/Identifier/test/Identifier_test.cxx

Adding @apsallid ,@jojungge ,@rbianchi as watchers

added Geometry main review-pending-level-1 labels

CI Result SUCCESS (hash 3df74d8c)

	Athena	AthSimulation	AthGeneration	AthAnalysis
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
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 15174] (remote access info)

Looks good to me, approving.

Cheers, Philipp (L1)

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

mentioned in commit dcbd7ff3

merged