Properly local()ize several $SIG{__WARN__} handlers
Peter Rabbitson [Thu, 21 Aug 2008 15:22:58 +0000 (15:22 +0000)]
t/100extra_source.t
t/94versioning.t
t/cdbi-t/15-accessor.t

index ac92209..5c531fd 100644 (file)
@@ -26,7 +26,7 @@ my $new_source = DBICTest::ResultSource::OtherSource->new({
 $new_source->add_column('other_col' => { data_type => 'integer', default_value => 1 });
 
 my $warn = '';
-$SIG{__WARN__} = sub { $warn = shift };
+local $SIG{__WARN__} = sub { $warn = shift };
 
 {
   $schema->register_extra_source( 'artist->extra' => $new_source );
index d9e2c41..633100e 100644 (file)
@@ -95,7 +95,7 @@ eval "use DBICVersionNew";
 
 
   my $warn = '';
-  $SIG{__WARN__} = sub { $warn = shift };
+  local $SIG{__WARN__} = sub { $warn = shift };
   $schema_version = DBICVersion::Schema->connect($dsn, $user, $pass);
   like($warn, qr/Your DB is currently unversioned/, 'warning detected without env var or attr');
 
index ad28a63..30f61c8 100644 (file)
@@ -213,5 +213,4 @@ eval {
        like $@, qr/read only/, "And can't delete 4 Days in July";
        my $abigail = eval { Film->create({ title => "Abigail's Party" }) };
        like $@, qr/read only/, "Or create new films";
-       $SIG{__WARN__} = sub { };
 }