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

➕ pytest-cov is installed. 75% threshhold is set #13

Conversation

parfeniukink
Copy link
Contributor

@parfeniukink parfeniukink commented Jul 21, 2024

Summary

  • 🔗package link
  • The package is added to the dependencies list in pyproject.toml
  • Commands in Makefile are updated with pytest-cov to generate the
    report and return the failure status code for the CI process

P.S. That branch has the base from another dev branch.

Dmytro Parfeniuk added 4 commits July 17, 2024 16:47
* Executor is refactored in order to track cancelled tasks on the level
  of the scheduler instead of tracking that on the Task level itself
* Report generation with an executor is tested
* Some redundancy code is removed
@parfeniukink parfeniukink self-assigned this Jul 21, 2024
@parfeniukink parfeniukink changed the base branch from main to parfeniukink/tests/executor July 21, 2024 20:12
https://pypi.org/project/pytest-cov/

* The package is added to the dependencies list
* The `pytest` command is configured in `pyproject.toml` file
* All tests are covered in `tox.ini` file
* `Makefile` is updated
@parfeniukink parfeniukink force-pushed the parfeniukink/tests/pytest-cov-installation branch from 17820fa to f017043 Compare July 21, 2024 20:21
@parfeniukink parfeniukink force-pushed the parfeniukink/tests/executor branch 2 times, most recently from d7d3846 to 52366a5 Compare July 22, 2024 05:53
@parfeniukink
Copy link
Contributor Author

It was closed due to changes in Executor. Created another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant