timeout should be of type Duration? instead of Long?

Which version of Kotest are you using version : 5.5.3

class TimeOutTest : StringSpec({
    timeout = 100.milliseconds //error: type Mismatch Type mismatch. Required:Long? Found:Duration
    "test case".config(timeout = 400.milliseconds) {
        println("hello world!")
    }
})

[AnouarDoukkali]

sorry the 1.22.0-RC2 version was for Detekt plugin. I’m using 5.5.3 version for Kotest

[sksamuel]

We can’t change it now without breaking everyone’s tests. It’s just a historical artifact that kotest came out before durations existed.

On Wed, Nov 2, 2022, 11:49 AM Anouar Doukkali @.***> wrote:

@Kantis https://github.com/Kantis timeout and invocationTimeout in project level config ,spec config , and test config are all related since each one override the one on top of it . I think it is best if they had all the most obvious type in kotest 6.0 to avoid problems for example for the same duration :

timeout:Duration? = 100.nanoseconds timeout:Long?= // I wouldn’t know what to put here

— Reply to this email directly, view it on GitHub https://github.com/kotest/kotest/issues/3273#issuecomment-1300028781, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFVSGT7ZRAALPQVAT7JMX3WGJBMVANCNFSM6AAAAAARSONSG4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>