search

jmarnold / EmbeddedMail

Integration testing against e-mail almost always sucks. EmbeddedMail doesn't remove all of that pain, but it certainly helps it suck less by providing an in-memory SMTP server that you can assert on.
http://nuget.org/packages/EmbeddedMail
Other
25 stars 4 forks source link

MessageParser always expected there to be a CC header #1

Closed dmeenhuis closed 12 years ago

dmeenhuis commented 12 years ago

When attempting to send a message without CC recipients to the embedded mail server, the message parser would throw an exception that the given key ("Cc") was not present in the dictionary.

Fixed it by throwing in a quick ContainsKey check around the parsing code and included a test for it as well.

jmarnold commented 12 years ago

Thanks for catching this!

dmeenhuis commented 12 years ago

No problem! I'm just working on a project where I need to do some integration tests regarding e-mail and this library is very helpful in that regard, so I'm glad I could contribute something :)

jmarnold commented 12 years ago

I'll get a nuget update out shortly

dmeenhuis commented 12 years ago

Awesome, much appreciated! Gotta love github :)

jmarnold commented 12 years ago

It's now setup in CI with auto-nuget-publishing and the new nuget is out.

dmeenhuis commented 12 years ago

Thanks for getting the update out so quickly :)