You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I learned that testing private methods is unnecessary and introduces coupling to the implementation details, which breaks encapsulation. It makes refactoring significantly more tedious because the tests for the private methods need to also be updated (also risk of test duplication with public methods). Instead, tests should revolve around public methods and their behaviors/outputs.
Feature Request Criteria
xarray
Is your feature request related to a problem?
The test suite includes tests for private methods and functions.
Per my comment here #236 (comment):
Describe the solution you'd like
Update tests for public APIs to cover tests for private APIs, and then remove the private API tests.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: