Skip to content

Conversation

@pp-mo
Copy link
Member

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

No description provided.

@pp-mo pp-mo requested a review from bjlittle March 20, 2024 20:04
@codecov
Copy link

codecov bot commented Mar 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.69%. Comparing base (58bb4b1) to head (04ed352).
Report is 23 commits behind head on FEATURE_pytest_conversion.

Additional details and impacted files
@@                      Coverage Diff                      @@
##           FEATURE_pytest_conversion    #5873      +/-   ##
=============================================================
- Coverage                      89.75%   89.69%   -0.07%     
=============================================================
  Files                             92       92              
  Lines                          22940    22940              
  Branches                        5462     5462              
=============================================================
- Hits                           20590    20575      -15     
- Misses                          1619     1638      +19     
+ Partials                         731      727       -4     

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

Copy link
Member

@bjlittle bjlittle left a comment

Choose a reason for hiding this comment

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

@pp-mo Nice one.

Some questions to consider.

class Tests:
# TODO: unit tests for existing functionality pre 2021-03-11.
def setUp(self):
def setup_method(self):
Copy link
Member

Choose a reason for hiding this comment

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

@pp-mo Adopt fixture pattern?

expected_names = [var.cf_name for var in (aux_var, coord_var, coord_var2)]
expected = set(expected_names)
self.assertEqual(expected, self.cf_group.non_data_variable_names)
assert expected == self.cf_group.non_data_variable_names
Copy link
Member

Choose a reason for hiding this comment

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

@pp-mo Swap order here i.e., actual == expected

@bjlittle bjlittle merged commit 6b9b53a into SciTools:FEATURE_pytest_conversion Mar 22, 2024
@pp-mo pp-mo deleted the pytests_unit_fileformats_cf branch November 29, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

2 participants