fix LArConditions2Ntuple.py, remove its legacy counterpart

• Fix sqlite reading in LArConditions2Ntuple.py
• Add a test for this script
• Remove LArCommConditions2Ntuple.py (fun-fact: 'Comm' stands for commissioning, as opposed to testbeam)

This merge request affects 1 package:

• LArCalorimeter/LArCalibTools

This merge request affects 3 files:

• LArCalorimeter/LArCalibTools/CMakeLists.txt
• LArCalorimeter/LArCalibTools/share/LArCommConditions2Ntuple.py
• LArCalorimeter/LArCalibTools/share/LArConditions2Ntuple.py

Adding @pavol as watcher

added 24.0 LAr review-pending-level-1 labels

CI Result SUCCESS (hash c46acaa5)

	Athena
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
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 5419] (remote access info)

Looks good to me. L1

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

added review-approved-point1 label

added review-approved-tier0 label

merged

mentioned in commit 5e2a1336

mentioned in merge request !69320 (merged)