Chore/organize tests#539
Conversation
Thank you for the pull request! 💙The Scribe-Android team will do our best to address your contribution as soon as we can. If you're not already a member of our public Matrix community, please consider joining! We'd suggest that you use the Element client as well as Element X for a mobile app, and definitely join the Note Scribe uses Conventional Comments in reviews to make sure that communication is as clear as possible. |
Maintainer ChecklistThe following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)
|
angrezichatterbox
left a comment
There was a problem hiding this comment.
Thanks for the PR @bhanu-dev82
Everything looks good to me. Thanks for the quick work.
Contributor checklist
./gradlew lintKotlin detekt testcommand as directed in the testing section of the contributing guideDescription
This PR addresses issue #509 by organizing the test files into the correct package structure and cleaning up unused/invalid tests. It also adds new unit tests to improve code coverage.
Changes:
AlphanumericComparatorTestandKeyboardTestto their correct packages (be.scri.helpers) to match the source code structure.ComprehensiveCoverageTest.ktas it was a dummy test providing no real value.DatabaseTesttoPluralFormsManagerTestto better reflect its purpose and location.StringUtilsTestto improve unit test coverage for theisWordCapitalizedfunction.Verification:
./gradlew testCoreDebugUnitTestto ensure all unit tests pass (Build Successful).Related issue