fix (jkube-kit) : lastModified timestamps should be preserved while p…

…reparing assembly

We don't seem to be preserving lastModified timestamps of files that are
being copied while preparing the assembly for image. Make sure we're
copying all the metadata of files/directories as well while copying
them.

Signed-off-by: Rohan Kumar <[email protected]>

jkube-kit/common/src/main/java/org/eclipse/jkube/kit/common/util/FileUtil.java

@@ -36,6 +36,7 @@
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
+import static java.nio.file.StandardCopyOption.COPY_ATTRIBUTES;
 import static java.nio.file.StandardCopyOption.REPLACE_EXISTING;
 
 /**
@@ -201,7 +202,7 @@ public static void copy(File sourceFile, File targetFile) throws IOException {
     }
 
     public static void copy(Path sourcePath, Path targetPath) throws IOException {
-        Files.copy(sourcePath, targetPath, REPLACE_EXISTING);
+        Files.copy(sourcePath, targetPath, REPLACE_EXISTING, COPY_ATTRIBUTES);
     }
 
     public static void copyDirectoryIfNotExists(File sourceDir, File targetDir) throws IOException {

jkube-kit/common/src/test/java/org/eclipse/jkube/kit/common/util/FileUtilTest.java

@@ -15,7 +15,10 @@
 
 import java.io.File;
 import java.io.IOException;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.Date;
 import java.util.List;
 
 import org.junit.jupiter.api.Test;
@@ -151,6 +154,23 @@ void testGetRelativePath() throws IOException {
     assertThat(relativeFile.getPath()).isEqualTo("foo" + File.separator + "fileInFoo1");
   }
 
+  @Test
+  void copy_whenFileCopied_shouldPreserveLastModifiedTimestamp() throws IOException {
+    // Given
+    File sourceFile = new File(folder, "source");
+    Files.write(sourceFile.toPath(), "testdata".getBytes(StandardCharsets.UTF_8));
+    long originalTimestamp = new Date().getTime() - 10;
+    assertThat(sourceFile.setLastModified(originalTimestamp)).isTrue();
+    Path targetFilePath = folder.toPath().resolve("target");
+
+    // When
+    FileUtil.copy(sourceFile.toPath(), targetFilePath);
+
+    // Then
+    assertThat(targetFilePath.toFile()).hasSameTextualContentAs(sourceFile);
+    assertThat(targetFilePath.toFile().lastModified()).isEqualTo(originalTimestamp);
+  }
+
   private void prepareDirectory() throws IOException {
     final File dir1 = Files.createDirectories(folder.toPath().resolve("foo")).toFile();
     assertThat(new File(dir1, "fileInFoo1").createNewFile()).isTrue();