There should be tests for mongodb_watchdog.

[17:47] dereine: i am not sure whether we have tests for the dblog admin screen , if not then the work is instnatly reusable as a core test. if there is such a core test then that just needs to be adjusted

Comments

Status:Active» Needs work
StatusFileSize
new21.36 KB

Totally broken. Sadly currently together with parts of http://drupal.org/node/660460

StatusFileSize
new26.64 KB

To test this patch, apply first http://drupal.org/files/issues/mongodb_watchdog-capped.patch

Down to:

84 passes, 9 fails, 0 exceptions, and 25 debug messages

Status:Needs work» Needs review
StatusFileSize
new27.65 KB

Tests are passing now, missing "feature" of the watchdog admin ui convertion.

TODO: Convert admin/reports/(not_found|access_denied) to mongodb_watchdog

This patch is sure togehter with the capped stuff.

Component:Code» Watchdog

Reassigning.

I think I committed this :) I need to run 'em to see whether they pass :p commit early, commit often :D

Status:Needs review» Needs work

No longer applies (CVS format ?)

The committed versions do not run currently.

The D6/D7 patches included in #1355808: Implement watchdog filter-on-insert in mongodb_watchdog include working tests for the new filter-on-insert feature.

Improvement added: tests no longer choke on qa.drupal.org: they are bypassed when no connection is available, so qa can still check well-formedness but not waste resources on tests that can only fail.

@chx - did you commit this?

@fgm - Can we close this issue?