Focus not working in FunSpecs

[ps-feng]

This is not working for FunSpecs:

internal class SomeTest : FunSpec() {
    init {
        context("f:my test") {
            test("something") { ... }
        }
    }
}

According to the documentation, wouldn't context be considered a top-level scope and thus should work? Or is this a special case?

[sksamuel]

What is the expected behavior in this case. You're focusing the only test, so it should run?

[ps-feng]

The example was not explicit enough perhaps so let me try again. Ideally I'd be able to focus on a single test but I was under the impression this is supported:

internal class SomeTest : FunSpec() {
    init {
        context("f:my test 1") {
            test("something 1") { ... }
            test("something 2") { ... }
        }
        context("my test 2") {
            test("blabla") { ... }
            test("lala") { ... }
        }
    }
}

I'd expect something 1 and something 2 to be executed, as the top-level f:my test 1 is focused.

[sksamuel]

Yes that should work. I'll add your test and see.

…

On Sat, 11 Apr 2020 at 13:42, Pin-Sho Feng ***@***.***> wrote: The example was not explicit enough perhaps so let me try again. Ideally I'd be able to focus on a single test but I was under the impression this is supported: internal class SomeTest : FunSpec() { init { context("f:my test 1") { test("something 1") { ... } test("something 2") { ... } } context("my test 2") { test("blabla") { ... } test("lala") { ... } } } } I'd expect something 1 and something 2 to be executed, as the top-level f:my test 1 is focused. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1376 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFVSGRUD4L43ZTA3IWU3MLRMC2Y3ANCNFSM4MGDBRPA> .

[sksamuel]

I've added test and they pass. Are you on 4.0.2 ?

[ps-feng]

I am on 4.0.2. I've just tested this and it doesn't work:

internal class BlaBlaTest : FunSpec() {

    init {
        context("f:some test") {
            test("should run") {
                1 shouldBe 1
            }
            test("should run 2") {
                2 shouldBe 1
            }
        }
        context("other test") {
            test("should never run") {
                3 shouldBe 4
            }
        }
    }
}

[sksamuel]

Thanks for this bug report. It is now fixed in master and will be included in 4.0.5.