From c367565d14459f5c9ac3529f4518d4e2b9b12e0d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ivan=20=E2=80=9CCLOVIS=E2=80=9D=20Canet?= Date: Sun, 1 Mar 2026 19:47:01 +0100 Subject: [PATCH] test(bson-multiplatform): Remove the timing tests for eager() They are too sensitive to other stuff going on on the CI runner, and fail too often. --- .../kotlin/MultiplatformBsonTests.kt | 21 ++++--------------- 1 file changed, 4 insertions(+), 17 deletions(-) diff --git a/bson-multiplatform/src/commonTest/kotlin/MultiplatformBsonTests.kt b/bson-multiplatform/src/commonTest/kotlin/MultiplatformBsonTests.kt index 85e8e206..e576658f 100644 --- a/bson-multiplatform/src/commonTest/kotlin/MultiplatformBsonTests.kt +++ b/bson-multiplatform/src/commonTest/kotlin/MultiplatformBsonTests.kt @@ -22,7 +22,6 @@ import opensavvy.ktmongo.bson.validateBsonFactory import opensavvy.ktmongo.dsl.LowLevelApi import opensavvy.prepared.runner.testballoon.preparedSuite import opensavvy.prepared.suite.prepared -import kotlin.time.measureTime val context by prepared { BsonFactory() @@ -45,18 +44,12 @@ val MultiplatformBsonWriterTest by preparedSuite { } } - val lazyTime = measureTime { - check(document.reader().read("baz")?.readArray()?.read(0)?.readString() == "a") - } + check(document.reader().read("baz")?.readArray()?.read(0)?.readString() == "a") println("*** Initializing everything eagerly ***") document.eager() - val eagerTime = measureTime { - check(document.reader().read("bad")?.readArray()?.read(1)?.readString() == "b") - } - - check(eagerTime <= lazyTime) + check(document.reader().read("bad")?.readArray()?.read(1)?.readString() == "b") document.eager() // Allowed, does nothing } @@ -72,18 +65,12 @@ val MultiplatformBsonWriterTest by preparedSuite { } } - val lazyTime = measureTime { - check(document.reader().read(0)?.readDocument()?.read("foo")?.readString() == "foo") - } + check(document.reader().read(0)?.readDocument()?.read("foo")?.readString() == "foo") println("*** Initializing everything eagerly ***") document.eager() - val eagerTime = measureTime { - check(document.reader().read(2)?.readDocument()?.read("bar")?.readString() == "bar") - } - - check(eagerTime <= lazyTime) + check(document.reader().read(2)?.readDocument()?.read("bar")?.readString() == "bar") document.eager() // Allowed, does nothing } -- 2.51.2