cleanup EnabledForRepository.GIT

Author: Vladimir Kotal

dev/before

@@ -3,7 +3,6 @@
 echo "SCM versions:"
 if command -v bk; then bk --version; fi
 if command -v hg; then hg --version; fi
-if command -v git; then git --version; fi
 if command -v svn; then svn --version; fi
 if command -v cvs; then cvs --version; fi
 if command -v mtn; then mtn --version; fi

opengrok-indexer/src/test/java/org/opengrok/indexer/condition/RepositoryInstalled.java

@@ -27,7 +27,6 @@
 import org.opengrok.indexer.history.BazaarRepository;
 import org.opengrok.indexer.history.BitKeeperRepository;
 import org.opengrok.indexer.history.CVSRepository;
-import org.opengrok.indexer.history.GitRepository;
 import org.opengrok.indexer.history.MercurialRepository;
 import org.opengrok.indexer.history.PerforceRepository;
 import org.opengrok.indexer.history.RCSRepository;
@@ -42,7 +41,6 @@ public class RepositoryInstalled {
     public enum Type {
         BITKEEPER(new BitKeeperRepository()),
         MERCURIAL(new MercurialRepository()),
-        GIT(new GitRepository()),
         RCS(new RCSRepository()),
         BAZAAR(new BazaarRepository()),
         CVS(new CVSRepository()),

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

@@ -26,7 +26,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.opengrok.indexer.condition.EnabledForRepository;
 import org.opengrok.indexer.configuration.RuntimeEnvironment;
 import org.opengrok.indexer.util.TestRepository;
 
@@ -35,7 +34,6 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 
 /**
  * Test file-based history cache with for git-octopus.
@@ -65,7 +63,6 @@ public void tearDown() {
         cache = null;
     }
 
-    @EnabledForRepository(GIT)
     @Test
     public void testStoreAndGet() throws Exception {
         File reposRoot = new File(repositories.getSourceRoot(), "git-octopus");

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

@@ -26,12 +26,10 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 
 import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
-import org.opengrok.indexer.condition.EnabledForRepository;
 import org.opengrok.indexer.configuration.RuntimeEnvironment;
 import org.opengrok.indexer.util.TestRepository;
 import org.opengrok.indexer.web.Util;
@@ -45,7 +43,6 @@
 /**
  * @author austvik
  */
-@EnabledForRepository(GIT)
 public class GitRepositoryOctopusTest {
 
     private static TestRepository repository = new TestRepository();

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

@@ -28,7 +28,6 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.CVS;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.MERCURIAL;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.SUBVERSION;
 
@@ -152,7 +151,6 @@ public void getCacheInfo() throws HistoryException {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testAddRemoveRepositories() {
         HistoryGuru instance = HistoryGuru.getInstance();
         final int numReposOrig = instance.getRepositories().size();
@@ -177,7 +175,7 @@ public void testAddRemoveRepositories() {
     }
 
     @Test
-    @EnabledForRepository({CVS, GIT})
+    @EnabledForRepository(CVS)
     public void testAddSubRepositoryNotNestable() {
         HistoryGuru instance = HistoryGuru.getInstance();
 
@@ -251,7 +249,6 @@ private static void certainlyMkdirs(File file) throws IOException {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testScanningDepth() throws IOException {
         String topLevelDirName = "scanDepthTest";
         File repoRoot = new File(repository.getSourceRoot(), topLevelDirName);

opengrok-indexer/src/test/java/org/opengrok/indexer/index/IndexDatabaseSymlinksTest.java

@@ -53,7 +53,7 @@
  * Represents a container for additional tests of {@link IndexDatabase} for symlinks.
  */
 @EnabledOnOs({OS.LINUX, OS.MAC, OS.SOLARIS, OS.AIX, OS.OTHER})
-@EnabledForRepository({RepositoryInstalled.Type.GIT, RepositoryInstalled.Type.MERCURIAL})
+@EnabledForRepository(RepositoryInstalled.Type.MERCURIAL)
 public class IndexDatabaseSymlinksTest {
 
     private static RuntimeEnvironment env;

opengrok-indexer/src/test/java/org/opengrok/indexer/index/IndexerRepoTest.java

@@ -37,7 +37,6 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.MERCURIAL;
 
 import org.junit.jupiter.api.AfterEach;
@@ -77,7 +76,7 @@ public void tearDown() {
      * Test it is possible to disable history per project.
      */
     @Test
-    @EnabledForRepository({MERCURIAL, GIT})
+    @EnabledForRepository(MERCURIAL)
     public void testPerProjectHistoryGlobalOn() throws IndexerException, IOException, HistoryException {
         testPerProjectHistory(true);
     }
@@ -86,7 +85,7 @@ public void testPerProjectHistoryGlobalOn() throws IndexerException, IOException
      * Test it is possible to enable history per project.
      */
     @Test
-    @EnabledForRepository({MERCURIAL, GIT})
+    @EnabledForRepository(MERCURIAL)
     public void testPerProjectHistoryGlobalOff() throws IndexerException, IOException, HistoryException {
         testPerProjectHistory(false);
     }

opengrok-indexer/src/test/java/org/opengrok/indexer/index/IndexerTest.java

@@ -29,7 +29,6 @@
 import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.MERCURIAL;
 
 import java.io.BufferedWriter;
@@ -366,7 +365,7 @@ public void testRemoveFileOnFileChange() throws Exception {
     }
 
     @Test
-    @EnabledForRepository({MERCURIAL, GIT})
+    @EnabledForRepository(MERCURIAL)
     public void testSetRepositories() throws Exception {
         RuntimeEnvironment env = RuntimeEnvironment.getInstance();
 

opengrok-web/src/test/java/org/opengrok/web/PageConfigTest.java

@@ -57,7 +57,6 @@
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.junit.jupiter.api.Assumptions.assumeTrue;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 import static org.opengrok.indexer.condition.RepositoryInstalled.Type.MERCURIAL;
 
 /**
@@ -257,7 +256,6 @@ public String getParameter(String name) {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testGetLatestRevisionValid() {
         DummyHttpServletRequest req1 = new DummyHttpServletRequest() {
             @Override
@@ -273,7 +271,6 @@ public String getPathInfo() {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testGetRevisionLocation() {
         DummyHttpServletRequest req1 = new DummyHttpServletRequest() {
             @Override
@@ -300,7 +297,6 @@ public String getQueryString() {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testGetRevisionLocationNullQuery() {
         DummyHttpServletRequest req1 = new DummyHttpServletRequest() {
             @Override
@@ -327,7 +323,6 @@ public String getQueryString() {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testGetLatestRevisionNotValid() {
         DummyHttpServletRequest req2 = new DummyHttpServletRequest() {
             @Override
@@ -368,7 +363,6 @@ public String getParameter(String name) {
     }
 
     @Test
-    @EnabledForRepository(GIT)
     public void testGetAnnotation() {
         final String[] revisions = {"aa35c258", "bb74b7e8"};
 

opengrok-web/src/test/java/org/opengrok/web/api/v1/controller/AnnotationControllerTest.java

@@ -29,7 +29,6 @@
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
-import org.opengrok.indexer.condition.EnabledForRepository;
 import org.opengrok.indexer.configuration.RuntimeEnvironment;
 import org.opengrok.indexer.history.HistoryGuru;
 import org.opengrok.indexer.history.RepositoryFactory;
@@ -50,9 +49,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.opengrok.indexer.condition.RepositoryInstalled.Type.GIT;
 
-@EnabledForRepository(GIT)
 public class AnnotationControllerTest extends OGKJerseyTest {
 
     private final RuntimeEnvironment env = RuntimeEnvironment.getInstance();