I m getting complaints about 2 `should` blocks having the sa kotlinlang #kotest

Channels

100daysofcode

100daysofkotlin

100daysofkotlin-2021

androidgithubprojects

cincinnati-user-group

competitive-programming

event21-community-content

java-to-kotlin-refactoring

jvm-ir-backend-feedback

kotlin-multiplatform-contest

kotlinultimatechallenge

myndocs-oauth2-server

naming

navigation-architecture-component

practical-functional-programming

proguard

prozis-android-backup

refactoring-to-kotlin

russian-kotlinasfirst

skia-wasm-interop-temp

stayhungrystayfoolish

swagger-gradle-codegen

tricity-kotlin-user-group

Title

bbaldino

07/21/2020, 9:49 PM

I'm getting complaints about 2

should

blocks having the same text for: "duplicated test name" (even though they are within different parent scopes. i.e.:

"one" {
    "a" {
        should("c") {}
    }
    "b" {
        should("c") {}
    }
}

This complains about duplicate test names

Is this something that changed recently in the plugin? I swear I've done this before

sam

07/21/2020, 9:50 PM

which versions of kotest and plugin

bbaldino

07/21/2020, 9:50 PM

This is older kotest...kotlintest 3.4.2. Checking plugin version.

plugin 1.0.5

sam

07/21/2020, 9:51 PM

the plugin doesn't work with older versions of kotest

well not properly :)

in 4.x the string parent tests are gone, try doing

bbaldino

07/21/2020, 9:52 PM

Ok...I've got some projects which are on newer kotest and some still on older. It does look like I can still run them in intellij fine, so I guess it's just some annoying warnings/highlighting

sam

07/21/2020, 9:52 PM

context("one") {
    context("a") {
        should("c") {}
    }
    context("b") {
        should("c") {}
    }
}

If you try that the highlighting should be fixed

that's the style you have to use in 4.x

bbaldino

07/21/2020, 9:53 PM

Yeah, I've used that in 4.x, but I don't think

context

exists in this version.

Not for

ShouldSpec

, at least.

sam

07/21/2020, 9:53 PM

hmmm ok

the plugin doesn't understand the raw string parent tests, so it thinks you have two root tests called "c"

bbaldino

07/21/2020, 9:54 PM

I see

Alright, I'll just deal with it until upgrading.

sam

07/21/2020, 9:54 PM

bbaldino

07/21/2020, 9:54 PM

Thanks, Sam

👍🏻 1

3 Views

#kotest Join Slack