Author: britter Date: Sat Mar 28 11:40:25 2015 New Revision: 1669745 URL: http://svn.apache.org/r1669745 Log: This test should fail instead of writing to std out so that we can add missing OS
Modified: commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/SystemUtilsTest.java Modified: commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/SystemUtilsTest.java URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/SystemUtilsTest.java?rev=1669745&r1=1669744&r2=1669745&view=diff ============================================================================== --- commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/SystemUtilsTest.java (original) +++ commons/proper/lang/trunk/src/test/java/org/apache/commons/lang3/SystemUtilsTest.java Sat Mar 28 11:40:25 2015 @@ -19,9 +19,13 @@ package org.apache.commons.lang3; -import org.junit.Test; -import static org.junit.Assert.*; import static org.apache.commons.lang3.JavaVersion.JAVA_1_4; +import static org.junit.Assert.assertEquals; +import static org.junit.Assert.assertFalse; +import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertNull; +import static org.junit.Assert.assertTrue; +import static org.junit.Assert.fail; import java.io.File; import java.lang.reflect.Constructor; @@ -29,6 +33,7 @@ import java.lang.reflect.Modifier; import java.util.Locale; import org.junit.Assert; +import org.junit.Test; /** * Unit tests {@link org.apache.commons.lang3.SystemUtils}. @@ -208,7 +213,7 @@ public class SystemUtilsTest { assertTrue(SystemUtils.IS_OS_UNIX); assertFalse(SystemUtils.IS_OS_WINDOWS); } else { - System.out.println("Can't test IS_OS value: "+osName); + fail("Can't test IS_OS value: " + osName); } }