Added a test to verify the instrumentation support for the Spring HTTP client RestClient. #3442
Conversation
github-actions
bot
added
agent-java
community
|
👋 @videnkz Thanks a lot for your contribution! It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it. Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it. |
videnkz
changed the title
| import static com.github.tomakehurst.wiremock.client.WireMock.urlEqualTo; | ||
| import static org.assertj.core.api.Assertions.assertThat; | ||
|
|
||
| public class SprintRestClientIntegration extends AbstractInstrumentationTest { |
There was a problem hiding this comment.
Does this test bring any benefit In addition to the SpringRestClientInstrumentationTest? To me it seems like they cover the same things.
If it doesn't cover anything I'm missing here, let's keep it simple and remove SpringRestClientIntegration
|
@elasticmachine run elasticsearch-ci/docs |
|
💚 CLA has been signed |
videnkz
force-pushed
the
spring6-restclient
branch
from
923899a to
e578892
Compare
|
@elasticmachine run elasticsearch-ci/docs |
What does this PR do?
Added a test to verify the instrumentation support for the Spring HTTP client
RestClient.closes #3441
Checklist
I have updated CHANGELOG.asciidoc