Merge pull request #808 from m-tmatma/feature/comeback-unittest-IsMai…

…lAddress-by-PR421

PR 421 によって導入されたけど revert された IsMailAddress の単体テストを復活する

tests/unittests/test-is_mailaddress.cpp

@@ -0,0 +1,206 @@
+/*! @file */
+/*
+	Copyright (C) 2018-2019 Sakura Editor Organization
+
+	This software is provided 'as-is', without any express or implied
+	warranty. In no event will the authors be held liable for any damages
+	arising from the use of this software.
+
+	Permission is granted to anyone to use this software for any purpose,
+	including commercial applications, and to alter it and redistribute it
+	freely, subject to the following restrictions:
+
+		1. The origin of this software must not be misrepresented;
+		   you must not claim that you wrote the original software.
+		   If you use this software in a product, an acknowledgment
+		   in the product documentation would be appreciated but is
+		   not required.
+
+		2. Altered source versions must be plainly marked as such,
+		   and must not be misrepresented as being the original software.
+
+		3. This notice may not be removed or altered from any source
+		   distribution.
+*/
+#include <gtest/gtest.h>
+
+#define NOMINMAX
+#include <tchar.h>
+#include <Windows.h>
+#include "parse/CWordParse.h"
+
+// テスト対象関数のヘッダファイル
+//#include "util/string_ex.h" //依存関係が多いのでテスト対象の関数定義のみ抜き出し
+BOOL IsMailAddress(const wchar_t* pszBuf, int nBufLen, int* pnAddressLenfth);
+
+//////////////////////////////////////////////////////////////////////
+// テストマクロ
+
+// 新動作 = PR #421 導入によって変更されるはずだった IsMailAddress()
+// 旧動作 = PR #421 導入前、revert 後の IsMailAddress()
+//
+// PR #421 導入によって IsMailAddress() の単体テストが実装されたが、
+// PR #421 の revert によって消された単体テストを復活する。
+// 将来 IsMailAddress() の仕様を変更する場合はこの単体テストを必要に応じて修正すればよい。
+
+// 新旧動作比較用マクロ1(TRUE, FALSE向け)
+// ASSERT_SAME: 旧実装と新実装で動作が変わらないことを期待
+#define ASSERT_SAME(expected, szTarget, cchTarget, pchMatchedLen) \
+		EXPECT_##expected(IsMailAddress(szTarget, cchTarget, pchMatchedLen))
+
+// 新旧動作比較用マクロ2(TRUE, FALSE向け)
+// ASSERT_CHANGE: 旧実装と新実装で動作が変わることを期待
+#define ASSERT_CHANGE(expected, szTarget, cchTarget, pchMatchedLen) \
+		EXPECT_NE(expected, IsMailAddress(szTarget, cchTarget, pchMatchedLen))
+
+//////////////////////////////////////////////////////////////////////
+// テストコード
+
+TEST(testIsMailAddress, CheckBlank)
+{
+	wchar_t szTest[] = L""; //空文字
+	ASSERT_SAME(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckExample)
+{
+	wchar_t szTest[] = L"[email protected]"; //標準的なサンプルメールアドレス
+	ASSERT_SAME(TRUE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckExampleCoJp)
+{
+	wchar_t szTest[] = L"[email protected]"; //標準的なサンプルメールアドレス
+	ASSERT_SAME(TRUE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckTrailingSpace)
+{
+	wchar_t szTest[] = L"[email protected] "; //標準的なサンプルメールアドレス
+	int mailboxLength;
+	ASSERT_SAME(TRUE, szTest, _countof(szTest) - 1, &mailboxLength);
+	ASSERT_EQ(_countof(szTest) - 2, mailboxLength);
+}
+
+TEST(testIsMailAddress, CheckPunctuation)
+{
+	wchar_t szTest[] = L"test!#$%&'*+-/=?^_`{|}[email protected]"; //記号類を含む
+	ASSERT_SAME(TRUE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckMaxLocalPart)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szTest, _countof(szTest), L"%s%s%s%[email protected]", szSeed, szSeed, szSeed, szSeed); //4個繋げて64文字にする
+	ASSERT_SAME(TRUE, szTest, ::wcslen(szTest), NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckExceedMaxLocalPart)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szTest, _countof(szTest), L"%s%s%s%[email protected]", szSeed, szSeed, szSeed, szSeed); //4個繋げて64文字 + 1
+	ASSERT_CHANGE(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckMaxMailbox)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed64[64 + 1];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szSeed64, _countof(szSeed64), L"%s%s%s%s", szSeed, szSeed, szSeed, szSeed); //4個繋げて64文字にする
+	::swprintf_s(szTest, _countof(szTest), L"%s@%.63s.%.63s.%.58s.com", szSeed64, szSeed64, szSeed64, szSeed64); //最大255文字のチェック
+	int mailboxLength;
+	ASSERT_SAME(TRUE, szTest, _countof(szTest) - 1, &mailboxLength);
+	ASSERT_EQ(255, mailboxLength);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckMaxExceedMailbox)
+{
+	wchar_t szTest[256 + 1];
+	wchar_t szSeed64[64 + 1];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szSeed64, _countof(szSeed64), L"%s%s%s%s", szSeed, szSeed, szSeed, szSeed); //4個繋げて64文字にする
+	::swprintf_s(szTest, _countof(szTest), L"%s@%.63s.%.63s.%.58s0.com", szSeed64, szSeed64, szSeed64, szSeed64); //最大255文字オーバーのチェック
+	ASSERT_CHANGE(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckTooLongDomain)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed64[64 + 1];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szSeed64, _countof(szSeed64), L"%s%s%s%s", szSeed, szSeed, szSeed, szSeed); //4個繋げて64文字にする
+	::swprintf_s(szTest, _countof(szTest), L"%s@%s.com", szSeed64, szSeed64); //63文字を超えるドメイン
+	ASSERT_CHANGE(FALSE, szTest, ::wcslen(szTest), NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckTooShortDomain)
+{
+	wchar_t szTest[] = L"[email protected]"; //ドメイン部は3文字以上
+	ASSERT_CHANGE(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckTooShortCCTLD)
+{
+	wchar_t szTest[] = L"[email protected]"; //CCTLD部は2文字以上
+	ASSERT_CHANGE(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckDomainIncludesUnderScore)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf(szTest, _countof(szTest), L"%s@test_domain.com", szSeed); //_を含むドメイン
+	ASSERT_CHANGE(FALSE, szTest, ::wcslen(szTest), NULL);
+}
+
+TEST(testIsMailAddress, CheckDomainIncludesSingleHyphen)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szTest, _countof(szTest), L"%[email protected]", szSeed); //途中に-を含むドメイン
+	ASSERT_SAME(TRUE, szTest, ::wcslen(szTest), NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckDomainIncludesDoubleHyphen)
+{
+	wchar_t szTest[256];
+	wchar_t szSeed[] = L"0123456789ABCDEF"; // 16文字の素片
+	::swprintf_s(szTest, _countof(szTest), L"%[email protected]", szSeed); //途中に-を含むドメイン
+	ASSERT_CHANGE(FALSE, szTest, ::wcslen(szTest), NULL);
+}
+
+// 動作変更あり。新実装では条件を厳しくして高速化している
+TEST(testIsMailAddress, CheckQuotedLocalPart)
+{
+	wchar_t szTest[] = L"\"test\\@c\"@test.com";
+	ASSERT_CHANGE(TRUE, szTest, _countof(szTest) - 1, NULL);
+}
+
+TEST(testIsMailAddress, CheckBadQuotedLocalPart)
+{
+	wchar_t szTest[] = L"\"[email protected]";
+	ASSERT_SAME(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+// レビューコメントにより追試。動作変えていない部分だが、誤って動作が変わっていた。
+TEST(testIsMailAddress, CheckAwithAtmark)
+{
+	wchar_t szTest[] = L"a@";
+	ASSERT_SAME(FALSE, szTest, _countof(szTest) - 1, NULL);
+}
+
+//////////////////////////////////////////////////////////////////////
+// テストマクロの後始末
+
+#undef ASSERT_SAME
+#undef ASSERT_CHANGE