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

Improve user compset error checking #1546

Merged
merged 6 commits into from
May 16, 2017

Conversation

jedwards4b
Copy link
Contributor

Add individual component specs to README.case, determine pesfile for user-compsets if it is not provided. replace principle_component with primary_component.

Test suite: scripts_regression_tests.py
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #608
Fixes #714

User interface changes?:

Code review:

Copy link
Contributor

@jgfouca jgfouca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One thing I've noticed is that we have a lot of bug-fix PRs going in without adding a test to make sure the bug doesn't re-emerge. I'd like to see us doing more of that if possible. If that doesn't make sense with this PR, let me know and I'll merge.

@jedwards4b
Copy link
Contributor Author

Added a bunch of tests of primary component logic.

@jgfouca
Copy link
Contributor

jgfouca commented May 16, 2017

That looks great, thanks!

@jgfouca jgfouca merged commit 98d386f into ESMCI:master May 16, 2017
jgfouca added a commit that referenced this pull request Jun 2, 2017
Adds test mods for RTM=NULL

[BFB] - Bit-For-Bit

* origin/singhbalwinder/atm/rtm-null:
  Adds test mods for RTM=NULL
jgfouca added a commit that referenced this pull request Feb 23, 2018
Adds test mods for RTM=NULL

[BFB] - Bit-For-Bit

* origin/singhbalwinder/atm/rtm-null:
  Adds test mods for RTM=NULL
jgfouca added a commit that referenced this pull request Mar 13, 2018
Adds test mods for RTM=NULL

[BFB] - Bit-For-Bit

* origin/singhbalwinder/atm/rtm-null:
  Adds test mods for RTM=NULL
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.

2 participants