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 <[email protected]>
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);
+ }
}
}