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

Add unit testing get/set functionality to for protected parameters #1293

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

glemieux
Copy link
Contributor

@glemieux glemieux commented Dec 2, 2024

Description:

Collaborators:

Expectation of Answer Changes:

Checklist

If this is your first time contributing, please read the CONTRIBUTING document.

All checklist items must be checked to enable merging this pull request:

Contributor

  • The in-code documentation has been updated with descriptive comments
  • The documentation has been assessed to determine if updates are necessary

Integrator

  • FATES PASS/FAIL regression tests were run
  • Evaluation of test results for answer changes was performed and results provided

Documentation

Test Results:

CTSM (or) E3SM (specify which) test hash-tag:

CTSM (or) E3SM (specify which) baseline hash-tag:

FATES baseline hash-tag:

Test Output:

@glemieux glemieux added PR status: Not Ready The author is signaling that this PR is a work in progress and not ready for integration. testing Related to improvements in code testing (e.g. unit, regression, etc) labels Dec 2, 2024
@glemieux glemieux changed the title Add unit testing get/set function to parameter read modules Add unit testing get/set functionality to for protected parameters Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR status: Not Ready The author is signaling that this PR is a work in progress and not ready for integration. testing Related to improvements in code testing (e.g. unit, regression, etc)
Projects
Development

Successfully merging this pull request may close these issues.

1 participant