papermill icon indicating copy to clipboard operation
papermill copied to clipboard

Upgrade tests to moto v5

Open bnavigator opened this issue 1 year ago • 1 comments

Moto v5 replaced mock_s3 with mock_aws

Supersedes #775

bnavigator avatar Feb 14 '24 12:02 bnavigator

Codecov Report

Merging #779 (e6f60c5) into main (cb2eb37) will decrease coverage by 0.28%. Report is 4 commits behind head on main. The diff coverage is n/a.

:exclamation: Current head e6f60c5 differs from pull request most recent head 1185c15. Consider uploading reports for the commit 1185c15 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #779      +/-   ##
==========================================
- Coverage   91.54%   91.27%   -0.28%     
==========================================
  Files          17       17              
  Lines        1621     1627       +6     
==========================================
+ Hits         1484     1485       +1     
- Misses        137      142       +5     

codecov[bot] avatar Feb 15 '24 19:02 codecov[bot]

Failure that's left is already fixed in main -- going to merge instead of trying to pull the base up to the PR

MSeal avatar Mar 28 '24 18:03 MSeal