wu-sheng commented on a change in pull request #5493:
URL: https://github.com/apache/skywalking/pull/5493#discussion_r493661714
##########
File path:
test/plugin/scenarios/webflux-scenario/webflux-projectB-scenario/src/main/java/test/apache/skywalking/apm/testcase/sc/webflux/projectB/controller/TestAnnotationController.java
##########
@@ -44,6 +44,11 @@ public String error() {
}
return "1";
}
+
+ @RequestMapping("/testcase/webclient/server")
Review comment:
OK, this screenshot proved the context propagation works, but, we still
need in the plugin test, otherwise, we can't be sure this plugin also works.
So, I think the things you mentioned are caused by the complexity of this
scenario. It has too many request, so, some parts may be missed when the
verification begins.
How about setting up a specific tests for webflux-webclient only?
----------------------------------------------------------------
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]
