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

Improve testing coverage to at least 80% #15

Closed
kylejb opened this issue May 30, 2022 · 2 comments · Fixed by #19
Closed

Improve testing coverage to at least 80% #15

kylejb opened this issue May 30, 2022 · 2 comments · Fixed by #19
Assignees
@kylejb
Labels
enhancement Improvements or additions to existing feature

Comments

@kylejb
Copy link
Member

kylejb commented May 30, 2022 •
edited
Loading

What

Improve testing coverage from 64% to at least 80%.

current coverage report

How

Add more tests to extend coverage

  • CRUD classes
  • Seed scripts

Why

Improve maintainability of code and enable CI/CD pipeline processes.
@kylejb kylejb added the enhancement Improvements or additions to existing feature label May 30, 2022
@kylejb kylejb self-assigned this May 31, 2022
@kylejb kylejb added the in-progress Issue is being worked on label May 31, 2022
@kylejb
Copy link
Member Author

kylejb commented May 31, 2022 •
edited
Loading

I will also create a database for testing purposes, so as to not interfere with our development/production data. The test database will be reset on every run only when PYTHON_ENV=test.

@kylejb
Copy link
Member Author

kylejb commented Jun 5, 2022

I'll stop at 82% for the moment. We should slowly work our way to 100% after #6 and #7 are merged up.

coverage_06-04-22

@kylejb kylejb mentioned this issue Jun 5, 2022
Merged
11 tasks
@kylejb kylejb removed the in-progress Issue is being worked on label Jun 5, 2022
@kylejb kylejb closed this as completed in #19 Jun 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kylejb kylejb

Labels
enhancement Improvements or additions to existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: improve coverage to 82% clairBuoyant/server
1 participant
@kylejb