andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Add 'toString' method into IdleStateEvent (#9695)

Browse Source 
IdleStateEvent is very convenient and frequently used type of events. However both in runtime (logs) and in debug you need some manual steps to see their actual content. Default implementation generates worthless trash like this:

    io.netty.handler.timeout.IdleStateEvent@27f674d

There are examples already, where event has convenient and useful toString implementation:

* io.netty.handler.proxy.ProxyConnectionEvent
* io.netty.handler.ssl.SslCompletionEvent

* Implement 'IdleStateEvent.toString' method.
* Unit test.

More useful String representation of IdleStateEvent
This commit is contained in:
ursa 2019-10-23 09:28:18 +01:00 committed by Norman Maurer
parent b17371c198
commit aa78d49de1
2 changed files with 45 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
handler/src/main/java/io/netty/handler/timeout/IdleStateEvent.java
View File

@ -19,6 +19,8 @@ import static java.util.Objects.requireNonNull;
import io.netty.channel.Channel; import io.netty.channel.Channel;
import io.netty.util.internal.StringUtil;
/** /**
* A user event triggered by {@link IdleStateHandler} when a {@link Channel} is idle. * A user event triggered by {@link IdleStateHandler} when a {@link Channel} is idle.
*/ */
@ -32,6 +34,7 @@ public class IdleStateEvent {
private final IdleState state; private final IdleState state;
private final boolean first; private final boolean first;
private String strVal;
/** /**
* Constructor for sub-classes. * Constructor for sub-classes.
@ -57,4 +60,12 @@ public class IdleStateEvent {
public boolean isFirst() { public boolean isFirst() {
return first; return first;
} }
@Override
public String toString() {
if (strVal == null) {
strVal = StringUtil.simpleClassName(this) + "(" + state + (first ? ", first" : "") + ')';
}
return strVal;
}
} }

34
handler/src/test/java/io/netty/handler/timeout/IdleStateEventTest.java Normal file
View File

@ -0,0 +1,34 @@
/*
* Copyright 2019 The Netty Project
*
* The Netty Project licenses this file to you under the Apache License,
* version 2.0 (the "License"); you may not use this file except in compliance
* with the License. You may obtain a copy of the License at:
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
* WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
* License for the specific language governing permissions and limitations
* under the License.
*/
package io.netty.handler.timeout;
import org.junit.Test;
import static io.netty.handler.timeout.IdleStateEvent.*;
import static org.hamcrest.Matchers.hasToString;
import static org.junit.Assert.*;
public class IdleStateEventTest {
@Test
public void testHumanReadableToString() {
assertThat(FIRST_READER_IDLE_STATE_EVENT, hasToString("IdleStateEvent(READER_IDLE, first)"));
assertThat(READER_IDLE_STATE_EVENT, hasToString("IdleStateEvent(READER_IDLE)"));
assertThat(FIRST_WRITER_IDLE_STATE_EVENT, hasToString("IdleStateEvent(WRITER_IDLE, first)"));
assertThat(WRITER_IDLE_STATE_EVENT, hasToString("IdleStateEvent(WRITER_IDLE)"));
assertThat(FIRST_ALL_IDLE_STATE_EVENT, hasToString("IdleStateEvent(ALL_IDLE, first)"));
assertThat(ALL_IDLE_STATE_EVENT, hasToString("IdleStateEvent(ALL_IDLE)"));
}
}