I'm seeing an issue with the plugin where tests still show as in-progress even though they've all fi...

# kotest

[bbaldino]

I'm seeing an issue with the plugin where tests still show as in-progress even though they've all finished:

[bbaldino]

Doesn't look like it happens for every test though, haven't noticed any pattern yet

[sam]

do you have that many nested tests ?

[sam]

or is that a bug

[bbaldino]

You're right, my tests aren't actually that deeply nested

[bbaldino]

I don't go deeper than context -> context -> should

[bbaldino]

In that test

[sam]

do any of the test names have characters other than az09

[bbaldino]

there is use of apostrophe in some context blocks

[sam]

ok that will be breaking the escaping that intellij uses

[bbaldino]

when jibri isn't busy in that chain

[sam]

can you ping me the test names or if they're sensitive obsecure them, but I need to see the special characters so I can check kotest is escaping it properly

[sam]

when jibri isn't busy is that the only one

[bbaldino]

In that test I think, yeah

[bbaldino]

In another file (where I've seen the issue) I use a dash: "before the clients-muted timeout"

[sam]

https://github.com/kotest/kotest/blob/fbc7934e99a6fcaed22c9a050c8223444c6d2607/kotest-runner/kotest-runner-console/src/jvmTest/kotlin/com/sksamuel/kotest/runner/console/TeamCityMessagesTest.kt#L20

[sam]

I have a test for that use case

[sam]

hmmm I don't test the name just the message

[bbaldino]

These are both in context

[sam]

which plugin vers

[bbaldino]

1.0.4

[sam]

kotest version ?

[bbaldino]

4.1.1

[sam]

the test passes with your test name

[sam]

no other special chars

[bbaldino]

Yeah the tests do pass...it's just that the icon doesn't change to a check

[bbaldino]

It keeps spinning like it's in progress

[sam]

It's because intellij isn't parsing the output from kotest because of escaping. But I can't figure out why your test name would cause that as the ' is escaped

[bbaldino]

Ah

[sam]

https://www.jetbrains.com/help/teamcity/build-script-interaction-with-teamcity.html#BuildScriptInteractionwithTeamCity-Escapedvalues

[bbaldino]

The project is open source if you want to try and repro...I'm on a branch right now but I can double check it happens on master

[bbaldino]

https://github.com/jitsi/jibri

[sam]

can you link me to the test

[bbaldino]

Yeah...let me double check I see it on master before I send you off on a goose chase

[sam]

👍🏻

[bbaldino]

Ok, I see it in this test: https://github.com/jitsi/jibri/blob/master/src/test/kotlin/org/jitsi/jibri/selenium/status_checks/EmptyCallStatusCheckTest.kt

[bbaldino]

Which doesn't have special characters in names, actually.

[sam]

what happens if you turn instance per leaf off

[sam]

ok I can duplicate it - with instance per leaf on

[sam]

It looks like a bug in the parallel handling

[sam]

I'll fix it tonight or tomorrow

[bbaldino]

👍

[thanksforallthefish]

I don’t know how much this could help, but I have the same problem with BehaviorSpec. I can reproduce with

class AnotherTest : BehaviorSpec() {
  override fun isolationMode(): IsolationMode? = IsolationMode.InstancePerLeaf

  init {
    given("test1") {
      `when`("when1") {
        then("then1") {
          1 shouldBe 1
        }
        then("then1.1") {
          1 shouldBe 1
        }
        then("then1.2") {
          1 shouldBe 1
        }
      }
    }

    given("test2") {
      `when`("when2") {
        then("then2") {
          1 shouldBe 1
        }
      }
    }

    given("test3") {
      `when`("when3") {
        then("then3") {
          1 shouldBe 1
        }
      }
    }

    given("test4") {
      `when`("when4") {
        then("then4") {
          1 shouldBe 1
        }
      }
    }
  }
}

as you already noticed @sam InstancePerLeaf seems to be the culprit. I am on kotest 4.1.1 and plugin 1.0.5

[sam]

This is fixed in master, will be releasing 4.1.2 next day or two