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

Side by Side Diff: chrome/browser/metrics/chromeos_metrics_provider_unittest.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Rebased. Created 5 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/metrics/chromeos_metrics_provider.h" 5 #include "chrome/browser/metrics/chromeos_metrics_provider.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h" 10 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 FakeBluetoothAdapterClient* fake_bluetooth_adapter_client_; 108 FakeBluetoothAdapterClient* fake_bluetooth_adapter_client_;
109 FakeBluetoothDeviceClient* fake_bluetooth_device_client_; 109 FakeBluetoothDeviceClient* fake_bluetooth_device_client_;
110 110
111 private: 111 private:
112 content::TestBrowserThreadBundle thread_bundle_; 112 content::TestBrowserThreadBundle thread_bundle_;
113 113
114 DISALLOW_COPY_AND_ASSIGN(ChromeOSMetricsProviderTest); 114 DISALLOW_COPY_AND_ASSIGN(ChromeOSMetricsProviderTest);
115 }; 115 };
116 116
117 TEST_F(ChromeOSMetricsProviderTest, MultiProfileUserCount) { 117 TEST_F(ChromeOSMetricsProviderTest, MultiProfileUserCount) {
118 std::string user1("user1@example.com"); 118 const AccountId user1(AccountId::FromUserEmail("user1@example.com"));
119 std::string user2("user2@example.com"); 119 const AccountId user2(AccountId::FromUserEmail("user2@example.com"));
120 std::string user3("user3@example.com"); 120 const AccountId user3(AccountId::FromUserEmail("user3@example.com"));
121 121
122 // |scoped_enabler| takes over the lifetime of |user_manager|. 122 // |scoped_enabler| takes over the lifetime of |user_manager|.
123 chromeos::FakeChromeUserManager* user_manager = 123 chromeos::FakeChromeUserManager* user_manager =
124 new chromeos::FakeChromeUserManager(); 124 new chromeos::FakeChromeUserManager();
125 chromeos::ScopedUserManagerEnabler scoped_enabler(user_manager); 125 chromeos::ScopedUserManagerEnabler scoped_enabler(user_manager);
126 user_manager->AddKioskAppUser(user1); 126 user_manager->AddKioskAppUser(user1);
127 user_manager->AddKioskAppUser(user2); 127 user_manager->AddKioskAppUser(user2);
128 user_manager->AddKioskAppUser(user3); 128 user_manager->AddKioskAppUser(user3);
129 129
130 user_manager->LoginUser(user1); 130 user_manager->LoginUser(user1);
131 user_manager->LoginUser(user3); 131 user_manager->LoginUser(user3);
132 132
133 ChromeOSMetricsProvider provider; 133 ChromeOSMetricsProvider provider;
134 provider.OnDidCreateMetricsLog(); 134 provider.OnDidCreateMetricsLog();
135 metrics::SystemProfileProto system_profile; 135 metrics::SystemProfileProto system_profile;
136 provider.ProvideSystemProfileMetrics(&system_profile); 136 provider.ProvideSystemProfileMetrics(&system_profile);
137 EXPECT_EQ(2u, system_profile.multi_profile_user_count()); 137 EXPECT_EQ(2u, system_profile.multi_profile_user_count());
138 } 138 }
139 139
140 TEST_F(ChromeOSMetricsProviderTest, MultiProfileCountInvalidated) { 140 TEST_F(ChromeOSMetricsProviderTest, MultiProfileCountInvalidated) {
141 std::string user1("user1@example.com"); 141 const AccountId user1(AccountId::FromUserEmail("user1@example.com"));
142 std::string user2("user2@example.com"); 142 const AccountId user2(AccountId::FromUserEmail("user2@example.com"));
143 std::string user3("user3@example.com"); 143 const AccountId user3(AccountId::FromUserEmail("user3@example.com"));
144 144
145 // |scoped_enabler| takes over the lifetime of |user_manager|. 145 // |scoped_enabler| takes over the lifetime of |user_manager|.
146 chromeos::FakeChromeUserManager* user_manager = 146 chromeos::FakeChromeUserManager* user_manager =
147 new chromeos::FakeChromeUserManager(); 147 new chromeos::FakeChromeUserManager();
148 chromeos::ScopedUserManagerEnabler scoped_enabler(user_manager); 148 chromeos::ScopedUserManagerEnabler scoped_enabler(user_manager);
149 user_manager->AddKioskAppUser(user1); 149 user_manager->AddKioskAppUser(user1);
150 user_manager->AddKioskAppUser(user2); 150 user_manager->AddKioskAppUser(user2);
151 user_manager->AddKioskAppUser(user3); 151 user_manager->AddKioskAppUser(user3);
152 152
153 user_manager->LoginUser(user1); 153 user_manager->LoginUser(user1);
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
257 EXPECT_EQ(0x030DU, device1.product_id()); 257 EXPECT_EQ(0x030DU, device1.product_id());
258 EXPECT_EQ(0x0306U, device1.device_id()); 258 EXPECT_EQ(0x0306U, device1.device_id());
259 259
260 // The Confirm Passkey object, this has no Device ID information. 260 // The Confirm Passkey object, this has no Device ID information.
261 EXPECT_EQ(FakeBluetoothDeviceClient::kConfirmPasskeyClass, 261 EXPECT_EQ(FakeBluetoothDeviceClient::kConfirmPasskeyClass,
262 device2.bluetooth_class()); 262 device2.bluetooth_class());
263 EXPECT_EQ(PairedDevice::DEVICE_PHONE, device2.type()); 263 EXPECT_EQ(PairedDevice::DEVICE_PHONE, device2.type());
264 EXPECT_EQ(0x207D74U, device2.vendor_prefix()); 264 EXPECT_EQ(0x207D74U, device2.vendor_prefix());
265 EXPECT_EQ(PairedDevice::VENDOR_ID_UNKNOWN, device2.vendor_id_source()); 265 EXPECT_EQ(PairedDevice::VENDOR_ID_UNKNOWN, device2.vendor_id_source());
266 } 266 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698