Would suggest putting the resetting of the old logger in the tearDown()
method...

I have to do this a lot a the moment where all unit tests are run through
irrespective of the result of others and this means you don't run any risk
of losing the output.

Finally will do the same but Junit provides this sort of method for a
reason...

-----Original Message-----
From: Simon Kitching [mailto:[EMAIL PROTECTED] 
Sent: 01 October 2003 02:14
To: Jakarta Commons Developers List
Subject: Re: [digester] turning off commons-logging for unit tests


On Wed, 2003-10-01 at 13:06, Simon Kitching wrote:
>  
>  try {
>      Log oldLog = digester.getLogger();
>      digester.setLogger(new NoOpLog());
>      digester.parse(
>         TestAll.getInputStream(this, "test2.xml"));
>      digester.setLogger(oldLog);
>   }
>   catch(Exception e) {
>     // yay - exception was correctly generated
>     // ....
>   }

hmm..try

 Log oldLog = digester.getLogger();
 try {
    digester.setLogger(new NoOpLog());
    digester.parse(
      TestAll.getInputStream(this, "test2.xml"));
 }
 catch(Exception e) {
     // yay ... etc
 }
 finally {
   digester.setLogger(oldLog);
 }

:-)

In fact, it probably doesn't matter a damn about restoring the old logger,
as the test has failed. Still, don't want anyone thinking I can't handle
exceptions properly!


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
 
**************************************************************************
The information transmitted herewith is sensitive information intended only
for use by the individual or entity to which it is addressed. If the reader
of this message is not the intended recipient, you are hereby notified that
any review, retransmission, dissemination, distribution, copying or other
use of, or taking of any action in reliance upon this information is
strictly prohibited. If you have received this communication in error,
please contact the sender and delete the material from your computer.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to