Need to add unit tests for utility schemes #82
Labels
enhancement
New feature or request
Comments
mwaxmonsky
added a commit
that referenced
this issue
Nov 14, 2024
Originator(s): @mwaxmonsky Summary (include the keyword ['closes', 'fixes', 'resolves'] and issue number): Sets up the basic infrastructure to start enabling unit testing. Addresses #82 Describe any changes made to the namelist: N/A List all files eliminated and why: N/A List all files added and what they do: A .github/workflows/code-coverage.yaml A test/unit-test/CMakeLists.txt A test/unit-test/include/ccpp_kinds.F90 A test/unit-test/tests/CMakeLists.txt A test/unit-test/tests/utilities/CMakeLists.txt A test/unit-test/tests/utilities/test_state_converters.pf - Initial round of infrastructure to start enabling unit tests. List all existing files that have been modified, and describe the changes: N/A (Helpful git command: `git diff --name-status development...<your_branch_name>`) List any test failures: N/A Is this a science-changing update? New physics package, algorithm change, tuning changes, etc? No
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Given the implementation of a unit testing framework for the atmospheric_physics repo in PR #78, there is now an opportunity to create some unit tests for the various physics schemes inside the
utilitiesdirectory.This would be particularly beneficial because the tests done in CAM-SIMA are usually for an entire physics suite, where the utility schemes are grouped together, as opposed to each individual utility scheme as proposed here. Plus it would allow for additional test coverage for a PR without necessarily requiring the user to run CAM-SIMA.
The text was updated successfully, but these errors were encountered: