Yet one more thing: I just realized that `beforeTe...

# kotest

[Sebastian Schuberth]

Yet one more thing: I just realized that beforeTest() in a WordSpec also gets called for the outer WordSpecShouldContainerScope. What can I use to just have something called for the contained actual tests, i.e. the WordSpecTerminalScope?

[Sebastian Schuberth]

Looks like beforeEach() is the way to go...

[sam]

beforeTest has always been "before every nested lambda". If you weren't seeing that before, sounds like a bug in 4.x with WordSpec

[sam]

beforeEach is "before only leaf nodeS"

[Sebastian Schuberth]

I believe the behavior was that like that before in Kotest 4, I just didn't realize until. as I was debugging other things...

[Sebastian Schuberth]

As a side note, I believe the naming of the callback to be a bit unfortunate... beforeEach() sounds to me as if it would be called more often than beforeTest(), whereas the opposite is the case.

[sam]

Yes the naming is shit

[sam]

beforeTest should be leaf's only but has existed since 1.0

[sam]

As part of the docs work I am doing, I am going to add more pages on listeners as the docs don't do a good job of all the extensions atm

[Sebastian Schuberth]

Yes, I agree calling beforeTest() on leaves only would be the more expected behavior.

[sam]

beforeTest was meant to be deprecated in 5.0 due to this confusing behavior, because we added beforeAll as well which is the same thing.

[sam]

But I decided not to deprecate it as I'm cautious about too many deprecations on something as fundamental as a test framework.

[sam]

You could use beforeAll yourself, and forget beforeTest exists if you want.

[Sebastian Schuberth]

Thanks, I'll probably do that.

[Sebastian Schuberth]

What about a "soft deprecation" of beforeTest()?

[sam]

I'm against deprecating anything that's used in 1000s of projects really. What I will do is just stop referencing beforeTest in docs and things, so over time it will fall out of use.