Added support for Kotest tests

orchestr7 / ktoml

Kotlin Multiplatform parser and compile-time serializer/deserializer for TOML format (Native, JS, JVM) based on KxS

https://akuleshov7.github.io/ktoml

MIT License

455 stars 25 forks source link

Added support for Kotest tests #243

Closed ILikeYourHat closed 2 days ago

ILikeYourHat commented 1 year ago

Addresses #123

Added Kotest framework to project
Rewritten all tests in ktoml-file and ktoml-source modules
Rewritten a few tests in ktoml-core module
Sometimes making minor improvements in tests, like migrating away from TomlTable.fullTableName method or removing unused classes

Hope that you would like the way tests look now, and accept my humble contribution :)

orchestr7 commented 1 year ago

Hey, thank you, you are awesome, we had big discussions about kotest https://github.com/akuleshov7/ktoml/issues/123 I will have a look and merge it

ILikeYourHat commented 1 year ago

One more thing: I did this PR because I'm participating in the Hacktoberfest. I would be grateful if you add "hacktoberfest-accepted" label to this PR to make it count. But if for any reason you don't want to do it, I'm also ok with it 👍

orchestr7 commented 1 year ago

One more thing: I did this PR because I'm participating in the Hacktoberfest. I would be grateful if you add "hacktoberfest-accepted" label to this PR to make it count. But if for any reason you don't want to do it, I'm also ok with it 👍

Added, am I right?

ILikeYourHat commented 1 year ago

Added, am I right?

Yes, thank you!

ILikeYourHat commented 1 year ago

Any update on this PR? Do you need anything more from me?