add some waypoint related E2E test cases#580
Conversation
Signed-off-by: YaoZengzeng <yaozengzeng@huawei.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Continue to review full report in Codecov by Sentry.
|
hzxuzhonghu
left a comment
There was a problem hiding this comment.
Since most codes are from istio, we need to add some document, suggest add a readme in e2e directory
| opt.Count = 10 | ||
| c := singleHost | ||
| if shouldBalance { | ||
| c = multipleHost |
There was a problem hiding this comment.
do we have workload level waypoint?
There was a problem hiding this comment.
Not yet, relevant test cases will be added later
test/e2e/baseline_test.go
Outdated
| - headers: | ||
| request: | ||
| add: | ||
| istio-custom-header: user-defined-value |
test/e2e/baseline_test.go
Outdated
| check.RequestHeader("Istio-Custom-Header", "user-defined-value")) | ||
| src.CallOrFail(t, opt) | ||
| }) | ||
| t.NewSubTest("subset").Run(func(t framework.TestContext) { |
There was a problem hiding this comment.
replace subset with a more meaningful name
Signed-off-by: YaoZengzeng <yaozengzeng@huawei.com>
|
@hzxuzhonghu updated |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hzxuzhonghu The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind enhancement
What this PR does / why we need it:
migrate some waypoint related E2E test cases
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: