functional: adjust kotlin tests with collection fields in struct

This commit adjusts the tests and the usage of collection
fields of structures. Instead of calling 'add()' / 'put()'
we override the field with new collection.

Signed-off-by: Patryk Wrobel <[email protected]>

Author: pwrobeldev

functional-tests/functional/android-kotlin/src/test/kotlin/com/here/android/test/EquatableNullableTest.kt

@@ -106,7 +106,7 @@ class EquatableNullableTest {
     @org.junit.Test
     fun equatableNullableStructNotEqualsMapField() {
         val otherStruct: Equatable.EquatableNullableStruct = createEquatableNullableStruct()
-        otherStruct.mapField!!.put(2, "foo")
+        otherStruct.mapField = otherStruct.mapField!! + mapOf(2 to "foo")
 
         assertNotEquals(mainStruct, otherStruct)
         assertNotEquals(mainStruct.hashCode(), otherStruct.hashCode())
@@ -115,7 +115,7 @@ class EquatableNullableTest {
     @org.junit.Test
     fun equatableNullableStructNotEqualsArrayField() {
         val otherStruct: Equatable.EquatableNullableStruct = createEquatableNullableStruct()
-        otherStruct.arrayField!!.add("foo")
+        otherStruct.arrayField = otherStruct.arrayField!! + listOf("foo")
 
         assertNotEquals(mainStruct, otherStruct)
         assertNotEquals(mainStruct.hashCode(), otherStruct.hashCode())
@@ -135,15 +135,15 @@ class EquatableNullableTest {
     @org.junit.Test
     fun equatableNullableStructNotEqualsCpp() {
         val otherStruct: Equatable.EquatableNullableStruct = createEquatableNullableStruct()
-        otherStruct.arrayField!!.add("foo")
+        otherStruct.arrayField = otherStruct.arrayField!! + listOf("foo")
 
         assertFalse(EquatableClass.areEqual(mainStruct, otherStruct))
     }
 
     private fun createEquatableNullableStruct(): Equatable.EquatableNullableStruct {
         return Equatable.EquatableNullableStruct(
             true, -42, 6542, 3.14f, "nonsense", Equatable.NestedEquatableStruct("foo"),
-            Equatable.SomeSomeEnum.BAR, mutableMapOf(0 to "one", 1 to "two"), mutableListOf("one", "two")
+            Equatable.SomeSomeEnum.BAR, mapOf(0 to "one", 1 to "two"), listOf("one", "two")
         )
     }
 }

functional-tests/functional/android-kotlin/src/test/kotlin/com/here/android/test/EquatableTest.kt

@@ -115,7 +115,7 @@ class EquatableTest {
     @org.junit.Test
     fun equatableStructNotEqualsMapField() {
         val otherStruct: Equatable.EquatableStruct = createEquatableStruct()
-        otherStruct.mapField.put(2, "foo")
+        otherStruct.mapField = otherStruct.mapField + mapOf(2 to "foo")
 
         assertNotEquals(mainStruct, otherStruct)
         assertNotEquals(mainStruct.hashCode(), otherStruct.hashCode())
@@ -124,7 +124,7 @@ class EquatableTest {
     @org.junit.Test
     fun equatableStructNotEqualsArrayField() {
         val otherStruct: Equatable.EquatableStruct = createEquatableStruct()
-        otherStruct.arrayField.add("foo")
+        otherStruct.arrayField = otherStruct.arrayField + listOf("foo")
 
         assertNotEquals(mainStruct, otherStruct)
         assertNotEquals(mainStruct.hashCode(), otherStruct.hashCode())
@@ -163,7 +163,7 @@ class EquatableTest {
     @org.junit.Test
     fun equatableStructNotEqualsCpp() {
         val otherStruct: Equatable.EquatableStruct = createEquatableStruct()
-        otherStruct.arrayField.add("foo")
+        otherStruct.arrayField = otherStruct.arrayField + listOf("foo")
 
         assertFalse(EquatableClass.areEqual(mainStruct, otherStruct))
     }
@@ -387,7 +387,7 @@ class EquatableTest {
     private fun createEquatableStruct(): Equatable.EquatableStruct {
         return Equatable.EquatableStruct(
             true, 65542, 2147484000L, 1.0f, 2.0, "nonsense", Equatable.NestedEquatableStruct("foo"),
-            Equatable.SomeSomeEnum.BAR, mutableMapOf(0 to "one", 1 to "two"), mutableListOf("one", "two")
+            Equatable.SomeSomeEnum.BAR, mapOf(0 to "one", 1 to "two"), listOf("one", "two")
         )
     }
 }