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);
         }
     }
 


Reply via email to