Is there a reason for not making this `deleteOldFiles` directly? I don't see another use case for getting old files other than deleting them. Is this because we need it for testing?