Skip Menu |

This queue is for tickets about the DynGig-Multiplex CPAN distribution.

Report information
The Basics
Id: 68257
Status: open
Priority: 0/
Queue: DynGig-Multiplex

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
cpan [...] zoffix.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.03
Fixed in: (no value)



Subject: Fails if Pod::Coverage is installed
As per subject. Example fail report: http://www.cpantesters.org/cpan/report/11815522 HTH, Regards,
Another sample fail report: http://www.cpantesters.org/cpan/report/37912073 HTH (Disclaimer: this was discovered by statistical analysis, I'm not speaking as a user)
The POD tests should be moved to xt/ and not run for regular users. Another issue (or annoyance) I notice with the pod is the DynGig::Multiplex pod references the two sub modules, but there's no clickable link to easily access them. Attached is the patch that moves pod tests to xt/, updates MANIFEST, and fixes the clickability issue. I have no idea what the ->new() method accepts, even after looking at the current documentation, so the pod-coverage test is still failing due to the missing docs on that part. -- Cheers, ZZ [ https://metacpan.org/author/ZOFFIX ]
Subject: DynGig-Multiplex-1.03.patch
diff -Naur DynGig-Multiplex-1.03_orig/lib/DynGig/Multiplex.pm DynGig-Multiplex-1.03/lib/DynGig/Multiplex.pm --- DynGig-Multiplex-1.03_orig/lib/DynGig/Multiplex.pm 2014-01-23 10:03:32.788120024 -0500 +++ DynGig-Multiplex-1.03/lib/DynGig/Multiplex.pm 2014-01-23 10:07:57.257431447 -0500 @@ -16,11 +16,11 @@ =head2 DynGig::Multiplex::CMD -Multiplexed Fork Client +Multiplexed Fork Client. See L<DynGig::Multiplex::CMD> =head2 DynGig::Multiplex::TCP -Multiplexed TCP Client +Multiplexed TCP Client. See L<DynGig::Multiplex::TCP> =head1 AUTHOR diff -Naur DynGig-Multiplex-1.03_orig/MANIFEST DynGig-Multiplex-1.03/MANIFEST --- DynGig-Multiplex-1.03_orig/MANIFEST 2014-01-23 10:03:32.788120024 -0500 +++ DynGig-Multiplex-1.03/MANIFEST 2014-01-23 10:04:10.056304824 -0500 @@ -6,7 +6,7 @@ lib/DynGig/Multiplex/TCP.pm lib/DynGig/Multiplex.pm t/00-load.t -t/pod-coverage.t -t/pod.t +xt/pod-coverage.t +xt/pod.t META.yml Module meta-data (added by MakeMaker) META.json Module JSON meta-data (added by MakeMaker) diff -Naur DynGig-Multiplex-1.03_orig/t/pod-coverage.t DynGig-Multiplex-1.03/t/pod-coverage.t --- DynGig-Multiplex-1.03_orig/t/pod-coverage.t 2014-01-23 10:03:32.788120024 -0500 +++ DynGig-Multiplex-1.03/t/pod-coverage.t 1969-12-31 19:00:00.000000000 -0500 @@ -1,18 +0,0 @@ -use strict; -use warnings; -use Test::More; - -# Ensure a recent version of Test::Pod::Coverage -my $min_tpc = 1.08; -eval "use Test::Pod::Coverage $min_tpc"; -plan skip_all => "Test::Pod::Coverage $min_tpc required for testing POD coverage" - if $@; - -# Test::Pod::Coverage doesn't require a minimum Pod::Coverage version, -# but older versions don't recognize some common documentation styles -my $min_pc = 0.18; -eval "use Pod::Coverage $min_pc"; -plan skip_all => "Pod::Coverage $min_pc required for testing POD coverage" - if $@; - -all_pod_coverage_ok(); diff -Naur DynGig-Multiplex-1.03_orig/t/pod.t DynGig-Multiplex-1.03/t/pod.t --- DynGig-Multiplex-1.03_orig/t/pod.t 2014-01-23 10:03:32.788120024 -0500 +++ DynGig-Multiplex-1.03/t/pod.t 1969-12-31 19:00:00.000000000 -0500 @@ -1,12 +0,0 @@ -#!perl -T - -use strict; -use warnings; -use Test::More; - -# Ensure a recent version of Test::Pod -my $min_tp = 1.22; -eval "use Test::Pod $min_tp"; -plan skip_all => "Test::Pod $min_tp required for testing POD" if $@; - -all_pod_files_ok(); diff -Naur DynGig-Multiplex-1.03_orig/xt/pod-coverage.t DynGig-Multiplex-1.03/xt/pod-coverage.t --- DynGig-Multiplex-1.03_orig/xt/pod-coverage.t 1969-12-31 19:00:00.000000000 -0500 +++ DynGig-Multiplex-1.03/xt/pod-coverage.t 2011-04-17 21:25:03.000000000 -0400 @@ -0,0 +1,18 @@ +use strict; +use warnings; +use Test::More; + +# Ensure a recent version of Test::Pod::Coverage +my $min_tpc = 1.08; +eval "use Test::Pod::Coverage $min_tpc"; +plan skip_all => "Test::Pod::Coverage $min_tpc required for testing POD coverage" + if $@; + +# Test::Pod::Coverage doesn't require a minimum Pod::Coverage version, +# but older versions don't recognize some common documentation styles +my $min_pc = 0.18; +eval "use Pod::Coverage $min_pc"; +plan skip_all => "Pod::Coverage $min_pc required for testing POD coverage" + if $@; + +all_pod_coverage_ok(); diff -Naur DynGig-Multiplex-1.03_orig/xt/pod.t DynGig-Multiplex-1.03/xt/pod.t --- DynGig-Multiplex-1.03_orig/xt/pod.t 1969-12-31 19:00:00.000000000 -0500 +++ DynGig-Multiplex-1.03/xt/pod.t 2011-04-17 21:25:03.000000000 -0400 @@ -0,0 +1,12 @@ +#!perl -T + +use strict; +use warnings; +use Test::More; + +# Ensure a recent version of Test::Pod +my $min_tp = 1.22; +eval "use Test::Pod $min_tp"; +plan skip_all => "Test::Pod $min_tp required for testing POD" if $@; + +all_pod_files_ok();