tests

Author: vladak

src/org/opensolaris/opengrok/web/PageConfig.java

@@ -1193,15 +1193,15 @@ public File findDataFile() {
                 path, env.isCompressXref());
     }
 
-    private String getLatestRevision() {
-        if (!env.isHistoryEnabled()) {
+    protected String getLatestRevision() {
+        if (!getEnv().isHistoryEnabled()) {
             return null;
         }
 
         History hist;
         try {
             hist = HistoryGuru.getInstance().
-                    getHistory(new File(env.getSourceRootFile(), path));
+                    getHistory(new File(getEnv().getSourceRootFile(), getPath()));
         } catch (HistoryException ex) {
             return null;
         }

test/org/opensolaris/opengrok/web/PageConfigTest.java

@@ -241,6 +241,36 @@ public String getParameter(String name) {
         }
     }
 
+    @Test
+    public void testGetLatestRevisionValid() {
+        DummyHttpServletRequest req1 = new DummyHttpServletRequest() {
+            @Override
+                public String getPathInfo() {
+                    return "/git/main.c";
+            }
+        };
+
+        PageConfig cfg = PageConfig.get(req1);
+        String rev = cfg.getLatestRevision();
+
+        assertEquals("aa35c258", rev);
+    }
+
+    @Test
+    public void testGetLatestRevisionNotValid() {
+        DummyHttpServletRequest req2 = new DummyHttpServletRequest() {
+            @Override
+                public String getPathInfo() {
+                    return "/git/nonexistent_file";
+            }
+        };
+
+        PageConfig cfg = PageConfig.get(req2);
+        String rev = cfg.getLatestRevision();
+
+        assertNull(rev);
+    }
+
     @Test
     public void testGetRequestedRevision() {
         final String[] params = {"r", "h", "r", "r", "r"};