Merge pull request #854 from fengxx/feature/netty_skip_non_linux

KostyaSha · web-flow · commit 3180671fafc3 · 2017-06-03T17:02:58.000+03:00
skip instead of throw error on non-linux os
diff --git a/src/test/java/com/github/dockerjava/netty/exec/AttachContainerCmdExecTest.java b/src/test/java/com/github/dockerjava/netty/exec/AttachContainerCmdExecTest.java
@@ -10,8 +10,10 @@
 import java.io.PipedInputStream;
 import java.io.PipedOutputStream;
 import java.lang.reflect.Method;
+import java.util.Locale;
 
 import org.testng.ITestResult;
+import org.testng.SkipException;
 import org.testng.annotations.AfterMethod;
 import org.testng.annotations.AfterTest;
 import org.testng.annotations.BeforeMethod;
@@ -30,6 +32,10 @@ public class AttachContainerCmdExecTest extends AbstractNettyDockerClientTest {
 
     @BeforeTest
     public void beforeTest() throws Exception {
+        // io.netty.channel.epollNative#loadNativeLibrary only support Linux
+        if (!System.getProperty("os.name").toLowerCase(Locale.UK).trim().startsWith("linux")) {
+            throw new SkipException("only support Linux");
+        }
         super.beforeTest();
     }
 
