diff options
author | Bjørn Christian Seime <bjorncs@yahoo-inc.com> | 2017-04-24 20:43:05 +0200 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@yahoo-inc.com> | 2017-04-24 20:43:05 +0200 |
commit | 9295b4f5f04bba79de95c45391af9769badf0722 (patch) | |
tree | 359d6e0f088be0480bd34080681c86e4deb621ff /jdisc_http_service/src/test/java/com/yahoo/jdisc/http | |
parent | 4023d56315ea917d798018754a3f5a91e7bccbd5 (diff) |
Remove non-working tracing test
The trace tests have never been enabled and there is no sign of the
trace headers in the source code except the unit tests.
Diffstat (limited to 'jdisc_http_service/src/test/java/com/yahoo/jdisc/http')
3 files changed, 1 insertions, 41 deletions
diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpHeadersTestCase.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpHeadersTestCase.java index 1472c411c38..9be39ddc464 100644 --- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpHeadersTestCase.java +++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpHeadersTestCase.java @@ -13,7 +13,5 @@ public class HttpHeadersTestCase { @Test public void requireThatHeadersDoNotChange() { assertEquals("X-JDisc-Disable-Chunking", HttpHeaders.Names.X_DISABLE_CHUNKING); - assertEquals("X-JDisc-Enable-TraceId", HttpHeaders.Names.X_ENABLE_TRACE_ID); - assertEquals("X-JDisc-TraceId", HttpHeaders.Names.X_TRACE_ID); } } diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpRequestTestCase.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpRequestTestCase.java index 021a14b2ae7..8f26d9c38b0 100644 --- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpRequestTestCase.java +++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/HttpRequestTestCase.java @@ -4,7 +4,6 @@ package com.yahoo.jdisc.http; import com.yahoo.jdisc.Container; import com.yahoo.jdisc.Request; import com.yahoo.jdisc.service.CurrentContainer; -import com.yahoo.jdisc.test.TestDriver; import org.jboss.netty.handler.codec.http.HttpHeaders; import org.jboss.netty.handler.codec.http.HttpMethod; import org.jboss.netty.handler.codec.http.HttpVersion; @@ -198,12 +197,6 @@ public class HttpRequestTestCase { } @Test - public void requireThatTraceIsDisabledByDefault() throws Exception { - final HttpRequest request = newRequest(HttpRequest.Version.HTTP_1_0); - assertFalse(request.headers().contains(com.yahoo.jdisc.http.HttpHeaders.Names.X_ENABLE_TRACE_ID, "true")); - } - - @Test public void requireThatCookieHeaderCanBeEncoded() throws Exception { final HttpRequest request = newRequest(HttpRequest.Version.HTTP_1_0); final List<Cookie> cookies = Collections.singletonList(new Cookie("foo", "bar")); diff --git a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java index febd6f5d926..465bd34f0c0 100644 --- a/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java +++ b/jdisc_http_service/src/test/java/com/yahoo/jdisc/http/server/jetty/HttpServerTest.java @@ -4,6 +4,7 @@ package com.yahoo.jdisc.http.server.jetty; import com.google.inject.AbstractModule; import com.google.inject.Module; import com.yahoo.jdisc.HeaderFields; +import com.yahoo.jdisc.References; import com.yahoo.jdisc.Request; import com.yahoo.jdisc.Response; import com.yahoo.jdisc.application.BindingSetSelector; @@ -19,7 +20,6 @@ import com.yahoo.jdisc.http.HttpRequest; import com.yahoo.jdisc.http.HttpResponse; import com.yahoo.jdisc.http.ServerConfig; import com.yahoo.jdisc.service.BindingSetNotFoundException; -import com.yahoo.jdisc.References; import org.apache.http.entity.ContentType; import org.apache.http.entity.mime.FormBodyPart; import org.apache.http.entity.mime.content.StringBody; @@ -48,7 +48,6 @@ import static com.yahoo.jdisc.http.HttpHeaders.Names.CONNECTION; import static com.yahoo.jdisc.http.HttpHeaders.Names.CONTENT_TYPE; import static com.yahoo.jdisc.http.HttpHeaders.Names.COOKIE; import static com.yahoo.jdisc.http.HttpHeaders.Names.X_DISABLE_CHUNKING; -import static com.yahoo.jdisc.http.HttpHeaders.Names.X_TRACE_ID; import static com.yahoo.jdisc.http.HttpHeaders.Values.APPLICATION_X_WWW_FORM_URLENCODED; import static com.yahoo.jdisc.http.HttpHeaders.Values.CLOSE; import static com.yahoo.jdisc.http.server.jetty.SimpleHttpClient.ResponseValidator; @@ -373,36 +372,6 @@ public class HttpServerTest { assertThat(driver.close(), is(true)); } - @Test(enabled = false) - public void requireThatGeneratedTraceIdIsSet() throws Exception { - final TestDriver driver = TestDrivers.newInstance(new EchoRequestHandler()); - final SimpleHttpClient client1 = driver.client(); - final SimpleHttpClient client2 = driver.newClient(); - - client1.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000000")); - client1.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000001")); - client2.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000000")); - client1.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000002")); - client2.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000001")); - client2.newGet("/status.html").addHeader(X_TRACE_ID, "true").execute() - .expectHeader("X-JDisc-TraceId", matchesPattern("\\w+00000002")); - - assertThat(driver.close(), is(true)); - } - - @Test(enabled = false) - public void requireThatClientTraceIdIsSet() throws Exception { - final TestDriver driver = TestDrivers.newInstance(new EchoRequestHandler()); - driver.client().newGet("/status.html").addHeader(X_TRACE_ID, "foo").execute() - .expectHeader(X_TRACE_ID, is("foo")); - assertThat(driver.close(), is(true)); - } - @Test public void requireThatTimeoutWorks() throws Exception { final UnresponsiveHandler requestHandler = new UnresponsiveHandler(); |