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

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

Issue 2482533002: [Payments] Display warning messages for incomplete shipping address and update editor title (Closed)
Patch Set: refactor function Created 4 years, 1 month 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/PaymentRequestDynamicShippingMultipleAddressesTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java
index 8aba7413a2e7acea029a945f9b69d18493f04fc6..229a1a6e809c9e412f4cd44cd3eede4770233d9d 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestDynamicShippingMultipleAddressesTest.java
@@ -41,14 +41,19 @@ public class PaymentRequestDynamicShippingMultipleAddressesTest extends PaymentR
"Maggie Simpson", "Acme Inc.", "123 Main", "California", "Los Angeles", "",
"90210", "", "Uzbekistan", "555 123-4567", "maggie@simpson.com", ""),
- // Incomplete profile.
+ // Incomplete profile (invalid address).
new AutofillProfile("" /* guid */, "https://www.example.com" /* origin */,
- "Marge Simpson", "Acme Inc.", "123 Main", "California", "Los Angeles", "",
- "90210", "", "US", "", "marge@simpson.com", ""),
+ "Marge Simpson", "Acme Inc.", "123 Main", "California", "", "",
+ "90210", "", "US", "555 123-4567", "marge@simpson.com", ""),
- // Incomplete profile.
+ // Incomplete profile (missing recipient).
new AutofillProfile("" /* guid */, "https://www.example.com" /* origin */,
- "Professor Frink", "Acme Inc.", "123 Main", "California", "", "",
+ "", "Acme Inc.", "123 Main", "California", "Los Angeles", "",
+ "90210", "", "US", "555 123-4567", "lisa@simpson.com", ""),
+
+ // Incomplete profile (need more information).
+ new AutofillProfile("" /* guid */, "https://www.example.com" /* origin */,
+ "", "Acme Inc.", "123 Main", "California", "", "",
"90210", "", "US", "555 123-4567", "lisa@simpson.com", ""),
};
@@ -179,4 +184,28 @@ public class PaymentRequestDynamicShippingMultipleAddressesTest extends PaymentR
R.string.payments_unsupported_shipping_address);
assertEquals(expectedString, actualString);
}
+
+ /**
+ * Make sure the information required message has been displayed for incomplete profile
+ * correctly.
+ */
+ @MediumTest
+ @Feature({"Payments"})
+ public void testShippingAddressEditRequiredMessage()
+ throws InterruptedException, ExecutionException, TimeoutException {
+ // Create four incomplete profiles with different missing information.
+ mProfilesToAdd = new AutofillProfile[] {AUTOFILL_PROFILES[0], AUTOFILL_PROFILES[4],
+ AUTOFILL_PROFILES[5], AUTOFILL_PROFILES[6]};
+ mCountsToSet = new int[] {15, 10, 5, 1};
+ mDatesToSet = new int[] {5000, 5000, 5000, 1};
+
+ triggerUIAndWait(mReadyForInput);
+ clickInShippingSummaryAndWait(R.id.payments_section, mReadyForInput);
+
+ assertEquals(4, getNumberOfShippingAddressSuggestions());
+ assertTrue(getShippingAddressSuggestionLabel(0).contains("Phone number required"));
+ assertTrue(getShippingAddressSuggestionLabel(1).contains("Invalid address"));
+ assertTrue(getShippingAddressSuggestionLabel(2).contains("Recipient required"));
+ assertTrue(getShippingAddressSuggestionLabel(3).contains("More information required"));
+ }
}

Powered by Google App Engine
This is Rietveld 408576698