Add unit test suite and enable branch coverage#112
Merged
Conversation
Cover the API client, endpoints, query layer, response/record handling, models, tokens and helpers with unit tests, and report branch coverage. - Add unit tests across api, endpoint, query, response, models, token and util - Mint auth tokens dynamically in tests instead of shipping static JWT fixtures - Fix Record list diffing so items containing the separator are not conflated - Count single-object responses as one result - Remove dead code: Request.get_openapi, Record pickle hooks, Token.__str__ - Correct the cat() docstring - Enable --cov-branch and upload coverage to Codecov with a token
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds a unit test suite covering the API client, endpoints, query layer, response/record handling, models, tokens and helpers. Enables branch coverage and uploads it to Codecov.
Why
The library had little automated testing. This raises confidence in refactors and guards against regressions in URL building, auth token routing, response parsing and diffing.
Changes
Requires the CODECOV_TOKEN secret to be set on the repository.
Closes #12