Initialise transactionTemplate if needed

cBioPortal / cbioportal-core

Externalized cBioPortal Core

2 stars 15 forks source link

Initialise transactionTemplate if needed #55

Closed forus closed 3 months ago

forus commented 3 months ago

JdbcUtil.getTransactionTemplate() returned null causing null pointer exception for all commands that used transaction programatically:

RemoveSamples
RemovePatients
ImportCnaDiscreteLongData
ImportTabDelimData

forus commented 3 months ago

I think it would be good to include a test covering getTransactionTemplate in this case. Apparently the existing set of tests were not sufficient for catching this in CI 🤔

@pieterlukasse this case is covered by tests in cbiportal https://github.com/cBioPortal/cbioportal-core/issues/36