Skip to content

Commit 113e96b

Browse files
nan-linan-li
committed
Fix unit test compilation for identity verification parameters
Made-with: Cursor
1 parent 457b745 commit 113e96b

File tree

13 files changed

+132
-53
lines changed

13 files changed

+132
-53
lines changed

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/operations/OperationRepoTests.kt‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import com.onesignal.debug.LogLevel
1212
import com.onesignal.debug.internal.logging.Logging
1313
import com.onesignal.mocks.MockHelper
1414
import com.onesignal.mocks.MockPreferencesService
15+
import com.onesignal.user.internal.identity.JwtTokenStore
1516
import com.onesignal.user.internal.operations.ExecutorMocks.Companion.getNewRecordState
1617
import com.onesignal.user.internal.operations.LoginUserOperation
1718
import io.kotest.core.spec.style.FunSpec
@@ -72,6 +73,8 @@ private class Mocks {
7273
configModelStore,
7374
Time(),
7475
getNewRecordState(configModelStore),
76+
mockk<JwtTokenStore>(relaxed = true),
77+
MockHelper.identityModelStore(),
7578
),
7679
recordPrivateCalls = true,
7780
)
@@ -97,6 +100,8 @@ class OperationRepoTests : FunSpec({
97100
mocks.configModelStore,
98101
Time(),
99102
getNewRecordState(mocks.configModelStore),
103+
mockk<JwtTokenStore>(relaxed = true),
104+
MockHelper.identityModelStore(),
100105
),
101106
)
102107

@@ -913,6 +918,8 @@ class OperationRepoTests : FunSpec({
913918
every { operation.modifyComparisonKey } returns modifyComparisonKey
914919
every { operation.translateIds(any()) } just runs
915920
every { operation.applyToRecordId } returns applyToRecordId
921+
every { operation.externalId } returns null
922+
every { operation.externalId = any() } just runs
916923

917924
return operation
918925
}

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/LoginHelperTests.kt‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import com.onesignal.debug.LogLevel
66
import com.onesignal.debug.internal.logging.Logging
77
import com.onesignal.mocks.MockHelper
88
import com.onesignal.user.internal.identity.IdentityModel
9+
import com.onesignal.user.internal.identity.JwtTokenStore
910
import com.onesignal.user.internal.operations.LoginUserOperation
1011
import com.onesignal.user.internal.properties.PropertiesModel
1112
import io.kotest.core.spec.style.FunSpec
@@ -56,6 +57,7 @@ class LoginHelperTests : FunSpec({
5657
userSwitcher = mockUserSwitcher,
5758
operationRepo = mockOperationRepo,
5859
configModel = mockConfigModel,
60+
jwtTokenStore = mockk<JwtTokenStore>(relaxed = true),
5961
lock = loginLock,
6062
)
6163

@@ -108,6 +110,7 @@ class LoginHelperTests : FunSpec({
108110
userSwitcher = mockUserSwitcher,
109111
operationRepo = mockOperationRepo,
110112
configModel = mockConfigModel,
113+
jwtTokenStore = mockk<JwtTokenStore>(relaxed = true),
111114
lock = loginLock,
112115
)
113116

@@ -173,6 +176,7 @@ class LoginHelperTests : FunSpec({
173176
userSwitcher = mockUserSwitcher,
174177
operationRepo = mockOperationRepo,
175178
configModel = mockConfigModel,
179+
jwtTokenStore = mockk<JwtTokenStore>(relaxed = true),
176180
lock = loginLock,
177181
)
178182

@@ -234,6 +238,7 @@ class LoginHelperTests : FunSpec({
234238
userSwitcher = mockUserSwitcher,
235239
operationRepo = mockOperationRepo,
236240
configModel = mockConfigModel,
241+
jwtTokenStore = mockk<JwtTokenStore>(relaxed = true),
237242
lock = loginLock,
238243
)
239244

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/LogoutHelperTests.kt‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import com.onesignal.debug.LogLevel
66
import com.onesignal.debug.internal.logging.Logging
77
import com.onesignal.mocks.MockHelper
88
import com.onesignal.user.internal.operations.LoginUserOperation
9+
import com.onesignal.user.internal.subscriptions.SubscriptionModelStore
910
import io.kotest.core.spec.style.FunSpec
1011
import io.kotest.matchers.shouldBe
1112
import io.mockk.every
@@ -49,6 +50,7 @@ class LogoutHelperTests : FunSpec({
4950
userSwitcher = mockUserSwitcher,
5051
operationRepo = mockOperationRepo,
5152
configModel = mockConfigModel,
53+
subscriptionModelStore = mockk<SubscriptionModelStore>(relaxed = true),
5254
lock = logoutLock,
5355
)
5456

@@ -79,6 +81,7 @@ class LogoutHelperTests : FunSpec({
7981
userSwitcher = mockUserSwitcher,
8082
operationRepo = mockOperationRepo,
8183
configModel = mockConfigModel,
84+
subscriptionModelStore = mockk<SubscriptionModelStore>(relaxed = true),
8285
lock = logoutLock,
8386
)
8487

@@ -118,6 +121,7 @@ class LogoutHelperTests : FunSpec({
118121
userSwitcher = mockUserSwitcher,
119122
operationRepo = mockOperationRepo,
120123
configModel = mockConfigModel,
124+
subscriptionModelStore = mockk<SubscriptionModelStore>(relaxed = true),
121125
lock = logoutLock,
122126
)
123127

@@ -150,6 +154,7 @@ class LogoutHelperTests : FunSpec({
150154
userSwitcher = mockUserSwitcher,
151155
operationRepo = mockOperationRepo,
152156
configModel = mockConfigModel,
157+
subscriptionModelStore = mockk<SubscriptionModelStore>(relaxed = true),
153158
lock = logoutLock,
154159
)
155160

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/migrations/RecoverFromDroppedLoginBugTests.kt‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import com.onesignal.core.internal.time.impl.Time
66
import com.onesignal.debug.LogLevel
77
import com.onesignal.debug.internal.logging.Logging
88
import com.onesignal.mocks.MockHelper
9+
import com.onesignal.user.internal.identity.JwtTokenStore
910
import com.onesignal.user.internal.operations.ExecutorMocks
1011
import com.onesignal.user.internal.operations.LoginUserOperation
1112
import io.kotest.core.spec.style.FunSpec
@@ -38,6 +39,8 @@ private class Mocks {
3839
configModelStore,
3940
Time(),
4041
ExecutorMocks.getNewRecordState(configModelStore),
42+
mockk<JwtTokenStore>(relaxed = true),
43+
MockHelper.identityModelStore(),
4144
),
4245
)
4346

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/operations/CustomEventOperationExecutorTests.kt‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import com.onesignal.core.internal.operations.ExecutionResult
99
import com.onesignal.core.internal.operations.Operation
1010
import com.onesignal.mocks.MockHelper
1111
import com.onesignal.user.internal.customEvents.ICustomEventBackendService
12+
import com.onesignal.user.internal.identity.JwtTokenStore
1213
import com.onesignal.user.internal.operations.impl.executors.CustomEventOperationExecutor
1314
import io.kotest.core.spec.style.FunSpec
1415
import io.kotest.matchers.equals.shouldBeEqual
@@ -36,7 +37,7 @@ class CustomEventOperationExecutorTests : FunSpec({
3637
val properties = JSONObject().put("key", "value").toString()
3738

3839
val customEventOperationExecutor =
39-
CustomEventOperationExecutor(mockCustomEventBackendService, mockApplicationService, mockDeviceService)
40+
CustomEventOperationExecutor(mockCustomEventBackendService, mockApplicationService, mockDeviceService, mockk<JwtTokenStore>(relaxed = true))
4041
val operations = listOf<Operation>(TrackCustomEventOperation("appId", "onesignalId", null, 1, "event-name", properties))
4142

4243
// When

‎OneSignalSDK/onesignal/core/src/test/java/com/onesignal/user/internal/operations/IdentityOperationExecutorTests.kt‎

Lines changed: 11 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import com.onesignal.user.internal.backend.IdentityConstants
1010
import com.onesignal.user.internal.builduser.IRebuildUserService
1111
import com.onesignal.user.internal.identity.IdentityModel
1212
import com.onesignal.user.internal.identity.IdentityModelStore
13+
import com.onesignal.user.internal.identity.JwtTokenStore
1314
import com.onesignal.user.internal.operations.ExecutorMocks.Companion.getNewRecordState
1415
import com.onesignal.user.internal.operations.impl.executors.IdentityOperationExecutor
1516
import io.kotest.core.spec.style.FunSpec
@@ -39,7 +40,7 @@ class IdentityOperationExecutorTests : FunSpec({
3940
val mockBuildUserService = mockk<IRebuildUserService>()
4041

4142
val identityOperationExecutor =
42-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
43+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
4344
val operations = listOf<Operation>(SetAliasOperation("appId", "onesignalId", "aliasKey1", "aliasValue1"))
4445

4546
// When
@@ -69,7 +70,7 @@ class IdentityOperationExecutorTests : FunSpec({
6970
val mockBuildUserService = mockk<IRebuildUserService>()
7071

7172
val identityOperationExecutor =
72-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
73+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
7374
val operations = listOf<Operation>(SetAliasOperation("appId", "onesignalId", "aliasKey1", "aliasValue1"))
7475

7576
// When
@@ -90,7 +91,7 @@ class IdentityOperationExecutorTests : FunSpec({
9091
val mockBuildUserService = mockk<IRebuildUserService>()
9192

9293
val identityOperationExecutor =
93-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
94+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
9495
val operations = listOf<Operation>(SetAliasOperation("appId", "onesignalId", "aliasKey1", "aliasValue1"))
9596

9697
// When
@@ -111,7 +112,7 @@ class IdentityOperationExecutorTests : FunSpec({
111112
every { mockBuildUserService.getRebuildOperationsIfCurrentUser(any(), any()) } returns null
112113

113114
val identityOperationExecutor =
114-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
115+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
115116
val operations = listOf<Operation>(SetAliasOperation("appId", "onesignalId", "aliasKey1", "aliasValue1"))
116117

117118
// When
@@ -134,7 +135,7 @@ class IdentityOperationExecutorTests : FunSpec({
134135
val mockConfigModelStore = MockHelper.configModelStore().also { it.model.opRepoPostCreateRetryUpTo = 1_000 }
135136
val newRecordState = getNewRecordState(mockConfigModelStore).also { it.add("onesignalId") }
136137
val identityOperationExecutor =
137-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, newRecordState)
138+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, newRecordState, mockConfigModelStore, mockk<JwtTokenStore>(relaxed = true))
138139
val operations = listOf<Operation>(SetAliasOperation("appId", "onesignalId", "aliasKey1", "aliasValue1"))
139140

140141
// When
@@ -160,7 +161,7 @@ class IdentityOperationExecutorTests : FunSpec({
160161
val mockBuildUserService = mockk<IRebuildUserService>()
161162

162163
val identityOperationExecutor =
163-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
164+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
164165
val operations = listOf<Operation>(DeleteAliasOperation("appId", "onesignalId", "aliasKey1"))
165166

166167
// When
@@ -183,7 +184,7 @@ class IdentityOperationExecutorTests : FunSpec({
183184
val mockBuildUserService = mockk<IRebuildUserService>()
184185

185186
val identityOperationExecutor =
186-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
187+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
187188
val operations = listOf<Operation>(DeleteAliasOperation("appId", "onesignalId", "aliasKey1"))
188189

189190
// When
@@ -203,7 +204,7 @@ class IdentityOperationExecutorTests : FunSpec({
203204
val mockBuildUserService = mockk<IRebuildUserService>()
204205

205206
val identityOperationExecutor =
206-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
207+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
207208
val operations = listOf<Operation>(DeleteAliasOperation("appId", "onesignalId", "aliasKey1"))
208209

209210
// When
@@ -225,7 +226,7 @@ class IdentityOperationExecutorTests : FunSpec({
225226
val mockBuildUserService = mockk<IRebuildUserService>()
226227

227228
val identityOperationExecutor =
228-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState())
229+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, getNewRecordState(), MockHelper.configModelStore(), mockk<JwtTokenStore>(relaxed = true))
229230
val operations = listOf<Operation>(DeleteAliasOperation("appId", "onesignalId", "aliasKey1"))
230231

231232
// When
@@ -250,7 +251,7 @@ class IdentityOperationExecutorTests : FunSpec({
250251
val mockConfigModelStore = MockHelper.configModelStore().also { it.model.opRepoPostCreateRetryUpTo = 1_000 }
251252
val newRecordState = getNewRecordState(mockConfigModelStore).also { it.add("onesignalId") }
252253
val identityOperationExecutor =
253-
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, newRecordState)
254+
IdentityOperationExecutor(mockIdentityBackendService, mockIdentityModelStore, mockBuildUserService, newRecordState, mockConfigModelStore, mockk<JwtTokenStore>(relaxed = true))
254255
val operations = listOf<Operation>(DeleteAliasOperation("appId", "onesignalId", "aliasKey1"))
255256

256257
// When

0 commit comments

Comments
 (0)