commit perl-Test-FailWarnings for openSUSE:Factory
Hello community, here is the log from the commit of package perl-Test-FailWarnings for openSUSE:Factory checked in at 2013-09-27 17:57:02 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/perl-Test-FailWarnings (Old) and /work/SRC/openSUSE:Factory/.perl-Test-FailWarnings.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "perl-Test-FailWarnings" Changes: -------- --- /work/SRC/openSUSE:Factory/perl-Test-FailWarnings/perl-Test-FailWarnings.changes 2013-08-27 21:04:26.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.perl-Test-FailWarnings.new/perl-Test-FailWarnings.changes 2013-09-27 17:57:04.000000000 +0200 @@ -1,0 +2,10 @@ +Mon Sep 2 11:12:09 UTC 2013 - joschibrauchle@gmx.de + +- Update to version 0.007 + * In some situations, the source of a local warning can have an + absolute filename and would not be reported if ALLOW_DEPS were true. + Now absolute filenames are made relative to the original cwd before + checking if they are local or not. + * Fixed synopsis to match new failing test label + +------------------------------------------------------------------- Old: ---- Test-FailWarnings-0.006.tar.gz New: ---- Test-FailWarnings-0.007.tar.gz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ perl-Test-FailWarnings.spec ++++++ --- /var/tmp/diff_new_pack.eBr5c3/_old 2013-09-27 17:57:04.000000000 +0200 +++ /var/tmp/diff_new_pack.eBr5c3/_new 2013-09-27 17:57:04.000000000 +0200 @@ -17,7 +17,7 @@ Name: perl-Test-FailWarnings -Version: 0.006 +Version: 0.007 Release: 0 %define cpan_name Test-FailWarnings Summary: Add test failures if warnings are caught ++++++ Test-FailWarnings-0.006.tar.gz -> Test-FailWarnings-0.007.tar.gz ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/Changes new/Test-FailWarnings-0.007/Changes --- old/Test-FailWarnings-0.006/Changes 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/Changes 2013-08-30 15:53:50.000000000 +0200 @@ -1,5 +1,18 @@ Revision history for Test-FailWarnings +0.007 2013-08-30 09:53:47 America/New_York + + [FIXED] + + - In some situations, the source of a local warning can have an + absolute filename and would not be reported if ALLOW_DEPS were true. + Now absolute filenames are made relative to the original cwd before + checking if they are local or not. + + [DOCUMENTED] + + - Fixed synopsis to match new failing test label + 0.006 2013-08-22 10:41:11 America/New_York [CHANGED] diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/META.json new/Test-FailWarnings-0.007/META.json --- old/Test-FailWarnings-0.006/META.json 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/META.json 2013-08-30 15:53:50.000000000 +0200 @@ -4,7 +4,7 @@ "David Golden <dagolden@cpan.org>" ], "dynamic_config" : 0, - "generated_by" : "Dist::Zilla version 4.300035, CPAN::Meta::Converter version 2.132140", + "generated_by" : "Dist::Zilla version 4.300037, CPAN::Meta::Converter version 2.132140", "license" : [ "apache_2_0" ], @@ -41,6 +41,8 @@ "runtime" : { "requires" : { "Carp" : "0", + "Cwd" : "0", + "File::Spec" : "0", "Test::More" : "0.86", "perl" : "5.008001", "strict" : "0", @@ -51,11 +53,13 @@ "requires" : { "Capture::Tiny" : "0.12", "ExtUtils::MakeMaker" : "0", + "File::Spec" : "0", "File::Spec::Functions" : "0", "File::Temp" : "0", + "IO::Handle" : "0", + "IPC::Open3" : "0", "List::Util" : "0", "Test::More" : "0.96", - "blib" : "0", "constant" : "0", "lib" : "0" } @@ -64,7 +68,7 @@ "provides" : { "Test::FailWarnings" : { "file" : "lib/Test/FailWarnings.pm", - "version" : "0.006" + "version" : "0.007" } }, "release_status" : "stable", @@ -79,7 +83,7 @@ "web" : "https://github.com/dagolden/test-failwarnings" } }, - "version" : "0.006", + "version" : "0.007", "x_authority" : "cpan:DAGOLDEN" } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/META.yml new/Test-FailWarnings-0.007/META.yml --- old/Test-FailWarnings-0.006/META.yml 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/META.yml 2013-08-30 15:53:50.000000000 +0200 @@ -5,17 +5,19 @@ build_requires: Capture::Tiny: 0.12 ExtUtils::MakeMaker: 0 + File::Spec: 0 File::Spec::Functions: 0 File::Temp: 0 + IO::Handle: 0 + IPC::Open3: 0 List::Util: 0 Test::More: 0.96 - blib: 0 constant: 0 lib: 0 configure_requires: ExtUtils::MakeMaker: 6.30 dynamic_config: 0 -generated_by: 'Dist::Zilla version 4.300035, CPAN::Meta::Converter version 2.132140' +generated_by: 'Dist::Zilla version 4.300037, CPAN::Meta::Converter version 2.132140' license: apache meta-spec: url: http://module-build.sourceforge.net/META-spec-v1.4.html @@ -32,9 +34,11 @@ provides: Test::FailWarnings: file: lib/Test/FailWarnings.pm - version: 0.006 + version: 0.007 requires: Carp: 0 + Cwd: 0 + File::Spec: 0 Test::More: 0.86 perl: 5.008001 strict: 0 @@ -43,5 +47,5 @@ bugtracker: https://github.com/dagolden/test-failwarnings/issues homepage: https://metacpan.org/release/Test-FailWarnings repository: git://github.com/dagolden/test-failwarnings.git -version: 0.006 +version: 0.007 x_authority: cpan:DAGOLDEN diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/Makefile.PL new/Test-FailWarnings-0.007/Makefile.PL --- old/Test-FailWarnings-0.006/Makefile.PL 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/Makefile.PL 2013-08-30 15:53:50.000000000 +0200 @@ -21,6 +21,8 @@ "NAME" => "Test::FailWarnings", "PREREQ_PM" => { "Carp" => 0, + "Cwd" => 0, + "File::Spec" => 0, "Test::More" => "0.86", "strict" => 0, "warnings" => 0 @@ -28,15 +30,17 @@ "TEST_REQUIRES" => { "Capture::Tiny" => "0.12", "ExtUtils::MakeMaker" => 0, + "File::Spec" => 0, "File::Spec::Functions" => 0, "File::Temp" => 0, + "IO::Handle" => 0, + "IPC::Open3" => 0, "List::Util" => 0, "Test::More" => "0.96", - "blib" => 0, "constant" => 0, "lib" => 0 }, - "VERSION" => "0.006", + "VERSION" => "0.007", "test" => { "TESTS" => "t/*.t" } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/README new/Test-FailWarnings-0.007/README --- old/Test-FailWarnings-0.006/README 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/README 2013-08-30 15:53:50.000000000 +0200 @@ -2,7 +2,7 @@ Test::FailWarnings - Add test failures if warnings are caught VERSION - version 0.006 + version 0.007 SYNOPSIS Test file: @@ -20,8 +20,8 @@ Output: ok 1 - first test - not ok 2 - Caught warning - # Failed test 'Caught warning' + not ok 2 - Test::FailWarnings should catch no warnings + # Failed test 'Test::FailWarnings should catch no warnings' # at t/bin/main-warn.pl line 7. # Warning was 'Argument "lkadjaks" isn't numeric in addition (+) at t/bin/main-warn.pl line 7.' ok 3 - add non-numeric @@ -29,7 +29,7 @@ # Looks like you failed 1 test of 3. DESCRIPTION - This module hooks $SIG{__WARN__} and converts warnings to Test::More's + This module hooks $SIG{__WARN__} and converts warnings to Test::More "fail()" calls. It is designed to be used with "done_testing", when you don't need to know the test count in advance. diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/lib/Test/FailWarnings.pm new/Test-FailWarnings-0.007/lib/Test/FailWarnings.pm --- old/Test-FailWarnings-0.006/lib/Test/FailWarnings.pm 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/lib/Test/FailWarnings.pm 2013-08-30 15:53:50.000000000 +0200 @@ -4,14 +4,18 @@ package Test::FailWarnings; # ABSTRACT: Add test failures if warnings are caught -our $VERSION = '0.006'; # VERSION +our $VERSION = '0.007'; # VERSION use Test::More 0.86; +use Cwd qw/getcwd/; +use File::Spec; use Carp; our $ALLOW_DEPS = 0; our @ALLOW_FROM = (); +my $ORIG_DIR = getcwd(); # cache in case handler runs after a chdir + sub import { my ( $class, @args ) = @_; croak("import arguments must be key/value pairs") @@ -33,7 +37,11 @@ # shortcut if ignoring dependencies and warning did not # come from something local - return if $ALLOW_DEPS && $filename !~ /^(?:t|xt|lib|blib)/; + if ( $ALLOW_DEPS ) { + $filename = File::Spec->abs2rel( $filename, $ORIG_DIR ) + if File::Spec->file_name_is_absolute( $filename ); + return if $filename !~ /^(?:t|xt|lib|blib)/; + } return if grep { $package eq $_ } @ALLOW_FROM; @@ -76,7 +84,7 @@ =head1 VERSION -version 0.006 +version 0.007 =head1 SYNOPSIS @@ -95,8 +103,8 @@ Output: ok 1 - first test - not ok 2 - Caught warning - # Failed test 'Caught warning' + not ok 2 - Test::FailWarnings should catch no warnings + # Failed test 'Test::FailWarnings should catch no warnings' # at t/bin/main-warn.pl line 7. # Warning was 'Argument "lkadjaks" isn't numeric in addition (+) at t/bin/main-warn.pl line 7.' ok 3 - add non-numeric @@ -105,7 +113,7 @@ =head1 DESCRIPTION -This module hooks C<$SIG{__WARN__}> and converts warnings to L<Test::More>'s +This module hooks C<$SIG{__WARN__}> and converts warnings to L<Test::More> C<fail()> calls. It is designed to be used with C<done_testing>, when you don't need to know the test count in advance. diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/t/00-compile.t new/Test-FailWarnings-0.007/t/00-compile.t --- old/Test-FailWarnings-0.006/t/00-compile.t 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/t/00-compile.t 2013-08-30 15:53:50.000000000 +0200 @@ -1,19 +1,17 @@ use strict; use warnings; -# This test was generated via Dist::Zilla::Plugin::Test::Compile 2.018 +# this test was generated with Dist::Zilla::Plugin::Test::Compile 2.022 use Test::More 0.88; -use Capture::Tiny qw{ capture }; - -my @module_files = qw( -Test/FailWarnings.pm +my @module_files = ( + 'Test/FailWarnings.pm' ); -my @scripts = qw( +my @scripts = ( ); @@ -22,15 +20,26 @@ local $ENV{HOME} = File::Temp::tempdir( CLEANUP => 1 ); +use IPC::Open3; +use IO::Handle; +use File::Spec; + my @warnings; for my $lib (@module_files) { - my ($stdout, $stderr, $exit) = capture { - system($^X, '-Mblib', '-e', qq{require q[$lib]}); - }; - is($?, 0, "$lib loaded ok"); - warn $stderr if $stderr; - push @warnings, $stderr if $stderr; + open my $stdout, '>', File::Spec->devnull or die $!; + open my $stdin, '<', File::Spec->devnull or die $!; + my $stderr = IO::Handle->new; + + my $pid = open3($stdin, $stdout, $stderr, qq{$^X -Mblib -e"require q[$lib]"}); + waitpid($pid, 0); + is($? >> 8, 0, "$lib loaded ok"); + + if (my @_warnings = <$stderr>) + { + warn @_warnings; + push @warnings, @_warnings; + } } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Test-FailWarnings-0.006/t/00-report-prereqs.t new/Test-FailWarnings-0.007/t/00-report-prereqs.t --- old/Test-FailWarnings-0.006/t/00-report-prereqs.t 2013-08-22 16:41:14.000000000 +0200 +++ new/Test-FailWarnings-0.007/t/00-report-prereqs.t 2013-08-30 15:53:50.000000000 +0200 @@ -12,12 +12,15 @@ my @modules = qw( Capture::Tiny Carp + Cwd ExtUtils::MakeMaker + File::Spec File::Spec::Functions File::Temp + IO::Handle + IPC::Open3 List::Util Test::More - blib constant lib perl -- To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-commit+help@opensuse.org
participants (1)
-
root@hilbert.suse.de