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

Unified Diff: chrome/browser/chromeos/policy/system_log_uploader_unittest.cc

Issue 1610123003: Added anonymizer to SystemLogUploader. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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
Index: chrome/browser/chromeos/policy/system_log_uploader_unittest.cc
diff --git a/chrome/browser/chromeos/policy/system_log_uploader_unittest.cc b/chrome/browser/chromeos/policy/system_log_uploader_unittest.cc
index e30311cb60c1bae8e3bf17caab2bd36b3cf7d610..9b2c267ab43086a8428abb1ca0a926de56205477 100644
--- a/chrome/browser/chromeos/policy/system_log_uploader_unittest.cc
+++ b/chrome/browser/chromeos/policy/system_log_uploader_unittest.cc
@@ -286,25 +286,4 @@ TEST_F(SystemLogUploaderTest, DisableLogUpload) {
SystemLogUploader::kDefaultUploadDelayMs));
}
-// Test RemovePII function.
-TEST_F(SystemLogUploaderTest, TestPII) {
Thiemo Nagel 2016/01/21 16:54:35 I'd prefer to keep that test to prove that the alg
Polina Bondarenko 2016/01/21 17:40:06 Done.
- std::string data =
- "aaaaaaaaSSID=123aaaaaaaaaaa\n" // SSID.
- "aaaaaaaahttp://tets.comaaaaaaa\n" // URL.
- "aaaaaemail@example.comaaa\n" // Email address.
- "example@@1234\n" // No PII, it is not valid email address.
- "255.255.355.255\n" // No PII, it is not valid IP address format.
- "aaaa123.123.45.4aaa\n" // IP address.
- "11:11;11::11\n" // IP address.
- "11::11\n" // IP address.
- "11:11:abcdef:0:0:0:0:0"; // No PII, it is not valid IP address format.
-
- std::string result =
- "example@@1234\n"
- "255.255.355.255\n"
- "11:11:abcdef:0:0:0:0:0\n";
-
- EXPECT_EQ(result, SystemLogUploader::RemoveSensitiveData(data));
-}
-
} // namespace policy

Powered by Google App Engine
This is Rietveld 408576698