Fix database tests

Author: shepeliev

firebase-database/src/commonTest/kotlin/dev.gitlive.firebase.database/database.kt

@@ -40,6 +40,8 @@ class FirebaseDatabaseTest {
 
     @Test
     fun testSetValue() = runTest {
+        awaitDatabaseConnection()
+
         val testValue = "test"
         val testReference = Firebase.database.reference("testPath")
 
@@ -52,9 +54,11 @@ class FirebaseDatabaseTest {
 
         assertEquals(testValue, testReferenceValue)
     }
-    
+
     @Test
     fun testChildCount() = runTest {
+        awaitDatabaseConnection()
+
         setupRealtimeData()
         val dataSnapshot = Firebase.database
             .reference("FirebaseRealtimeDatabaseTest")
@@ -65,6 +69,11 @@ class FirebaseDatabaseTest {
         assertEquals(3, firebaseDatabaseChildCount)
     }
 
+    private suspend fun awaitDatabaseConnection() {
+        // workaround to avoid "Database not connected" exception with Firebase emulator
+        Firebase.database.reference(".info/connected").valueEvents.first { it.value() }
+    }
+
     private suspend fun setupRealtimeData() {
         val firebaseDatabaseTestReference = Firebase.database
             .reference("FirebaseRealtimeDatabaseTest")
@@ -77,4 +86,4 @@ class FirebaseDatabaseTest {
         firebaseDatabaseTestReference.child("2").setValue(firebaseDatabaseChildTest2)
         firebaseDatabaseTestReference.child("3").setValue(firebaseDatabaseChildTest3)
     }
-}
+}

firebase-database/src/jsMain/kotlin/dev/gitlive/firebase/database/database.kt

@@ -165,7 +165,7 @@ actual class DataSnapshot internal constructor(val js: JsDataSnapshot) {
 
     actual val children: Iterable<DataSnapshot> = rethrow {
         ArrayList<DataSnapshot>(js.size).also {
-            js.forEach { snapshot -> it.add(DataSnapshot(snapshot)) }
+            js.forEach { snapshot -> it.add(DataSnapshot(snapshot)); false /* don't cancel enumeration */ }
         }
     }