Skip to content
This repository was archived by the owner on May 28, 2018. It is now read-only.

Commit ef3f060

Browse files
committed
JERSEY-2291: Un-ignored shutdown hook leak test which serves as a good evidence whether enabled immediate scope services do not mess up the client.
When immediate scope services are enabled, this test creates new thread for each iteration, which is probably something we don't like. Change-Id: I3874e5ce0e5e813c36d2c0a45cf066a19223b498
1 parent 5217803 commit ef3f060

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

core-client/src/test/java/org/glassfish/jersey/client/ShutdownHookLeakTest.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@
3939
*/
4040
package org.glassfish.jersey.client;
4141

42-
4342
import java.lang.ref.WeakReference;
4443
import java.lang.reflect.Field;
4544
import java.util.Collection;
@@ -48,13 +47,11 @@
4847
import javax.ws.rs.client.ClientBuilder;
4948
import javax.ws.rs.client.WebTarget;
5049

50+
import org.junit.Test;
5151
import static org.hamcrest.CoreMatchers.is;
5252
import static org.hamcrest.number.OrderingComparison.lessThan;
5353
import static org.junit.Assert.assertThat;
5454

55-
import org.junit.Ignore;
56-
import org.junit.Test;
57-
5855
/**
5956
* Reproducer for JERSEY-2786.
6057
*
@@ -67,7 +64,6 @@ public class ShutdownHookLeakTest {
6764

6865
@SuppressWarnings("unchecked")
6966
@Test
70-
@Ignore("intermittent failures.")
7167
public void testShutdownHookDoesNotLeak() throws Exception {
7268
final Client client = ClientBuilder.newClient();
7369
final WebTarget target = client.target("http://example.com");

0 commit comments

Comments
 (0)