wu-sheng commented on a change in pull request #5493:
URL: https://github.com/apache/skywalking/pull/5493#discussion_r491643236
##########
File path:
test/plugin/scenarios/webflux-scenario/webflux-projectA-scenario/src/main/java/org/apache/skywalking/apm/testcase/sc/webflux/projectA/controller/TestController.java
##########
@@ -43,6 +45,7 @@ public String testcase() throws IOException {
visit("http://"; + hostBAddress + "/testcase/route/error");
visit("http://"; + hostBAddress + "/notFound");
visit("http://"; + hostBAddress + "/testcase/annotation/mono/hello");
+ visit("http://"; + clientHostAddress + "/testcase/webclient/testGet");
Review comment:
What is the difference between this new link and above L45-L47? You seem
only to change the target address. Why you do `WebClient` in another new
service rather than doing directly here? I am feeling the new service in the
test is not necessary.
Please correct me if I am wrong.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
