Skip to content

Commit 7511d23

Browse files
committed
fix failing tests
1 parent 8b4334e commit 7511d23

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

OneSignalSDK/onesignal/in-app-messages/src/test/java/com/onesignal/inAppMessages/internal/InAppMessagesManagerTests.kt

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -305,6 +305,7 @@ class InAppMessagesManagerTests : FunSpec({
305305

306306
// When
307307
mocks.inAppMessagesManager.start()
308+
awaitIO()
308309

309310
// Then
310311
verify { mocks.subscriptionManager.subscribe(any()) }
@@ -384,6 +385,7 @@ class InAppMessagesManagerTests : FunSpec({
384385
// When
385386
iamManager.addClickListener(mockListener)
386387
iamManager.onMessageActionOccurredOnMessage(message, mockClickResult)
388+
awaitIO()
387389

388390
// Then
389391
// Verify listener callback was called
@@ -401,6 +403,7 @@ class InAppMessagesManagerTests : FunSpec({
401403
iamManager.addClickListener(mockListener)
402404
iamManager.removeClickListener(mockListener)
403405
iamManager.onMessageActionOccurredOnMessage(message, mockClickResult)
406+
awaitIO()
404407

405408
// Then
406409
// Listener should not be called after removal
@@ -445,6 +448,7 @@ class InAppMessagesManagerTests : FunSpec({
445448

446449
// When
447450
mocks.inAppMessagesManager.onModelUpdated(args, "tag")
451+
awaitIO()
448452

449453
// Then
450454
coVerify(exactly = 0) { mocks.backend.listInAppMessages(any(), any(), any(), any()) }
@@ -511,6 +515,7 @@ class InAppMessagesManagerTests : FunSpec({
511515

512516
// When
513517
iamManager.onSubscriptionChanged(mockSubscription, args)
518+
awaitIO()
514519

515520
// Then
516521
coVerify(exactly = 0) { mocks.backend.listInAppMessages(any(), any(), any(), any()) }
@@ -530,6 +535,7 @@ class InAppMessagesManagerTests : FunSpec({
530535

531536
// When
532537
iamManager.onSubscriptionChanged(mocks.pushSubscription, args)
538+
awaitIO()
533539

534540
// Then
535541
coVerify(exactly = 0) { mocks.backend.listInAppMessages(any(), any(), any(), any()) }
@@ -740,7 +746,7 @@ class InAppMessagesManagerTests : FunSpec({
740746

741747
// Then
742748
// Should trigger re-evaluation
743-
verify { mocks.triggerController.evaluateMessageTriggers(any()) }
749+
coVerify { mocks.triggerController.evaluateMessageTriggers(any()) }
744750
}
745751

746752
test("onTriggerChanged makes redisplay messages available and re-evaluates") {
@@ -1012,10 +1018,10 @@ class InAppMessagesManagerTests : FunSpec({
10121018

10131019
// Fetch messages
10141020
mocks.inAppMessagesManager.onSessionStarted()
1015-
awaitIO()
10161021

10171022
// Dismiss the message
10181023
mocks.inAppMessagesManager.onMessageWasDismissed(message)
1024+
awaitIO()
10191025

10201026
// When - trigger evaluation
10211027
mocks.inAppMessagesManager.paused = false
@@ -1031,6 +1037,7 @@ class InAppMessagesManagerTests : FunSpec({
10311037

10321038
// When
10331039
mocks.inAppMessagesManager.onMessageActionOccurredOnMessage(mocks.testInAppMessage, mocks.inAppMessageClickResult)
1040+
awaitIO()
10341041

10351042
// Then - wait for async operations
10361043
coVerify { mocks.outcomeEventsController.sendOutcomeEvent("outcome-name") }
@@ -1155,6 +1162,7 @@ class InAppMessagesManagerTests : FunSpec({
11551162

11561163
// When
11571164
mocks.inAppMessagesManager.onMessageActionOccurredOnMessage(mocks.testInAppMessage, mocks.inAppMessageClickResult)
1165+
awaitIO()
11581166

11591167
// Then
11601168
coVerify(exactly = 0) { mocks.inAppDisplayer.dismissCurrentInAppMessage() }
@@ -1171,6 +1179,7 @@ class InAppMessagesManagerTests : FunSpec({
11711179

11721180
// When
11731181
mocks.inAppMessagesManager.onMessageWasDismissed(message)
1182+
awaitIO()
11741183

11751184
// Then
11761185
coVerify { mocks.repository.saveInAppMessage(message) }

0 commit comments

Comments
 (0)