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

Merge remote-tracking branch 'origin/jetty-9.4.x' into jetty-9.4.x-3681-HttpFieldsOptimize

This commit is contained in:
Greg Wilkins 2019-05-26 16:18:13 +02:00
parent 53dc425bde a3b755858d
commit cf9c238721
17 changed files with 403 additions and 60 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Jmh_Jenkinsfile
View File

@ -4,20 +4,15 @@ def branch = params.get("JETTY_BRANCH" ,"jetty-9.4.x")
def owner = params.get("REPO_OWNER", "eclipse")
def jdk = params.get("JDK", "jdk8")
def jmhJarPath = params.get("jmhJarPath","jetty-jmh/target/benchmarks.jar")
currentBuild.description = "Build branch $branch with jdk $jdk"
node("linux") {
// System Dependent Locations
def mvntool = tool name: 'maven3.5', type: 'hudson.tasks.Maven$MavenInstallation'
def jdktool = tool name: "$jdk", type: 'hudson.model.JDK'
def mvnName = 'maven3.5'
def localRepo = "${env.JENKINS_HOME}/${env.EXECUTOR_NUMBER}"
def settingsName = 'oss-settings.xml'
def mavenOpts = '-Xms1g -Xmx4g -Djava.awt.headless=true'
// Environment
List mvnEnv = ["PATH+MVN=${mvntool}/bin", "PATH+JDK=${jdktool}/bin", "JAVA_HOME=${jdktool}/", "MAVEN_HOME=${mvntool}"]
mvnEnv.add("MAVEN_OPTS=$mavenOpts")
stage("Checkout") {
git url: "https://github.com/$owner/jetty.project.git", branch: "$branch"
@ -41,15 +36,14 @@ node("linux") {
}
// jmh run
stage("jmh-run") {
node( 'jmh-build-node' ) {
node( 'jmh-build-node' ) {
stage("jmh-run") {
timeout( time: 180, unit: 'MINUTES' ) {
withEnv( ["JAVA_HOME=${tool "$jdk"}"] ) {
unstash name: 'perf-tests'
sh "rm -rf jmh_results"
sh "mkdir jmh_results"
sh "${env.JAVA_HOME}/bin/java -jar $jmhJarPath -rff jmh_results/jmh_result.json -rf json"
sh "${env.JAVA_HOME}/bin/java -jar $jmhJarPath -rff jmh_results/jmh_result.json -rf json -foe true"
jmhReport 'jmh_results/jmh_result.json'
}
}

3
jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientTLSTest.java
View File

@ -45,6 +45,7 @@ import org.eclipse.jetty.io.ssl.SslHandshakeListener;
import org.eclipse.jetty.server.Handler;
import org.eclipse.jetty.server.Server;
import org.eclipse.jetty.server.ServerConnector;
import org.eclipse.jetty.util.StringUtil;
import org.eclipse.jetty.util.ssl.SslContextFactory;
import org.eclipse.jetty.util.thread.ExecutorThreadPool;
import org.eclipse.jetty.util.thread.QueuedThreadPool;
@ -501,7 +502,7 @@ public class HttpClientTLSTest
while (true)
{
String line = reader.readLine();
if (line == null || line.isEmpty())
if (StringUtil.isEmpty(line))
break;
}

3
jetty-client/src/test/java/org/eclipse/jetty/client/HttpClientURITest.java
View File

@ -44,6 +44,7 @@ import org.eclipse.jetty.http.HttpStatus;
import org.eclipse.jetty.server.handler.AbstractHandler;
import org.eclipse.jetty.toolchain.test.Net;
import org.eclipse.jetty.util.Fields;
import org.eclipse.jetty.util.StringUtil;
import org.eclipse.jetty.util.URIUtil;
import org.junit.jupiter.api.Assumptions;
import org.junit.jupiter.params.ParameterizedTest;
@ -609,7 +610,7 @@ public class HttpClientURITest extends AbstractHttpClientServerTest
while (true)
{
String line = reader.readLine();
if (line == null || line.isEmpty())
if (StringUtil.isEmpty(line))
break;
}

3
jetty-fcgi/fcgi-server/src/main/java/org/eclipse/jetty/fcgi/server/HttpChannelOverFCGI.java
View File

@ -35,6 +35,7 @@ import org.eclipse.jetty.server.Connector;
import org.eclipse.jetty.server.HttpChannel;
import org.eclipse.jetty.server.HttpConfiguration;
import org.eclipse.jetty.server.HttpTransport;
import org.eclipse.jetty.util.StringUtil;
import org.eclipse.jetty.util.log.Log;
import org.eclipse.jetty.util.log.Logger;
@ -87,7 +88,7 @@ public class HttpChannelOverFCGI extends HttpChannel
public void onRequest()
{
String uri = path;
if (query != null && !query.isEmpty())
if (!StringUtil.isEmpty(query))
uri += "?" + query;
// TODO https?
onRequest(new MetaData.Request(method, HttpScheme.HTTP.asString(), hostPort, uri, HttpVersion.fromString(version), fields,Long.MIN_VALUE));

3
jetty-http/src/main/java/org/eclipse/jetty/http/HttpParser.java
View File

@ -28,6 +28,7 @@ import org.eclipse.jetty.http.HttpTokens.EndOfContent;
import org.eclipse.jetty.util.ArrayTernaryTrie;
import org.eclipse.jetty.util.ArrayTrie;
import org.eclipse.jetty.util.BufferUtil;
import org.eclipse.jetty.util.StringUtil;
import org.eclipse.jetty.util.Trie;
import org.eclipse.jetty.util.Utf8StringBuilder;
import org.eclipse.jetty.util.log.Log;
@ -1132,7 +1133,7 @@ public class HttpParser
throw new BadMessageException(HttpStatus.BAD_REQUEST_400,"Header Folding");
// header value without name - continuation?
if (_valueString==null || _valueString.isEmpty())
if ( StringUtil.isEmpty(_valueString))
{
_string.setLength(0);
_length=0;

3
jetty-http2/http2-server/src/main/java/org/eclipse/jetty/http2/server/HTTP2ServerConnectionFactory.java
View File

@ -38,6 +38,7 @@ import org.eclipse.jetty.server.Connector;
import org.eclipse.jetty.server.HttpConfiguration;
import org.eclipse.jetty.server.NegotiatingServerConnection.CipherDiscriminator;
import org.eclipse.jetty.util.Callback;
import org.eclipse.jetty.util.StringUtil;
import org.eclipse.jetty.util.annotation.Name;
import org.eclipse.jetty.util.log.Log;
import org.eclipse.jetty.util.log.Logger;
@ -116,7 +117,7 @@ public class HTTP2ServerConnectionFactory extends AbstractHTTP2ServerConnectionF
public void onClose(Session session, GoAwayFrame frame, Callback callback)
{
String reason = frame.tryConvertPayload();
if (reason != null && !reason.isEmpty())
if (!StringUtil.isEmpty(reason))
reason = " (" + reason + ")";
getConnection().onSessionFailure(new EofException(String.format("Close %s/%s", ErrorCode.toString(frame.getError(), null), reason)), callback);
}

32
jetty-jmh/src/main/java/org/eclipse/jetty/http/jmh/MultiPartBenchmark.java
View File

@ -26,7 +26,7 @@ import static org.hamcrest.Matchers.notNullValue;
import org.eclipse.jetty.http.MultiPartFormInputStream;
import org.eclipse.jetty.toolchain.test.Hex;
import org.eclipse.jetty.toolchain.test.IO;
import org.eclipse.jetty.util.IO;
import org.eclipse.jetty.util.QuotedStringTokenizer;
import org.eclipse.jetty.util.StringUtil;
import org.hamcrest.Matchers;
@ -57,6 +57,7 @@ import java.io.InputStream;
import java.io.OutputStream;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.nio.file.StandardOpenOption;
import java.security.DigestOutputStream;
import java.security.MessageDigest;
@ -177,7 +178,7 @@ public class MultiPartBenchmark
public long testLargeGenerated() throws Exception
{
Path multipartRawFile = _file.toPath();
Path outputDir = new File("/tmp").toPath();
Path outputDir = Files.createTempDirectory( "jetty_multipart_benchmark");
MultipartConfigElement config = newMultipartConfigElement(outputDir);
@ -188,11 +189,14 @@ public class MultiPartBenchmark
case "HTTP":
{
MultiPartFormInputStream parser = new MultiPartFormInputStream(in, _contentType, config, outputDir.toFile());
parser.setDeleteOnExit(true);
if (parser.getParts().size() != _numSections)
throw new IllegalStateException("Incorrect Parsing");
for (Part p : parser.getParts())
{
count += p.getSize();
if(p instanceof MultiPartFormInputStream.MultiPart) ((MultiPartFormInputStream.MultiPart)p).cleanUp();
else p.delete();
}
}
break;
@ -200,11 +204,14 @@ public class MultiPartBenchmark
case "UTIL":
{
org.eclipse.jetty.util.MultiPartInputStreamParser parser = new org.eclipse.jetty.util.MultiPartInputStreamParser(in, _contentType, config, outputDir.toFile());
parser.setDeleteOnExit(true);
if (parser.getParts().size() != _numSections)
throw new IllegalStateException("Incorrect Parsing");
for (Part p : parser.getParts())
{
count += p.getSize();
if(p instanceof org.eclipse.jetty.util.MultiPartInputStreamParser.MultiPart) ((org.eclipse.jetty.util.MultiPartInputStreamParser.MultiPart)p).cleanUp();
else p.delete();
}
}
break;
@ -213,7 +220,7 @@ public class MultiPartBenchmark
throw new IllegalStateException("Unknown parserType Parameter");
}
}
IO.delete(outputDir.toFile());
return count;
}
@ -236,18 +243,16 @@ public class MultiPartBenchmark
{
for (String multiPart : data)
{
//Path multipartRawFile = MavenTestingUtils.getTestResourcePathFile("multipart/" + multiPart + ".raw");
String expectationPath = "multipart/" + multiPart + ".expected.txt";
//Path expectationPath = MavenTestingUtils.getTestResourcePathFile("multipart/" + multiPart + ".expected.txt");
File expectationFile = File.createTempFile( expectationPath, ".tmp" );
try(InputStream inputStream = Thread.currentThread().getContextClassLoader().getResourceAsStream(expectationPath);
OutputStream os = Files.newOutputStream( expectationFile.toPath() )) {
IO.copy( inputStream, os );
try(InputStream inputStream = Thread.currentThread().getContextClassLoader().getResourceAsStream(expectationPath); //
OutputStream os = Files.newOutputStream( expectationFile.toPath())) {
IO.copy(inputStream, os);
}
Path outputDir = Files.createTempDirectory( "expected_output_jmh_jetty" );// new File("/tmp").toPath();
Path outputDir = Files.createTempDirectory("expected_output_jmh_jetty");
MultipartExpectations multipartExpectations = new MultipartExpectations(expectationFile.toPath());
MultipartConfigElement config = newMultipartConfigElement(outputDir);
@ -259,18 +264,24 @@ public class MultiPartBenchmark
case "HTTP":
{
MultiPartFormInputStream parser = new MultiPartFormInputStream(in, multipartExpectations.contentType, config, outputDir.toFile());
parser.setDeleteOnExit(true);
for (Part p : parser.getParts())
{
count += p.getSize();
if(p instanceof MultiPartFormInputStream.MultiPart) ((MultiPartFormInputStream.MultiPart)p).cleanUp();
else p.delete();
}
}
break;
case "UTIL":
{
org.eclipse.jetty.util.MultiPartInputStreamParser parser = new org.eclipse.jetty.util.MultiPartInputStreamParser(in, multipartExpectations.contentType, config, outputDir.toFile());
parser.setDeleteOnExit(true);
for (Part p : parser.getParts())
{
count += p.getSize();
if(p instanceof org.eclipse.jetty.util.MultiPartInputStreamParser.MultiPart) ((org.eclipse.jetty.util.MultiPartInputStreamParser.MultiPart)p).cleanUp();
else p.delete();
}
}
break;
@ -278,7 +289,8 @@ public class MultiPartBenchmark
throw new IllegalStateException("Unknown parserType Parameter");
}
}
Files.deleteIfExists(expectationFile.toPath());
IO.delete(outputDir.toFile());
}
return count;
}

27
jetty-jmh/src/main/java/org/eclipse/jetty/server/jmh/ForwardBenchmark.java
View File

@ -18,28 +18,9 @@
package org.eclipse.jetty.server.jmh;
import java.io.IOException;
import java.lang.invoke.MethodHandle;
import java.lang.invoke.MethodHandles;
import java.lang.invoke.MethodType;
import java.util.Arrays;
import java.util.concurrent.ThreadLocalRandom;
import java.util.concurrent.TimeUnit;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import org.eclipse.jetty.http.HttpField;
import org.eclipse.jetty.http.HttpFields;
import org.eclipse.jetty.http.HttpHeader;
import org.eclipse.jetty.server.ForwardedRequestCustomizer;
import org.eclipse.jetty.server.HttpConfiguration;
import org.eclipse.jetty.server.HttpConnectionFactory;
import org.eclipse.jetty.server.LocalConnector;
import org.eclipse.jetty.server.Request;
import org.eclipse.jetty.server.Server;
import org.eclipse.jetty.server.handler.AbstractHandler;
import org.eclipse.jetty.util.ArrayTrie;
import org.eclipse.jetty.util.Trie;
import org.openjdk.jmh.annotations.Benchmark;
@ -56,8 +37,12 @@ import org.openjdk.jmh.runner.RunnerException;
import org.openjdk.jmh.runner.options.Options;
import org.openjdk.jmh.runner.options.OptionsBuilder;
import static java.lang.invoke.MethodHandles.dropArguments;
import static java.lang.invoke.MethodHandles.foldArguments;
import java.lang.invoke.MethodHandle;
import java.lang.invoke.MethodHandles;
import java.lang.invoke.MethodType;
import java.util.Arrays;
import java.util.concurrent.TimeUnit;
import static java.lang.invoke.MethodType.methodType;

66
jetty-jmh/src/main/java/org/eclipse/jetty/util/StringIsEmptyBenchmark.java Normal file
View File

@ -0,0 +1,66 @@
//
// ========================================================================
// Copyright (c) 1995-2019 Mort Bay Consulting Pty. Ltd.
// ------------------------------------------------------------------------
// All rights reserved. This program and the accompanying materials
// are made available under the terms of the Eclipse Public License v1.0
// and Apache License v2.0 which accompanies this distribution.
//
// The Eclipse Public License is available at
// http://www.eclipse.org/legal/epl-v10.html
//
// The Apache License v2.0 is available at
// http://www.opensource.org/licenses/apache2.0.php
//
// You may elect to redistribute this code under either of these licenses.
// ========================================================================
//
package org.eclipse.jetty.util;
import org.openjdk.jmh.annotations.Benchmark;
import org.openjdk.jmh.annotations.BenchmarkMode;
import org.openjdk.jmh.annotations.Measurement;
import org.openjdk.jmh.annotations.Mode;
import org.openjdk.jmh.annotations.Scope;
import org.openjdk.jmh.annotations.State;
import org.openjdk.jmh.annotations.Threads;
import org.openjdk.jmh.annotations.Warmup;
import java.util.concurrent.TimeUnit;
@State( Scope.Benchmark)
@Threads(4)
@Warmup(iterations = 7, time = 500, timeUnit = TimeUnit.MILLISECONDS)
@Measurement(iterations = 9, time = 800, timeUnit = TimeUnit.MILLISECONDS)
public class StringIsEmptyBenchmark
{
private static final String SHORT = "beer.com/foo";
private static final String MEDIUM = "beer.com/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde";
private static final String LONG = "beer.com/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde/foobarbeeerbe/bebbebbebebbe/bebbeghdegde";
@Benchmark
@BenchmarkMode( Mode.Throughput)
public void shortIsEmpty()
{
StringUtil.isEmpty(SHORT);
}
@Benchmark
@BenchmarkMode( Mode.Throughput)
public void mediumIsEmpty()
{
StringUtil.isEmpty(MEDIUM);
}
@Benchmark
@BenchmarkMode( Mode.Throughput)
public void longIsEmpty()
{
StringUtil.isEmpty(LONG);
}
}

29
jetty-jmh/src/main/java/org/eclipse/jetty/util/thread/strategy/jmh/EWYKBenchmark.java
View File

@ -19,11 +19,16 @@
package org.eclipse.jetty.util.thread.strategy.jmh;
import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.CompletableFuture;
import java.util.concurrent.TimeUnit;
import org.eclipse.jetty.util.IO;
import org.eclipse.jetty.util.component.LifeCycle;
import org.eclipse.jetty.util.thread.ExecutionStrategy;
import org.eclipse.jetty.util.thread.Invocable;
@ -51,7 +56,7 @@ public class EWYKBenchmark
{
static TestServer server;
static ReservedThreadExecutor reserved;
static File directory;
static Path directory;
@Param({"PC","PEC","EWYK"})
public static String strategyName;
@ -66,21 +71,15 @@ public class EWYKBenchmark
public static void setupServer() throws Exception
{
// Make a test directory
directory = File.createTempFile("ewyk","dir");
if (directory.exists())
directory.delete();
directory.mkdir();
directory.deleteOnExit();
directory = Files.createTempDirectory("ewyk");
// Make some test files
for (int i=0;i<75;i++)
{
File file =new File(directory,i+".txt");
file.createNewFile();
file.deleteOnExit();
File.createTempFile( "ewyk_benchmark", i + ".txt" , directory.toFile());
}
server=new TestServer(directory);
server=new TestServer(directory.toFile());
server.start();
reserved = new ReservedThreadExecutor(server,20);
reserved.start();
@ -89,6 +88,14 @@ public class EWYKBenchmark
@TearDown(Level.Trial)
public static void stopServer() throws Exception
{
try
{
IO.delete(directory.toFile());
}
catch ( Exception e )
{
System.out.println("cannot delete directory:"+directory);
}
reserved.stop();
server.stop();
}

3
jetty-jmh/src/main/java/org/eclipse/jetty/util/thread/strategy/jmh/TestServer.java
View File

@ -19,6 +19,7 @@
package org.eclipse.jetty.util.thread.strategy.jmh;
import java.io.File;
import java.nio.file.Files;
import java.util.HashMap;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
@ -44,7 +45,7 @@ public class TestServer implements Executor, TryExecutor
TestServer()
{
this(new File("/tmp/"));
this(new File(System.getProperty("java.io.tmpdir")));
}

2
jetty-server/src/main/java/org/eclipse/jetty/server/session/SessionHandler.java
View File

@ -725,7 +725,7 @@ public class SessionHandler extends ScopedHandler
if (isUsingCookies())
{
String sessionPath = (_cookieConfig.getPath()==null) ? contextPath : _cookieConfig.getPath();
sessionPath = (sessionPath==null||sessionPath.isEmpty()) ? "/" : sessionPath;
sessionPath = (StringUtil.isEmpty(sessionPath)) ? "/" : sessionPath;
String id = getExtendedId(session);
HttpCookie cookie = null;
if (_sessionComment == null)

25
jetty-util/src/main/java/org/eclipse/jetty/util/StringUtil.java
View File

@ -470,7 +470,30 @@ public class StringUtil
// only whitespace
return true;
}
/**
* <p>Checks if a String is empty ("") or null.</p>
*
* <pre>
* isEmpty(null) == true
* isEmpty("") == true
* isEmpty("\r\n") == false
* isEmpty("\t") == false
* isEmpty(" ") == false
* isEmpty("a") == false
* isEmpty(".") == false
* isEmpty(";\n") == false
* </pre>
*
* @param str
* the string to test.
* @return true if string is null or empty.
*/
public static boolean isEmpty(String str)
{
return str == null || str.isEmpty();
}
/* ------------------------------------------------------------ */
/**
* Test if a string is not null and contains at least 1 non-whitespace characters in it.

103
jetty-util/src/test/java/org/eclipse/jetty/util/ArrayUtilTest.java Normal file
View File

@ -0,0 +1,103 @@
//
// ========================================================================
// Copyright (c) 1995-2019 Mort Bay Consulting Pty. Ltd.
// ------------------------------------------------------------------------
// All rights reserved. This program and the accompanying materials
// are made available under the terms of the Eclipse Public License v1.0
// and Apache License v2.0 which accompanies this distribution.
//
// The Eclipse Public License is available at
// http://www.eclipse.org/legal/epl-v10.html
//
// The Apache License v2.0 is available at
// http://www.opensource.org/licenses/apache2.0.php
//
// You may elect to redistribute this code under either of these licenses.
// ========================================================================
//
package org.eclipse.jetty.util;
import org.junit.jupiter.api.Test;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotSame;
import static org.junit.jupiter.api.Assertions.assertNull;
import static org.junit.jupiter.api.Assertions.assertSame;
/**
* Unit tests for class {@link ArrayUtil}.
*
* @see ArrayUtil
*/
public class ArrayUtilTest
{
@Test
public void testAddToArrayWithEmptyArray()
{
String[] stringArray = new String[0];
String[] resultArray = ArrayUtil.addToArray(stringArray, "Ca?", Object.class);
assertEquals(0, stringArray.length);
assertEquals(1, resultArray.length);
assertNotSame(stringArray, resultArray);
assertNotSame(resultArray, stringArray);
assertFalse(resultArray.equals(stringArray));
assertEquals(String.class, resultArray[0].getClass());
}
@Test
public void testAddUsingNull()
{
String[] stringArray = new String[7];
String[] stringArrayTwo = ArrayUtil.add(stringArray, null);
assertEquals(7, stringArray.length);
assertEquals(7, stringArrayTwo.length);
assertSame(stringArray, stringArrayTwo);
assertSame(stringArrayTwo, stringArray);
}
@Test
public void testAddWithNonEmptyArray()
{
Object[] objectArray = new Object[3];
Object[] objectArrayTwo = ArrayUtil.add(objectArray, objectArray);
assertEquals(3, objectArray.length);
assertEquals(6, objectArrayTwo.length);
assertNotSame(objectArray, objectArrayTwo);
assertNotSame(objectArrayTwo, objectArray);
assertFalse(objectArrayTwo.equals(objectArray));
}
@Test
public void testRemoveFromNullArrayReturningNull()
{
assertNull(ArrayUtil.removeFromArray((Integer[])null, new Object()));
}
@Test
public void testRemoveNulls()
{
Object[] objectArray = new Object[2];
objectArray[0] = new Object();
Object[] resultArray = ArrayUtil.removeNulls(objectArray);
assertEquals(2, objectArray.length);
assertEquals(1, resultArray.length);
assertNotSame(objectArray, resultArray);
assertNotSame(resultArray, objectArray);
assertFalse(resultArray.equals(objectArray));
}
}

76
jetty-util/src/test/java/org/eclipse/jetty/util/IntrospectionUtilTest.java Normal file
View File

@ -0,0 +1,76 @@
//
// ========================================================================
// Copyright (c) 1995-2019 Mort Bay Consulting Pty. Ltd.
// ------------------------------------------------------------------------
// All rights reserved. This program and the accompanying materials
// are made available under the terms of the Eclipse Public License v1.0
// and Apache License v2.0 which accompanies this distribution.
//
// The Eclipse Public License is available at
// http://www.eclipse.org/legal/epl-v10.html
//
// The Apache License v2.0 is available at
// http://www.opensource.org/licenses/apache2.0.php
//
// You may elect to redistribute this code under either of these licenses.
// ========================================================================
//
package org.eclipse.jetty.util;
import org.junit.jupiter.api.Test;
import java.lang.reflect.Array;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertThrows;
import static org.junit.jupiter.api.Assertions.assertTrue;
/**
* Unit tests for class {@link IntrospectionUtil}.
*
* @see IntrospectionUtil
*/
public class IntrospectionUtilTest
{
@Test
public void testIsTypeCompatibleWithTwoTimesString()
{
assertTrue(IntrospectionUtil.isTypeCompatible(String.class, String.class, true));
}
@Test
public void testIsSameSignatureWithNull()
{
assertFalse(IntrospectionUtil.isSameSignature(null, null));
}
@Test
public void testFindMethodWithEmptyString()
{
assertThrows(NoSuchMethodException.class,
() -> IntrospectionUtil.findMethod(Integer.class, "", null, false, false));
}
@Test
public void testFindMethodWithNullMethodParameter()
{
assertThrows(NoSuchMethodException.class,
() -> IntrospectionUtil.findMethod(String.class, null, (Class<Integer>[])Array.newInstance(Class.class, 3), true, true));
}
@Test
public void testFindMethodWithNullClassParameter() throws NoSuchMethodException
{
assertThrows(NoSuchMethodException.class,
() -> IntrospectionUtil.findMethod(null, "subSequence", (Class<Object>[])Array.newInstance(Class.class, 9), false, false));
}
@Test
public void testIsJavaBeanCompliantSetterWithNull()
{
assertFalse(IntrospectionUtil.isJavaBeanCompliantSetter(null));
}
}

55
jetty-util/src/test/java/org/eclipse/jetty/util/LoaderTest.java Normal file
View File

@ -0,0 +1,55 @@
//
// ========================================================================
// Copyright (c) 1995-2019 Mort Bay Consulting Pty. Ltd.
// ------------------------------------------------------------------------
// All rights reserved. This program and the accompanying materials
// are made available under the terms of the Eclipse Public License v1.0
// and Apache License v2.0 which accompanies this distribution.
//
// The Eclipse Public License is available at
// http://www.eclipse.org/legal/epl-v10.html
//
// The Apache License v2.0 is available at
// http://www.opensource.org/licenses/apache2.0.php
//
// You may elect to redistribute this code under either of these licenses.
// ========================================================================
//
package org.eclipse.jetty.util;
import org.junit.jupiter.api.Test;
import java.util.Locale;
import java.util.MissingResourceException;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertThrows;
/**
* Unit tests for class {@link Loader}.
*
* @see Loader
*/
public class LoaderTest
{
@Test
public void testGetResourceBundleThrowsMissingResourceException()
{
assertThrows(MissingResourceException.class, () -> Loader.getResourceBundle("nothing", true, Locale.ITALIAN));
}
@Test
public void testLoadClassThrowsClassNotFoundException()
{
assertThrows(ClassNotFoundException.class, () -> Loader.loadClass(Object.class, "String"));
}
@Test
public void testLoadClassSucceeds() throws ClassNotFoundException
{
assertEquals(LazyList.class, Loader.loadClass(Object.class, "org.eclipse.jetty.util.LazyList"));
}
}

16
jetty-util/src/test/java/org/eclipse/jetty/util/StringUtilTest.java
View File

@ -261,6 +261,22 @@ public class StringUtilTest
assertTrue(StringUtil.isNotBlank("."));
assertTrue(StringUtil.isNotBlank(";\n"));
}
@Test
public void testIsEmpty()
{
assertTrue(StringUtil.isEmpty(null));
assertTrue(StringUtil.isEmpty(""));
assertFalse(StringUtil.isEmpty("\r\n"));
assertFalse(StringUtil.isEmpty("\t"));
assertFalse(StringUtil.isEmpty(" "));
assertFalse(StringUtil.isEmpty("a"));
assertFalse(StringUtil.isEmpty(" a"));
assertFalse(StringUtil.isEmpty("a "));
assertFalse(StringUtil.isEmpty("."));
assertFalse(StringUtil.isEmpty(";\n"));
}
@Test
public void testSanitizeHTML()