Map null in map to undefined for JS database

[Daeda88]

No description provided.

[Daeda88]

@nbransby re #448 (comment)
If its specific to updateChildren on JS, this should fix it.

Im not sure how JS is supposed to handle null in a map though. Like in EncodersTest.encodeDecodeGenericClass, is it supposed to be
"nullableBool": null or "nullableBool": undefined?

[Daeda88]

I should note that if I go back to the commit before the Serialization changes where merged, I dont see anything that would do this mapping either.

[nbransby]

Yes I had a look myself and couldn't see anything either. All I know is rolling back fixes it, I'll have to investigate it more

[Daeda88]

Lets see if we can write a test that reproduces it I guess

[Daeda88]

So far, on master. I havent been able to reproduce using:

@Test
    fun testUpdateChildren() = runTest {
        setupRealtimeData()
        val reference = database
            .reference("FirebaseRealtimeDatabaseTest")
        reference.updateChildren(mapOf("lastActivity" to null))
    }

    private suspend fun setupRealtimeData() {
        ensureDatabaseConnected()
        val firebaseDatabaseTestReference = database
            .reference("FirebaseRealtimeDatabaseTest")

        val firebaseDatabaseChildTest1 = FirebaseDatabaseChildTest("aaa")
        val firebaseDatabaseChildTest2 = FirebaseDatabaseChildTest("bbb")
        val firebaseDatabaseChildTest3 = FirebaseDatabaseChildTest("ccc")

        val values = firebaseDatabaseTestReference.child("values")
        values.child("1").setValue(firebaseDatabaseChildTest1)
        values.child("2").setValue(firebaseDatabaseChildTest2)
        values.child("3").setValue(firebaseDatabaseChildTest3)
        firebaseDatabaseTestReference.child("lastActivity").setValue(1)
    }

given

{
  "rules": {
    ".read": true,
    "FirebaseRealtimeDatabaseTest": {
      "values": {
        ".write": true
      },
      "lastActivity": {
        ".validate": "!newData.exists() || newData.isNumber()",
        ".write": true
      }
    }
  }
}

Whereas reference.updateChildren(mapOf("lastActivity" to "bla")) does fail

[Daeda88]

I found a bug in OnDisconnect specifically, will make a patch for that once I have testing figured out

[Daeda88]

Closing cause real fix is here #494