1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
|
// Copyright 2023 The Chromium Authors
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
#include "chrome/browser/ui/profiles/profile_customization_util.h"
#include <string>
#include "base/run_loop.h"
#include "base/strings/strcat.h"
#include "base/strings/utf_string_conversions.h"
#include "base/test/task_environment.h"
#include "base/test/test_future.h"
#include "components/signin/public/identity_manager/identity_test_environment.h"
#include "components/signin/public/identity_manager/signin_constants.h"
#include "google_apis/gaia/gaia_id.h"
#include "testing/gtest/include/gtest/gtest.h"
using signin::constants::kNoHostedDomainFound;
namespace {
AccountInfo FillAccountInfo(
const CoreAccountInfo& core_info,
const std::string& given_name,
const std::string& hosted_domain = kNoHostedDomainFound) {
AccountInfo account_info;
account_info.email = core_info.email;
account_info.gaia = core_info.gaia;
account_info.account_id = core_info.account_id;
account_info.is_under_advanced_protection =
core_info.is_under_advanced_protection;
account_info.full_name = base::StrCat({given_name, " Doe"});
account_info.given_name = given_name;
account_info.hosted_domain = hosted_domain;
account_info.locale = "en";
account_info.picture_url = base::StrCat({"https://picture.url/", given_name});
return account_info;
}
} // namespace
class ProfileNameResolverTest : public testing::Test {
protected:
const std::string kTestGivenName = "Jane";
const std::string kTestEmail = "jane@bar.baz";
const GaiaId kTestGaiaId = GaiaId("123456");
signin::IdentityTestEnvironment* identity_test_env() {
return &identity_test_env_;
}
private:
base::test::SingleThreadTaskEnvironment task_environment_;
signin::IdentityTestEnvironment identity_test_env_;
};
TEST_F(ProfileNameResolverTest, RunWithProfileName) {
CoreAccountInfo core_account_info =
identity_test_env()->MakeAccountAvailable(kTestEmail);
base::test::TestFuture<std::u16string> profile_name_future;
base::test::TestFuture<std::u16string> profile_name_future_2;
ProfileNameResolver resolver{identity_test_env()->identity_manager(),
core_account_info};
// `RunWithProfileName` should not run the callback as no profile name is
// available.
resolver.RunWithProfileName(profile_name_future.GetCallback());
base::RunLoop().RunUntilIdle();
EXPECT_FALSE(profile_name_future.IsReady());
// Simulate the account info being updated, should result in the callback
// getting called.
resolver.OnExtendedAccountInfoUpdated(
FillAccountInfo(core_account_info, kTestGivenName));
EXPECT_EQ(base::ASCIIToUTF16(kTestGivenName), profile_name_future.Get());
// Calling `RunWithProfileName` again should make it get called right away,
// synchronously.
resolver.RunWithProfileName(profile_name_future_2.GetCallback());
EXPECT_TRUE(profile_name_future_2.IsReady());
EXPECT_EQ(base::ASCIIToUTF16(kTestGivenName), profile_name_future_2.Get());
}
TEST_F(ProfileNameResolverTest, RunWithProfileName_InfoAvailable) {
CoreAccountInfo core_account_info =
identity_test_env()->MakeAccountAvailable(kTestEmail);
identity_test_env()->UpdateAccountInfoForAccount(
FillAccountInfo(core_account_info, kTestGivenName));
base::test::TestFuture<std::u16string> profile_name_future;
ProfileNameResolver resolver{identity_test_env()->identity_manager(),
core_account_info};
// The information is available, the callback should run right away.
resolver.RunWithProfileName(profile_name_future.GetCallback());
EXPECT_TRUE(profile_name_future.IsReady());
EXPECT_EQ(base::ASCIIToUTF16(kTestGivenName), profile_name_future.Get());
}
TEST_F(ProfileNameResolverTest, RunWithProfileName_InfoUnvailable) {
auto scoped_timeout_override =
ProfileNameResolver::CreateScopedInfoFetchTimeoutOverrideForTesting(
base::TimeDelta());
base::test::TestFuture<std::u16string> profile_name_future;
// The account is not known from the identity manager.
CoreAccountInfo core_account_info;
core_account_info.gaia = kTestGaiaId;
core_account_info.account_id =
CoreAccountId::FromGaiaId(core_account_info.gaia);
core_account_info.email = kTestEmail;
ProfileNameResolver resolver{identity_test_env()->identity_manager(),
core_account_info};
resolver.RunWithProfileName(profile_name_future.GetCallback());
// Simulate the account info being updated.
resolver.OnExtendedAccountInfoUpdated(
FillAccountInfo(core_account_info, kTestGivenName));
EXPECT_TRUE(profile_name_future.IsReady());
EXPECT_EQ(base::ASCIIToUTF16(kTestGivenName), profile_name_future.Get());
}
// Regression test for https://crbug.com/1481902
TEST_F(ProfileNameResolverTest, RunWithProfileName_InfoMissing) {
auto scoped_timeout_override =
ProfileNameResolver::CreateScopedInfoFetchTimeoutOverrideForTesting(
base::TimeDelta());
base::test::TestFuture<std::u16string> profile_name_future;
// The account is not known from the identity manager.
CoreAccountInfo core_account_info;
core_account_info.gaia = kTestGaiaId;
core_account_info.account_id =
CoreAccountId::FromGaiaId(core_account_info.gaia);
core_account_info.email = kTestEmail;
ProfileNameResolver resolver{identity_test_env()->identity_manager(),
core_account_info};
resolver.RunWithProfileName(profile_name_future.GetCallback());
// Simulate the account info being updated with invalid info. Should
// result in the callback not getting called, and waiting until the
// timeout to get a fallback value as name.
resolver.OnExtendedAccountInfoUpdated(AccountInfo());
EXPECT_FALSE(profile_name_future.IsReady());
EXPECT_EQ(base::ASCIIToUTF16(kTestEmail), profile_name_future.Get());
}
|
