[Jifty-commit] r6396 - in jifty/trunk: lib/Jifty

Jifty commits jifty-commit at lists.jifty.org
Fri Feb 20 22:35:25 EST 2009


Author: alexmv
Date: Fri Feb 20 22:35:25 2009
New Revision: 6396

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Test.pm

Log:
 r42407 at kohr-ah:  chmrr | 2009-02-20 22:34:56 -0500
  * Warnings avoidance for tests which don't set up the mailbox or an app


Modified: jifty/trunk/lib/Jifty/Test.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Test.pm	(original)
+++ jifty/trunk/lib/Jifty/Test.pm	Fri Feb 20 22:35:25 2009
@@ -531,6 +531,7 @@
 =cut
 
 sub messages {
+    return () unless -f mailbox();
     return Email::Folder->new(mailbox())->messages;
 }
 
@@ -612,8 +613,9 @@
 sub _ending {
     my $Test = Jifty::Test->builder;
 
-    my $plugin = Jifty->find_plugin("Jifty::Plugin::TestServerWarnings");
-    warn "Uncaught warning: $_" for $plugin->stashed_warnings;
+    if (my $plugin = Jifty->find_plugin("Jifty::Plugin::TestServerWarnings")) {
+        warn "Uncaught warning: $_" for $plugin->stashed_warnings;
+    }
 
     # Such a hack -- try to detect if this is a forked copy and don't
     # do cleanup in that case.


More information about the Jifty-commit mailing list