diff --git a/hadoop-common-project/hadoop-common/CHANGES.txt b/hadoop-common-project/hadoop-common/CHANGES.txt index 16002d53079..0af0beb903a 100644 --- a/hadoop-common-project/hadoop-common/CHANGES.txt +++ b/hadoop-common-project/hadoop-common/CHANGES.txt @@ -184,6 +184,9 @@ Trunk (Unreleased) HADOOP-11593. Convert site documentation from apt to markdown (stragglers) (Masatake Iwasaki via aw) + HADOOP-11673. Skip using JUnit Assume in TestCodec. (Brahma Reddy Battula + via cdouglas) + BUG FIXES HADOOP-11473. test-patch says "-1 overall" even when all checks are +1 diff --git a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java index 98b3934ea48..7246bf50bb1 100644 --- a/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java +++ b/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/io/compress/TestCodec.java @@ -74,6 +74,7 @@ import org.junit.Assert; import org.junit.Assume; import org.junit.Test; import static org.junit.Assert.*; +import static org.junit.Assume.*; public class TestCodec { @@ -364,10 +365,7 @@ public class TestCodec { public void testCodecPoolGzipReuse() throws Exception { Configuration conf = new Configuration(); conf.setBoolean(CommonConfigurationKeys.IO_NATIVE_LIB_AVAILABLE_KEY, true); - if (!ZlibFactory.isNativeZlibLoaded(conf)) { - LOG.warn("testCodecPoolGzipReuse skipped: native libs not loaded"); - return; - } + assumeTrue(ZlibFactory.isNativeZlibLoaded(conf)); GzipCodec gzc = ReflectionUtils.newInstance(GzipCodec.class, conf); DefaultCodec dfc = ReflectionUtils.newInstance(DefaultCodec.class, conf); Compressor c1 = CodecPool.getCompressor(gzc); @@ -723,10 +721,7 @@ public class TestCodec { public void testNativeGzipConcat() throws IOException { Configuration conf = new Configuration(); conf.setBoolean(CommonConfigurationKeys.IO_NATIVE_LIB_AVAILABLE_KEY, true); - if (!ZlibFactory.isNativeZlibLoaded(conf)) { - LOG.warn("skipped: native libs not loaded"); - return; - } + assumeTrue(ZlibFactory.isNativeZlibLoaded(conf)); GzipConcatTest(conf, GzipCodec.GzipZlibDecompressor.class); } @@ -840,10 +835,7 @@ public class TestCodec { Configuration conf = new Configuration(); conf.setBoolean(CommonConfigurationKeys.IO_NATIVE_LIB_AVAILABLE_KEY, useNative); if (useNative) { - if (!ZlibFactory.isNativeZlibLoaded(conf)) { - LOG.warn("testGzipCodecWrite skipped: native libs not loaded"); - return; - } + assumeTrue(ZlibFactory.isNativeZlibLoaded(conf)); } else { assertFalse("ZlibFactory is using native libs against request", ZlibFactory.isNativeZlibLoaded(conf));