[HADOOP-11673] Use org.junit.Assume to skip tests instead of return

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha1
• Component/s: test
• Hadoop Flags: Reviewed
• Assignee: Brahma Reddy Battula
• Reporter: Akira Ajisaka
• Created: 05/Mar/15 03:31
• Updated: 12/May/16 18:21
• Resolved: 09/Mar/15 02:24

Description

We see the following code many times:

TestCodec.java

    if (!ZlibFactory.isNativeZlibLoaded(conf)) {
      LOG.warn("skipped: native libs not loaded");
      return;
    }

If ZlibFactory.isNativeZlibLoaded(conf) is false, the test will pass, with a warn log. I'd like to skip this test case by using org.junit.Assume.