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

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

Issue 2538543002: [Payments] Add detailed error messages and title for card unmask prompt. (Closed)
Patch Set: Addressed Rouslan's comments Created 4 years 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/PaymentRequestExpiredLocalCardTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestExpiredLocalCardTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestExpiredLocalCardTest.java
index cb18b020d5b57f39805fc2fbaa9e68249d6fd09e..08c11fec2238212cb5831987fcc1e47dd1aca4b1 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestExpiredLocalCardTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestExpiredLocalCardTest.java
@@ -112,4 +112,48 @@ public class PaymentRequestExpiredLocalCardTest extends PaymentRequestTestBase {
clickInCardEditorAndWait(R.id.payments_edit_done_button, mReadyToPay);
}
+
+ /**
+ * Tests the different card unmask error messages for an expired card.
+ */
+ @MediumTest
+ @Feature({"Payments"})
+ public void testPromptErrorMessages()
+ throws InterruptedException, ExecutionException, TimeoutException {
+ // Click pay to get to the card unmask prompt.
+ triggerUIAndWait(mReadyToPay);
+ clickAndWait(R.id.button_primary, mReadyForUnmaskInput);
+
+ // Set valid arguments.
+ setTextInExpiredCardUnmaskDialogAndWait(
+ new int[] {R.id.expiration_month, R.id.expiration_year, R.id.card_unmask_input},
+ new String[] {"10", "26", "123"}, mUnmaskValidationDone);
+ assertTrue(getUnmaskPromptErrorMessage().equals(""));
+
+ // Set an invalid expiration date.
+ setTextInExpiredCardUnmaskDialogAndWait(
+ new int[] {R.id.expiration_month, R.id.expiration_year, R.id.card_unmask_input},
+ new String[] {"10", "14", "123"}, mUnmaskValidationDone);
+ assertTrue(getUnmaskPromptErrorMessage().equals(
+ "Check your expiration date and try again"));
+
+ // Set an invalid CVC and expiration date.
+ setTextInExpiredCardUnmaskDialogAndWait(
+ new int[] {R.id.expiration_month, R.id.expiration_year, R.id.card_unmask_input},
+ new String[] {"10", "14", "12312"}, mUnmaskValidationDone);
+ assertTrue(getUnmaskPromptErrorMessage().equals(
+ "Check your expiration date and CVC and try again"));
+
+ // Set an invalid CVC.
+ setTextInExpiredCardUnmaskDialogAndWait(
+ new int[] {R.id.expiration_month, R.id.expiration_year, R.id.card_unmask_input},
+ new String[] {"10", "26", "12312"}, mUnmaskValidationDone);
+ assertTrue(getUnmaskPromptErrorMessage().equals("Check your CVC and try again"));
+
+ // Set valid arguments again.
+ setTextInExpiredCardUnmaskDialogAndWait(
+ new int[] {R.id.expiration_month, R.id.expiration_year, R.id.card_unmask_input},
+ new String[] {"10", "26", "123"}, mUnmaskValidationDone);
+ assertTrue(getUnmaskPromptErrorMessage().equals(""));
+ }
}

Powered by Google App Engine
This is Rietveld 408576698