Statistics testing party

From the test coverage data for statistics.admin.inc and statistics.module, the following things most likely need testing:

- "Recent hits" page (entire page) [important]
- "Top pages" page (entire page) [important]
- "Top visitors" page with no available statistics [minor]
- "Top referrers" page (entire page) [important]
- "Access log" page (entire page) [important]
- Using the "access log settings" page to change statistics' settings [moderately important]
- Viewing a node with a certain number of views - does the "1 read" / "@count reads" text show up properly? [moderately important]
- Test proper handling of deleting a user - is the accesslog properly updated to reflect that the user id no longer exists? [minor]
- Proper clearing of day counts and expired access logs [moderately important]
- Statistics' "popular content" block [important]
- Test correct handling of deleting a node - is the node's counter also deleted? [minor]

Hope this helps! :) It would be great to see a few more thorough tests for statistics.

Taking a look at statistics.admin #drupalcon

"Recent hits" page (entire page) [important]
"Top pages" page (entire page) [important]
"Top referrers" page (entire page) [important]
"Access log" page (entire page) [important]

The above were already written for #659578: Top Pages and Top Referrers reports giving errors

I have (finally) written tests for the rest of the stuff.

This looks like a great patch. Some minor issues:

+++ modules/statistics/statistics.test	3 Jan 2010 22:01:37 -0000
@@ -88,12 +86,41 @@ class StatisticsReportsTestCase extends 
+    // Get some page and check if the block is displayed

Make sure your comments end with a point (.). There are a couple of instances where you forgot the trailing point.

+++ modules/statistics/statistics.test	3 Jan 2010 22:01:37 -0000
@@ -145,3 +172,140 @@ class StatisticsBlockVisitorsTestCase ex
+   * Tests that cron clears daycounts and expired accesslogs.

Let's write 'day counts' and 'access logs' here.

+++ modules/statistics/statistics.test	3 Jan 2010 22:01:37 -0000
@@ -145,3 +172,140 @@ class StatisticsBlockVisitorsTestCase ex
+    // Wait two secs before running cron, so the access log timer will time 
+    // out.
+    sleep(2); 

Could use some extra documentation. It's not immediately clear what the timer is for, and why we need to wait 2 seconds.

Put some missing points in the comments and corrected the wording in comments a bit.

And.... Uploaded the wrong patch.. Here is the right one

I'm marking this RTBC because this looks good.

Agreed! :)

Committed to HEAD.