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.

Sign up for GitHub

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

Jump to bottom

Increase Test Coverage #107

Closed
2 tasks
ctuguinay opened this issue Jun 29, 2023 · 0 comments · Fixed by #115
Closed
2 tasks

Increase Test Coverage #107

ctuguinay opened this issue Jun 29, 2023 · 0 comments · Fixed by #115
Assignees
@ctuguinay
Labels
enhancement tests
Milestone
v0.1.0 (refactor ...

Comments

@ctuguinay
Copy link
Collaborator

ctuguinay commented Jun 29, 2023 •
edited
Loading

With the new functionality and refactoring, there exists a lot of code that is not hit by testing coverage. Testing coverage is in the mid 70%s. We want to increase this to at least 80%.

  • Add failure tests where not tested.
  • Use fixtures when possible.

This issue should be addressed in the same PR that addresses #58.
@ctuguinay ctuguinay mentioned this issue Jun 30, 2023
Closed
3 tasks
@ctuguinay ctuguinay self-assigned this Jun 30, 2023
@ctuguinay ctuguinay moved this to Todo in Echoregions Jul 3, 2023
@ctuguinay ctuguinay moved this from Todo to In Progress in Echoregions Jul 3, 2023
@ctuguinay ctuguinay mentioned this issue Jul 7, 2023
Merged
@ctuguinay ctuguinay linked a pull request Jul 7, 2023 that will close this issue
Improve Testing #115
Merged
@ctuguinay ctuguinay moved this from In Progress to To-Be-Updated in Echoregions Jul 7, 2023
@ctuguinay ctuguinay moved this from To-Be-Updated to Review Needed in Echoregions Jul 25, 2023
ctuguinay added a commit that referenced this issue Aug 2, 2023
@ctuguinay @pre-commit-ci
Improve Testing (#115)
36d7eaf 
## Improve Testing

This PR closes #58, #107, #85, #112, #113, #99, #53, #28, #120:

- Created and used a new testing suite: New EVL, new EVR, new Zarr file
- New testing suite resolved the warning that arose from #85
- Refactored tests
- Added fixtures
- Added hints
- Added marks
- Added/improved upon numpy docstrings for tests and fixtures
- Added further tests for Lines and Regions2D
- Added functionality and tests for EVR parser to handle missing bbox values and no regions
- Removed the unused convert_points function
- Fixes docstrings that appeared as errors in the readthedocs
- Used Pathlib instead of OS

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this from Review Needed to Done in Echoregions Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ctuguinay ctuguinay

Labels
enhancement tests
Projects
Echoregions
Status: Done
Milestone
v0.1.0 (refactor echoregions)
Development

Successfully merging a pull request may close this issue.

Improve Testing ctuguinay/echoregions
1 participant
@ctuguinay