Skip to content

Commit 1123f69

Browse files
committed
Update unit test
1 parent 0921bb2 commit 1123f69

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

PayPalWebPayments/src/main/java/com/paypal/android/paypalwebpayments/PayPalWebCheckoutClient.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ class PayPalWebCheckoutClient internal constructor(
4848

4949
private val activityReference = WeakReference(activity)
5050

51-
private var observer = PayPalWebCheckoutLifeCycleObserver(this)
51+
internal var observer = PayPalWebCheckoutLifeCycleObserver(this)
5252

5353
init {
5454
activity.lifecycle.addObserver(observer)

PayPalWebPayments/src/test/java/com/paypal/android/paypalwebpayments/PayPalWebCheckoutClientUnitTest.kt

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,15 @@ import com.paypal.android.corepayments.CoreConfig
66
import com.paypal.android.corepayments.PayPalSDKError
77
import com.paypal.android.corepayments.analytics.AnalyticsService
88
import io.mockk.*
9-
import junit.framework.TestCase.assertNotNull
10-
import junit.framework.TestCase.assertEquals
119
import junit.framework.TestCase.assertSame
1210
import kotlinx.coroutines.ExperimentalCoroutinesApi
1311
import kotlinx.coroutines.test.*
1412
import org.junit.Before
1513
import org.junit.Test
1614
import org.junit.runner.RunWith
1715
import org.robolectric.RobolectricTestRunner
16+
import strikt.api.expectThat
17+
import strikt.assertions.isNull
1818

1919
@ExperimentalCoroutinesApi
2020
@RunWith(RobolectricTestRunner::class)
@@ -209,13 +209,11 @@ class PayPalWebCheckoutClientUnitTest {
209209
fun `when client is complete, lifecycle observer is removed`() {
210210
val lifeCycle = mockk<Lifecycle>(relaxed = true)
211211
every { activity.lifecycle } returns lifeCycle
212-
213-
assertNotNull(sut.listener)
214-
assertNotNull(sut.vaultListener)
215212

216213
sut.removeObservers()
217214

218-
assertEquals(sut.listener, null)
219-
assertEquals(sut.vaultListener, null)
215+
verify { lifeCycle.removeObserver(sut.observer) }
216+
expectThat(sut.listener).isNull()
217+
expectThat(sut.vaultListener).isNull()
220218
}
221219
}

0 commit comments

Comments
 (0)