Fix :+ operator

leviysoft · Jan 5, 2025 · 4f9b336 · 4f9b336
1 parent 6f63492
commit 4f9b336
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 1 deletion.
diff --git a/oolong-bson/src/main/scala/oolong/bson/bson.scala b/oolong-bson/src/main/scala/oolong/bson/bson.scala
@@ -76,7 +76,7 @@ extension (bv: BsonValue)
    *
    * In a case of key collision bson1 values takes priority
    */
-  @inline def :+(other: BsonValue): BsonValue = merge(other, bv, false)
+  @inline def :+(other: BsonValue): BsonValue = merge(bv, other, false)
 
   /**
    * Merges two bson values

diff --git a/oolong-bson/src/test/scala/oolong/bson/BsonMergeSpec.scala b/oolong-bson/src/test/scala/oolong/bson/BsonMergeSpec.scala
@@ -0,0 +1,25 @@
+package oolong.bson
+
+import org.mongodb.scala.bson.*
+import org.scalatest.funsuite.AnyFunSuite
+import org.scalatest.matchers.should.Matchers
+
+class BsonMergeSpec extends AnyFunSuite with Matchers {
+  test(":+ document merge") {
+    val doc1 = BsonDocument("field1" -> 10, "field2" -> 30)
+    val doc2 = BsonDocument("field1" -> 20, "field3" -> 40)
+
+    val res = doc1 :+ doc2
+
+    res shouldBe BsonDocument("field1" -> 10, "field2" -> 30, "field3" -> 40)
+  }
+
+  test("+: document merge") {
+    val doc1 = BsonDocument("field1" -> 10, "field2" -> 30)
+    val doc2 = BsonDocument("field1" -> 20, "field3" -> 40)
+
+    val res = doc1 +: doc2
+
+    res shouldBe BsonDocument("field1" -> 20, "field2" -> 30, "field3" -> 40)
+  }
+}