diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2021-10-04 12:27:03 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2021-10-04 12:27:03 +0200 |
commit | 93b789c66b883d5ce2dc9ad9edcd164bd4539ff3 (patch) | |
tree | 6b7ebecb2c778a1911b28983d4cee4e04ba96ec9 /jdisc_core | |
parent | 84051a519ba5741edc897328775db8b36d87dbd5 (diff) |
Avoid use of deprecated Request.setServerRequest.
Diffstat (limited to 'jdisc_core')
-rw-r--r-- | jdisc_core/src/main/java/com/yahoo/jdisc/Request.java | 6 | ||||
-rw-r--r-- | jdisc_core/src/test/java/com/yahoo/jdisc/core/ContainerSnapshotTestCase.java | 19 |
2 files changed, 10 insertions, 15 deletions
diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/Request.java b/jdisc_core/src/main/java/com/yahoo/jdisc/Request.java index c1431ff8ba8..e2cde8e806c 100644 --- a/jdisc_core/src/main/java/com/yahoo/jdisc/Request.java +++ b/jdisc_core/src/main/java/com/yahoo/jdisc/Request.java @@ -79,9 +79,13 @@ public class Request extends AbstractResource { * @param uri The identifier of this request. */ public Request(CurrentContainer current, URI uri) { + this(current, uri, true); + } + + public Request(CurrentContainer current, URI uri, boolean isServerRequest) { parent = null; parentReference = null; - serverRequest = true; + serverRequest = isServerRequest; setUri(uri); container = current.newReference(uri); creationTime = container.currentTimeMillis(); diff --git a/jdisc_core/src/test/java/com/yahoo/jdisc/core/ContainerSnapshotTestCase.java b/jdisc_core/src/test/java/com/yahoo/jdisc/core/ContainerSnapshotTestCase.java index d8f8087bd46..ac2efafbba5 100644 --- a/jdisc_core/src/test/java/com/yahoo/jdisc/core/ContainerSnapshotTestCase.java +++ b/jdisc_core/src/test/java/com/yahoo/jdisc/core/ContainerSnapshotTestCase.java @@ -31,7 +31,6 @@ import static org.junit.Assert.assertTrue; public class ContainerSnapshotTestCase { @Test - @SuppressWarnings("deprecation") public void requireThatServerHandlerCanBeResolved() { TestDriver driver = TestDriver.newSimpleApplicationInstanceWithoutOsgi(); ContainerBuilder builder = driver.newContainerBuilder(); @@ -43,8 +42,7 @@ public class ContainerSnapshotTestCase { assertNotNull(request.getBindingMatch()); request.release(); - request = new Request(driver, URI.create("http://foo/")); - request.setServerRequest(false); + request = new Request(driver, URI.create("http://foo/"), false); assertNull(request.container().resolveHandler(request)); assertNull(request.getBindingMatch()); request.release(); @@ -54,8 +52,7 @@ public class ContainerSnapshotTestCase { assertNull(request.getBindingMatch()); request.release(); - request = new Request(driver, URI.create("http://bar/")); - request.setServerRequest(false); + request = new Request(driver, URI.create("http://bar/"), false); assertNull(request.container().resolveHandler(request)); assertNull(request.getBindingMatch()); request.release(); @@ -64,7 +61,6 @@ public class ContainerSnapshotTestCase { } @Test - @SuppressWarnings("deprecation") public void requireThatClientHandlerCanBeResolved() { TestDriver driver = TestDriver.newSimpleApplicationInstanceWithoutOsgi(); ContainerBuilder builder = driver.newContainerBuilder(); @@ -76,8 +72,7 @@ public class ContainerSnapshotTestCase { assertNull(request.getBindingMatch()); request.release(); - request = new Request(driver, URI.create("http://foo/")); - request.setServerRequest(false); + request = new Request(driver, URI.create("http://foo/"), false); assertNotNull(request.container().resolveHandler(request)); assertNotNull(request.getBindingMatch()); request.release(); @@ -87,8 +82,7 @@ public class ContainerSnapshotTestCase { assertNull(request.getBindingMatch()); request.release(); - request = new Request(driver, URI.create("http://bar/")); - request.setServerRequest(false); + request = new Request(driver, URI.create("http://bar/"), false); assertNull(request.container().resolveHandler(request)); assertNull(request.getBindingMatch()); request.release(); @@ -97,15 +91,12 @@ public class ContainerSnapshotTestCase { } @Test - @SuppressWarnings("deprecation") public void requireThatClientBindingsAreUsed() { TestDriver driver = TestDriver.newSimpleApplicationInstanceWithoutOsgi(); ContainerBuilder builder = driver.newContainerBuilder(); builder.clientBindings().bind("http://host/path", MyRequestHandler.newInstance()); driver.activateContainer(builder); - Request request = new Request(driver, URI.create("http://host/path")); - assertNull(request.container().resolveHandler(request)); - request.setServerRequest(false); + Request request = new Request(driver, URI.create("http://host/path"), false); assertNotNull(request.container().resolveHandler(request)); request.release(); assertTrue(driver.close()); |