james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rouaz...@apache.org
Subject [05/11] james-project git commit: JAMES-1862 Remove TODO note on commons-net fix
Date Tue, 06 Dec 2016 09:02:50 GMT
JAMES-1862 Remove TODO note on commons-net fix


Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/b20a7d4c
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/b20a7d4c
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/b20a7d4c

Branch: refs/heads/master
Commit: b20a7d4c6e76fc6fa6fe3fc76c9b63ae75300a48
Parents: 737b687
Author: Antoine Duprat <aduprat@apache.org>
Authored: Wed Nov 30 10:04:30 2016 +0100
Committer: Antoine Duprat <aduprat@apache.org>
Committed: Tue Dec 6 09:56:34 2016 +0100

----------------------------------------------------------------------
 .../james/protocols/smtp/AbstractStartTlsSMTPServerTest.java  | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/james-project/blob/b20a7d4c/protocols/smtp/src/test/java/org/apache/james/protocols/smtp/AbstractStartTlsSMTPServerTest.java
----------------------------------------------------------------------
diff --git a/protocols/smtp/src/test/java/org/apache/james/protocols/smtp/AbstractStartTlsSMTPServerTest.java
b/protocols/smtp/src/test/java/org/apache/james/protocols/smtp/AbstractStartTlsSMTPServerTest.java
index 626cb17..02f87ec 100644
--- a/protocols/smtp/src/test/java/org/apache/james/protocols/smtp/AbstractStartTlsSMTPServerTest.java
+++ b/protocols/smtp/src/test/java/org/apache/james/protocols/smtp/AbstractStartTlsSMTPServerTest.java
@@ -97,11 +97,8 @@ public abstract class AbstractStartTlsSMTPServerTest {
             
             assertTrue(client.execTLS());
             
-            // TODO: Add back once commons-net 3.1.0 was released.
-            // See: NET-421
-            //
-            //client.quit();
-            //assertTrue("Reply="+ client.getReplyString(), SMTPReply.isPositiveCompletion(client.getReplyCode()));
+            client.quit();
+            assertTrue("Reply="+ client.getReplyString(), SMTPReply.isPositiveCompletion(client.getReplyCode()));
             
             client.disconnect();
 


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message