Skip to content

Conversation

@pp-mo
Copy link
Member

@pp-mo pp-mo commented Mar 14, 2024

This is a follow-on to #5834
Following a discussion on simpler solutions to migrating setUp functions..
This effectively reverts some of the tweaks in the previous : makes fewer diffs from the original unittest form.
May be slightly pointless, but the new way will be easier to follow in the eventual feature merge-back.

@pp-mo pp-mo requested a review from bjlittle March 14, 2024 14:32
@pp-mo pp-mo force-pushed the pytests_unit_auxfact__atmossigma branch from 74bf624 to b49a508 Compare March 14, 2024 14:35
@codecov
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.74%. Comparing base (b3eb2d1) to head (b49a508).
Report is 4 commits behind head on FEATURE_pytest_conversion.

Additional details and impacted files
@@                      Coverage Diff                      @@
##           FEATURE_pytest_conversion    #5838      +/-   ##
=============================================================
- Coverage                      89.74%   89.74%   -0.01%     
=============================================================
  Files                             92       92              
  Lines                          22942    22940       -2     
  Branches                        5464     5462       -2     
=============================================================
- Hits                           20590    20588       -2     
  Misses                          1620     1620              
  Partials                         732      732              

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

@bjlittle bjlittle merged commit bed9093 into SciTools:FEATURE_pytest_conversion Mar 14, 2024
@pp-mo pp-mo deleted the pytests_unit_auxfact__atmossigma branch November 29, 2025 11:33
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.

2 participants