Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Increase Test Coverage to >90% #27

Open
3 of 4 tasks
jdisho opened this issue Dec 26, 2024 · 4 comments
Open
3 of 4 tasks

Increase Test Coverage to >90% #27

jdisho opened this issue Dec 26, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@jdisho
Copy link
Member

jdisho commented Dec 26, 2024

Problem

SpeziFHIR currently has only 33% test coverage. For a library of this size, we should aim to have at least 90% coverage to make sure things are maintainable.

Solution

First, explore the code coverage at Codecov for SpeziFHIR. Then, write unit tests each component to improve coverage.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct and Contributing Guidelines
@jdisho jdisho added the enhancement New feature or request label Dec 26, 2024
@jdisho
Copy link
Member Author

jdisho commented Dec 26, 2024

I have started writing tests for FHIRResource but I don't have access to create and push to branches. Same for managing this issues (assigning, set labels, etc).

cc: @philippzagar 🙏

@philippzagar
Copy link
Member

I have started writing tests for FHIRResource but I don't have access to create and push to branches. Same for managing this issues (assigning, set labels, etc).

cc: @philippzagar 🙏

Thanks for creating the issue! Just gave you access to the repo.

@philippzagar
Copy link
Member

#28 was just merged, a good first step into increasing code coverage! 🚀

@PSchmiedmayer
Copy link
Member

Amazing; thank you @jdisho!

@jdisho jdisho mentioned this issue Jan 30, 2025
1 task
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

3 participants