-
Notifications
You must be signed in to change notification settings - Fork 83
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
try run tests in check-mode, too #328
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #328 +/- ##
==========================================
+ Coverage 70.79% 70.84% +0.05%
==========================================
Files 18 18
Lines 1883 1883
Branches 327 327
==========================================
+ Hits 1333 1334 +1
Misses 405 405
+ Partials 145 144 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
adf6a37
to
d3f5817
Compare
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.
great improvement!
I left suggestions for fqcn in site.yml files, don't know if it's necessary
tests/integration/targets/grafana_notification_channel/site.yml
Outdated
Show resolved
Hide resolved
when a runme.sh exists in the test-path, it will be run. then the roles will be run in check-mode and normal-mode to test if they work in check mode (at least somewhat)
e9cf1aa
to
3690867
Compare
SUMMARY
when a runme.sh exists in the test-path, it will be run. then the roles will be run in check-mode and normal-mode to test if they work in check mode (at least somewhat)
ISSUE TYPE