X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=xt%2Fpodcoverage.t;h=7a7804eb04c301199722396efaf4e2f57dd23edb;hb=f7ed0ed356e3a637e8de759731fbf9421a47d3c6;hp=ba30f3180f1b68e20470b31b17944849c81adb47;hpb=1f870d5a08fa1794734380d7c8bd6753cdcf8f6d;p=dbsrgits%2FDBIx-Class.git

diff --git a/xt/podcoverage.t b/xt/podcoverage.t
index ba30f31..7a7804e 100644
--- a/xt/podcoverage.t
+++ b/xt/podcoverage.t
@@ -3,23 +3,21 @@ use strict;
 
 use Test::More;
 use List::Util 'first';
-use lib qw(t/lib);
+use lib qw(t/lib maint/.Generated_Pod/lib);
 use DBICTest;
 use namespace::clean;
 
-# Don't run tests for installs
-if ( DBICTest::RunMode->is_plain ) {
-  plan( skip_all => "Author tests not required for installation" );
-}
-
 require DBIx::Class;
 unless ( DBIx::Class::Optional::Dependencies->req_ok_for ('test_podcoverage') ) {
   my $missing = DBIx::Class::Optional::Dependencies->req_missing_for ('test_podcoverage');
-  $ENV{RELEASE_TESTING} || DBICTest::AuthorCheck->is_author
+  $ENV{RELEASE_TESTING}
     ? die ("Failed to load release-testing module requirements: $missing")
     : plan skip_all => "Test needs: $missing"
 }
 
+# this has already been required but leave it here for CPANTS static analysis
+require Test::Pod::Coverage;
+
 # Since this is about checking documentation, a little documentation
 # of what this is doing might be in order.
 # The exceptions structure below is a hash keyed by the module
@@ -42,14 +40,14 @@ my $exceptions = {
             mk_classaccessor
         /]
     },
-    'DBIx::Class::Row' => {
+    'DBIx::Class::Carp' => {
         ignore => [qw/
-            MULTICREATE_DEBUG
-        /],
+            unimport
+        /]
     },
-    'DBIx::Class::Storage::TxnScopeGuard' => {
+    'DBIx::Class::Row' => {
         ignore => [qw/
-            IS_BROKEN_PERL
+            MULTICREATE_DEBUG
         /],
     },
     'DBIx::Class::FilterColumn' => {
@@ -68,6 +66,14 @@ my $exceptions = {
             resolve_condition
             resolve_join
             resolve_prefetch
+            STORABLE_freeze
+            STORABLE_thaw
+        /],
+    },
+    'DBIx::Class::ResultSet' => {
+        ignore => [qw/
+            STORABLE_freeze
+            STORABLE_thaw
         /],
     },
     'DBIx::Class::ResultSourceHandle' => {
@@ -110,11 +116,15 @@ my $exceptions = {
     'DBIx::Class::Admin::*'                         => { skip => 1 },
     'DBIx::Class::ClassResolver::PassThrough'       => { skip => 1 },
     'DBIx::Class::Componentised'                    => { skip => 1 },
+    'DBIx::Class::AccessorGroup'                    => { skip => 1 },
     'DBIx::Class::Relationship::*'                  => { skip => 1 },
     'DBIx::Class::ResultSetProxy'                   => { skip => 1 },
     'DBIx::Class::ResultSourceProxy'                => { skip => 1 },
+    'DBIx::Class::ResultSource::*'                  => { skip => 1 },
     'DBIx::Class::Storage::Statistics'              => { skip => 1 },
     'DBIx::Class::Storage::DBI::Replicated::Types'  => { skip => 1 },
+    'DBIx::Class::GlobalDestruction'                => { skip => 1 },
+    'DBIx::Class::Storage::BlockRunner'             => { skip => 1 }, # temporary
 
 # test some specific components whose parents are exempt below
     'DBIx::Class::Relationship::Base'               => {},
@@ -122,16 +132,21 @@ my $exceptions = {
 
 # internals
     'DBIx::Class::SQLMaker*'                        => { skip => 1 },
+    'DBIx::Class::SQLAHacks*'                       => { skip => 1 },
     'DBIx::Class::Storage::DBI*'                    => { skip => 1 },
     'SQL::Translator::*'                            => { skip => 1 },
 
 # deprecated / backcompat stuff
+    'DBIx::Class::Serialize::Storable'              => { skip => 1 },
     'DBIx::Class::CDBICompat*'                      => { skip => 1 },
     'DBIx::Class::ResultSetManager'                 => { skip => 1 },
     'DBIx::Class::DB'                               => { skip => 1 },
 
 # skipped because the synopsis covers it clearly
     'DBIx::Class::InflateColumn::File'              => { skip => 1 },
+
+# internal subclass, nothing to POD
+    'DBIx::Class::ResultSet::Pager'                 => { skip => 1 },
 };
 
 my $ex_lookup = {};
@@ -142,7 +157,7 @@ for my $string (keys %$exceptions) {
   $ex_lookup->{$re} = $ex;
 }
 
-my @modules = sort { $a cmp $b } (Test::Pod::Coverage::all_modules());
+my @modules = sort { $a cmp $b } Test::Pod::Coverage::all_modules('lib');
 
 foreach my $module (@modules) {
   SKIP: {