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/05/15 13:49:46 UTC

svn commit: r656596 - /mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java

Author: ngn
Date: Thu May 15 04:49:45 2008
New Revision: 656596

URL: http://svn.apache.org/viewvc?rev=656596&view=rev
Log:
Adding methods for setting the passive port

Modified:
    mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java

Modified: mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java
URL: http://svn.apache.org/viewvc/mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java?rev=656596&r1=656595&r2=656596&view=diff
==============================================================================
--- mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java (original)
+++ mina/ftpserver/trunk/core/src/test/java/org/apache/ftpserver/PassivePortsTest.java Thu May 15 04:49:45 2008
@@ -25,21 +25,21 @@
 public class PassivePortsTest extends TestCase {
 
     public void testParseSingleValue() {
-        PassivePorts ports = PassivePorts.parse("123");
+        PassivePorts ports = new PassivePorts("123");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(-1, ports.reserveNextPort());
     }
 
     public void testParseMaxValue() {
-        PassivePorts ports = PassivePorts.parse("65535");
+        PassivePorts ports = new PassivePorts("65535");
         
         assertEquals(65535, ports.reserveNextPort());
         assertEquals(-1, ports.reserveNextPort());
     }
 
     public void testParseMinValue() {
-        PassivePorts ports = PassivePorts.parse("0");
+        PassivePorts ports = new PassivePorts("0");
         
         assertEquals(0, ports.reserveNextPort());
         assertEquals(0, ports.reserveNextPort());
@@ -50,7 +50,7 @@
 
     public void testParseTooLargeValue() {
         try {
-            PassivePorts.parse("65536");
+            new PassivePorts("65536");
             fail("Must fail due to too high port number");
         } catch(IllegalArgumentException e) {
             // ok
@@ -59,7 +59,7 @@
 
     public void testParseNonNumericValue() {
         try {
-            PassivePorts.parse("foo");
+            new PassivePorts("foo");
             fail("Must fail due to non numerical port number");
         } catch(IllegalArgumentException e) {
             // ok
@@ -67,7 +67,7 @@
     }
     
     public void testParseListOfValues() {
-        PassivePorts ports = PassivePorts.parse("123, 456,\t\n789");
+        PassivePorts ports = new PassivePorts("123, 456,\t\n789");
 
         assertEquals(123, ports.reserveNextPort());
         assertEquals(456, ports.reserveNextPort());
@@ -76,7 +76,7 @@
     }
 
     public void testParseListOfValuesOrder() {
-        PassivePorts ports = PassivePorts.parse("123, 789, 456");
+        PassivePorts ports = new PassivePorts("123, 789, 456");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(789, ports.reserveNextPort());
@@ -85,7 +85,7 @@
     }
     
     public void testParseListOfValuesDuplicate() {
-        PassivePorts ports = PassivePorts.parse("123, 789, 456, 789");
+        PassivePorts ports = new PassivePorts("123, 789, 456, 789");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(789, ports.reserveNextPort());
@@ -94,7 +94,7 @@
     }
     
     public void testParseSimpleRange() {
-        PassivePorts ports = PassivePorts.parse("123-125");
+        PassivePorts ports = new PassivePorts("123-125");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(124, ports.reserveNextPort());
@@ -103,7 +103,7 @@
     }
 
     public void testParseMultipleRanges() {
-        PassivePorts ports = PassivePorts.parse("123-125, 127-128, 130-132");
+        PassivePorts ports = new PassivePorts("123-125, 127-128, 130-132");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(124, ports.reserveNextPort());
@@ -117,7 +117,7 @@
     }
     
     public void testParseMixedRangeAndSingle() {
-        PassivePorts ports = PassivePorts.parse("123-125, 126, 128-129");
+        PassivePorts ports = new PassivePorts("123-125, 126, 128-129");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(124, ports.reserveNextPort());
@@ -129,7 +129,7 @@
     }
 
     public void testParseOverlapingRanges() {
-        PassivePorts ports = PassivePorts.parse("123-125, 124-126");
+        PassivePorts ports = new PassivePorts("123-125, 124-126");
         
         assertEquals(123, ports.reserveNextPort());
         assertEquals(124, ports.reserveNextPort());
@@ -139,7 +139,7 @@
     }
 
     public void testParseOverlapingRangesorder() {
-        PassivePorts ports = PassivePorts.parse("124-126, 123-125");
+        PassivePorts ports = new PassivePorts("124-126, 123-125");
         
         assertEquals(124, ports.reserveNextPort());
         assertEquals(125, ports.reserveNextPort());
@@ -149,7 +149,7 @@
     }
 
     public void testParseOpenLowerRange() {
-        PassivePorts ports = PassivePorts.parse("9, -3");
+        PassivePorts ports = new PassivePorts("9, -3");
         
         assertEquals(9, ports.reserveNextPort());
         assertEquals(1, ports.reserveNextPort());
@@ -159,7 +159,7 @@
     }
 
     public void testParseOpenUpperRange() {
-        PassivePorts ports = PassivePorts.parse("65533-");
+        PassivePorts ports = new PassivePorts("65533-");
         
         assertEquals(65533, ports.reserveNextPort());
         assertEquals(65534, ports.reserveNextPort());
@@ -168,7 +168,7 @@
     }
     
     public void testParseOpenUpperRange3() {
-        PassivePorts ports = PassivePorts.parse("65533-, 65532-");
+        PassivePorts ports = new PassivePorts("65533-, 65532-");
         
         assertEquals(65533, ports.reserveNextPort());
         assertEquals(65534, ports.reserveNextPort());
@@ -178,7 +178,7 @@
     }
     
     public void testParseOpenUpperRange2() {
-        PassivePorts ports = PassivePorts.parse("65533-, 1");
+        PassivePorts ports = new PassivePorts("65533-, 1");
         
         assertEquals(65533, ports.reserveNextPort());
         assertEquals(65534, ports.reserveNextPort());
@@ -188,7 +188,7 @@
     }
 
     public void testParseRelease() {
-        PassivePorts ports = PassivePorts.parse("123, 456,789");
+        PassivePorts ports = new PassivePorts("123, 456,789");
         
         
         assertEquals(123, ports.reserveNextPort());