Skip to content

Commit c6f8b25

Browse files
committed
add tests for PageConfig's getHistoryTitle() and getPathTitle()
1 parent c1e3fa2 commit c6f8b25

File tree

1 file changed

+49
-0
lines changed

1 file changed

+49
-0
lines changed

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

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@
6464
import org.opengrok.indexer.index.Indexer;
6565
import org.opengrok.indexer.util.TestRepository;
6666
import org.opengrok.indexer.web.DummyHttpServletRequest;
67+
import org.opengrok.indexer.web.QueryParameters;
6768

6869
import static org.junit.jupiter.api.Assertions.assertEquals;
6970
import static org.junit.jupiter.api.Assertions.assertFalse;
@@ -722,4 +723,52 @@ void testIsNotModifiedNotModified() {
722723
assertTrue(cfg.isNotModified(req, resp));
723724
verify(resp).setStatus(eq(HttpServletResponse.SC_NOT_MODIFIED));
724725
}
726+
727+
@Test
728+
void testGetPathTitle() {
729+
HttpServletRequest req = new DummyHttpServletRequest() {
730+
@Override
731+
public String getPathInfo() {
732+
return "path <foo>"; // use HTML reserved characters
733+
}
734+
};
735+
736+
PageConfig cfg = PageConfig.get(req);
737+
assertEquals("path &lt;foo&gt; - OpenGrok cross reference for /path &lt;foo&gt;", cfg.getPathTitle());
738+
}
739+
740+
741+
@Test
742+
void testGetPathTitleRevision() {
743+
HttpServletRequest req = new DummyHttpServletRequest() {
744+
@Override
745+
public String getPathInfo() {
746+
return "bar";
747+
}
748+
749+
@Override
750+
public String getParameter(String name) {
751+
if (name.equals(QueryParameters.REVISION_PARAM)) {
752+
return "42";
753+
}
754+
return null;
755+
}
756+
};
757+
758+
PageConfig cfg = PageConfig.get(req);
759+
assertEquals("bar (revision 42) - OpenGrok cross reference for /bar", cfg.getPathTitle());
760+
}
761+
762+
@Test
763+
void testGetHistoryTitle() {
764+
HttpServletRequest req = new DummyHttpServletRequest() {
765+
@Override
766+
public String getPathInfo() {
767+
return "foo <bar>"; // use HTML reserved characters
768+
}
769+
};
770+
771+
PageConfig cfg = PageConfig.get(req);
772+
assertEquals("foo &lt;bar&gt; - OpenGrok history log for /foo &lt;bar&gt;", cfg.getHistoryTitle());
773+
}
725774
}

0 commit comments

Comments
 (0)