geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jdil...@apache.org
Subject svn commit: r420367 - in /geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system: configuration/ServerOverrideTest.java repository/AbstractRepositoryTest.java rmi/RMIClassLoaderSpiImplTest.java
Date Sun, 09 Jul 2006 23:06:51 GMT
Author: jdillon
Date: Sun Jul  9 16:06:51 2006
New Revision: 420367

URL: http://svn.apache.org/viewvc?rev=420367&view=rev
Log:
Use logging instead of System.*

Modified:
    geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/configuration/ServerOverrideTest.java
    geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/repository/AbstractRepositoryTest.java
    geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/rmi/RMIClassLoaderSpiImplTest.java

Modified: geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/configuration/ServerOverrideTest.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/configuration/ServerOverrideTest.java?rev=420367&r1=420366&r2=420367&view=diff
==============================================================================
--- geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/configuration/ServerOverrideTest.java
(original)
+++ geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/configuration/ServerOverrideTest.java
Sun Jul  9 16:06:51 2006
@@ -45,10 +45,15 @@
 import java.util.Map;
 import java.util.Set;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /**
  * @version $Rev$ $Date$
  */
 public class ServerOverrideTest extends TestCase {
+    private static final Log log = LogFactory.getLog(ServerOverrideTest.class);
+    
     public void testBasics() throws Exception {
         GBeanOverride pizza = new GBeanOverride("Pizza", true);
         assertTrue(pizza.isLoad());
@@ -305,9 +310,7 @@
         xform.setOutputProperty(OutputKeys.INDENT, "yes");
         xform.setOutputProperty("{http://xml.apache.org/xslt}indent-amount", "2");
         xform.transform(new DOMSource(e), new StreamResult(out));
-        System.out.println();
-        System.out.println();
-        System.out.println(new String(out.toByteArray()));
+        log.debug(new String(out.toByteArray()));
         ByteArrayInputStream in = new ByteArrayInputStream(out.toByteArray());
         return parseXml(in, name);
     }

Modified: geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/repository/AbstractRepositoryTest.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/repository/AbstractRepositoryTest.java?rev=420367&r1=420366&r2=420367&view=diff
==============================================================================
--- geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/repository/AbstractRepositoryTest.java
(original)
+++ geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/repository/AbstractRepositoryTest.java
Sun Jul  9 16:06:51 2006
@@ -26,10 +26,15 @@
 import org.apache.geronimo.kernel.repository.ListableRepository;
 import org.apache.geronimo.kernel.repository.Version;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /**
  * @version $Rev$ $Date$
  */
 public class AbstractRepositoryTest extends TestCase {
+    private static final Log log = LogFactory.getLog(AbstractRepositoryTest.class);
+    
     protected ListableRepository repository;
     protected File rootRepoDir;
 
@@ -78,7 +83,7 @@
 
     public void testListAll() {
         SortedSet artifacts = repository.list();
-        System.out.println("Matched artifacts: "+artifacts);
+        log.debug("Matched artifacts: "+artifacts);
 
         assertTrue(artifacts.contains(new Artifact("org.foo", "test", "2.0.1", "properties")));
         assertFalse(artifacts.contains(new Artifact("Unknown", "artifact", "2.0.1", "properties")));

Modified: geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/rmi/RMIClassLoaderSpiImplTest.java
URL: http://svn.apache.org/viewvc/geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/rmi/RMIClassLoaderSpiImplTest.java?rev=420367&r1=420366&r2=420367&view=diff
==============================================================================
--- geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/rmi/RMIClassLoaderSpiImplTest.java
(original)
+++ geronimo/sandbox/svkmerge/m2migration/modules/system/src/test/org/apache/geronimo/system/rmi/RMIClassLoaderSpiImplTest.java
Sun Jul  9 16:06:51 2006
@@ -27,13 +27,17 @@
 
 import junit.framework.TestCase;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
 /**
  * Unit tests for {@link RMIClassLoaderSpiImpl} class.
  *
  * @version $Rev$ $Date$
  */
-public class RMIClassLoaderSpiImplTest
-        extends TestCase {
+public class RMIClassLoaderSpiImplTest extends TestCase {
+    private static final Log log = LogFactory.getLog(RMIClassLoaderSpiImplTest.class);
+    
     private String baseURL;
     private String normalizedBaseURL;
 
@@ -50,8 +54,8 @@
             normalizedBaseURL = normalizedBaseURL.substring(0, normalizedBaseURL.length()
- 1);
         }
 
-        System.out.println("Using base URL: " + baseURL);
-        System.out.println("Using normalized base URL: " + normalizedBaseURL);
+        log.debug("Using base URL: " + baseURL);
+        log.debug("Using normalized base URL: " + normalizedBaseURL);
     }
 
     public void testNormalizeURL() throws MalformedURLException {



Mime
View raw message