Merge branch 'master' into update-jvm-target

# Conflicts:
#	firebase-common/src/jsMain/kotlin/dev/gitlive/firebase/externals.kt
#	firebase-firestore/src/iosMain/kotlin/dev/gitlive/firebase/firestore/firestore.kt
#	settings.gradle.kts

Author: nbransby

README.md

@@ -88,6 +88,7 @@ db.collection("cities").document("LA").set(City.serializer(), city, encodeDefaul
 ```
 
 The `encodeDefaults` parameter is optional and defaults to `true`, set this to false to omit writing optional properties if they are equal to theirs default values.
+Using [@EncodeDefault](https://kotlinlang.org/api/kotlinx.serialization/kotlinx-serialization-core/kotlinx.serialization/-encode-default/) on properties is a recommended way to locally override the behavior set with `encodeDefaults`.
 
 You can also omit the serializer but this is discouraged due to a [current limitation on Kotlin/JS and Kotlin/Native](https://github.com/Kotlin/kotlinx.serialization/issues/1116#issuecomment-704342452)
 
@@ -110,6 +111,21 @@ data class Post(
 )
 ```
 
+In addition `firebase-firestore` provides [GeoPoint] and [DocumentReference] classes which allow persisting
+geo points and document references in a native way:
+
+```kotlin
+@Serializable
+data class PointOfInterest(
+    val reference: DocumentReference, 
+    val location: GeoPoint
+)
+val document = PointOfInterest(
+    reference = Firebase.firestore.collection("foo").document("bar"),
+    location = GeoPoint(51.939, 4.506)
+)
+```
+
 <h4>Polymorphic serialization (sealed classes)</h4>
 
 This sdk will handle polymorphic serialization automatically if you have a sealed class and its children marked as `Serializable`. It will include a `type` property that will be used to discriminate which child class is the serialized.

build.gradle.kts

@@ -73,7 +73,28 @@ subprojects {
         onlyIf { !project.gradle.startParameter.taskNames.any { "MavenLocal" in it } && !project.gradle.startParameter.taskNames.contains("publishToGitHubPackagesRepository") }
     }
 
+    val skipPublishing = project.name == "test-utils" // skip publishing for test utils
+
     tasks {
+        withType<Test> {
+            testLogging {
+                showExceptions = true
+                exceptionFormat = TestExceptionFormat.FULL
+                showStandardStreams = true
+                showCauses = true
+                showStackTraces = true
+                events = setOf(
+                    TestLogEvent.STARTED,
+                    TestLogEvent.FAILED,
+                    TestLogEvent.PASSED,
+                    TestLogEvent.SKIPPED,
+                    TestLogEvent.STANDARD_OUT,
+                    TestLogEvent.STANDARD_ERROR
+                )
+            }
+        }
+
+        if (skipPublishing) return@tasks
 
         val updateVersion by registering(Exec::class) {
             commandLine("npm", "--allow-same-version", "--prefix", projectDir, "version", "${project.property("${project.name}.version")}")
@@ -144,24 +165,6 @@ subprojects {
                 commandLine("npm", "publish")
             }
         }
-
-        withType<Test> {
-            testLogging {
-                showExceptions = true
-                exceptionFormat = TestExceptionFormat.FULL
-                showStandardStreams = true
-                showCauses = true
-                showStackTraces = true
-                events = setOf(
-                    TestLogEvent.STARTED,
-                    TestLogEvent.FAILED,
-                    TestLogEvent.PASSED,
-                    TestLogEvent.SKIPPED,
-                    TestLogEvent.STANDARD_OUT,
-                    TestLogEvent.STANDARD_ERROR
-                )
-            }
-        }
     }
 
     afterEvaluate  {
@@ -193,8 +196,10 @@ subprojects {
         }
     }
 
-    apply(plugin="maven-publish")
-    apply(plugin="signing")
+    if (skipPublishing) return@subprojects
+
+    apply(plugin = "maven-publish")
+    apply(plugin = "signing")
 
 
     configure<PublishingExtension> {
@@ -247,6 +252,7 @@ subprojects {
                 }
             }
         }
+
     }
 }
 

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

@@ -19,8 +19,9 @@ import platform.Foundation.*
 actual val Firebase.auth
     get() = FirebaseAuth(FIRAuth.auth())
 
-actual fun Firebase.auth(app: FirebaseApp): FirebaseAuth = TODO("Come back to issue")
-//    FirebaseAuth(FIRAuth.authWithApp(app.ios))
+actual fun Firebase.auth(app: FirebaseApp): FirebaseAuth = FirebaseAuth(
+    FIRAuth.authWithApp(app.ios as objcnames.classes.FIRApp)
+)
 
 actual class FirebaseAuth internal constructor(val ios: FIRAuth) {
 

firebase-common/build.gradle.kts

@@ -103,6 +103,12 @@ kotlin {
             }
         }
 
+        val commonTest by getting {
+            dependencies {
+                implementation(project(":test-utils"))
+            }
+        }
+
         val androidMain by getting {
             dependencies {
                 api("com.google.firebase:firebase-common")

firebase-common/src/commonMain/kotlin/dev/gitlive/firebase/serializers.kt

@@ -108,7 +108,7 @@ class FirebaseListSerializer : KSerializer<Iterable<Any?>> {
  * A special case of serializer for values natively supported by Firebase and
  * don't require an additional encoding/decoding.
  */
-abstract class SpecialValueSerializer<T>(
+class SpecialValueSerializer<T>(
     serialName: String,
     private val toNativeValue: (T) -> Any?,
     private val fromNativeValue: (Any?) -> T

firebase-common/src/commonTest/kotlin/dev/gitlive/firebase/EncodersTest.kt

@@ -11,10 +11,6 @@ import kotlin.test.Test
 import kotlin.test.assertEquals
 import kotlin.test.assertNull
 
-expect fun nativeMapOf(vararg pairs: Pair<String, Any?>): Any
-expect fun nativeListOf(vararg elements: Any): Any
-expect fun nativeAssertEquals(expected: Any?, actual: Any?): Unit
-
 @Serializable
 data class TestData(val map: Map<String, String>, val bool: Boolean = false, val nullableBool: Boolean? = null)
 

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

@@ -276,7 +276,6 @@ external object firebase {
     object functions {
         class Functions {
             fun httpsCallable(name: String, options: Json?): HttpsCallable
-            fun useFunctionsEmulator(origin: String)
             fun useEmulator(host: String, port: Int)
         }
         interface HttpsCallableResult {
@@ -455,6 +454,7 @@ external object firebase {
             fun update(field: FieldPath, value: Any?, vararg moreFieldsAndValues: Any?): Promise<Unit>
             fun delete(): Promise<Unit>
             fun onSnapshot(options: Json, next: (snapshot: DocumentSnapshot) -> Unit, error: (error: Error) -> Unit): ()->Unit
+            fun isEqual(other: DocumentReference): Boolean
         }
 
         open class WriteBatch {
@@ -479,6 +479,8 @@ external object firebase {
             companion object {
                 val documentId: FieldPath
             }
+
+            fun isEqual(other: FieldPath): Boolean
         }
 
         abstract class FieldValue {
@@ -492,6 +494,13 @@ external object firebase {
 
             fun isEqual(other: FieldValue): Boolean
         }
+
+        open class GeoPoint(latitude: Double, longitude: Double) {
+            val latitude: Double
+            val longitude: Double
+
+            fun isEqual(other: GeoPoint): Boolean
+        }
     }
 
     fun remoteConfig(app: App? = definedExternally): remoteConfig.RemoteConfig

firebase-config/src/iosMain/kotlin/dev/gitlive/firebase/remoteconfig/FirebaseRemoteConfig.kt

@@ -19,8 +19,9 @@ import platform.Foundation.timeIntervalSince1970
 actual val Firebase.remoteConfig: FirebaseRemoteConfig
     get() = FirebaseRemoteConfig(FIRRemoteConfig.remoteConfig())
 
-actual fun Firebase.remoteConfig(app: FirebaseApp): FirebaseRemoteConfig = TODO("Come back to issue")
-//    FirebaseRemoteConfig(FIRRemoteConfig.remoteConfigWithApp(Firebase.app.ios))
+actual fun Firebase.remoteConfig(app: FirebaseApp): FirebaseRemoteConfig = FirebaseRemoteConfig(
+    FIRRemoteConfig.remoteConfigWithApp(Firebase.app.ios as objcnames.classes.FIRApp)
+)
 
 actual class FirebaseRemoteConfig internal constructor(val ios: FIRRemoteConfig) {
     actual val all: Map<String, FirebaseRemoteConfigValue>

firebase-database/src/commonMain/kotlin/dev/gitlive/firebase/database/ServerValue.kt

@@ -3,6 +3,7 @@ package dev.gitlive.firebase.database
 import dev.gitlive.firebase.FirebaseDecoder
 import dev.gitlive.firebase.FirebaseEncoder
 import dev.gitlive.firebase.SpecialValueSerializer
+import kotlinx.serialization.KSerializer
 import kotlinx.serialization.Serializable
 import kotlinx.serialization.SerializationException
 
@@ -18,7 +19,7 @@ expect class ServerValue internal constructor(nativeValue: Any){
 }
 
 /** Serializer for [ServerValue]. Must be used with [FirebaseEncoder]/[FirebaseDecoder].*/
-object ServerValueSerializer: SpecialValueSerializer<ServerValue>(
+object ServerValueSerializer: KSerializer<ServerValue> by SpecialValueSerializer(
     serialName = "ServerValue",
     toNativeValue = ServerValue::nativeValue,
     fromNativeValue = { raw ->

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

@@ -33,11 +33,13 @@ actual val Firebase.database
 actual fun Firebase.database(url: String) =
     FirebaseDatabase(FIRDatabase.databaseWithURL(url))
 
-actual fun Firebase.database(app: FirebaseApp): FirebaseDatabase = TODO("Come back to issue")
-//    FirebaseDatabase(FIRDatabase.databaseForApp(app.ios))
+actual fun Firebase.database(app: FirebaseApp): FirebaseDatabase = FirebaseDatabase(
+    FIRDatabase.databaseForApp(app.ios as objcnames.classes.FIRApp)
+)
 
-actual fun Firebase.database(app: FirebaseApp, url: String): FirebaseDatabase = TODO("Come back to issue")
-//    FirebaseDatabase(FIRDatabase.databaseForApp(app.ios, url))
+actual fun Firebase.database(app: FirebaseApp, url: String): FirebaseDatabase = FirebaseDatabase(
+    FIRDatabase.databaseForApp(app.ios as objcnames.classes.FIRApp, url)
+)
 
 actual class FirebaseDatabase internal constructor(val ios: FIRDatabase) {