Eclipse
/
jetty.project
mirror of https://github.com/jetty/jetty.project.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

cleanup dumps from review 

Signed-off-by: Greg Wilkins <gregw@webtide.com>
This commit is contained in:
Greg Wilkins 2019-05-02 10:57:51 +02:00
parent aafb0fcec1
commit 604f2379bd
2 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
jetty-server/src/main/java/org/eclipse/jetty/server/handler/InetAccessHandler.java
View File

@ -34,6 +34,7 @@ import org.eclipse.jetty.util.IncludeExclude;
import org.eclipse.jetty.util.IncludeExcludeSet;
import org.eclipse.jetty.util.InetAddressSet;
import org.eclipse.jetty.util.component.Dumpable;
import org.eclipse.jetty.util.component.DumpableCollection;
import org.eclipse.jetty.util.log.Log;
import org.eclipse.jetty.util.log.Logger;
@ -202,9 +203,9 @@ public class InetAccessHandler extends HandlerWrapper
public void dump(Appendable out, String indent) throws IOException
{
dumpObjects(out, indent,
Dumpable.labelled("included", _addrs.getIncluded()),
Dumpable.labelled("excluded", _addrs.getExcluded()),
Dumpable.labelled("includedConnectorNames", _names.getIncluded()),
Dumpable.labelled("excludedConnectorNames", _names.getExcluded()));
new DumpableCollection("included", _addrs.getIncluded()),
new DumpableCollection("excluded", _addrs.getExcluded()),
new DumpableCollection("includedConnectorNames", _names.getIncluded()),
new DumpableCollection("excludedConnectorNames", _names.getExcluded()));
}
}

12
jetty-util/src/test/java/org/eclipse/jetty/util/component/DumpableCollectionTest.java → jetty-util/src/test/java/org/eclipse/jetty/util/component/DumpableTest.java
View File

@ -26,9 +26,8 @@ import java.util.Collection;
import org.hamcrest.Matchers;
import org.junit.jupiter.api.Test;
public class DumpableCollectionTest
public class DumpableTest
{
@Test
public void testNullDumpableCollection () throws Exception
{
@ -51,4 +50,13 @@ public class DumpableCollectionTest
assertThat(dump, Matchers.containsString("two"));
assertThat(dump, Matchers.containsString("three"));
}
@Test
public void testDumpableLabelled() throws Exception
{
String dump = Dumpable.labelled("label", "Item").dump();
assertThat(dump, Matchers.containsString("label:"));
assertThat(dump, Matchers.containsString("Item"));
}
}