Actually, maybe I should explicitly ask before getting too deep into this: Fixing this involved changing a couple of management commands: massemail and massemailtxt.  These commands don't already have any unit tests associated with them, which makes test coverage for this patch look pretty bad.  They're pretty thin wrappers around things that are tested.  Is the expectation that we should be adding tests in cases like these, so we have them going forward, or are these command scripts deliberately left untested?