Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1135)

Unified Diff: core/src/fpdftext/fpdf_text_int_unittest.cpp

Issue 1800523005: Move core/src/ up to core/. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « core/src/fpdftext/fpdf_text_int.cpp ('k') | core/src/fpdftext/text_int.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/src/fpdftext/fpdf_text_int_unittest.cpp
diff --git a/core/src/fpdftext/fpdf_text_int_unittest.cpp b/core/src/fpdftext/fpdf_text_int_unittest.cpp
deleted file mode 100644
index b482f02be46daf49efc4aa3348b35d3d2107a24b..0000000000000000000000000000000000000000
--- a/core/src/fpdftext/fpdf_text_int_unittest.cpp
+++ /dev/null
@@ -1,56 +0,0 @@
-// Copyright 2015 PDFium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "testing/gtest/include/gtest/gtest.h"
-
-#include "core/src/fpdftext/text_int.h"
-
-// Class to help test functions in CPDF_LinkExtract class.
-class CPDF_TestLinkExtract : public CPDF_LinkExtract {
- private:
- // Add test cases as friends to access protected member functions.
- // Access CheckMailLink.
- FRIEND_TEST(fpdf_text_int, CheckMailLink);
-};
-
-TEST(fpdf_text_int, CheckMailLink) {
- CPDF_TestLinkExtract extractor;
- // Check cases that fail to extract valid mail link.
- const wchar_t* invalid_strs[] = {
- L"",
- L"peter.pan" // '@' is required.
- L"abc@server", // Domain name needs at least one '.'.
- L"abc.@gmail.com", // '.' can not immediately precede '@'.
- L"abc@xyz&q.org", // Domain name should not contain '&'.
- L"abc@.xyz.org", // Domain name should not start with '.'.
- L"fan@g..com" // Domain name should not have consecutive '.'
- };
- for (int i = 0; i < FX_ArraySize(invalid_strs); ++i) {
- CFX_WideString text_str(invalid_strs[i]);
- EXPECT_FALSE(extractor.CheckMailLink(text_str));
- }
-
- // Check cases that can extract valid mail link.
- // An array of {input_string, expected_extracted_email_address}.
- const wchar_t* valid_strs[][2] = {
- {L"peter@abc.d", L"peter@abc.d"},
- {L"red.teddy.b@abc.com", L"red.teddy.b@abc.com"},
- {L"abc_@gmail.com", L"abc_@gmail.com"}, // '_' is ok before '@'.
- {L"dummy-hi@gmail.com",
- L"dummy-hi@gmail.com"}, // '-' is ok in user name.
- {L"a..df@gmail.com", L"df@gmail.com"}, // Stop at consecutive '.'.
- {L".john@yahoo.com", L"john@yahoo.com"}, // Remove heading '.'.
- {L"abc@xyz.org?/", L"abc@xyz.org"}, // Trim ending invalid chars.
- {L"fan{abc@xyz.org", L"abc@xyz.org"}, // Trim beginning invalid chars.
- {L"fan@g.com..", L"fan@g.com"}, // Trim the ending periods.
- {L"CAP.cap@Gmail.Com", L"CAP.cap@Gmail.Com"}, // Keep the original case.
- };
- for (int i = 0; i < FX_ArraySize(valid_strs); ++i) {
- CFX_WideString text_str(valid_strs[i][0]);
- CFX_WideString expected_str(L"mailto:");
- expected_str += valid_strs[i][1];
- EXPECT_TRUE(extractor.CheckMailLink(text_str));
- EXPECT_STREQ(text_str.c_str(), expected_str.c_str());
- }
-}
« no previous file with comments | « core/src/fpdftext/fpdf_text_int.cpp ('k') | core/src/fpdftext/text_int.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698