test(integration): add dedicated NPI test suite to e2e test runner - #5015
test(integration): add dedicated NPI test suite to e2e test runner#5015kislaykishore wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces support for running Network Protocol Integration (NPI / Pirlo) end-to-end tests. It adds a new --npi flag to improved_run_e2e_tests.sh and handles the RUN_TESTS_FOR_NPI environment variable in build.sh to trigger the NPI test suite across both HNS and FLAT bucket layouts. I have no feedback to provide as there are no review comments.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #5015 +/- ##
==========================================
- Coverage 83.81% 0 -83.82%
==========================================
Files 174 0 -174
Lines 21376 0 -21376
==========================================
- Hits 17916 0 -17916
+ Misses 2779 0 -2779
+ Partials 681 0 -681
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Hi @meet2mky, @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! |
3 similar comments
|
Hi @meet2mky, @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, @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, @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! |
Description
This PR introduces a dedicated integration test configuration for NPI related tests: CLI Runner Integration (
--npi)Added
--npioption toimproved_run_e2e_tests.shto trigger NPI test groups across Flat and HNS regional buckets.Link to the issue in case of a bug fix.
b/530771587
Testing details
make build. Validated shell scripts syntax withbash -n.Any backward incompatible change? If so, please explain.
N/A