-
Notifications
You must be signed in to change notification settings - Fork 69
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(ci): add e2e test framework #72
Conversation
Codecov Report
@@ Coverage Diff @@
## master #72 +/- ##
==========================================
- Coverage 80.84% 79.49% -1.36%
==========================================
Files 11 11
Lines 710 746 +36
==========================================
+ Hits 574 593 +19
- Misses 100 114 +14
- Partials 36 39 +3
Continue to review full report at Codecov.
|
|
||
- name: startup runner | ||
run: | | ||
APISIX_LISTEN_ADDRESS=unix:/tmp/runner.sock APISIX_CONF_EXPIRE_TIME=3600 ./go-runner run & |
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.
Is it possible to let APISIX manage the go-runner, just like in the product environment?
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.
I will solve it by creating a new PR.
image: mendhak/http-https-echo | ||
environment: | ||
HTTP_PORT: 8888 | ||
HTTPS_PORT: 9999 |
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.
The HTTPS port is unused.
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.
Clear!
Headers: map[string]string{"X-API-KEY": tools.GetAdminToken()}, | ||
ExpectStatusRange: httpexpect.Status2xx, | ||
}), | ||
table.Entry("Test if limit-req plugin work.", tools.HttpTestCase{ |
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.
The limit-req test doesn't test the limit req behavior...
add e2e test framework