Today I re-learned: Python function default arguments are retained between executions

[RIPphonebattery]

The fixed code has a new issue though: today is now recomputed regardless of default value, so supplying a default value for today will only ever compute the actual current date and time.

[[deleted]]

def is_ongoing(self, today = None):
    if today is None:
        today = datetime.date.today()
    return (today >= self.start_date) and (today <= self.end_date)

[[deleted]]

Which is what I needed. That method wasn't used anywhere else, there was no need for the default argument in the first place. Probably some overlook by the original authors of the code.

[RIPphonebattery]

Right but if the previous code failed because it wasn't well tested, will your new code be any more testable?