Skip to content

Commit

Permalink
Fix Jagger operation name
Browse files Browse the repository at this point in the history
Jagger service `vertx-web-client` is tracing operating called `/trace/ping`, however tests were looking for `trace/ping` operation and thus failed. Fixed tests are `ChuckNorrisResourceIT#endpointShouldTrace` and `ChuckNorrisResourceIT#httpClientShouldHaveHisOwnSpan`.
  • Loading branch information
michalvavrik committed May 27, 2022
1 parent 73e912e commit 58d6620
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ public void getTimeoutWhenResponseItsTooSlow() {
@Test
public void endpointShouldTrace() {
final int pageLimit = 50;
final String expectedOperationName = "trace/ping";
final String expectedOperationName = "/trace/ping";
await().atMost(1, TimeUnit.MINUTES).pollInterval(Duration.ofSeconds(1)).untilAsserted(() -> {
whenIMakePingRequest();
thenRetrieveTraces(pageLimit, "1h", getServiceName(), expectedOperationName);
Expand All @@ -117,7 +117,7 @@ public void endpointShouldTrace() {
@Test
public void httpClientShouldHaveHisOwnSpan() {
final int pageLimit = 50;
final String expectedOperationName = "trace/ping";
final String expectedOperationName = "/trace/ping";
await().atMost(1, TimeUnit.MINUTES).pollInterval(Duration.ofSeconds(1)).untilAsserted(() -> {
whenIMakePingRequest();
thenRetrieveTraces(pageLimit, "1h", getServiceName(), expectedOperationName);
Expand Down

0 comments on commit 58d6620

Please sign in to comment.