Skip to content

Commit 2f0364e

Browse files
authored
Replace field with local variable in grizzly test (#12297)
1 parent 1b5d47e commit 2f0364e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

instrumentation/grizzly-2.3/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/grizzly/GrizzlyFilterchainServerTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,10 @@ class GrizzlyFilterchainServerTest extends AbstractHttpServerTest<Transport> {
5050
@RegisterExtension
5151
static final InstrumentationExtension testing = HttpServerInstrumentationExtension.forAgent();
5252

53-
private TCPNIOTransport transport;
54-
5553
@Override
5654
protected Transport setupServer() throws Exception {
5755
FilterChain filterChain = setUpFilterChain();
58-
setUpTransport(filterChain);
56+
TCPNIOTransport transport = setUpTransport(filterChain);
5957

6058
transport.bind("127.0.0.1", port);
6159
transport.start();
@@ -88,7 +86,7 @@ protected void configure(HttpServerTestOptions options) {
8886
options.setTestErrorBody(false);
8987
}
9088

91-
private void setUpTransport(FilterChain filterChain) {
89+
private static TCPNIOTransport setUpTransport(FilterChain filterChain) {
9290
TCPNIOTransportBuilder transportBuilder =
9391
TCPNIOTransportBuilder.newInstance().setOptimizedForMultiplexing(true);
9492

@@ -98,8 +96,10 @@ private void setUpTransport(FilterChain filterChain) {
9896
transportBuilder.setServerConnectionBackLog(50);
9997
transportBuilder.setServerSocketSoTimeout(80000);
10098

101-
transport = transportBuilder.build();
99+
TCPNIOTransport transport = transportBuilder.build();
102100
transport.setProcessor(filterChain);
101+
102+
return transport;
103103
}
104104

105105
@SuppressWarnings(

0 commit comments

Comments
 (0)