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 to 90% #20

Open
jpmckinney opened this issue May 6, 2021 · 4 comments
Open

Increase test coverage to 90% #20

jpmckinney opened this issue May 6, 2021 · 4 comments
Labels
testing
Milestone
Cleanup

Comments

@jpmckinney
Copy link
Member

Testing these 8-10 files would get us there.

Screen Shot 2021-05-06 at 3 07 07 PM
@jpmckinney
Copy link
Member Author

jpmckinney commented Oct 27, 2021 •
edited
Loading

Test coverage was being reported as 76%, but this number was being inflated, because tests directories inside the application code were being counted. The real number is currently 59%. (2024: now 70%.)

https://coveralls.io/github/open-contracting/pelican-backend?branch=main

@jpmckinney
Copy link
Member Author

We can mock Pika methods to test the workers.

@jpmckinney jpmckinney added this to the Cleanup milestone Jan 17, 2023
@jpmckinney
Copy link
Member Author

  • consistent.tender_value: Add test fixture for non-number amount

@jpmckinney
Copy link
Member Author

  • Convert more of the test files to the new-style tests, using CompiledReleaseTests (and any new helper classes to be created for dataset-level and/or time-based checks).

@jpmckinney jpmckinney mentioned this issue Jan 19, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
testing
Projects
None yet
Milestone
Cleanup
Development

No branches or pull requests
1 participant
@jpmckinney