You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by cc...@apache.org on 2009/06/12 06:03:08 UTC

svn commit: r783976 - /servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java

Author: ccustine
Date: Fri Jun 12 04:03:08 2009
New Revision: 783976

URL: http://svn.apache.org/viewvc?rev=783976&view=rev
Log:
SMXCOMP-556 NullPointerException activating FTP endpoint when setting client pool host /port.

Modified:
    servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java

Modified: servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java
URL: http://svn.apache.org/viewvc/servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java?rev=783976&r1=783975&r2=783976&view=diff
==============================================================================
--- servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java (original)
+++ servicemix/components/bindings/servicemix-ftp/trunk/src/test/java/org/apache/servicemix/ftp/FtpClientPoolTest.java Fri Jun 12 04:03:08 2009
@@ -86,20 +86,13 @@
 		endpoint.setEndpoint("endpoint");
 		endpoint.setTargetService(receiver.getService());
 		endpoint.setClientPool(clientPool);
-
-		// TODO Need this until SMXCOMP-556 is fixed
-		endpoint.setUri(new URI("ftp://testuser1:password@localhost:"
-				+ getListenerPort() + "/"));
-
-		endpoint.validate();
 		return endpoint;
 	}
 
 	private FTPClientPool createClientPool() {
 		FTPClientPool pool = new FTPClientPool();
-		// TODO Causes NPE, seeSMXCOMP-556
-		// pool.setHost("localhost");
-		// pool.setPort(getListenerPort());
+        pool.setHost("localhost");
+        pool.setPort(getListenerPort());
 		pool.setUsername("testuser2");
 		pool.setPassword("password");
 		return pool;