Improved Javadoc and added test for isPage method.

src/fr/devinsy/statoolinfos/metrics/httpaccess/HttpAccessLogAnalyzer.java

@@ -307,7 +307,21 @@ public class HttpAccessLogAnalyzer
 
     /**
      * Checks if is page.
-     *
+     * 
+     * <pre>
+     * null             => false
+     * ""               => false
+     * "     "          => false
+     * "GET /foo"       => true
+     * "GET /foo42"     => true
+     * "GET /foo.cgi"   => true
+     * "GET /foo.htm"  => true
+     * "GET /foo.html"  => true
+     * "GET /foo.php"   => true
+     * "GET /foo.xhtml"  => true
+     * "GET /foo.jpg"   => false
+     * </pre>
+     * 
      * @param request
      *            the request
      * @return true, if is page

test/fr/devinsy/statoolinfos/metrics/httpaccess/HttpAccessLogsAnalyzerTest.java

@@ -38,6 +38,27 @@ import fr.devinsy.statoolinfos.util.FilesUtils;
  */
 public class HttpAccessLogsAnalyzerTest
 {
+    /**
+     * Checks if is page test 01.
+     */
+    @Test
+    public void isPageTest01()
+    {
+        Assert.assertFalse(HttpAccessLogAnalyzer.isPage(null));
+        Assert.assertFalse(HttpAccessLogAnalyzer.isPage(""));
+        Assert.assertFalse(HttpAccessLogAnalyzer.isPage("     "));
+
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo"));
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo42"));
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo.html"));
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo.HTML"));
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo.HtMl"));
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo.HtMl"));
+
+        Assert.assertTrue(HttpAccessLogAnalyzer.isPage("GET /foo/bar/foo.php"));
+        Assert.assertFalse(HttpAccessLogAnalyzer.isPage("GET /foo.jpg"));
+    }
+
     /**
      * @throws Exception
      */