Commit 6cace0c5 authored by Parastoo Geranmayeh's avatar Parastoo Geranmayeh Committed by Commit Bot

[AF] add then remove a profile/test.

To show that this case is not currently supported.

Bug: 909730
Change-Id: Iaca51858fa9987ef95cd9c4341386472dc55cb57
Reviewed-on: https://chromium-review.googlesource.com/c/1352507Reviewed-by: default avatarSebastien Seguin-Gagnon <sebsg@chromium.org>
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#611926}
parent 55420a27
......@@ -493,6 +493,19 @@ TEST_F(PersonalDataManagerTest, AddProfile) {
ExpectSameElements(profiles, personal_data_->GetProfiles());
}
// TODO(crbug.com/909730): If you add a profile and then remove it right away,
// the profile will not be removed, but it should.
TEST_F(PersonalDataManagerTest, AddRemoveProfile) {
AutofillProfile profile(test::GetFullProfile());
personal_data_->AddProfile(profile);
personal_data_->RemoveByGUID(profile.guid());
WaitForOnPersonalDataChanged();
auto profiles = personal_data_->GetProfiles();
ASSERT_EQ(1U, profiles.size()); // the correct size is 0.
}
// Test that a new profile has its basic information set.
TEST_F(PersonalDataManagerTest, AddProfile_BasicInformation) {
// Create the test clock and set the time to a specific value.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment