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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java

Issue 2680143002: Use dropdown list for admin areas in pr form. (Closed)
Patch Set: Touch ups Created 3 years, 10 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/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java
index 0ee19c748b56e51073f69f81fbaa66119d034112..71dd69708da70a0a4ba8e2679cd84ca8a144a69d 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBillingAddressTest.java
@@ -251,8 +251,12 @@ public class PaymentRequestBillingAddressTest extends PaymentRequestTestBase {
// Add a new billing address.
setSpinnerSelectionsInCardEditorAndWait(
new int[] {DECEMBER, NEXT_YEAR, ADD_BILLING_ADDRESS}, mReadyToEdit);
+
sebsg 2017/02/28 16:13:59 Nit: remove new-line
Parastoo 2017/03/21 14:30:44 Done.
setTextInEditorAndWait(new String[] {"Seb Doe", "Google", "340 Main St", "Los Angeles",
- "CA", "90291", "999-999-9999"}, mEditorTextUpdate);
+ "90291", "999-999-9999"},
+ mEditorTextUpdate);
+ setSpinnerSelectionsInEditorAndWait(new int[] {US, CA}, mEditorTextUpdate);
+
clickInEditorAndWait(R.id.payments_edit_done_button, mReadyToEdit);
// There should be 9 suggestions, the 7 initial addresses, the newly added address and the
@@ -288,7 +292,10 @@ public class PaymentRequestBillingAddressTest extends PaymentRequestTestBase {
clickInShippingSummaryAndWait(R.id.payments_section, mReadyForInput);
clickInShippingAddressAndWait(R.id.payments_add_option_button, mReadyToEdit);
setTextInEditorAndWait(new String[] {"Seb Doe", "Google", "340 Main St", "Los Angeles",
- "CA", "90291", "999-999-9999"}, mEditorTextUpdate);
+ "90291", "999-999-9999"},
+ mEditorTextUpdate);
+ setSpinnerSelectionsInEditorAndWait(new int[] {US, CA}, mEditorTextUpdate);
+
clickInEditorAndWait(R.id.payments_edit_done_button, mReadyToPay);
// Navigate to the card editor UI.
@@ -330,8 +337,11 @@ public class PaymentRequestBillingAddressTest extends PaymentRequestTestBase {
// Selects the fourth billing addresss that misses phone number brings up the address
// editor.
setSpinnerSelectionsInCardEditorAndWait(new int[] {DECEMBER, NEXT_YEAR, 3}, mReadyToEdit);
+
sebsg 2017/02/28 16:13:59 Nit: Remove new-line
Parastoo 2017/03/21 14:30:44 Done.
setTextInEditorAndWait(new String[] {"Lisa Doe", "Google", "340 Main St", "Los Angeles",
- "CA", "90291", "999-999-9999"}, mEditorTextUpdate);
+ "90291", "999-999-9999"},
+ mEditorTextUpdate);
+
clickInEditorAndWait(R.id.payments_edit_done_button, mReadyToEdit);
// The newly completed address must be selected and put at the top of the dropdown.

Powered by Google App Engine
This is Rietveld 408576698