fix the test case and syntax of instrument test cases

Author: Mansi-mParticle

.github/workflows/pull-request.yml

@@ -59,15 +59,10 @@ jobs:
           script: |
             # Disable animations and clean up test packages
             bash -eo pipefail -c '
-              adb shell settings put global transition_animation_scale 0.0
-              adb shell settings put global animator_duration_scale 0.0
+            adb shell settings put global transition_animation_scale 0.0
+            adb shell settings put global animator_duration_scale 0.0
 
-              for pkg in com.mparticle.kittests com.mparticle.kits.test com.mparticle.legacyTest com.mparticle.test; do
-                if adb shell pm list packages | grep -q "$pkg"; then
-                  adb uninstall "$pkg"
-                fi
-              done
-            '
+            for pkg in com.mparticle.kittests com.mparticle.kits.test com.mparticle.legacyTest com.mparticle.test; do if adb shell pm list packages | grep -q "$pkg"; then adb uninstall "$pkg"; fi; done
 
             ./gradlew connectedCheck --info --stacktrace
 

android-kit-base/src/androidTest/kotlin/com/mparticle/kits/testkits/UserAttributeListenerTestKit.kt

@@ -15,11 +15,11 @@ open class UserAttributeListenerTestKit :
     var onSetUserTag: ((key: String?, user: FilteredMParticleUser?) -> Unit)? = null
     var onSetUserAttributeList: ((attributeKey: String?, attributeValueList: List<String?>?, user: FilteredMParticleUser?) -> Unit)? =
         null
-    var onSetAllUserAttributes: (
+    var onSetAllUserAttributes: ((
         userAttributes: Map<String?, String?>?,
         userAttributeLists: Map<String?, List<String?>?>?,
         user: FilteredMParticleUser?,
-    ) -> Unit? = null
+    ) -> Unit)? = null
     var supportsAttributeLists: (() -> Boolean)? = null
     var onConsentStateUpdated: ((oldState: ConsentState?, newState: ConsentState?, user: FilteredMParticleUser?) -> Unit)? =
         null