Skip to content

WIP: FreeAccumulator for unit tests

Rupert Tombs requested to merge rtombs/athena:FreeAccumulator into master

Proposition for a way to clearly allow ComponentAccumulators to be deleted without error.

Create TestUtility.py defining FreeAccumulator. Use FreeAccumulator for SCT_Digitization config test.

This is intended to remove the need for acc.popPrivateTools(); acc.wasMerged() lines in unit tests which create but do not need to merge ComponentAccumulators.

Minor cleanup to SCT_DigitizationConfigNew_test.py.

Q:

Does this give enough clarity?

Is there a better location, such as inside ComponentAccumulator class?

Should acc be returned?

Edited by Rupert Tombs

Merge request reports