diff --git a/opengrok-web/src/test/java/org/opengrok/web/PageConfigTest.java b/opengrok-web/src/test/java/org/opengrok/web/PageConfigTest.java index af7f5c4aa06..8a5cab58db5 100644 --- a/opengrok-web/src/test/java/org/opengrok/web/PageConfigTest.java +++ b/opengrok-web/src/test/java/org/opengrok/web/PageConfigTest.java @@ -64,6 +64,7 @@ import org.opengrok.indexer.index.Indexer; import org.opengrok.indexer.util.TestRepository; import org.opengrok.indexer.web.DummyHttpServletRequest; +import org.opengrok.indexer.web.QueryParameters; import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertFalse; @@ -722,4 +723,52 @@ void testIsNotModifiedNotModified() { assertTrue(cfg.isNotModified(req, resp)); verify(resp).setStatus(eq(HttpServletResponse.SC_NOT_MODIFIED)); } + + @Test + void testGetPathTitle() { + HttpServletRequest req = new DummyHttpServletRequest() { + @Override + public String getPathInfo() { + return "path "; // use HTML reserved characters + } + }; + + PageConfig cfg = PageConfig.get(req); + assertEquals("path <foo> - OpenGrok cross reference for /path <foo>", cfg.getPathTitle()); + } + + + @Test + void testGetPathTitleRevision() { + HttpServletRequest req = new DummyHttpServletRequest() { + @Override + public String getPathInfo() { + return "bar"; + } + + @Override + public String getParameter(String name) { + if (name.equals(QueryParameters.REVISION_PARAM)) { + return "42"; + } + return null; + } + }; + + PageConfig cfg = PageConfig.get(req); + assertEquals("bar (revision 42) - OpenGrok cross reference for /bar", cfg.getPathTitle()); + } + + @Test + void testGetHistoryTitle() { + HttpServletRequest req = new DummyHttpServletRequest() { + @Override + public String getPathInfo() { + return "foo "; // use HTML reserved characters + } + }; + + PageConfig cfg = PageConfig.get(req); + assertEquals("foo <bar> - OpenGrok history log for /foo <bar>", cfg.getHistoryTitle()); + } }