Skip to content
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

Add code coverage metrics #198

Open
1 of 2 tasks
LeoColman opened this issue Dec 25, 2022 · 0 comments
Open
1 of 2 tasks

Add code coverage metrics #198

LeoColman opened this issue Dec 25, 2022 · 0 comments
Labels
Feature A new feature Good First Issue Defines a good issue for a new contributor to try Maintenance Fixing a dependency, code-cleaning

Comments

@LeoColman
Copy link
Owner

LeoColman commented Dec 25, 2022

Using kotlinx-kover https://github.com/Kotlin/kotlinx-kover we should try to estimate our current coverage by unit tests.

For example. I know that plenty of the database is covered in unit tests, but UI tests are lacking. How lacking? We don't know.

  • Add Kover to the project
  • Create a github action to run it on every push
@LeoColman LeoColman added Maintenance Fixing a dependency, code-cleaning Good First Issue Defines a good issue for a new contributor to try labels Dec 25, 2022
LeoColman added a commit that referenced this issue Jan 28, 2023
When running Kover (related: #198), PauseTest showed as not having one
of the branches properly covered. While addressing this issue directly
(and bringing coverage on this class to 100%), this commit also
rearranges the test class so assertions are clearer and less repetitive.

Signed-off-by: Leonardo Colman Lopes <dev@leonardo.colman.com.br>
@LeoColman LeoColman added the Feature A new feature label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature Good First Issue Defines a good issue for a new contributor to try Maintenance Fixing a dependency, code-cleaning
Projects
None yet
Development

No branches or pull requests

1 participant