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

#105 Improving Test Coverage #109

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

AnHoff
Copy link
Collaborator

@AnHoff AnHoff commented Jul 18, 2023

Motivation

Hygia had 85% of test coverage but the ideal is to have 90%. Some methods weren't tested and others had incomplete tests.

Warning

I could improve the test coverage in Hygia from 85% to 89%. There is still 1% enhancement needed.

Changes

  • Tests to YAML parser;
  • Tests to pre-processing parser;
  • Tests to annotate parser;
  • Complementing tests to annotate data;
  • Complementing tests to random forest;

Status Checklist

  • Enhance 5% in test coverage.
  • Test all files needed.

Screenshots

Before After
image image

Testing

  • First activate your env and install the dependancies;
  • Then run "pytest --cov".

@AnHoff AnHoff added the enhancement New feature or request label Jul 18, 2023
@AnHoff AnHoff requested a review from RochaCarla July 18, 2023 16:39
@AnHoff AnHoff self-assigned this Jul 18, 2023
@AnHoff AnHoff linked an issue Jul 18, 2023 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve test coverage
1 participant