Test cases

Agreed, this would be nice. Will probably need support though.

I don't imagine test cases will be much help if the testbot can't actually retrieve any mail... #1038922: Verify basic mailbox operations in simpletest testcase.

This is now theoretically possible, following: #275926: Message retrieval library plugins (in order to avoid PHP IMAP dependency)

We'd have to write a 'test' retrieval library that just supplies dummy text.

Another good test would be automatic coder review.

http://drupalcode.org/project/mailhandler.git/commit/3d4fd6f
http://drupalcode.org/project/mailhandler.git/commit/4385164

I guess this isn't quite done... 7.x-2.x works, but 6.x-2.x still has issues. I can't figure out why.

The errors in 6.x-2.x might be related to #923746: Notices given from FeedsProcessor::map and FeedsBatch::getTotal

Nope, turns out it's this: #1344316: Use recommended release rather than just sorting

Additionally, more test cases need to be written to verify that importing messages works.

Try to add a test case to make sure messages get purged: #1363372: Authenticated messages not being purged

I updated the summary to take care of suggestions to this point.

Updated summary

Oh man... I just spent so long discovering terrible things about imap_open() and local folders.

Okay, so apparently the library underlying imap_open() has changed somewhat recently- it used to allow you to pass any absolute file path to an mbox file, but due to security reasons it now takes paths relative to the home directory of the web server user (www-data, so /var/www, for instance).

Thanks to this post for the info: http://www.webmasterworld.com/php/3738080.htm

Obviously the docs need to be updated to reflect this. I don't know if it can be worked around for testing purposes.

a

Finally got test imports from local folders working: http://drupalcode.org/project/mailhandler.git/commit/619a66e

More tests can be added later.

http://drupalcode.org/project/mailhandler.git/commit/219425b