-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch the testing framework to Kotest? #123
Comments
Good idea, we can try it. But actually you should know, that several people in JetBrains do not like this framework :) Unfortunately I am busy right now with the support for Custom deserialisers and Inline (VALUE) classes, so won't be able to make it in the nearest future... :( |
@bishiboosh @petertrr what do you think about kotest? |
@akuleshov7 I would be interested to know why some people in Jetbrains do not like the framework? :) |
As do I. It's great |
Understandable :) |
I haven't tried kotest myself just yet, but I've read about it. Indeed, API looks nice and more kotlin-idiomatic. I was a little bit concerned about multiplatform support and IDE support, though. As they say in kotest docs, IDE support for multiplafrom is limited and tests can't be run from IDE directly (though, it usualy doesn't work with kotlin.test too :| ). I think it's worth a shot, and their library for property-based testing can be useful for testing ktoml, where there are a lot of similar test scenarios for different input data. |
Currently kotlin.test is used for testing. Have you considered Kotest instead? Like Kotlin's test framework, it's built on JUnit on the JVM, but tests are written in a more idiomatic way. There's also an IntelliJ plugin that can display the tests being run in real time, rather than producing a report at the end.
Here's an example of one style it provides:
This is, in my opinion, way easier to read than:
Transitioning would take some effort, but I think it's worth considering. Thoughts?
The text was updated successfully, but these errors were encountered: