Commit 87a25187 authored by Boris Sazonov's avatar Boris Sazonov Committed by Commit Bot

[Android][Sheriff] Disable PaymentHandlerChangePaymentMethodTest on tablets

Disables testReject, testRejectBasicCard and testDetails on tablets.
Reason: flaky (builder links: https://crbug.com/1136100#c1).

TBR=rouslan@chromium.org

Bug: 1136100
Change-Id: I818fe70318c063755a7329b473117bbef779b3f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2456847Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#814721}
parent f0005f63
...@@ -16,6 +16,7 @@ import org.junit.Test; ...@@ -16,6 +16,7 @@ import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.DisableIf;
import org.chromium.base.test.util.DisabledTest; import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.chrome.R; import org.chromium.chrome.R;
...@@ -25,6 +26,7 @@ import org.chromium.components.payments.PaymentFeatureList; ...@@ -25,6 +26,7 @@ import org.chromium.components.payments.PaymentFeatureList;
import org.chromium.net.test.EmbeddedTestServer; import org.chromium.net.test.EmbeddedTestServer;
import org.chromium.net.test.ServerCertificate; import org.chromium.net.test.ServerCertificate;
import org.chromium.ui.test.util.DisableAnimationsTestRule; import org.chromium.ui.test.util.DisableAnimationsTestRule;
import org.chromium.ui.test.util.UiDisableIf;
/** An integration test for PaymentRequestEvent.changePaymentMethod(). */ /** An integration test for PaymentRequestEvent.changePaymentMethod(). */
@RunWith(ChromeJUnit4ClassRunner.class) @RunWith(ChromeJUnit4ClassRunner.class)
...@@ -101,6 +103,7 @@ public class PaymentHandlerChangePaymentMethodTest { ...@@ -101,6 +103,7 @@ public class PaymentHandlerChangePaymentMethodTest {
*/ */
@Test @Test
@Feature({"Payments"}) @Feature({"Payments"})
@DisableIf.Device(type = {UiDisableIf.TABLET}) // See https://crbug.com/1136100.
public void testReject() throws Throwable { public void testReject() throws Throwable {
installPaymentHandler(); installPaymentHandler();
mRule.clickNodeAndWait("testReject", mRule.getDismissed()); mRule.clickNodeAndWait("testReject", mRule.getDismissed());
...@@ -114,6 +117,7 @@ public class PaymentHandlerChangePaymentMethodTest { ...@@ -114,6 +117,7 @@ public class PaymentHandlerChangePaymentMethodTest {
*/ */
@Test @Test
@Feature({"Payments"}) @Feature({"Payments"})
@DisableIf.Device(type = {UiDisableIf.TABLET}) // See https://crbug.com/1136100.
public void testRejectBasicCard() throws Throwable { public void testRejectBasicCard() throws Throwable {
mRule.clickNode("basicCardMethodName"); mRule.clickNode("basicCardMethodName");
installPaymentHandler(); installPaymentHandler();
...@@ -157,6 +161,7 @@ public class PaymentHandlerChangePaymentMethodTest { ...@@ -157,6 +161,7 @@ public class PaymentHandlerChangePaymentMethodTest {
*/ */
@Test @Test
@Feature({"Payments"}) @Feature({"Payments"})
@DisableIf.Device(type = {UiDisableIf.TABLET}) // See https://crbug.com/1136100.
public void testDetails() throws Throwable { public void testDetails() throws Throwable {
installPaymentHandler(); installPaymentHandler();
mRule.clickNodeAndWait("testDetails", mRule.getDismissed()); mRule.clickNodeAndWait("testDetails", mRule.getDismissed());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment