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

C unit testing with Check #49

Closed

Conversation

ma595
Copy link

@ma595 ma595 commented Mar 15, 2023

Introduce the 'C' unit testing Framework 'Check' into ONEFlux. This follows the python test PR #45 convention where tests are developed separately to the source. A new directory is created called 'tests/C' containing test_meteo.c file with a Makefile that compiles the test code.

Cannot close until:

  • Demonstrated use of Check on more interesting functions.
  • Currently an attempt is made to test static functions. Avoid this all together, or implement (potentially undesirable) workaround. Fix error in Tests makefile.
  • Coverage analysis
  • Implement change to Project makefile that gives option to compile tests as well.
  • Integrate 'C' testing into automated workflow. Build Check atomatically as part of github workflow.
  • Ensure generated tests pass.

Fixes #48

…ing meteo_proc is_valid_era_filename function
…rd and import source directly in test_meteo.c file
@ma595 ma595 marked this pull request as draft March 15, 2023 10:04
@ma595 ma595 self-assigned this Mar 15, 2023
@ma595 ma595 changed the base branch from main to tests_framework March 15, 2023 10:04
@ma595 ma595 closed this May 4, 2023
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.

1 participant