Author: sebb
Date: Thu Dec 11 15:36:58 2014
New Revision: 1644681

URL: http://svn.apache.org/r1644681
Log:
Oops - also need to check previous entry

Modified:
    
commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/parser/FTPParseTestFramework.java

Modified: 
commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/parser/FTPParseTestFramework.java
URL: 
http://svn.apache.org/viewvc/commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/parser/FTPParseTestFramework.java?rev=1644681&r1=1644680&r2=1644681&view=diff
==============================================================================
--- 
commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/parser/FTPParseTestFramework.java
 (original)
+++ 
commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/parser/FTPParseTestFramework.java
 Thu Dec 11 15:36:58 2014
@@ -149,6 +149,8 @@ public abstract class FTPParseTestFramew
     }
 
     // associate Calendar unit ints with a readable string
+    // MUST be listed least significant first, as the routine needs to
+    // find the previous - less significant - entry
     protected enum CalendarUnit {
         MILLISECOND(Calendar.MILLISECOND),
         SECOND(Calendar.SECOND),
@@ -170,6 +172,11 @@ public abstract class FTPParseTestFramew
         Calendar stamp = file.getTimestamp();
         assertNotNull("Failed to parse time in "+listEntry, stamp);
         assertTrue("Expected set "+expectedPrecision+" in "+listEntry, 
stamp.isSet(expectedPrecision.unit));
+        final int ordinal = expectedPrecision.ordinal();
+        if (ordinal > 0) {
+            final CalendarUnit prevUnit = CalendarUnit.values()[ordinal-1];
+            assertFalse("Expected not set "+prevUnit+" in "+listEntry, 
stamp.isSet(prevUnit.unit));            
+        }
     }
     
 }


Reply via email to