I found myself messing around with pinax again this week and decided
to resubmit the simple text+html support that I hashed out back in
October. This time I managed to add a few very simple tests. It is
committed on github:
http://github.com/istruble/django-mailer/commit/6bb3601fcbc225aad5a7c694ad88996eb91b6013
I actually addressed issues #2 and #4 on github. They started in two
separate branches but unfortunately I merged one back into the other
before I added tests today. The tests are very basic but do exercise
the text+html stuff. People can add more tests when they touch an
untested piece of the code. Are their any suggestions for better ways
to go about adding tests to this project or is this a good start?
http://github.com/jtauber/django-mailer/issues#issue/2
http://github.com/jtauber/django-mailer/issues#issue/4
I am not sure if there is a feature freeze in place right now as I
gather their is for pinax. Neither issue is critical. Issue #2, use
single SMTP connection for sending e-mails, might not be considered a
feature change. If there is a feature freeze in place, it can still
be considered for 0.1.0.
The way I addressed issue #4 is nearly identical to the way that
FlashingPumpkin addressed it but we are both adding the text+html
functionality. It should not be considered for 0.1.0 if there is a
feature freeze in place.
Ian