1 files changed, 65 insertions, 10 deletions

diff --git a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/filedistribution/FileReferencesAndDownloadsMaintainerTest.java b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/filedistribution/FileReferencesAndDownloadsMaintainerTest.java
index fad021c0119..c41305b4dc8 100644
--- a/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/filedistribution/FileReferencesAndDownloadsMaintainerTest.java
+++ b/config-proxy/src/test/java/com/yahoo/vespa/config/proxy/filedistribution/FileReferencesAndDownloadsMaintainerTest.java
@@ -2,6 +2,7 @@
 package com.yahoo.vespa.config.proxy.filedistribution;
 
 import com.yahoo.io.IOUtils;
+import com.yahoo.vespa.config.util.ConfigUtils;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
@@ -10,6 +11,9 @@ import java.io.File;
 import java.io.IOException;
 import java.time.Duration;
 import java.time.Instant;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.stream.IntStream;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
@@ -19,9 +23,12 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
  */
 public class FileReferencesAndDownloadsMaintainerTest {
 
+    private static final Duration keepDuration = Duration.ofMinutes(1);
+    private static final int outDatedFilesToKeep = 9;
+
     private File cachedFileReferences;
     private File cachedDownloads;
-    private FileReferencesAndDownloadsMaintainer cachedFilesMaintainer;
+    private FileReferencesAndDownloadsMaintainer maintainer;
 
     @TempDir
     public File tempFolder;
@@ -30,22 +37,70 @@ public class FileReferencesAndDownloadsMaintainerTest {
     public void setup() throws IOException {
         cachedFileReferences = newFolder(tempFolder, "cachedFileReferences");
         cachedDownloads = newFolder(tempFolder, "cachedDownloads");
-        cachedFilesMaintainer = new FileReferencesAndDownloadsMaintainer(cachedFileReferences, cachedDownloads, Duration.ofMinutes(1));
     }
 
     @Test
-    void require_old_files_to_be_deleted() throws IOException {
+    void require_old_files_to_be_deleted() {
+        maintainer = new FileReferencesAndDownloadsMaintainer(cachedFileReferences, cachedDownloads, keepDuration, outDatedFilesToKeep,
+                                                              List.of("host1"));
         runMaintainerAndAssertFiles(0, 0);
 
-        File fileReference = writeFile(cachedFileReferences, "fileReference");
-        File download = writeFile(cachedDownloads, "download");
-        runMaintainerAndAssertFiles(1, 1);
+        var fileReferences = writeFiles(20);
+        var downloads = writeDownloads(21);
+        runMaintainerAndAssertFiles(20, 21);
+
+        updateLastModifiedTimestamp(0, 5, fileReferences, downloads);
+        runMaintainerAndAssertFiles(15, 16);
 
-        updateLastModifiedTimeStamp(fileReference, Instant.now().minus(Duration.ofMinutes(10)));
-        runMaintainerAndAssertFiles(0, 1);
+        updateLastModifiedTimestamp(6, 20, fileReferences, downloads);
+        // Should keep at least outDatedFilesToKeep file references and downloads even if there are more that are old
+        runMaintainerAndAssertFiles(outDatedFilesToKeep, outDatedFilesToKeep);
+    }
 
-        updateLastModifiedTimeStamp(download, Instant.now().minus(Duration.ofMinutes(10)));
+    @Test
+    void require_no_files_deleted_when_running_on_config_server_host() {
+        maintainer = new FileReferencesAndDownloadsMaintainer(cachedFileReferences, cachedDownloads, keepDuration,
+                                                              outDatedFilesToKeep, List.of(ConfigUtils.getCanonicalHostName()));
         runMaintainerAndAssertFiles(0, 0);
+
+        var fileReferences = writeFiles(10);
+        var downloads = writeDownloads(10);
+        runMaintainerAndAssertFiles(10, 10);
+
+        updateLastModifiedTimestamp(0, 10, fileReferences, downloads);
+        runMaintainerAndAssertFiles(10, 10);
+    }
+
+    private void updateLastModifiedTimestamp(int startInclusive, int endExclusive, List<File> fileReferences, List<File> downloads) {
+        IntStream.range(startInclusive, endExclusive).forEach(i -> {
+            Instant instant = Instant.now().minus(keepDuration.plus(Duration.ofMinutes(1)).minus(Duration.ofSeconds(i)));
+            updateLastModifiedTimeStamp(fileReferences.get(i), instant);
+            updateLastModifiedTimeStamp(downloads.get(i), instant);
+        });
+    }
+
+    private List<File> writeFiles(int count) {
+        List<File> files = new ArrayList<>();
+        IntStream.range(0, count).forEach(i -> {
+            try {
+                files.add(writeFile(cachedFileReferences, "fileReference" + i));
+            } catch (IOException e) {
+                throw new RuntimeException(e);
+            }
+        });
+        return files;
+    }
+
+    private List<File> writeDownloads(int count) {
+        List<File> files = new ArrayList<>();
+        IntStream.range(0, count).forEach(i -> {
+            try {
+                files.add(writeFile(cachedDownloads, "download" + i));
+            } catch (IOException e) {
+                throw new RuntimeException(e);
+            }
+        });
+        return files;
     }
 
     private void updateLastModifiedTimeStamp(File file, Instant instant) {
@@ -55,7 +110,7 @@ public class FileReferencesAndDownloadsMaintainerTest {
     }
 
     private void runMaintainerAndAssertFiles(int fileReferenceCount, int downloadCount) {
-        cachedFilesMaintainer.run();
+        maintainer.run();
         File[] fileReferences = cachedFileReferences.listFiles();
         assertNotNull(fileReferences);
         assertEquals(fileReferenceCount, fileReferences.length);