andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Enable netty-tcnative shading test again (#8492)

Browse Source 
Motivation:

We disabled the test at some point but it should work now without any problems.

Modifications:

Remove @Ignore from test.

Result:

Verify shading of netty-tcnative on CI.
This commit is contained in:
Norman Maurer 2018-11-11 07:18:21 +01:00 committed by GitHub
parent 0f8ce1b284
commit 35471a1a6f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
testsuite-shading/src/test/java/io/netty/testsuite/shading/ShadingIT.java
View File

@ -16,7 +16,6 @@
package io.netty.testsuite.shading; package io.netty.testsuite.shading;
import io.netty.util.internal.PlatformDependent; import io.netty.util.internal.PlatformDependent;
import org.junit.Ignore;
import org.junit.Test; import org.junit.Test;
import java.lang.reflect.Method; import java.lang.reflect.Method;
@ -34,7 +33,6 @@ public class ShadingIT {
testShading0(SHADING_PREFIX2, className); testShading0(SHADING_PREFIX2, className);
} }
@Ignore("Figure out why this sometimes fail on the CI")
@Test @Test
public void testShadingTcnative() throws Exception { public void testShadingTcnative() throws Exception {
String className = "io.netty.handler.ssl.OpenSsl"; String className = "io.netty.handler.ssl.OpenSsl";