-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Kotlin for the tests #1389
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
Comments
Agreed that we don't maintain the test suite as much as we should, and apologies on that front. :) I'm "in" if we can get a good foundation to start with (e.g. the entire test suite ported to Kotlin), and ideally a bit of documentation on how to get started with writing tests in Kotlin (perhaps in the wiki we can give an example of one of the tests in both Kotlin and Java?). As for whether or not we want to make it a requirement that all new tests should be in Kotlin, I'm undecided about that. For consistency it would indeed be better to do so, but on the other hand this would place a requirement on all core developers to learn Kotlin regardless of whether they like it or not. Personally I would not mind doing so (especially in light of the neverending Oracle vs Google battle ). But we probably want to see what others think about it as well. |
I ran across a blog post from Brian Button (an Agile leader based in the St Louis area) that talks about "Three Degrees of Unit Testing" that might be interesting to the current topic. https://blog.brianbutton.io/index.php/2018/04/01/the-degrees-of-unit-testing/ I love his comment
|
Tests are now done in Kotlin, builds are passing, coverage is increasing and there's a nice CI pipeline to alpha releases. Thanks to everyone who helped with this 🎉 🍾 |
Summary:
It has been our stated strategy that we would evaluate the use of Kotlin by using it for writing tests.
Expected behavior:
New tests would be written using Kotlin & our test coverage would increase.
Observed behavior:
Device and Android version:
One thing to note: we should update the version of Kotlin in our project.
Would you like to work on the issue?
I would be happy to Port the entire suite to Kotlin, for sure, including new ones for Nearby but this needs to be a team buy-in ... are we "in" or "out" of the Kotlin game? If we are "in" is there a more effective strategy for adoption?
The text was updated successfully, but these errors were encountered: