Skip to content

Commit

Permalink
upgrade dependencies
Browse files Browse the repository at this point in the history
I had to fix JettyServerTest because Jetty behaviour has been changed: now it throws IllegalArgumentException if maxThreads < minThreads
  • Loading branch information
asolntsev committed Jun 27, 2018
1 parent c697d98 commit e0c4a2f
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 27 deletions.
24 changes: 13 additions & 11 deletions pom.xml
Expand Up @@ -30,9 +30,9 @@


<properties> <properties>
<java.version>1.8</java.version> <java.version>1.8</java.version>
<jetty.version>9.4.8.v20171121</jetty.version> <jetty.version>9.4.11.v20180605</jetty.version>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
<powermock.version>1.6.6</powermock.version> <powermock.version>1.7.4</powermock.version>
<mockito.version>1.10.19</mockito.version> <mockito.version>1.10.19</mockito.version>
</properties> </properties>


Expand Down Expand Up @@ -103,19 +103,19 @@
<dependency> <dependency>
<groupId>org.apache.httpcomponents</groupId> <groupId>org.apache.httpcomponents</groupId>
<artifactId>httpclient</artifactId> <artifactId>httpclient</artifactId>
<version>4.5.1</version> <version>4.5.5</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
<dependency> <dependency>
<groupId>org.freemarker</groupId> <groupId>org.freemarker</groupId>
<artifactId>freemarker</artifactId> <artifactId>freemarker</artifactId>
<version>2.3.23</version> <version>2.3.28</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
<dependency> <dependency>
<groupId>com.google.code.gson</groupId> <groupId>com.google.code.gson</groupId>
<artifactId>gson</artifactId> <artifactId>gson</artifactId>
<version>2.4</version> <version>2.8.5</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
<dependency> <dependency>
Expand All @@ -124,12 +124,14 @@
<version>${jetty.version}</version> <version>${jetty.version}</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
<!--
<dependency> <dependency>
<groupId>org.powermock.tests</groupId> <groupId>org.powermock.tests</groupId>
<artifactId>powermock-tests-utils</artifactId> <artifactId>powermock-tests-utils</artifactId>
<version>${powermock.version}</version> <version>1.6.6</version>
<scope>test</scope> <scope>test</scope>
</dependency> </dependency>
-->
</dependencies> </dependencies>


<build> <build>
Expand All @@ -155,7 +157,7 @@
<plugin> <plugin>
<groupId>org.apache.maven.plugins</groupId> <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-compiler-plugin</artifactId> <artifactId>maven-compiler-plugin</artifactId>
<version>3.3</version> <version>3.7.0</version>
<configuration> <configuration>
<source>${java.version}</source> <source>${java.version}</source>
<target>${java.version}</target> <target>${java.version}</target>
Expand All @@ -166,7 +168,7 @@
<plugin> <plugin>
<groupId>org.apache.maven.plugins</groupId> <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId> <artifactId>maven-enforcer-plugin</artifactId>
<version>1.4</version> <version>1.4.1</version>
<executions> <executions>
<execution> <execution>
<id>enforce-java</id> <id>enforce-java</id>
Expand All @@ -186,7 +188,7 @@
<plugin> <plugin>
<groupId>org.apache.maven.plugins</groupId> <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId> <artifactId>maven-surefire-plugin</artifactId>
<version>2.18.1</version> <version>2.22.0</version>
<configuration> <configuration>
<forkCount>1</forkCount> <forkCount>1</forkCount>
<reuseForks>false</reuseForks> <reuseForks>false</reuseForks>
Expand All @@ -195,15 +197,15 @@
<plugin> <plugin>
<groupId>org.apache.maven.plugins</groupId> <groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-javadoc-plugin</artifactId> <artifactId>maven-javadoc-plugin</artifactId>
<version>2.10.3</version> <version>3.0.1</version>
<configuration> <configuration>
<failOnError>false</failOnError> <failOnError>false</failOnError>
</configuration> </configuration>
</plugin> </plugin>
<plugin> <plugin>
<groupId>org.apache.felix</groupId> <groupId>org.apache.felix</groupId>
<artifactId>maven-bundle-plugin</artifactId> <artifactId>maven-bundle-plugin</artifactId>
<version>2.5.3</version> <version>3.5.1</version>
<extensions>true</extensions> <extensions>true</extensions>
</plugin> </plugin>
</plugins> </plugins>
Expand Down
40 changes: 24 additions & 16 deletions src/test/java/spark/embeddedserver/jetty/JettyServerTest.java
Expand Up @@ -3,43 +3,51 @@
import org.eclipse.jetty.server.Server; import org.eclipse.jetty.server.Server;
import org.eclipse.jetty.util.thread.QueuedThreadPool; import org.eclipse.jetty.util.thread.QueuedThreadPool;
import org.junit.Test; import org.junit.Test;
import org.mockito.internal.util.reflection.Whitebox; import org.powermock.reflect.Whitebox;


import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;


public class JettyServerTest { public class JettyServerTest {

@Test @Test
public void testCreateServer_useDefaults() throws Exception { public void testCreateServer_useDefaults() {

Server server = new JettyServer().create(0, 0, 0); Server server = new JettyServer().create(0, 0, 0);


QueuedThreadPool threadPool = (QueuedThreadPool) server.getThreadPool(); QueuedThreadPool threadPool = (QueuedThreadPool) server.getThreadPool();


int minThreads = (int) Whitebox.getInternalState(threadPool, "_minThreads"); int minThreads = Whitebox.getInternalState(threadPool, "_minThreads");
int maxThreads = (int) Whitebox.getInternalState(threadPool, "_maxThreads"); int maxThreads = Whitebox.getInternalState(threadPool, "_maxThreads");
int idleTimeout = (int) Whitebox.getInternalState(threadPool, "_idleTimeout"); int idleTimeout = Whitebox.getInternalState(threadPool, "_idleTimeout");


assertEquals("Server thread pool default minThreads should be 8", 8, minThreads); assertEquals("Server thread pool default minThreads should be 8", 8, minThreads);
assertEquals("Server thread pool default maxThreads should be 200", 200, maxThreads); assertEquals("Server thread pool default maxThreads should be 200", 200, maxThreads);
assertEquals("Server thread pool default idleTimeout should be 60000", 60000, idleTimeout); assertEquals("Server thread pool default idleTimeout should be 60000", 60000, idleTimeout);

} }


@Test @Test
public void testCreateServer_whenNonDefaultMaxThreadOnly_thenUseDefaultMinThreadAndTimeout() throws Exception { public void testCreateServer_whenNonDefaultMaxThreadsOnly_thenUseDefaultMinThreadsAndTimeout() {

Server server = new JettyServer().create(9, 0, 0);
Server server = new JettyServer().create(1, 0, 0);


QueuedThreadPool threadPool = (QueuedThreadPool) server.getThreadPool(); QueuedThreadPool threadPool = (QueuedThreadPool) server.getThreadPool();


int minThreads = (int) Whitebox.getInternalState(threadPool, "_minThreads"); int minThreads = Whitebox.getInternalState(threadPool, "_minThreads");
int maxThreads = (int) Whitebox.getInternalState(threadPool, "_maxThreads"); int maxThreads = Whitebox.getInternalState(threadPool, "_maxThreads");
int idleTimeout = (int) Whitebox.getInternalState(threadPool, "_idleTimeout"); int idleTimeout = Whitebox.getInternalState(threadPool, "_idleTimeout");


assertEquals("Server thread pool default minThreads should be 1", 1, minThreads); assertEquals("Server thread pool default minThreads should be 8", 8, minThreads);
assertEquals("Server thread pool default maxThreads should be the same as specified", 1, maxThreads); assertEquals("Server thread pool default maxThreads should be the same as specified", 9, maxThreads);
assertEquals("Server thread pool default idleTimeout should be 60000", 60000, idleTimeout); assertEquals("Server thread pool default idleTimeout should be 60000", 60000, idleTimeout);


} }

@Test
public void testCreateServer_whenNonDefaultMaxThreads_isLessThanDefaultMinThreads() {
try {
new JettyServer().create(2, 0, 0);
fail("expected IllegalArgumentException");
}
catch (IllegalArgumentException expected) {
assertEquals("max threads (2) less than min threads (8)", expected.getMessage());
}
}
} }

0 comments on commit e0c4a2f

Please sign in to comment.