Skip to content

Commit 4577d88

Browse files
authored
Merge pull request #1574 from TestBoost/master
[UNDERTOW-2371] initialize the DefaultServer once to speed up test HttpContinueSslServletTestCase
2 parents 33448b2 + 35b21ef commit 4577d88

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

servlet/src/test/java/io/undertow/servlet/test/handlers/AbstractHttpContinueServletTestCase.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,6 @@
3232
import org.apache.http.params.HttpParams;
3333
import org.junit.Assert;
3434
import org.junit.Assume;
35-
import org.junit.Before;
3635
import org.junit.BeforeClass;
3736
import org.junit.Test;
3837

@@ -56,10 +55,10 @@ public abstract class AbstractHttpContinueServletTestCase {
5655
public static void setup() {
5756
DeploymentUtils.setupServlet(Servlets.servlet(ContinueConsumeServlet.class).addMappings("/path"),
5857
Servlets.servlet(ContinueIgnoreServlet.class).addMappings("/ignore"));
58+
Assume.assumeFalse(DefaultServer.isAjp());
5959
}
6060

61-
@Before
62-
public void before() throws Exception {
61+
public static void before() throws Exception {
6362
Assume.assumeFalse(DefaultServer.isAjp());
6463
}
6564

servlet/src/test/java/io/undertow/servlet/test/handlers/HttpContinueSslServletTestCase.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020

2121
import io.undertow.testutils.DefaultServer;
2222
import io.undertow.testutils.TestHttpClient;
23-
import org.junit.After;
24-
import org.junit.Before;
23+
import org.junit.AfterClass;
24+
import org.junit.BeforeClass;
2525
import org.junit.runner.RunWith;
2626

2727
import java.io.IOException;
@@ -42,14 +42,14 @@ protected TestHttpClient getClient() {
4242
return client;
4343
}
4444

45-
@Before
46-
public void before() throws Exception {
47-
super.before();
45+
@BeforeClass
46+
public static void before() throws Exception {
47+
AbstractHttpContinueServletTestCase.before();
4848
DefaultServer.startSSLServer();
4949
}
5050

51-
@After
52-
public void after() throws IOException {
51+
@AfterClass
52+
public static void after() throws IOException {
5353
DefaultServer.stopSSLServer();
5454
}
5555
}

0 commit comments

Comments
 (0)