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

Bump pytest and its plugins. #74

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

gmmajal
Copy link
Contributor

@gmmajal gmmajal commented Dec 4, 2024

Relevant issue: #73

Background info: # any relevant background info for additional context, references to documentations etc.

  • There are deprecation warnings being raised for unit tests running on python v3.12.
  • In order to remove these warnings pytest and its plugins: pytest-cov and pytest-mock need to be upgraded to versions which are compatible with python v3.12.

Changes introduced: # list changes to the code repo made in this pull request

  • Four packages have been bumped: pytest, pytest-cov, pytest-mock and safety.
  • The pyproject.toml and poetry.lock files have been changed.

@gmmajal gmmajal self-assigned this Dec 4, 2024
@gmmajal gmmajal linked an issue Dec 4, 2024 that may be closed by this pull request
@gmmajal gmmajal requested a review from annakasprzik December 4, 2024 14:51
Copy link

codecov bot commented Dec 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (b083b55) to head (31a066b).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master       #74   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           16        16           
  Lines          922       922           
=========================================
  Hits           922       922           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@annakasprzik annakasprzik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked all checks for all Python versions (test code, security) and found no issues left.

@gmmajal gmmajal merged commit 07561b5 into master Dec 4, 2024
6 checks passed
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.

Deal with warnings being raised for unit tests
2 participants