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

More unit tests - for initialization functions used in IO tests, plus doxygen updates #1170

Closed
wants to merge 7 commits into from

Conversation

edwardhartnett
Copy link
Contributor

Pull Request Summary

Unit testing and doxygen for some functions used to initialize WW3 code for the I/O tests.

Description

There are some functions used in test_io_point_bin.F90 which are themselves untested. Also they do not all have good doxygen documentation.

Let's discuss whether legacy documentation should be removed with doxygen updates. I can submit the doxygen updates as a stand-alone PR if that is preferable.

More testing is possible for these functions, but this is a good set of smoke tests for them.

Issue(s) addressed

Part of #682

Commit Message

More unit tests - for initialization functions used in IO tests, plus doxygen updates

Check list

Testing

Unit tests run on GitHub CI.

@edwardhartnett
Copy link
Contributor Author

I will close this and work on one PR at a time...

@edwardhartnett edwardhartnett deleted the ejh_test_more branch April 5, 2024 10:57
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.

1 participant