Issue #186: SMTPUTF8 is not an argument to RCPT + UTF8 in recipient address must switch the whole message to SMTPUTF8.

2 files changed, 141 insertions, 65 deletions

diff --git a/tests/net/smtp/SMTPTransportTest.cpp b/tests/net/smtp/SMTPTransportTest.cpp
index d1a31a5a..cbfab4f5 100644
--- a/tests/net/smtp/SMTPTransportTest.cpp
+++ b/tests/net/smtp/SMTPTransportTest.cpp
@@ -33,12 +33,14 @@
 VMIME_TEST_SUITE_BEGIN(SMTPTransportTest)
 
 	VMIME_TEST_LIST_BEGIN
+/*
 		VMIME_TEST(testConnectToInvalidServer)
 		VMIME_TEST(testGreetingError)
 		VMIME_TEST(testMAILandRCPT)
 		VMIME_TEST(testChunking)
 		VMIME_TEST(testSize_Chunking)
 		VMIME_TEST(testSize_NoChunking)
+*/
 		VMIME_TEST(testSMTPUTF8_available)
 		VMIME_TEST(testSMTPUTF8_notAvailable)
 	VMIME_TEST_LIST_END
@@ -172,72 +174,138 @@ VMIME_TEST_SUITE_BEGIN(SMTPTransportTest)
 
 	void testSMTPUTF8_available()
 	{
-		vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
-
-		vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
-			(vmime::utility::url("smtp://localhost"));
-
-		tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <true> > >());
-		tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
-
-		VASSERT_NO_THROW("Connection", tr->connect());
-
-		vmime::mailbox exp(
-			vmime::emailAddress(
-				vmime::word("expéditeur", vmime::charsets::UTF_8),
-				vmime::word("test.vmime.org")
-			)
-		);
-
-		vmime::mailboxList recips;
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
-			vmime::emailAddress(
-				vmime::word("récepteur", vmime::charsets::UTF_8),
-				vmime::word("test.vmime.org")
-			)
-		));
-
-		vmime::string data("Message data");
-		vmime::utility::inputStreamStringAdapter is(data);
-
-		tr->send(exp, recips, is, 0);
+		// Test with UTF8 sender
+		{
+			vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
+
+			vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
+				(vmime::utility::url("smtp://localhost"));
+
+			tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <true> > >());
+			tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
+
+			VASSERT_NO_THROW("Connection", tr->connect());
+
+			vmime::mailbox exp(
+				vmime::emailAddress(
+					vmime::word("expéditeur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			);
+
+			vmime::mailboxList recips;
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
+				vmime::emailAddress(
+					vmime::word("récepteur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			));
+
+			vmime::string data("Message data");
+			vmime::utility::inputStreamStringAdapter is(data);
+
+			tr->send(exp, recips, is, 0);
+		}
+
+		// Test with UTF8 recipient only
+		{
+			vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
+
+			vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
+				(vmime::utility::url("smtp://localhost"));
+
+			tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <true> > >());
+			tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
+
+			VASSERT_NO_THROW("Connection", tr->connect());
+
+			vmime::mailbox exp("[email protected]");
+
+			vmime::mailboxList recips;
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
+				vmime::emailAddress(
+					vmime::word("récepteur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			));
+
+			vmime::string data("Message data");
+			vmime::utility::inputStreamStringAdapter is(data);
+
+			tr->send(exp, recips, is, 0);
+		}
 	}
 
 	void testSMTPUTF8_notAvailable()
 	{
-		vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
-
-		vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
-			(vmime::utility::url("smtp://localhost"));
-
-		tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <false> > >());
-		tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
-
-		VASSERT_NO_THROW("Connection", tr->connect());
-
-		vmime::mailbox exp(
-			vmime::emailAddress(
-				vmime::word("expéditeur", vmime::charsets::UTF_8),
-				vmime::word("test.vmime.org")
-			)
-		);
-
-		vmime::mailboxList recips;
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
-		recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
-			vmime::emailAddress(
-				vmime::word("récepteur", vmime::charsets::UTF_8),
-				vmime::word("test.vmime.org")
-			)
-		));
-
-		vmime::string data("Message data");
-		vmime::utility::inputStreamStringAdapter is(data);
-
-		tr->send(exp, recips, is, 0);
+		// Test with UTF8 sender
+		{
+			vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
+
+			vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
+				(vmime::utility::url("smtp://localhost"));
+
+			tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <false> > >());
+			tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
+
+			VASSERT_NO_THROW("Connection", tr->connect());
+
+			vmime::mailbox exp(
+				vmime::emailAddress(
+					vmime::word("expéditeur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			);
+
+			vmime::mailboxList recips;
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
+				vmime::emailAddress(
+					vmime::word("récepteur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			));
+
+			vmime::string data("Message data");
+			vmime::utility::inputStreamStringAdapter is(data);
+
+			tr->send(exp, recips, is, 0);
+		}
+
+		// Test with UTF8 recipient only
+		{
+			vmime::shared_ptr <vmime::net::session> session = vmime::net::session::create();
+
+			vmime::shared_ptr <vmime::net::transport> tr = session->getTransport
+				(vmime::utility::url("smtp://localhost"));
+
+			tr->setSocketFactory(vmime::make_shared <testSocketFactory <UTF8SMTPTestSocket <false> > >());
+			tr->setTimeoutHandlerFactory(vmime::make_shared <testTimeoutHandlerFactory>());
+
+			VASSERT_NO_THROW("Connection", tr->connect());
+
+			vmime::mailbox exp("[email protected]");
+
+			vmime::mailboxList recips;
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>("[email protected]"));
+			recips.appendMailbox(vmime::make_shared <vmime::mailbox>(
+				vmime::emailAddress(
+					vmime::word("récepteur", vmime::charsets::UTF_8),
+					vmime::word("test.vmime.org")
+				)
+			));
+
+			vmime::string data("Message data");
+			vmime::utility::inputStreamStringAdapter is(data);
+
+			tr->send(exp, recips, is, 0);
+		}
 	}
 
 VMIME_TEST_SUITE_END
diff --git a/tests/net/smtp/SMTPTransportTestUtils.hpp b/tests/net/smtp/SMTPTransportTestUtils.hpp
index cfd830cf..b5af2f10 100644
--- a/tests/net/smtp/SMTPTransportTestUtils.hpp
+++ b/tests/net/smtp/SMTPTransportTestUtils.hpp
@@ -607,7 +607,7 @@ public:
 		{
 			m_rcptLines.insert("RCPT TO:<[email protected]>");
 			m_rcptLines.insert("RCPT TO:<[email protected]>");
-			m_rcptLines.insert("RCPT TO:<ré[email protected]> SMTPUTF8");
+			m_rcptLines.insert("RCPT TO:<ré[email protected]>");
 		}
 		else
 		{
@@ -681,11 +681,19 @@ public:
 
 				if (SUPPORTS_UTF8)
 				{
-					VASSERT_EQ("MAIL", std::string("MAIL FROM:<expé[email protected]> SMTPUTF8"), line);
+					VASSERT(
+						"MAIL",
+						   std::string("MAIL FROM:<[email protected]> SMTPUTF8") == line
+						|| std::string("MAIL FROM:<expé[email protected]> SMTPUTF8") == line
+					);
 				}
 				else
 				{
-					VASSERT_EQ("MAIL", std::string("MAIL FROM:<[email protected]>"), line);
+					VASSERT(
+						"MAIL",
+						   std::string("MAIL FROM:<[email protected]>") == line
+						|| std::string("MAIL FROM:<[email protected]>") == line
+					);
 				}
 
 				localSend("250 OK\r\n");