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.

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

Unit Testing #15

Open
anton-seaice opened this issue Apr 17, 2024 · 0 comments
Open

Unit Testing #15

anton-seaice opened this issue Apr 17, 2024 · 0 comments

Comments

@anton-seaice
Copy link
Contributor

In #6, we add an end-to-end/integration test for generating a cice grid from mom. This doesn't completely cover the CiceGrid class, however at this point there is probably limited value in adding unit tests for the class unless we need to do further development on the class.

We also add a couple of utils which should be unit tested:

  • md5sum function is covered by test_cice_grid.py but it would be good practice to unit test this
  • safe_version function is used, but we should be unit testing the string it returns (noting we can assume the version function is tested / works upstream).
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

No branches or pull requests

1 participant