This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-vfs.git


The following commit(s) were added to refs/heads/master by this push:
     new ed444fb  The Java 8/Linux combination on GitHub returns the time with 
000 milliseconds, IOW with second precision. Workaround OpenJDK 8 and 9 bug 
JDK-8177809 https://bugs.openjdk.java.net/browse/JDK-8177809
ed444fb is described below

commit ed444fb71bb0f95d570f86b8f96ade87f9763652
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Feb 10 23:02:36 2021 -0500

    The Java 8/Linux combination on GitHub returns the time with 000
    milliseconds, IOW with second precision.
    Workaround OpenJDK 8 and 9 bug JDK-8177809
    https://bugs.openjdk.java.net/browse/JDK-8177809
---
 .../commons/vfs2/provider/ftp/FtpMdtmOnLastModifiedTests.java  | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git 
a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/ftp/FtpMdtmOnLastModifiedTests.java
 
b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/ftp/FtpMdtmOnLastModifiedTests.java
index 0e5f0e1..efeefd3 100644
--- 
a/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/ftp/FtpMdtmOnLastModifiedTests.java
+++ 
b/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/ftp/FtpMdtmOnLastModifiedTests.java
@@ -22,6 +22,7 @@ import java.nio.file.Files;
 import java.nio.file.Paths;
 import java.nio.file.attribute.FileTime;
 
+import org.apache.commons.lang3.SystemUtils;
 import org.apache.commons.vfs2.AbstractTestSuite;
 import org.apache.commons.vfs2.FileObject;
 import org.apache.commons.vfs2.LastModifiedTests;
@@ -41,7 +42,14 @@ public class FtpMdtmOnLastModifiedTests extends 
LastModifiedTests {
         // now try to match
         final FileTime lastModifiedTime = Files
             .getLastModifiedTime(Paths.get(getTestDirectory(), 
AbstractTestSuite.READ_TESTS_FOLDER, fileName));
-        assertEqualMillis("getLastModified on File", 
lastModifiedTime.toMillis(), lastModifiedTimeMillis);
+        if (SystemUtils.IS_JAVA_1_8 && SystemUtils.IS_OS_LINUX) {
+            // The Java 8/Linux combination on GitHub returns the time with 
000 milliseconds, IOW with second precision.
+            // Workaround OpenJDK 8 and 9 bug JDK-8177809
+            // https://bugs.openjdk.java.net/browse/JDK-8177809
+            assertDeltaMillis(fileName, lastModifiedTimeMillis, 
lastModifiedTimeMillis, lastModifiedTimeMillis);
+        } else {
+            assertEqualMillis("getLastModified on " + fileObject, 
lastModifiedTime.toMillis(), lastModifiedTimeMillis);
+        }
     }
 
 }

Reply via email to