-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fix notebook tests #320
Fix notebook tests #320
Conversation
Also: - Add safe bash header - Make runnable locally
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev_master #320 +/- ##
===========================================
Coverage 77.04% 77.04%
===========================================
Files 57 57
Lines 7705 7705
===========================================
Hits 5936 5936
Misses 1769 1769 ☔ View full report in Codecov by Sentry. |
I'd say this works now |
@@ -1,53 +1,56 @@ | |||
#!/usr/bin/env bash | |||
|
|||
echo "# Running Notebooks Tests" >> $GITHUB_STEP_SUMMARY | |||
# https://betterdev.blog/minimal-safe-bash-script-template/ | |||
set -Eeuo pipefail |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uch, I messed this up, because I did not try to run the script without any parameters. If $1 is unset, the script breaks. See https://github.com/AstarVienna/ScopeSim_Data/actions/runs/7405748794
Rework the notebook tests to have a single workflow actually running the notebooks, with two separate workflows requesting it with or without git cloning the IRDB repo.