add createHistoryCache() test

Author: vladak

opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryGuru.java

@@ -135,6 +135,11 @@ private HistoryGuru() {
         repositoryLookup = RepositoryLookup.cached();
     }
 
+    @VisibleForTesting
+    HistoryCache getHistoryCache() {
+        return historyCache;
+    }
+
     /**
      * Set annotation cache to its default implementation.
      * @return {@link AnnotationCache} instance or {@code null} on error
@@ -1048,7 +1053,6 @@ public void storeHistory(File file, History history) {
     }
 
     private void createHistoryCache(Repository repository, String sinceRevision) throws CacheException, HistoryException {
-        // TODO: add test for this
         if (!repository.isHistoryCacheEnabled()) {
             LOGGER.log(Level.INFO,
                     "Skipping history cache creation for {0} and its subdirectories: history cache disabled",

opengrok-indexer/src/test/java/org/opengrok/indexer/history/HistoryGuruVsRepositoryCacheTest.java

@@ -0,0 +1,101 @@
+/*
+ * CDDL HEADER START
+ *
+ * The contents of this file are subject to the terms of the
+ * Common Development and Distribution License (the "License").
+ * You may not use this file except in compliance with the License.
+ *
+ * See LICENSE.txt included in this distribution for the specific
+ * language governing permissions and limitations under the License.
+ *
+ * When distributing Covered Code, include this CDDL HEADER in each
+ * file and include the License file at LICENSE.txt.
+ * If applicable, add the following below this CDDL HEADER, with the
+ * fields enclosed by brackets "[]" replaced with your own identifying
+ * information: Portions Copyright [yyyy] [name of copyright owner]
+ *
+ * CDDL HEADER END
+ */
+
+/*
+ * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
+ */
+package org.opengrok.indexer.history;
+
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.ValueSource;
+import org.opengrok.indexer.configuration.CommandTimeoutType;
+import org.opengrok.indexer.configuration.RuntimeEnvironment;
+import org.opengrok.indexer.util.IOUtils;
+import org.opengrok.indexer.util.TestRepository;
+
+import java.io.File;
+import java.net.URL;
+import java.nio.file.Path;
+import java.util.Collection;
+import java.util.HashMap;
+import java.util.List;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+class HistoryGuruVsRepositoryCacheTest {
+    private RuntimeEnvironment env;
+
+    private static TestRepository testRepository;
+
+    @BeforeEach
+    void setUpClass() throws Exception {
+        env = RuntimeEnvironment.getInstance();
+
+        testRepository = new TestRepository();
+        URL resourceURL = HistoryGuru.class.getResource("/repositories");
+        assertNotNull(resourceURL);
+        testRepository.create(resourceURL);
+
+        env.setSourceRoot(testRepository.getSourceRoot());
+        env.setDataRoot(testRepository.getDataRoot());
+        env.setHistoryEnabled(true);
+        env.setProjectsEnabled(true);
+        RepositoryFactory.initializeIgnoredNames(env);
+
+        // Restore the project and repository information.
+        env.setProjects(new HashMap<>());
+        env.setRepositories(testRepository.getSourceRoot());
+        HistoryGuru.getInstance().invalidateRepositories(env.getRepositories(), CommandTimeoutType.INDEXER);
+        env.generateProjectRepositoriesMap();
+    }
+
+    @AfterEach
+    void tearDownClass() throws Exception {
+        testRepository.destroy();
+    }
+
+    /**
+     * Test that {@link HistoryGuru#createHistoryCache(Collection)} honors
+     * the {@link Repository#isHistoryCacheEnabled()} setting.
+     */
+    @ParameterizedTest
+    @ValueSource(booleans = {false, true})
+    void testCreateCacheVsRepository(boolean historyCacheEnabled) throws Exception {
+        Path filePath = Path.of(env.getSourceRootPath(), "git", "main.c");
+        File file = filePath.toFile();
+        assertTrue(file.exists());
+        HistoryGuru histGuru = HistoryGuru.getInstance();
+        Repository repository = histGuru.getRepository(file);
+        assertNotNull(repository);
+
+        repository.setHistoryCacheEnabled(historyCacheEnabled);
+        histGuru.createHistoryCache(List.of(repository.getDirectoryNameRelative()));
+
+        HistoryCache historyCache = histGuru.getHistoryCache();
+        assertNotNull(historyCache);
+        assertEquals(historyCacheEnabled, historyCache.hasCacheForFile(file));
+
+        // cleanup
+        IOUtils.removeRecursive(Path.of(env.getDataRootPath()));
+    }
+}