check revisions

Author: vladak

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

@@ -79,7 +79,6 @@ public void processStream(InputStream input) throws IOException {
                     // Use the history index hash map to get the author.
                     String author = Optional.ofNullable(revs.get(fullRev)).map(HistoryEntry::getAuthor).
                             orElse("N/A");
-                    // TODO: add check that history stored in the index uses the same format of the revision
                     annotation.addLine(fullRev, Util.getEmail(author.trim()), true, displayRev);
                 } else {
                     LOGGER.log(Level.WARNING,

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

@@ -438,7 +438,7 @@ void testAnnotationNegative() throws Exception {
     }
 
     private static Stream<Triple<String, List<String>, List<String>>> provideParametersForPositiveAnnotationTest() {
-        return Stream.of(Triple.of("8:6a8c423f5624", List.of("7", "8"), List.of("7:db1394c05268", "8:6a8c423f5624")),
+        return Stream.of(Triple.of("8:6a8c423f5624", List.of("7", "8"), List.of("8:6a8c423f5624", "7:db1394c05268")),
                 Triple.of("7:db1394c05268", List.of("7"), List.of("7:db1394c05268")));
     }
 
@@ -467,6 +467,7 @@ void testAnnotationPositive(Triple<String, List<String>, List<String>> triple) t
                 collect(Collectors.toList());
         assertFalse(relevantEntries.isEmpty());
         for (HistoryEntry entry : relevantEntries) {
+            assertTrue(annotation.getRevisions().contains(entry.getRevision()));
             assertEquals(entry.getDescription(), annotation.getDesc(entry.getRevision()));
             assertTrue(annotation.getAuthors().contains(Util.getEmail(entry.getAuthor())));
         }