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

Separate example_scripts testing from unit_testing #423

Closed
jpolton opened this issue Apr 1, 2022 · 3 comments
Closed

Separate example_scripts testing from unit_testing #423

jpolton opened this issue Apr 1, 2022 · 3 comments

Comments

@jpolton
Copy link
Collaborator

jpolton commented Apr 1, 2022

Perhaps:

  • unit_testing is a requirement for committing to develop
  • example_scripts testing is a requirements for committing to master

Example scripts have two (three?) types:

  • Can be run with example_files
  • Can not be run with example_files
  • No data dependency?

Story: Seeking advice on how to structure the workflow that ends up with working code on master.

Aug Update We are moving example_scripts/*py to notebooks.ipynb Is there a workflow to verify these still work before a push to master?

@thogar-computer
Copy link
Member

thogar-computer commented Jun 9, 2022

Developer input on how testing and testing workflow might be setup for easier work (scalable) - This can be done via a meeeting

@lukegorman
Copy link
Contributor

maybe pick this up tomorrow

@jpolton
Copy link
Collaborator Author

jpolton commented Sep 16, 2022

example_scripts are physically removed from unit_testing
example_scripts/notebooks are tested during the documentation build.
Component functionality is tested during component testing.
A new issue to map notebook testing GitAction from COAsT-site to COAsT: Issue 552

@jpolton jpolton closed this as completed Sep 16, 2022
Repository owner moved this from BODC - Todo to Done in COAsT_dev Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants