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

update tests for mock, pytest-mock and procrunner #541

Merged
merged 3 commits into from
Nov 17, 2020
Merged

Conversation

Anthchirp
Copy link
Member

  • drop dependency on package mock and use the standard library implementation instead
  • allow running with newer versions of pytest-mock (unblocking Update pytest-mock dials/dials#1498)
  • fix deprecation warnings with current procrunner versions

* allow running with newer versions of pytest-mock
* fix deprecation warnings with current procrunner versions
and use the standard library implementation instead
@Anthchirp Anthchirp self-assigned this Nov 15, 2020
@codecov
Copy link

codecov bot commented Nov 15, 2020

Codecov Report

Merging #541 (f170869) into master (6511e5d) will increase coverage by 0.00%.
The diff coverage is 70.83%.

@@           Coverage Diff           @@
##           master     #541   +/-   ##
=======================================
  Coverage   26.85%   26.86%           
=======================================
  Files         228      228           
  Lines       23449    23459   +10     
  Branches     3913     3916    +3     
=======================================
+ Hits         6298     6302    +4     
- Misses      16678    16681    +3     
- Partials      473      476    +3     

@Anthchirp Anthchirp merged commit 510cb10 into master Nov 17, 2020
@Anthchirp Anthchirp deleted the update-tests branch November 17, 2020 18:06
Anthchirp added a commit that referenced this pull request Nov 17, 2020
* update tests for pytest-mock and procrunner
* allow running with newer versions of pytest-mock
* fix deprecation warnings with current procrunner versions
* Drop dependency on mock package
  and use the standard library implementation instead
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