fixed HTTP integration tests - trailing slashes were the issue (#120)

petabridge · Nov 18, 2019 · 59dd481 · 59dd481
1 parent e2d28bb
commit 59dd481
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Petabridge.Tracing.Zipkin.Integration.Tests/Http/ZipkinHttpIntegrationSpecs.cs b/src/Petabridge.Tracing.Zipkin.Integration.Tests/Http/ZipkinHttpIntegrationSpecs.cs
@@ -90,7 +90,7 @@ public async Task HttpReporterShouldPostCorrectlyToZipkin()
             resp2.IsSuccessStatusCode.Should().BeTrue(
                 $"Expected success status code, but instead found [{resp2.StatusCode}][{resp2.ReasonPhrase}]");
 
-            var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}/");
+            var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}");
             HttpResponseMessage traceResp = null;
             var retries = 3;
             for (var i = 1; i <= retries; i++)
@@ -132,7 +132,7 @@ public async Task ShouldPostTraceToZipkin()
                 traceId = active.Span.Context.AsInstanceOf<IZipkinSpanContext>().TraceId;
             }
 
-            var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}/");
+            var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}");
             HttpResponseMessage traceResp = null;
             var retries = 3;
             for (var i = 1; i <= retries; i++)

diff --git a/src/Petabridge.Tracing.Zipkin.Integration.Tests/Kafka/ZipkinKafkaIntegrationSpecs.cs b/src/Petabridge.Tracing.Zipkin.Integration.Tests/Kafka/ZipkinKafkaIntegrationSpecs.cs
@@ -70,7 +70,7 @@ public async Task ShouldPostTraceToZipkin()
                     traceId = active.Span.Context.AsInstanceOf<IZipkinSpanContext>().TraceId;
                 }
 
-                var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}/");
+                var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}");
 
                 try
                 {
-Original file line number
+Diff line change
@@ Expand Up / @@ -70,7 +70,7 @@ public async Task ShouldPostTraceToZipkin() @@
                         traceId = active.Span.Context.AsInstanceOf<IZipkinSpanContext>().TraceId;
                     }
-                    var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}/");
+                    var fullUri = new Uri(_httpBaseUri, $"api/v2/trace/{traceId}");
                     try
                     {
@@ Expand Down @@