-
Notifications
You must be signed in to change notification settings - Fork 542
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
feat: rewrite e2e test(route-with-valid-remote-addr-test) with ginkgo #1560
feat: rewrite e2e test(route-with-valid-remote-addr-test) with ginkgo #1560
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1560 +/- ##
==========================================
- Coverage 70.18% 69.12% -1.07%
==========================================
Files 130 49 -81
Lines 5340 3164 -2176
Branches 549 0 -549
==========================================
- Hits 3748 2187 -1561
+ Misses 1344 729 -615
Partials 248 248
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
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.
Using table.Entry
is better for us, and we could clearly see the comparison before and after the modification.
Here also need to create route and replace upstream_ id, |
the old test cases are all table driven |
It's my problem. I'll revise it |
merged. thanks a lot. @Jaycean |
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
Feat: Rewrite all E2E tests using the Ginkgo #1500
New feature or improvement