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.

Sign up for GitHub

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

Jump to bottom

[Feature]: Add more unit testing for VarHandler class #226

Open
tomvothecoder opened this issue Nov 8, 2023 · 0 comments
Open

[Feature]: Add more unit testing for VarHandler class #226

tomvothecoder opened this issue Nov 8, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@tomvothecoder
Copy link
Collaborator

Is your feature request related to a problem?

Currently, many of the methods in the VarHandler class are not covered by unit tests. I implemented this class without much coverage because I needed to reproduce the behavior of the old code and writing unit tests is really cumbersome when not implementing from scratch. Specifically, the VarHandler.cmorize() method involves calling CMOR which is another layer complexity in testing.

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response
@tomvothecoder tomvothecoder added the enhancement New feature or request label Nov 8, 2023
@tomvothecoder tomvothecoder changed the title [Feature]: Add more unit testing VarHandler class [Feature]: Add more unit testing for VarHandler class Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
E3SM to CMIP Development
Status: To do
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tomvothecoder