Don't retry tests

connect/src/androidTest/java/com/stripe/android/connect/FullScreenComponentTest.kt

@@ -52,7 +52,7 @@ class FullScreenComponentTest {
     @get:Rule
     val ruleChain: RuleChain = RuleChain
         .outerRule(activityRule)
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     private val rootView
         get() = isAssignableFrom(StripeComponentDialogFragmentView::class.java)

crypto-onramp-example/src/androidTest/java/com/stripe/android/crypto/onramp/example/OnrampFlowTest.kt

@@ -44,7 +44,7 @@ class OnrampFlowTest {
     val rule = RuleChain.emptyRuleChain()
         .around(composeRule)
         .around(attestationFeatureFlagTestRule)
-        .around(RetryRule(3))
+        .around(RetryRule())
         .around(activityRule)
 
     private val defaultTimeout: Duration = 30.seconds

payment-method-messaging/src/androidTest/java/com/stripe/android/paymentmethodmessaging/element/LearnMoreActivityTest.kt

@@ -26,7 +26,7 @@ internal class LearnMoreActivityTest {
     private val applicationContext = ApplicationProvider.getApplicationContext<Application>()
 
     @get:Rule
-    val retryRule = RetryRule(3)
+    val retryRule = RetryRule()
 
     @Test
     fun loadsUrl() {

payment-method-messaging/src/androidTest/java/com/stripe/android/paymentmethodmessaging/element/PaymentMethodMessagingElementTest.kt

@@ -29,7 +29,7 @@ class PaymentMethodMessagingElementTest {
     val testRule: RuleChain = RuleChain.emptyRuleChain()
         .around(composeTestRule)
         .around(networkRule)
-        .around(RetryRule(5))
+        .around(RetryRule())
         .around(AdvancedFraudSignalsTestRule())
 
     private val getConfigRequestMatcher = composite(

payments-core-testing/src/main/java/com/stripe/android/testing/RetryRule.kt

@@ -1,35 +1,15 @@
 package com.stripe.android.testing
 
-import android.util.Log
-import leakcanary.NoLeakAssertionFailedError
-import org.junit.AssumptionViolatedException
 import org.junit.rules.TestRule
 import org.junit.runner.Description
 import org.junit.runners.model.Statement
 
-class RetryRule(private val attempts: Int) : TestRule {
+class RetryRule() : TestRule {
     override fun apply(base: Statement, description: Description): Statement {
-        val logTag = description.className
         return object : Statement() {
             @Throws
             override fun evaluate() {
-                if (BuildConfig.IS_RUNNING_IN_CI) {
-                    for (attempt in 1..attempts) {
-                        val isLast = attempts == attempt
-                        runCatching {
-                            base.evaluate()
-                        }.onSuccess {
-                            return
-                        }.onFailure { error ->
-                            if (isLast || error is AssumptionViolatedException || error is NoLeakAssertionFailedError) {
-                                throw error
-                            }
-                            Log.d(logTag, "Failed attempt $attempt out of $attempts with error")
-                        }
-                    }
-                } else {
                     base.evaluate()
-                }
             }
         }
     }

payments-core/src/androidTest/java/com/stripe/android/RadarSessionTest.kt

@@ -56,7 +56,7 @@ class RadarSessionTest {
     val ruleChain: RuleChain = RuleChain
         .outerRule(CoroutineTestRule(testDispatcher))
         .around(scenarioRule)
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     @Test
     fun ensureRadarSessionsAttachHCaptchaToken(): Unit = runTest(testDispatcher) {

payments-core/src/androidTest/java/com/stripe/android/view/CardInputWidgetTest.kt

@@ -34,7 +34,7 @@ class CardInputWidgetTest {
     @get:Rule
     val rule = RuleChain.emptyRuleChain()
         .around(composeTestRule)
-        .around(RetryRule(3))
+        .around(RetryRule())
         .around(activityRule)
 
     @Test

paymentsheet-example/src/androidTest/java/com/stripe/android/utils/TestRules.kt

@@ -48,7 +48,7 @@ class TestRules private constructor(
                     }
                 }.let { chain ->
                     if (BuildConfig.IS_RUNNING_IN_CI && !BuildConfig.RUN_LATENCY_TESTS_IN_CI) {
-                        chain.around(RetryRule(retryCount))
+                        chain.around(RetryRule())
                     } else {
                         chain
                     }

paymentsheet/src/androidTest/java/com/stripe/android/paymentsheet/utils/TestRules.kt

@@ -35,7 +35,7 @@ class TestRules private constructor(
                 .around(DetectLeaksAfterTestSuccess())
                 .around(FakeGooglePayRepositoryRule())
                 .around(composeTestRule)
-                .around(RetryRule(5))
+                .around(RetryRule())
                 .around(networkRule)
                 .around(terminalTestRule)
                 .block()

paymentsheet/src/test/java/com/stripe/android/common/taptoadd/TapToAddActivityTest.kt

@@ -76,7 +76,7 @@ class TapToAddActivityTest {
         .around(imageLoaderTestRule)
         .around(FeatureFlagTestRule(FeatureFlags.forceTapToAddWithTerminal, isEnabled = true))
         .around(PaymentConfigurationTestRule(applicationContext))
-        .around(RetryRule(3))
+        .around(RetryRule())
         .around(intentsRule)
 
     private val linkHelper = TapToAddLinkTestHelper(composeTestRule, networkRule)

paymentsheet/src/test/java/com/stripe/android/paymentelement/confirmation/lpms/BaseLpmNetworkTest.kt

@@ -40,7 +40,7 @@ internal open class BaseLpmNetworkTest(
         .around(createTestActivityRule<LpmNetworkTestActivity>())
         .around(IntentsRule())
         .around(PaymentElementCallbackTestRule())
-        .around(RetryRule(attempts = 3))
+        .around(RetryRule())
 
     fun test(
         testType: TestType,

paymentsheet/src/test/java/com/stripe/android/paymentelement/embedded/sheet/EmbeddedSheetActivityTest.kt

@@ -67,7 +67,7 @@ internal class EmbeddedSheetActivityTest {
         .outerRule(composeTestRule)
         .around(networkRule)
         .around(PaymentConfigurationTestRule(applicationContext))
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     @Test
     fun `when launched without args should finish with error result`() {

paymentsheet/src/test/java/com/stripe/android/paymentsheet/PaymentOptionsActivityTest.kt

@@ -93,7 +93,7 @@ internal class PaymentOptionsActivityTest {
         .around(coroutineScopeCleanupRule)
         .around(composeTestRule)
         .around(networkRule)
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     @get:Rule
     val viewModelStoreRule = ViewModelStoreTestRule()

paymentsheet/src/test/java/com/stripe/android/paymentsheet/PaymentSheetViewModelTest.kt

@@ -191,7 +191,7 @@ internal class PaymentSheetViewModelTest {
         .around(SessionTestRule())
         .around(PaymentElementCallbackTestRule())
         .around(ResetMockRule(eventReporter))
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     @BeforeTest
     fun setup() {

paymentsheet/src/test/java/com/stripe/android/paymentsheet/VerticalModePaymentSheetActivityTest.kt

@@ -60,7 +60,7 @@ internal class VerticalModePaymentSheetActivityTest {
         .around(composeTestRule)
         .around(networkRule)
         .around(PaymentConfigurationTestRule(applicationContext))
-        .around(RetryRule(3))
+        .around(RetryRule())
 
     @Test
     fun `Allows paying with card`() = runTest(

stripecardscan/src/androidTest/java/com/stripe/android/stripecardscan/payment/ml/SSDOcrTest.kt

@@ -25,7 +25,7 @@ class SSDOcrTest {
     private val testContext = InstrumentationRegistry.getInstrumentation().context
 
     @get:Rule
-    val retryRule = RetryRule(3)
+    val retryRule = RetryRule()
 
     @Before
     fun initializeTfLite() {