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); + } } }