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

Include utility in test name in cmake #477

Closed
edwardhartnett opened this issue Apr 29, 2021 · 2 comments · Fixed by #478
Closed

Include utility in test name in cmake #477

edwardhartnett opened this issue Apr 29, 2021 · 2 comments · Fixed by #478
Assignees
Labels

Comments

@edwardhartnett
Copy link
Collaborator

edwardhartnett commented Apr 29, 2021

Our output looks like this:

Test project /Users/runner/work/UFS_UTILS/UFS_UTILS/ufs_utils/build
    Start 1: ftst_utils
1/5 Test #1: ftst_utils .......................   Passed    0.99 sec
    Start 2: ftst_program_setup
2/5 Test #2: ftst_program_setup ...............   Passed    0.45 sec
   Start 3: ftst_search_util
3/5 Test #3: ftst_search_util .................   Passed    0.20 sec
   Start 4: ftst_sfc_input_data
4/5 Test #4: ftst_sfc_input_data ..............   Passed    0.32 sec
   Start 5: tst_create_xgrid
5/5 Test #5: tst_create_xgrid .................   Passed    0.05 sec

It's not clear which tests apply to chgres_cube. When naming our tests in the cmake file, we should add the name of the utility, so it is clear what utilities are being tested.

@edwardhartnett
Copy link
Collaborator Author

OK, now it looks like this:


Test project /home/ed/UFS_UTILS/b
    Start 1: chgres_cube-ftst_utils
1/5 Test #1: chgres_cube-ftst_utils ............   Passed    0.01 sec
    Start 2: chgres_cube-ftst_program_setup
2/5 Test #2: chgres_cube-ftst_program_setup ....   Passed    0.05 sec
    Start 3: chgres_cube-ftst_search_util
3/5 Test #3: chgres_cube-ftst_search_util ......   Passed    0.04 sec
    Start 4: chgres_cube-ftst_sfc_input_data
4/5 Test #4: chgres_cube-ftst_sfc_input_data ...   Passed    0.05 sec
    Start 5: fre-nctools-tst_create_xgrid
5/5 Test #5: fre-nctools-tst_create_xgrid ......   Passed    0.00 sec

@edwardhartnett
Copy link
Collaborator Author

We can sneak this into the 1.3.0 release...

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

Successfully merging a pull request may close this issue.

1 participant