You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by ng...@apache.org on 2008/07/07 12:43:09 UTC
svn commit: r674443 -
/mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java
Author: ngn
Date: Mon Jul 7 03:43:08 2008
New Revision: 674443
URL: http://svn.apache.org/viewvc?rev=674443&view=rev
Log:
Making test error message more useful
Modified:
mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java
Modified: mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java
URL: http://svn.apache.org/viewvc/mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java?rev=674443&r1=674442&r2=674443&view=diff
==============================================================================
--- mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java (original)
+++ mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/clienttests/PasvAddressTest.java Mon Jul 7 03:43:08 2008
@@ -27,12 +27,13 @@
public class PasvAddressTest extends ClientTestTemplate {
protected FtpServer createServer() throws Exception {
- FtpServer server = super.createServer();
+ FtpServer server = super.createServer();
+
+ DefaultDataConnectionConfiguration ddcc = (DefaultDataConnectionConfiguration) server.getServerContext()
+ .getListener("default").getDataConnectionConfiguration();
+
+ ddcc.setPassiveAddress(InetAddress.getByName("127.0.0.200"));
- DefaultDataConnectionConfiguration ddcc = (DefaultDataConnectionConfiguration) server.getServerContext().getListener("default").getDataConnectionConfiguration();
-
- ddcc.setPassiveAddress(InetAddress.getByName("127.0.0.200"));
-
return server;
}
@@ -40,6 +41,9 @@
client.login(ADMIN_USERNAME, ADMIN_PASSWORD);
client.pasv();
- assertTrue(client.getReplyString().indexOf("(127,0,0,200,") > -1);
+ String reply = client.getReplyString();
+
+ assertTrue("The PASV address should contain \"127,0,0,200\" but was \"" + reply + "\"",
+ reply.indexOf("(127,0,0,200,") > -1);
}
}