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

SAT: compute unit test coverage #15441

Closed
alafanechere opened this issue Aug 9, 2022 · 0 comments · Fixed by #15443
Closed

SAT: compute unit test coverage #15441

alafanechere opened this issue Aug 9, 2022 · 0 comments · Fixed by #15443
Assignees
Labels
Acceptance Tests https://docs.google.com/document/d/1pFLR8eLP5VOj6vDtWe69KZL0aWWHEA6Vkbod6NYJon4/edit# area/connectors Connector related issues team/extensibility type/enhancement New feature or request

Comments

@alafanechere
Copy link
Contributor

Tell us about the problem you're trying to solve

(Not to be confused with https://github.com/airbytehq/airbyte-internal-issues/issues/326)
When iterating on SAT I want to measure the coverage of unit tests I wrote to test the behavior of new SAT tests.

Describe the solution you’d like

  • Install pytest-cov on source-acceptance-test
  • Add the --cov options by default to get coverage when running python -m pytest unit_tests
  • Get a coverage report on each test run (ignore fully covered or empty files from the report)
  • Show a failure if the coverage decreases.
@alafanechere alafanechere added type/enhancement New feature or request area/connectors Connector related issues Acceptance Tests https://docs.google.com/document/d/1pFLR8eLP5VOj6vDtWe69KZL0aWWHEA6Vkbod6NYJon4/edit# team/extensibility labels Aug 9, 2022
@alafanechere alafanechere self-assigned this Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Acceptance Tests https://docs.google.com/document/d/1pFLR8eLP5VOj6vDtWe69KZL0aWWHEA6Vkbod6NYJon4/edit# area/connectors Connector related issues team/extensibility type/enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant