-
Notifications
You must be signed in to change notification settings - Fork 448
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
Upgrade KotlinTest to Kotest #2405
Conversation
Task linked: CU-j98mr4 [ 1.x.x ] Upgrade KotlinTest to Kotest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AMAZING @nomisRev !! Just 2 small things:
- Move code Arrow-Core #2403 is ready to merge in case you want to check all the changes together
- Kotest version could be updated from 4.3.1 to 4.6.0
Congrats!!! 👏 🚀
@rachelcarmena bumped to 4.6.0. Now some conflcits to fix 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @nomisRev 🙏
In preparation for moving test to common module for the MPP setup this PR upgrades KotlinTest to Kotest. (With the exception of the old Arrow Meta Kapt module)
This PR keeps everything as is as much as possible.
In almost all files only the following changes were applied:
==
=>shouldBe
asforAll
expectedBoolean
butcheckAll
uses assertionsGen
=>Arb