Merge pull request #194 from tiagonuneslx/firestore-document

Add CollectionReference.document to firestore for all targets

Author: nbransby

firebase-common/src/jsMain/kotlin/dev/gitlive/firebase/externals.kt

@@ -381,7 +381,7 @@ external object firebase {
 
         open class CollectionReference : Query {
             val path: String
-            fun doc(path: String): DocumentReference
+            fun doc(path: String = definedExternally): DocumentReference
             fun add(data: Any): Promise<DocumentReference>
         }
 

firebase-firestore/src/androidMain/kotlin/dev/gitlive/firebase/firestore/firestore.kt

@@ -364,6 +364,9 @@ actual class CollectionReference(override val android: com.google.firebase.fires
     actual val path: String
         get() = android.path
 
+    actual val document: DocumentReference
+        get() = DocumentReference(android.document())
+
     actual fun document(documentPath: String) = DocumentReference(android.document(documentPath))
 
     actual suspend inline fun <reified T> add(data: T, encodeDefaults: Boolean) =

firebase-firestore/src/commonMain/kotlin/dev/gitlive/firebase/firestore/firestore.kt

@@ -128,6 +128,7 @@ expect class DocumentReference {
 
 expect class CollectionReference : Query {
     val path: String
+    val document: DocumentReference
 
     fun document(documentPath: String): DocumentReference
     suspend inline fun <reified T> add(data: T, encodeDefaults: Boolean = true): DocumentReference

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

@@ -124,6 +124,24 @@ class FirebaseFirestoreTest {
 
     }
 
+
+    @Test
+    fun testDocumentAutoId() = runTest {
+        val doc = Firebase.firestore
+            .collection("testDocumentAutoId")
+            .document
+
+        doc.set(FirestoreTest.serializer(), FirestoreTest("AutoId"))
+
+        val resultDoc = Firebase.firestore
+            .collection("testDocumentAutoId")
+            .document(doc.id)
+            .get()
+
+        assertEquals(true, resultDoc.exists)
+        assertEquals("AutoId", resultDoc.get("prop1"))
+    }
+
     private suspend fun setupFirestoreData() {
         Firebase.firestore.collection("FirebaseFirestoreTest")
             .document("one")

firebase-firestore/src/iosMain/kotlin/dev/gitlive/firebase/firestore/firestore.kt

@@ -279,6 +279,8 @@ actual class CollectionReference(override val ios: FIRCollectionReference) : Que
     actual val path: String
         get() = ios.path
 
+    actual val document get() = DocumentReference(ios.documentWithAutoID())
+
     actual fun document(documentPath: String) = DocumentReference(ios.documentWithPath(documentPath))
 
     actual suspend inline fun <reified T> add(data: T, encodeDefaults: Boolean) =

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

@@ -345,6 +345,8 @@ actual class CollectionReference(override val js: firebase.firestore.CollectionR
     actual val path: String
         get() =  rethrow { js.path }
 
+    actual val document get() = rethrow { DocumentReference(js.doc()) }
+
     actual fun document(documentPath: String) = rethrow { DocumentReference(js.doc(documentPath)) }
 
     actual suspend inline fun <reified T> add(data: T, encodeDefaults: Boolean) =