fixed create user test to ensure we omit aliases

PR #1794 made changes to omit aliases on user create, but the test was
never updated. Also the SubscriptionStatus arg part of this test wasn't
compare the corret types, causing it to fail the test.

Author: jkasten2

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

@@ -392,13 +392,18 @@ class LoginUserOperationExecutorTests : FunSpec({
         coVerify(exactly = 1) {
             mockUserBackendService.createUser(
                 appId,
-                mapOf("aliasLabel1" to "aliasValue1-2"),
+                // Aliases omitted intentionally in PR #1794, to avoid failed create user calls.
+                //  - Ideally we batch as much as possible into the create as most of the time it
+                //    should be successful. Then when there are failures omit the "bad data" and try
+                //    again, however this is more complex which is why it wasn't done initially.
+                mapOf(),
                 withArg {
                     it.count() shouldBe 1
-                    it[0].type shouldBe SubscriptionObjectType.ANDROID_PUSH
-                    it[0].enabled shouldBe true
-                    it[0].token shouldBe "pushToken2"
-                    it[0].notificationTypes shouldBe SubscriptionStatus.SUBSCRIBED
+                    val subscription = it[0]
+                    subscription.type shouldBe SubscriptionObjectType.ANDROID_PUSH
+                    subscription.enabled shouldBe true
+                    subscription.token shouldBe "pushToken2"
+                    SubscriptionStatus.fromInt(subscription.notificationTypes!!) shouldBe SubscriptionStatus.SUBSCRIBED
                 },
                 any(),
             )