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

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

Issue 2711263003: Don't cache canMakePayment results. (Closed)
Patch Set: Addressed comments 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
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBasicCardTest.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestPaymentAppCanMakePaymentQueryTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestPaymentAppCanMakePaymentQueryTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestPaymentAppCanMakePaymentQueryTest.java
index 46e28285e6a08d4504602265041b7bec38600f32..fd3b250288e33b9b27a5475b8ee7f6d2e0f5acec 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestPaymentAppCanMakePaymentQueryTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestPaymentAppCanMakePaymentQueryTest.java
@@ -28,10 +28,23 @@ public class PaymentRequestPaymentAppCanMakePaymentQueryTest extends PaymentRequ
public void testNoBobPayInstalled() throws InterruptedException, ExecutionException,
TimeoutException {
openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, false"});
+ expectResultContains(new String[] {"false, false"});
clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, QuotaExceededError"});
+ expectResultContains(new String[] {"false, QuotaExceededError"});
+ }
+
+ @MediumTest
+ @Feature({"Payments"})
+ public void testBobPayInstalledLater()
+ throws InterruptedException, ExecutionException, TimeoutException {
+ openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
+ expectResultContains(new String[] {"false, false"});
+
+ installPaymentApp(HAVE_INSTRUMENTS, IMMEDIATE_RESPONSE);
+
+ clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
+ expectResultContains(new String[] {"true, QuotaExceededError"});
}
@MediumTest
@@ -40,10 +53,10 @@ public class PaymentRequestPaymentAppCanMakePaymentQueryTest extends PaymentRequ
TimeoutException {
installPaymentApp(NO_INSTRUMENTS, IMMEDIATE_RESPONSE);
openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, false"});
+ expectResultContains(new String[] {"false, false"});
clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, QuotaExceededError"});
+ expectResultContains(new String[] {"false, QuotaExceededError"});
}
@MediumTest
@@ -52,10 +65,10 @@ public class PaymentRequestPaymentAppCanMakePaymentQueryTest extends PaymentRequ
TimeoutException {
installPaymentApp(NO_INSTRUMENTS, DELAYED_RESPONSE);
openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, false"});
+ expectResultContains(new String[] {"false, false"});
clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"false, QuotaExceededError"});
+ expectResultContains(new String[] {"false, QuotaExceededError"});
}
@MediumTest
@@ -64,10 +77,10 @@ public class PaymentRequestPaymentAppCanMakePaymentQueryTest extends PaymentRequ
TimeoutException {
installPaymentApp(HAVE_INSTRUMENTS, IMMEDIATE_RESPONSE);
openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"true, true"});
+ expectResultContains(new String[] {"true, true"});
clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"true, QuotaExceededError"});
+ expectResultContains(new String[] {"true, QuotaExceededError"});
}
@MediumTest
@@ -76,9 +89,9 @@ public class PaymentRequestPaymentAppCanMakePaymentQueryTest extends PaymentRequ
TimeoutException {
installPaymentApp(HAVE_INSTRUMENTS, DELAYED_RESPONSE);
openPageAndClickBuyAndWait(mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"true, true"});
+ expectResultContains(new String[] {"true, true"});
clickNodeAndWait("otherBuy", mCanMakePaymentQueryResponded);
- expectResultContains(new String[]{"true, QuotaExceededError"});
+ expectResultContains(new String[] {"true, QuotaExceededError"});
}
}
« no previous file with comments | « chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestBasicCardTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698