Okay forget the questions about Sunit. I found it. I assumed it was not there because there was no Test menu option in the browser. I can run my tests. TF On Sat, Dec 25, 2010 at 12:19 PM, Tony Fleig <tony.fleig@gmail.com> wrote:
I am confused.
I have some changes to MailMessage (a Pharo class) and WAEmailMessage (a Seaside class) and an extension to MailMessage (the method MailMessage>>seasideEmailMessage:from:to:subject:) that I would like to contribute. These changes allow multi-part MIME messages to be created and sent from a Seaside web applicationfo -- r example messages with an HTML part and a plain text part.
Following the instructions at http://code.google.com/p/pharo/wiki/HowToContribute, I am installing my changes into Pharo 1.2 in preparation for creating a Monticello package for PharoInbox. Here are my problems:
1. One of the steps is "File in your changes and test." I have test classes derived from Sunit. Pharo 1.2 core doesn't seem to include Sunit. Should I test manually and not include the Sunit tests in the package I submit?
2. I assume the MailMessage extension described above belongs in a Seaside update. Is this true?
3. The Seaside changes cannot load without first loading the Pharo changes since the multi-part message is created using Pharo's MailMessage . How is this kind of dependency handled? Should I wait for a Seaside distribution on a version of Pharo that includes my changes before submitting the Seaside package? It seems like this could take a long time.
4. Should I make this a Seaside-only package and extend Pharo's MailMessage in that package?
Any advice will be appreciated.
Cheers, TF