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

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

Issue 2841643002: When a new Payments address is created from the editor, use it as the chosen address (Closed)
Patch Set: Self CR Created 3 years, 8 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 1d5430fb760f785d95a7e66a48ef62f8f12cabff..5ba3ecd420d875d9c9bc308adeac7522f34b4b42 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
@@ -211,6 +211,11 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
EnteringValidDataWithDefaultCountry) {
Mathieu 2017/04/24 20:05:19 Could we have a test similar to https://cs.chromiu
MAD 2017/04/25 14:23:49 As discussed, will do this in another CL.
InvokePaymentRequestUI();
+ // No shipping profiles are available.
+ PaymentRequest* request = GetPaymentRequests(GetActiveWebContents()).front();
+ EXPECT_EQ(0U, request->state()->shipping_profiles().size());
+ EXPECT_EQ(nullptr, request->state()->selected_shipping_profile());
+
SetDefaultCountryData();
EnableAddressInputOverride();
@@ -242,6 +247,11 @@ IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,
EXPECT_EQ(base::ASCIIToUTF16(country_code),
profile->GetRawInfo(autofill::ADDRESS_HOME_COUNTRY));
ExpectExistingRequiredFields(nullptr);
+
+ // One shipping profile is available and selected.
+ EXPECT_EQ(1U, request->state()->shipping_profiles().size());
+ EXPECT_EQ(request->state()->shipping_profiles().back(),
+ request->state()->selected_shipping_profile());
}
IN_PROC_BROWSER_TEST_F(PaymentRequestShippingAddressEditorTest,

Powered by Google App Engine
This is Rietveld 408576698