search

qbdsoft / owasp-esapi-php

Automatically exported from code.google.com/p/owasp-esapi-php
Other
0 stars 0 forks source link

LoggerTest should check that records make their way into the log #4

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Please update testFatal, testWarning, etc. to check that the record
actually makes it into the log, with the expected details.

Original issue reported on code.google.com by mike.bob...@gmail.com on 9 Oct 2009 at 7:30

GoogleCodeExporter commented 9 years ago

Original comment by mike.bob...@gmail.com on 9 Oct 2009 at 7:42

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Deleted my previous comment on this issue.  It occurred to me that, with the
timestamp being the only variable we can't reliably set an expectation for and 
with
the caveat that the tests will need to be changed if any of the ESAPI properties
change, it should be possible to set expectations for the rest of the bits that 
make
up a log entry.  Working on it.

Original comment by jahboite@gmail.com on 14 Feb 2010 at 12:42

GoogleCodeExporter commented 9 years ago 
I've updated the tests as requested and committed them in r506.

Original comment by jahboite@gmail.com on 14 Feb 2010 at 8:22