fix: send receive receipt even when preventDefault is called (#2512)

Author: nan-li

OneSignalSDK/onesignal/notifications/src/main/java/com/onesignal/notifications/internal/generation/impl/NotificationGenerationProcessor.kt

@@ -232,9 +232,9 @@ internal class NotificationGenerationProcessor(
             // Notification channel disable or not displayed
             // save notification as dismissed to avoid user re-enabling channel and notification being displayed due to restore
             markNotificationAsDismissed(notificationJob)
-            return
         }
 
+        // Always call notificationReceived, regardless of wasDisplayed
         _lifecycleService.notificationReceived(notificationJob)
     }
 

OneSignalSDK/onesignal/notifications/src/test/java/com/onesignal/notifications/internal/generation/NotificationGenerationProcessorTests.kt

@@ -168,6 +168,10 @@ class NotificationGenerationProcessorTests : FunSpec({
         mocks.notificationGenerationProcessor.processNotificationData(context, 1, mocks.notificationPayload, false, 1111)
 
         // Then
+        // notificationReceived should be called
+        coVerify(exactly = 1) {
+            mocks.notificationLifecycleService.notificationReceived(any())
+        }
     }
 
     test("processNotificationData should display notification when external callback takes longer than 30 seconds") {
@@ -211,6 +215,10 @@ class NotificationGenerationProcessorTests : FunSpec({
         mocks.notificationGenerationProcessor.processNotificationData(context, 1, mocks.notificationPayload, false, 1111)
 
         // Then
+        // notificationReceived should be called
+        coVerify(exactly = 1) {
+            mocks.notificationLifecycleService.notificationReceived(any())
+        }
     }
 
     test("processNotificationData should display notification when foreground callback takes longer than 30 seconds") {