X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F03podcoverage.t;h=fabdd0ce94c6cd8d75ed136eaf2845f56d6d399e;hb=97f9f16ea3aff5d9e85d89e8a37ade594c4c778d;hp=022a1c91bfc766809e550441d474eea8de8a49b8;hpb=046ad905e50dab1348ffca956586d7f386cfaa7e;p=dbsrgits%2FDBIx-Class.git diff --git a/t/03podcoverage.t b/t/03podcoverage.t index 022a1c9..fabdd0c 100644 --- a/t/03podcoverage.t +++ b/t/03podcoverage.t @@ -1,16 +1,39 @@ +use warnings; +use strict; + use Test::More; +use List::Util (); +use lib qw(t/lib); +use DBICTest; -eval "use Test::Pod::Coverage 1.04"; -plan skip_all => 'Test::Pod::Coverage 1.04 required' if $@; -plan skip_all => 'set TEST_POD to enable this test' unless $ENV{TEST_POD}; +my @MODULES = ( + 'Test::Pod::Coverage 1.08', + 'Pod::Coverage 0.20', +); -my @modules = sort { $a cmp $b } (Test::Pod::Coverage::all_modules()); -plan tests => scalar(@modules); +# Don't run tests for installs +unless ( DBICTest::AuthorCheck->is_author || $ENV{AUTOMATED_TESTING} || $ENV{RELEASE_TESTING} ) { + plan( skip_all => "Author tests not required for installation" ); +} + +# Load the testing modules +foreach my $MODULE ( @MODULES ) { + eval "use $MODULE"; + if ( $@ ) { + $ENV{RELEASE_TESTING} + ? die( "Failed to load required release-testing module $MODULE" ) + : plan( skip_all => "$MODULE not available for testing" ); + } +} # Since this is about checking documentation, a little documentation -# of what this is doing might be in order... +# of what this is doing might be in order. # The exceptions structure below is a hash keyed by the module -# name. The value for each is a hash, which contains one or more +# name. Any * in a name is treated like a wildcard and will behave +# as expected. Modules are matched by longest string first, so +# A::B::C will match even if there is A::B* + +# The value for each is a hash, which contains one or more # (although currently more than one makes no sense) of the following # things:- # skip => a true value means this module is not checked @@ -18,72 +41,115 @@ plan tests => scalar(@modules); # do not need to be documented. my $exceptions = { 'DBIx::Class' => { - ignore => [ - qw/MODIFY_CODE_ATTRIBUTES - component_base_class - mk_classdata/ - ] + ignore => [qw/ + MODIFY_CODE_ATTRIBUTES + component_base_class + mk_classdata + mk_classaccessor + /] + }, + 'DBIx::Class::Row' => { + ignore => [qw/ + MULTICREATE_DEBUG + /], + }, + 'DBIx::Class::ResultSource' => { + ignore => [qw/ + compare_relationship_keys + pk_depends_on + resolve_condition + resolve_join + resolve_prefetch + /], + }, + 'DBIx::Class::ResultSourceHandle' => { + ignore => [qw/ + schema + source_moniker + /], + }, + 'DBIx::Class::Storage' => { + ignore => [qw/ + schema + cursor + /] + }, + 'DBIx::Class::Schema' => { + ignore => [qw/ + setup_connection_class + /] + }, + + 'DBIx::Class::Schema::Versioned' => { + ignore => [ qw/ + connection + /] + }, + + 'DBIx::Class::Storage::DBI::Replicated*' => { + ignore => [ qw/ + connect_call_do_sql + disconnect_call_do_sql + /] }, - 'DBIx::Class::CDBICompat::AccessorMapping' => { skip => 1 }, - 'DBIx::Class::CDBICompat::AttributeAPI' => { skip => 1 }, - 'DBIx::Class::CDBICompat::AutoUpdate' => { skip => 1 }, - 'DBIx::Class::CDBICompat::ColumnCase' => { skip => 1 }, - 'DBIx::Class::CDBICompat::ColumnGroups' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Constraints' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Constructor' => { skip => 1 }, - 'DBIx::Class::CDBICompat::DestroyWarning' => { skip => 1 }, - 'DBIx::Class::CDBICompat::GetSet' => { skip => 1 }, - 'DBIx::Class::CDBICompat::HasA' => { skip => 1 }, - 'DBIx::Class::CDBICompat::HasMany' => { skip => 1 }, - 'DBIx::Class::CDBICompat::ImaDBI' => { skip => 1 }, - 'DBIx::Class::CDBICompat::LazyLoading' => { skip => 1 }, - 'DBIx::Class::CDBICompat::LiveObjectIndex' => { skip => 1 }, - 'DBIx::Class::CDBICompat::MightHave' => { skip => 1 }, - 'DBIx::Class::CDBICompat::ObjIndexStubs' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Pager' => { skip => 1 }, - 'DBIx::Class::CDBICompat::ReadOnly' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Retrieve' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Stringify' => { skip => 1 }, - 'DBIx::Class::CDBICompat::TempColumns' => { skip => 1 }, - 'DBIx::Class::CDBICompat::Triggers' => { skip => 1 }, - 'DBIx::Class::ClassResolver::PassThrough' => { skip => 1 }, - 'DBIx::Class::Componentised' => { skip => 1 }, - 'DBIx::Class::Relationship::Accessor' => { skip => 1 }, - 'DBIx::Class::Relationship::BelongsTo' => { skip => 1 }, - 'DBIx::Class::Relationship::CascadeActions' => { skip => 1 }, - 'DBIx::Class::Relationship::HasMany' => { skip => 1 }, - 'DBIx::Class::Relationship::HasOne' => { skip => 1 }, - 'DBIx::Class::Relationship::Helpers' => { skip => 1 }, - 'DBIx::Class::Relationship::ManyToMany' => { skip => 1 }, - 'DBIx::Class::Relationship::ProxyMethods' => { skip => 1 }, - 'DBIx::Class::ResultSetProxy' => { skip => 1 }, - 'DBIx::Class::ResultSourceProxy' => { skip => 1 }, - 'DBIx::Class::Storage::DBI' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::DB2' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::MSSQL' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::MultiDistinctEmulation' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::ODBC400' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::ODBC::DB2_400_SQL' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::Oracle' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::Pg' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::SQLite' => { skip => 1 }, - 'DBIx::Class::Storage::DBI::mysql' => { skip => 1 }, - 'SQL::Translator::Parser::DBIx::Class' => { skip => 1 }, - 'SQL::Translator::Producer::DBIx::Class::File' => { skip => 1 }, + + 'DBIx::Class::Admin::*' => { skip => 1 }, + 'DBIx::Class::ClassResolver::PassThrough' => { skip => 1 }, + 'DBIx::Class::Componentised' => { skip => 1 }, + 'DBIx::Class::Relationship::*' => { skip => 1 }, + 'DBIx::Class::ResultSetProxy' => { skip => 1 }, + 'DBIx::Class::ResultSourceProxy' => { skip => 1 }, + 'DBIx::Class::Storage::Statistics' => { skip => 1 }, + 'DBIx::Class::Storage::DBI::Replicated::Types' => { skip => 1 }, + +# test some specific components whose parents are exempt below + 'DBIx::Class::Relationship::Base' => {}, + +# internals + 'DBIx::Class::SQLAHacks*' => { skip => 1 }, + 'DBIx::Class::Storage::DBI*' => { skip => 1 }, + 'SQL::Translator::*' => { skip => 1 }, + +# deprecated / backcompat stuff + '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 }, }; +my $ex_lookup = {}; +for my $string (keys %$exceptions) { + my $ex = $exceptions->{$string}; + $string =~ s/\*/'.*?'/ge; + my $re = qr/^$string$/; + $ex_lookup->{$re} = $ex; +} + +my @modules = sort { $a cmp $b } (Test::Pod::Coverage::all_modules()); + foreach my $module (@modules) { - SKIP: - { - skip "No real methods", 1 if ($exceptions->{$module}{skip}); - - # build parms up from ignore list - my $parms = {}; - $parms->{trustme} = - [ map { qr/^$_$/ } @{ $exceptions->{$module}{ignore} } ] - if exists($exceptions->{$module}{ignore}); - - # run the test with the potentially modified parm set - pod_coverage_ok($module, $parms, "$module POD coverage"); - } + SKIP: { + + my ($match) = List::Util::first + { $module =~ $_ } + (sort { length $b <=> length $a || $b cmp $a } (keys %$ex_lookup) ) + ; + + my $ex = $ex_lookup->{$match} if $match; + + skip ("$module exempt", 1) if ($ex->{skip}); + + # build parms up from ignore list + my $parms = {}; + $parms->{trustme} = + [ map { qr/^$_$/ } @{ $ex->{ignore} } ] + if exists($ex->{ignore}); + + # run the test with the potentially modified parm set + pod_coverage_ok($module, $parms, "$module POD coverage"); + } } + +done_testing;