ci(build): Add make targets for running integration tests #4207
Conversation
kislaykishore
requested review from
a team,
Tulsishah and
meet2mky
as code owners
github-actions
bot
added
the
remind-reviewers
kislaykishore
force-pushed
the
make-test
branch
from
09ac2ef to
7f302c4
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4207 +/- ##
==========================================
- Coverage 83.09% 83.08% -0.02%
==========================================
Files 154 154
Lines 18980 18980
==========================================
- Hits 15772 15770 -2
- Misses 2631 2633 +2
Partials 577 577
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi @meet2mky, @Tulsishah, @charith87, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
2 similar comments
|
Hi @meet2mky, @Tulsishah, @charith87, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @Tulsishah, @charith87, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
2 similar comments
|
Hi @meet2mky, @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
Hi @meet2mky, @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
|
@kislaykishore description still says unit tests but it seems you removed it? Is running integ_test with only regional expected? |
|
Hi @Tulsishah, your feedback is needed to move this pull request forward. This automated reminder was triggered because there has been no activity for over 24 hours. Please provide your input when you have a moment. Thank you! |
kislaykishore
changed the title
kislaykishore
force-pushed
the
make-test
branch
2 times, most recently
from
fa3c3dd to
d046b1f
Compare
kislaykishore
force-pushed
the
make-test
branch
from
d046b1f to
026551e
Compare
3 participants
Description
Add
maketargets for running integration tests. This allows one to executemake integ-testto run integration testsLink to the issue in case of a bug fix.
Testing details
Any backward incompatible change? If so, please explain.
No