add basic History test

Author: Vladimir Kotal

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

@@ -123,4 +123,22 @@ public boolean isRenamed(String file) {
     public Set<String> getRenamedFiles() {
         return renamedFiles;
     }
+
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
+        History that = (History) o;
+        return this.getHistoryEntries().equals(that.getHistoryEntries()) &&
+                this.getRenamedFiles().equals(that.getRenamedFiles());
+    }
+
+    @Override
+    public int hashCode() {
+        return getHistoryEntries().hashCode() * getRenamedFiles().hashCode();
+    }
 }

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

@@ -23,6 +23,7 @@
  */
 package org.opengrok.indexer.history;
 
+import java.util.Collection;
 import java.util.Date;
 import java.util.SortedSet;
 import java.util.TreeSet;
@@ -84,6 +85,12 @@ public HistoryEntry(String revision, Date date, String author,
         this.files = new TreeSet<>();
     }
 
+    public HistoryEntry(String revision, Date date, String author,
+                        String tags, String message, boolean active, Collection<String> files) {
+        this(revision, date, author, tags, message, active);
+        this.files.addAll(files);
+    }
+
     public String getLine() {
         return revision + " " + date + " " + author + " " + message + "\n";
     }
@@ -207,4 +214,28 @@ public void stripFiles() {
     public void stripTags() {
         tags = null;
     }
+
+    @Override
+    public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
+        HistoryEntry that = (HistoryEntry) o;
+        return this.getAuthor().equals(that.getAuthor()) &&
+                this.getRevision().equals(that.getRevision()) &&
+                this.getDate().equals(that.getDate()) &&
+                this.getMessage().equals(that.getMessage()) &&
+                this.getFiles().equals(that.getFiles()) &&
+                (this.getTags() != null && that.getTags() != null ?
+                        this.getTags().equals(that.getTags()) : this.getTags() == null && that.getTags() == null);
+    }
+
+    @Override
+    public int hashCode() {
+        return getAuthor().hashCode() + getRevision().hashCode() + getDate().hashCode() + getMessage().hashCode() +
+                getFiles().hashCode() + (getTags() != null ? getTags().hashCode() : 0);
+    }
 }

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

@@ -35,6 +35,7 @@
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
+import java.util.SortedSet;
 import java.util.TreeSet;
 
 import org.eclipse.jgit.api.Git;
@@ -506,12 +507,59 @@ private void runRenamedTest(String fname, String cset, String content) throws Ex
         }
     }
 
+    @Test
+    public void testHistory() throws Exception {
+        RuntimeEnvironment.getInstance().setHandleHistoryOfRenamedFiles(false);
+        File root = new File(repository.getSourceRoot(), "git");
+        GitRepository gitrepo = (GitRepository) RepositoryFactory.getRepository(root);
+
+        History history = gitrepo.getHistory(root);
+        assertNotNull(history);
+        assertNotNull(history.getHistoryEntries());
+        assertEquals(8, history.getHistoryEntries().size());
+        assertEquals(0, history.getRenamedFiles().size());
+
+        History expectedHistory = new History(List.of(
+                new HistoryEntry("84599b3c", new Date(1485438707000L),
+                        "Kryštof Tulinger <[email protected]>", null,
+                    "    renaming directories\n\n", true,
+                        Set.of("/git/moved2/renamed2.c")),
+                new HistoryEntry("67dfbe26", new Date(1485263397000L),
+                        "Kryštof Tulinger <[email protected]>", null,
+                        "    renaming renamed -> renamed2\n\n", true,
+                        Set.of("/git/moved/renamed2.c")),
+                new HistoryEntry("1086eaf5", new Date(1485263368000L),
+                        "Kryštof Tulinger <[email protected]>", null,
+                        "     adding some lines into renamed.c\n\n", true,
+                        Set.of("/git/moved/renamed.c")),
+                new HistoryEntry("b6413947", new Date(1485263264000L),
+                        "Kryštof Tulinger <[email protected]>", null,
+                        "    moved renamed.c to new location\n\n", true,
+                        Set.of("/git/moved/renamed.c")),
+                new HistoryEntry("ce4c98ec", new Date(1485263232000L),
+                        "Kryštof Tulinger <[email protected]>", null,
+                        "    adding simple file for renamed file testing\n\n", true,
+                        Set.of("/git/renamed.c")),
+                new HistoryEntry("aa35c258", new Date(1218571965000L),
+                        "Trond Norbye <[email protected]>", null,
+                        "    Add lint make target and fix lint warnings\n\n", true,
+                        Set.of("/git/Makefile", "/git/main.c")),
+                new HistoryEntry("84821564", new Date(1218571643000L),
+                        "Trond Norbye <[email protected]>", null,
+                        "    Add the result of a make on Solaris x86\n\n", true,
+                        Set.of("/git/main.o", "/git/testsprog")),
+                new HistoryEntry("bb74b7e8", new Date(1218571573000L),
+                        "Trond Norbye <[email protected]>", null,
+                        "    Added a small test program\n\n", true,
+                        Set.of("/git/Makefile", "/git/header.h", "/git/main.c"))));
+        assertEquals(expectedHistory, history);
+    }
+
     @Test
     public void testRenamedHistory() throws Exception {
         RuntimeEnvironment.getInstance().setHandleHistoryOfRenamedFiles(true);
         File root = new File(repository.getSourceRoot(), "git");
-        GitRepository gitrepo
-                = (GitRepository) RepositoryFactory.getRepository(root);
+        GitRepository gitrepo = (GitRepository) RepositoryFactory.getRepository(root);
 
         History history = gitrepo.getHistory(root);
         assertNotNull(history);