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

Unified Diff: chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc

Issue 2951743002: Merge-60[Payments] Make state combobox def. value work with state abbrv. (Closed)
Patch Set: Created 3 years, 6 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/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
diff --git a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
index 35ea784c527b21a3197267ae6ea8427e845b473f..c5db770bab4dcb430340eab090974af56e5ffe6d 100644
--- a/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
+++ b/chrome/browser/ui/views/payments/shipping_address_editor_view_controller_browsertest.cc
@@ -967,4 +967,66 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
EXPECT_TRUE(save_button->enabled());
}
+// Tests that the state dropdown is set to the right value if the value from the
+// profile is a region code.
+IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
+ DefaultRegion_RegionCode) {
+ // Add address without a country but a valid state for the default country.
+ autofill::AutofillProfile profile = autofill::test::GetFullProfile();
+ profile.SetInfo(autofill::AutofillType(autofill::ADDRESS_HOME_COUNTRY),
+ base::ASCIIToUTF16(""), kLocale);
+ profile.SetInfo(autofill::AutofillType(autofill::ADDRESS_HOME_STATE),
+ base::ASCIIToUTF16("ca"), kLocale);
+ AddAutofillProfile(profile);
+
+ InvokePaymentRequestUI();
+ SetRegionDataLoader(&test_region_data_loader_);
+
+ test_region_data_loader_.set_synchronous_callback(true);
+ std::vector<std::pair<std::string, std::string>> regions;
+ regions.push_back(std::make_pair("AL", "Alabama"));
+ regions.push_back(std::make_pair("CA", "California"));
+ test_region_data_loader_.SetRegionData(regions);
+ OpenShippingAddressSectionScreen();
+
+ ResetEventObserver(DialogEvent::SHIPPING_ADDRESS_EDITOR_OPENED);
+ ClickOnChildInListViewAndWait(/*child_index=*/0, /*num_children=*/1,
+ DialogViewID::SHIPPING_ADDRESS_SHEET_LIST_VIEW);
+
+ // Expect that the state was selected.
+ EXPECT_EQ(base::ASCIIToUTF16("California"),
+ GetComboboxValue(autofill::ADDRESS_HOME_STATE));
+}
+
+// Tests that the state dropdown is set to the right value if the value from the
+// profile is a region name.
+IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
+ DefaultRegion_RegionName) {
+ // Add address without a country but a valid state for the default country.
+ autofill::AutofillProfile profile = autofill::test::GetFullProfile();
+ profile.SetInfo(autofill::AutofillType(autofill::ADDRESS_HOME_COUNTRY),
+ base::ASCIIToUTF16(""), kLocale);
+ profile.SetInfo(autofill::AutofillType(autofill::ADDRESS_HOME_STATE),
+ base::ASCIIToUTF16("california"), kLocale);
+ AddAutofillProfile(profile);
+
+ InvokePaymentRequestUI();
+ SetRegionDataLoader(&test_region_data_loader_);
+
+ test_region_data_loader_.set_synchronous_callback(true);
+ std::vector<std::pair<std::string, std::string>> regions;
+ regions.push_back(std::make_pair("AL", "Alabama"));
+ regions.push_back(std::make_pair("CA", "California"));
+ test_region_data_loader_.SetRegionData(regions);
+ OpenShippingAddressSectionScreen();
+
+ ResetEventObserver(DialogEvent::SHIPPING_ADDRESS_EDITOR_OPENED);
+ ClickOnChildInListViewAndWait(/*child_index=*/0, /*num_children=*/1,
+ DialogViewID::SHIPPING_ADDRESS_SHEET_LIST_VIEW);
+
+ // Expect that the state was selected.
+ EXPECT_EQ(base::ASCIIToUTF16("California"),
+ GetComboboxValue(autofill::ADDRESS_HOME_STATE));
+}
+
} // namespace payments

Powered by Google App Engine
This is Rietveld 408576698