Skip to content

Commit 080a552

Browse files
committed
Clean up detekt errors.
1 parent 90722fd commit 080a552

File tree

4 files changed

+13
-7
lines changed

4 files changed

+13
-7
lines changed

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,4 +9,4 @@ fun interface PayPalWebStartCallback {
99
*/
1010
@MainThread
1111
fun onPayPalWebStartResult(result: PayPalPresentAuthChallengeResult)
12-
}
12+
}

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ internal class UpdateClientConfigAPI(
3939
)
4040

4141
is LoadRawResourceResult.Failure -> UpdateClientConfigResult.Failure(
42-
PayPalSDKError(123, "TODO: implement")
42+
PayPalSDKError(0, "TODO: implement")
4343
)
4444
}
4545
}
@@ -98,4 +98,4 @@ internal class UpdateClientConfigAPI(
9898
UpdateClientConfigResult.Failure(error)
9999
}
100100
}
101-
}
101+
}

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import com.paypal.android.corepayments.PayPalSDKError
44

55
sealed class UpdateClientConfigResult {
66

7-
data class Success(val clientConfig: String): UpdateClientConfigResult()
8-
data class Failure(val error: PayPalSDKError): UpdateClientConfigResult()
7+
data class Success(val clientConfig: String) : UpdateClientConfigResult()
8+
data class Failure(val error: PayPalSDKError) : UpdateClientConfigResult()
99
}
10-

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

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import io.mockk.mockk
99
import io.mockk.verify
1010
import junit.framework.TestCase.assertSame
1111
import junit.framework.TestCase.assertTrue
12+
import kotlinx.coroutines.Dispatchers
1213
import kotlinx.coroutines.ExperimentalCoroutinesApi
1314
import org.junit.Before
1415
import org.junit.Test
@@ -21,6 +22,7 @@ class PayPalWebCheckoutClientUnitTest {
2122

2223
private val activity: FragmentActivity = mockk(relaxed = true)
2324
private val analytics = mockk<PayPalWebAnalytics>(relaxed = true)
25+
private val updateClientConfigAPI = mockk<UpdateClientConfigAPI>(relaxed = true)
2426

2527
private val intent = Intent()
2628

@@ -30,7 +32,12 @@ class PayPalWebCheckoutClientUnitTest {
3032
@Before
3133
fun beforeEach() {
3234
payPalWebLauncher = mockk(relaxed = true)
33-
sut = PayPalWebCheckoutClient(analytics, payPalWebLauncher)
35+
sut = PayPalWebCheckoutClient(
36+
analytics = analytics,
37+
updateClientConfigAPI = updateClientConfigAPI,
38+
payPalWebLauncher = payPalWebLauncher,
39+
dispatcher = Dispatchers.Main
40+
)
3441
}
3542

3643
@Test

0 commit comments

Comments
 (0)