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

Testmods don't check that folder contains valid testmods #4601

Open
bartgol opened this issue Mar 18, 2024 · 1 comment
Open

Testmods don't check that folder contains valid testmods #4601

bartgol opened this issue Mar 18, 2024 · 1 comment

Comments

@bartgol
Copy link
Contributor

bartgol commented Mar 18, 2024

I recently made a typo when submitting a test, which got undetected. I had some testmods in a fews subfolders of a main folder. Let's call the parent folder "root" and the subfolders "1", "2', and so on. The correct testmod were then <test>.root-1, <test>.root-2, and so on. However, I accidentally just typed <test>.root. Since the "root" folder existed, CIME did not complain. However, it also did not add any testmod, since no shell_commands was in that folder (nor any user_nl_XYZ).

I think it would be nice if CIME checked that the testmod does something, precicely to avoid cases where the user specifies only part of the path/to/testmod. This could be implemented either as an error or warning, depending on how sure we are that not finding valid mods is indeed an error rather than "a coincidence" (maybe inside some general purpose script). But having at least a warning would allow one to see in the logs that a testmod they tought was added was not indeed added.

Copy link
Contributor

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 5 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants