Merge pull request #1994 from OneSignal/ci/fix-unit-tests-2024-02-14

CI - Fix failing test 2024-02-14; Includes fixes for RobolectricTest, updating to Android compile API 34, and fixing a variety of tests

Author: jkasten2

.github/workflows/ci.yml

@@ -26,7 +26,7 @@ jobs:
       - name: "[Test] SDK Unit Tests"
         working-directory: OneSignalSDK
         run: |
-          ./gradlew test --console=plain
+          ./gradlew testReleaseUnitTest --console=plain --continue
       - name: Unit tests results
         if: failure()
         uses: actions/upload-artifact@v3

OneSignalSDK/build.gradle

@@ -4,8 +4,8 @@ buildscript {
 
     ext {
         buildVersions = [
-                compileSdkVersion: 31,
-                targetSdkVersion: 31,
+                compileSdkVersion: 34,
+                targetSdkVersion: 34,
                 minSdkVersion: 19
         ]
         androidGradlePluginVersion = '7.2.0'

OneSignalSDK/onesignal/core/build.gradle

@@ -78,6 +78,7 @@ dependencies {
         }
     }
 
+    testImplementation(project(':OneSignal:testhelpers'))
     testImplementation("junit:junit:$junitVersion")
     testImplementation("io.kotest:kotest-runner-junit4:$kotestVersion")
     testImplementation("io.kotest:kotest-runner-junit4-jvm:$kotestVersion")

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/user/internal/backend/impl/SubscriptionBackendService.kt

@@ -17,9 +17,9 @@ internal class SubscriptionBackendService(
         aliasValue: String,
         subscription: SubscriptionObject,
     ): String? {
-        val requestJSON =
-            JSONObject()
-                .put("subscription", JSONConverter.convertToJSON(subscription))
+        val jsonSubscription = JSONConverter.convertToJSON(subscription)
+        jsonSubscription.remove("id")
+        val requestJSON = JSONObject().put("subscription", jsonSubscription)
 
         val response = _httpClient.post("apps/$appId/users/by/$aliasLabel/$aliasValue/subscriptions", requestJSON)
 

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/application/ApplicationServiceTests.kt

@@ -8,7 +8,7 @@ import com.onesignal.common.threading.suspendifyOnThread
 import com.onesignal.core.internal.application.impl.ApplicationService
 import com.onesignal.debug.LogLevel
 import com.onesignal.debug.internal.logging.Logging
-import com.onesignal.extensions.RobolectricTest
+import com.onesignal.testhelpers.extensions.RobolectricTest
 import io.kotest.core.spec.style.FunSpec
 import io.kotest.matchers.shouldBe
 import io.kotest.runner.junit4.KotestTestRunner

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/database/OSDatabaseTests.kt

@@ -6,8 +6,8 @@ import com.onesignal.core.internal.database.impl.OSDatabase
 import com.onesignal.core.internal.database.impl.OneSignalDbContract
 import com.onesignal.debug.LogLevel
 import com.onesignal.debug.internal.logging.Logging
-import com.onesignal.extensions.RobolectricTest
 import com.onesignal.session.internal.outcomes.impl.OutcomeTableProvider
+import com.onesignal.testhelpers.extensions.RobolectricTest
 import io.kotest.core.spec.style.FunSpec
 import io.kotest.matchers.shouldBe
 import io.kotest.runner.junit4.KotestTestRunner

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/core/internal/preferences/PreferencesServiceTests.kt

@@ -5,9 +5,9 @@ import androidx.test.core.app.ApplicationProvider
 import com.onesignal.core.internal.preferences.impl.PreferencesService
 import com.onesignal.debug.LogLevel
 import com.onesignal.debug.internal.logging.Logging
-import com.onesignal.extensions.RobolectricTest
 import com.onesignal.mocks.AndroidMockHelper
 import com.onesignal.mocks.MockHelper
+import com.onesignal.testhelpers.extensions.RobolectricTest
 import io.kotest.assertions.throwables.shouldThrowUnit
 import io.kotest.core.spec.style.FunSpec
 import io.kotest.matchers.shouldBe

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/extensions/RobolectricExtension.kt

@@ -2,7 +2,6 @@ package com.onesignal.session.internal.outcomes
 
 import com.onesignal.debug.LogLevel
 import com.onesignal.debug.internal.logging.Logging
-import com.onesignal.extensions.RobolectricTest
 import com.onesignal.mocks.DatabaseMockHelper
 import com.onesignal.session.internal.influence.Influence
 import com.onesignal.session.internal.influence.InfluenceChannel
@@ -13,6 +12,7 @@ import com.onesignal.session.internal.outcomes.impl.OutcomeEventsRepository
 import com.onesignal.session.internal.outcomes.impl.OutcomeEventsTable
 import com.onesignal.session.internal.outcomes.impl.OutcomeSource
 import com.onesignal.session.internal.outcomes.impl.OutcomeSourceBody
+import com.onesignal.testhelpers.extensions.RobolectricTest
 import io.kotest.core.spec.style.FunSpec
 import io.kotest.matchers.shouldBe
 import io.kotest.matchers.shouldNotBe
@@ -171,13 +171,15 @@ class OutcomeEventsRepositoryTests : FunSpec({
                     OutcomeEventsTable.COLUMN_NAME_NAME to "outcomeId1",
                     OutcomeEventsTable.COLUMN_NAME_WEIGHT to 0.2f,
                     OutcomeEventsTable.COLUMN_NAME_TIMESTAMP to 1111L,
+                    OutcomeEventsTable.COLUMN_NAME_SESSION_TIME to 1L,
                     OutcomeEventsTable.COLUMN_NAME_NOTIFICATION_INFLUENCE_TYPE to "unattributed",
                     OutcomeEventsTable.COLUMN_NAME_IAM_INFLUENCE_TYPE to "unattributed",
                 ),
                 mapOf(
                     OutcomeEventsTable.COLUMN_NAME_NAME to "outcomeId2",
                     OutcomeEventsTable.COLUMN_NAME_WEIGHT to 0.4f,
                     OutcomeEventsTable.COLUMN_NAME_TIMESTAMP to 2222L,
+                    OutcomeEventsTable.COLUMN_NAME_SESSION_TIME to 2L,
                     OutcomeEventsTable.COLUMN_NAME_NOTIFICATION_INFLUENCE_TYPE to "indirect",
                     OutcomeEventsTable.COLUMN_NAME_NOTIFICATION_IDS to "[\"notificationId1\",\"notificationId2\"]",
                     OutcomeEventsTable.COLUMN_NAME_IAM_INFLUENCE_TYPE to "indirect",
@@ -187,6 +189,7 @@ class OutcomeEventsRepositoryTests : FunSpec({
                     OutcomeEventsTable.COLUMN_NAME_NAME to "outcomeId3",
                     OutcomeEventsTable.COLUMN_NAME_WEIGHT to 0.6f,
                     OutcomeEventsTable.COLUMN_NAME_TIMESTAMP to 3333L,
+                    OutcomeEventsTable.COLUMN_NAME_SESSION_TIME to 3L,
                     OutcomeEventsTable.COLUMN_NAME_NOTIFICATION_INFLUENCE_TYPE to "direct",
                     OutcomeEventsTable.COLUMN_NAME_NOTIFICATION_IDS to "[\"notificationId3\"]",
                     OutcomeEventsTable.COLUMN_NAME_IAM_INFLUENCE_TYPE to "direct",
@@ -205,12 +208,14 @@ class OutcomeEventsRepositoryTests : FunSpec({
         events[0].outcomeId shouldBe "outcomeId1"
         events[0].weight shouldBe 0.2f
         events[0].timestamp shouldBe 1111L
+        events[0].sessionTime shouldBe 1L
         events[0].outcomeSource shouldNotBe null
         events[0].outcomeSource!!.directBody shouldBe null
         events[0].outcomeSource!!.indirectBody shouldBe null
         events[1].outcomeId shouldBe "outcomeId2"
         events[1].weight shouldBe 0.4f
         events[1].timestamp shouldBe 2222L
+        events[1].sessionTime shouldBe 2L
         events[1].outcomeSource shouldNotBe null
         events[1].outcomeSource!!.directBody shouldBe null
         events[1].outcomeSource!!.indirectBody shouldNotBe null
@@ -223,6 +228,7 @@ class OutcomeEventsRepositoryTests : FunSpec({
         events[2].outcomeId shouldBe "outcomeId3"
         events[2].weight shouldBe 0.6f
         events[2].timestamp shouldBe 3333L
+        events[2].sessionTime shouldBe 3L
         events[2].outcomeSource shouldNotBe null
         events[2].outcomeSource!!.indirectBody shouldBe null
         events[2].outcomeSource!!.directBody shouldNotBe null

OneSignalSDK/onesignal/core/src/test/java/com/onesignal/session/internal/outcomes/OutcomeEventsRepositoryTests.kt

@@ -5,7 +5,6 @@ import com.onesignal.core.internal.http.HttpResponse
 import com.onesignal.core.internal.http.IHttpClient
 import com.onesignal.debug.LogLevel
 import com.onesignal.debug.internal.logging.Logging
-import com.onesignal.extensions.RobolectricTest
 import com.onesignal.user.internal.backend.impl.SubscriptionBackendService
 import com.onesignal.user.internal.subscriptions.SubscriptionStatus
 import io.kotest.assertions.throwables.shouldThrowUnit
@@ -17,7 +16,8 @@ import io.mockk.coVerify
 import io.mockk.mockk
 import org.junit.runner.RunWith
 
-@RobolectricTest
+// WARNING: Adding @RobolectricTest will cause JSONObject.map() to stop working
+// at runtime.
 @RunWith(KotestTestRunner::class)
 class SubscriptionBackendServiceTests : FunSpec({
     beforeAny {
@@ -29,7 +29,7 @@ class SubscriptionBackendServiceTests : FunSpec({
         val aliasLabel = "onesignal_id"
         val aliasValue = "11111111-1111-1111-1111-111111111111"
         val spyHttpClient = mockk<IHttpClient>()
-        coEvery { spyHttpClient.post(any(), any()) } returns HttpResponse(202, "{id: \"subscriptionId\"}")
+        coEvery { spyHttpClient.post(any(), any()) } returns HttpResponse(202, "{ \"subscription\": { id: \"subscriptionId\" } }")
         val subscriptionBackendService = SubscriptionBackendService(spyHttpClient)
 
         // When
@@ -48,13 +48,14 @@ class SubscriptionBackendServiceTests : FunSpec({
         response shouldBe "subscriptionId"
         coVerify {
             spyHttpClient.post(
-                "apps/appId/user/by/$aliasLabel/$aliasValue/subscription",
+                "apps/appId/users/by/$aliasLabel/$aliasValue/subscriptions",
                 withArg {
-                    it.has("id") shouldBe false
-                    it.getString("type") shouldBe "AndroidPush"
-                    it.getString("token") shouldBe "pushToken"
-                    it.getBoolean("enabled") shouldBe true
-                    it.getInt("notification_types") shouldBe 1
+                    val sub = it.getJSONObject("subscription")
+                    sub.has("id") shouldBe false
+                    sub.getString("type") shouldBe "AndroidPush"
+                    sub.getString("token") shouldBe "pushToken"
+                    sub.getBoolean("enabled") shouldBe true
+                    sub.getInt("notification_types") shouldBe 1
                 },
             )
         }
@@ -92,13 +93,14 @@ class SubscriptionBackendServiceTests : FunSpec({
         // Then
         coVerify {
             spyHttpClient.post(
-                "apps/appId/user/by/$aliasLabel/$aliasValue/subscription",
+                "apps/appId/users/by/$aliasLabel/$aliasValue/subscriptions",
                 withArg {
-                    it.has("id") shouldBe false
-                    it.getString("type") shouldBe "AndroidPush"
-                    it.getString("token") shouldBe "pushToken"
-                    it.getBoolean("enabled") shouldBe true
-                    it.getInt("notification_types") shouldBe 1
+                    val sub = it.getJSONObject("subscription")
+                    sub.has("id") shouldBe false
+                    sub.getString("type") shouldBe "AndroidPush"
+                    sub.getString("token") shouldBe "pushToken"
+                    sub.getBoolean("enabled") shouldBe true
+                    sub.getInt("notification_types") shouldBe 1
                 },
             )
         }