Disallow Int64 as double and add more tests

Author: pulewicz

commons-mongo/jvm/src/main/scala/com/avsystem/commons/mongo/BsonReaderInput.scala

@@ -41,8 +41,7 @@ class BsonReaderInput(br: BsonReader, override val legacyOptionEncoding: Boolean
     bsonType match {
       case BsonType.DOUBLE => br.readDouble()
       case BsonType.INT32 => br.readInt32().toDouble
-      case BsonType.INT64 => br.readInt64().toDouble
-      case _ => wrongType(BsonType.DOUBLE, BsonType.INT32, BsonType.INT64)
+      case _ => wrongType(BsonType.DOUBLE, BsonType.INT32)
     }
   }
 

commons-mongo/jvm/src/main/scala/com/avsystem/commons/mongo/BsonValueInput.scala

@@ -37,8 +37,7 @@ class BsonValueInput(bsonValue: BsonValue, override val legacyOptionEncoding: Bo
     bsonType match {
       case BsonType.DOUBLE => bsonValue.asDouble().getValue
       case BsonType.INT32 => readInt().toDouble
-      case BsonType.INT64 => bsonValue.asInt64().getValue.toDouble
-      case _ => wrongType(BsonType.DOUBLE, BsonType.INT32, BsonType.INT64)
+      case _ => wrongType(BsonType.DOUBLE, BsonType.INT32)
     }
   }
 

commons-mongo/jvm/src/test/scala/com/avsystem/commons/mongo/BsonInputOutputTest.scala

@@ -9,9 +9,10 @@ import org.bson.types.Decimal128
 import org.scalactic.source.Position
 import org.scalatest.funsuite.AnyFunSuite
 import org.scalatestplus.scalacheck.ScalaCheckPropertyChecks
-
 import java.nio.ByteBuffer
 
+import com.avsystem.commons.serialization.GenCodec.ReadFailure
+
 class BinaryBsonGenCodecRoundtripTest extends GenCodecRoundtripTest {
   type Raw = Array[Byte]
 
@@ -82,10 +83,10 @@ class BsonValueGenCodecRoundtripTest extends GenCodecRoundtripTest {
     assert(result == 43D)
   }
 
-  test("Int64 to Double decoding") {
+  test("Int64 to Double decoding exception") {
     val input = createInput(new BsonInt64(44))
-    val result = input.readSimple().readDouble()
-    assert(result == 44D)
+
+    assertThrows[ReadFailure](input.readSimple().readDouble())
   }
 }
 
@@ -246,6 +247,26 @@ class BsonInputOutputTest extends AnyFunSuite with ScalaCheckPropertyChecks {
     }
   }
 
+  test("BsonBinaryReader Int types to Double decoding") {
+    val document = new BsonDocument()
+      .append("int", new BsonInt32(42))
+      .append("long", new BsonInt64(42L))
+
+      val rawJson = document.toJson(JsonWriterSettings.builder.outputMode(JsonMode.EXTENDED).build)
+      val input = new BsonReaderInput(new BsonBinaryReader(RawBsonDocument.parse(rawJson).getByteBuffer.asNIO()))
+
+      val objectInput = input.readObject()
+
+      val intField = objectInput.peekField("int")
+      assert(intField.nonEmpty)
+      assert(intField.get.readDouble() == 42D)
+
+
+      val longField = objectInput.peekField("long")
+      assert(longField.nonEmpty)
+      assertThrows[ReadFailure](longField.get.readDouble())
+  }
+
   test("BsonBinaryReader type metadata") {
     forAll(SomethingPlain.gen) { sth =>
       val document = somethingToBson(sth)