use warnings;
use strict;
-use Carp;
-use Scalar::Util qw/isweak weaken blessed reftype refaddr/;
+# this noop trick initializes the STDOUT, so that the TAP::Harness
+# issued IO::Select->can_read calls (which are blocking wtf wtf wtf)
+# keep spinning and scheduling jobs
+# This results in an overall much smoother job-queue drainage, since
+# the Harness blocks less
+# (ideally this needs to be addressed in T::H, but a quick patchjob
+# broke everything so tabling it for now)
+BEGIN {
+ if ($INC{'Test/Builder.pm'}) {
+ local $| = 1;
+ print "#\n";
+ }
+}
+
+use Module::Runtime 'module_notional_filename';
+BEGIN {
+ for my $mod (qw( SQL::Abstract::Test SQL::Abstract )) {
+ if ( $INC{ module_notional_filename($mod) } ) {
+ # FIXME this does not seem to work in BEGIN - why?!
+ #require Carp;
+ #$Carp::Internal{ (__PACKAGE__) }++;
+ #Carp::croak( __PACKAGE__ . " must be loaded before $mod" );
+
+ my ($fr, @frame) = 1;
+ while (@frame = caller($fr++)) {
+ last if $frame[1] !~ m|^t/lib/DBICTest|;
+ }
+
+ die __PACKAGE__ . " must be loaded before $mod (or modules using $mod) at $frame[1] line $frame[2]\n";
+ }
+ }
+}
+
use Config;
+use Carp 'confess';
+use Scalar::Util qw(blessed refaddr);
use base 'Exporter';
-our @EXPORT_OK = qw/local_umask stacktrace populate_weakregistry assert_empty_weakregistry/;
+our @EXPORT_OK = qw(local_umask stacktrace check_customcond_args);
sub local_umask {
return unless defined $Config{d_umask};
}
}
-
sub stacktrace {
my $frame = shift;
$frame++;
return join "\tinvoked as ", map { sprintf ("%s at %s line %d\n", @$_ ) } @stack;
}
-my $refs_traced = 0;
-sub populate_weakregistry {
- my ($reg, $target, $slot) = @_;
-
- croak 'Target is not a reference' unless defined ref $target;
+sub check_customcond_args ($) {
+ my $args = shift;
- $slot ||= (sprintf '%s%s(0x%x)', # so we don't trigger stringification
- (defined blessed $target) ? blessed($target) . '=' : '',
- reftype $target,
- refaddr $target,
- );
+ confess "Expecting a hashref"
+ unless ref $args eq 'HASH';
- if (defined $reg->{$slot}{weakref}) {
- if ( refaddr($reg->{$slot}{weakref}) != (refaddr $target) ) {
- print STDERR "Bail out! Weak Registry slot collision: $reg->{$slot}{weakref} / $target\n";
- exit 255;
- }
- }
- else {
- $refs_traced++;
- weaken( $reg->{$slot}{weakref} = $target );
- $reg->{$slot}{stacktrace} = stacktrace(1);
+ for (qw(rel_name foreign_relname self_alias foreign_alias)) {
+ confess "Custom condition argument '$_' must be a plain string"
+ if length ref $args->{$_} or ! length $args->{$_};
}
- $target;
-}
+ confess "Current and legacy rel_name arguments do not match"
+ if $args->{rel_name} ne $args->{foreign_relname};
-my $leaks_found;
-sub assert_empty_weakregistry {
- my ($weak_registry, $quiet) = @_;
+ confess "Custom condition argument 'self_resultsource' must be a rsrc instance"
+ unless defined blessed $args->{self_resultsource} and $args->{self_resultsource}->isa('DBIx::Class::ResultSource');
- croak 'Expecting a registry hashref' unless ref $weak_registry eq 'HASH';
+ confess "Passed resultsource has no record of the supplied rel_name - likely wrong \$rsrc"
+ unless ref $args->{self_resultsource}->relationship_info($args->{rel_name});
- return unless keys %$weak_registry;
+ my $rowobj_cnt = 0;
- my $tb = eval { Test::Builder->new }
- or croak 'Calling test_weakregistry without a loaded Test::Builder makes no sense';
+ if (defined $args->{self_result_object} or defined $args->{self_rowobj} ) {
+ $rowobj_cnt++;
+ for (qw(self_result_object self_rowobj)) {
+ confess "Custom condition argument '$_' must be a result instance"
+ unless defined blessed $args->{$_} and $args->{$_}->isa('DBIx::Class::Row');
+ }
- for my $slot (sort keys %$weak_registry) {
- next if ! defined $weak_registry->{$slot}{weakref};
- $tb->BAILOUT("!!!! WEAK REGISTRY SLOT $slot IS NOT A WEAKREF !!!!")
- unless isweak( $weak_registry->{$slot}{weakref} );
+ confess "Current and legacy self_result_object arguments do not match"
+ if refaddr($args->{self_result_object}) != refaddr($args->{self_rowobj});
}
+ if (defined $args->{foreign_result_object}) {
+ $rowobj_cnt++;
- for my $slot (sort keys %$weak_registry) {
- ! defined $weak_registry->{$slot}{weakref} and next if $quiet;
-
- $tb->ok (! defined $weak_registry->{$slot}{weakref}, "No leaks of $slot") or do {
- $leaks_found = 1;
-
- my $diag = '';
-
- $diag .= Devel::FindRef::track ($weak_registry->{$slot}{weakref}, 20) . "\n"
- if ( $ENV{TEST_VERBOSE} && eval { require Devel::FindRef });
-
- if (my $stack = $weak_registry->{$slot}{stacktrace}) {
- $diag .= " Reference first seen$stack";
- }
-
- $tb->diag($diag) if $diag;
- };
+ confess "Custom condition argument 'foreign_result_object' must be a result instance"
+ unless defined blessed $args->{foreign_result_object} and $args->{foreign_result_object}->isa('DBIx::Class::Row');
}
-}
-
-END {
- if ($INC{'Test/Builder.pm'}) {
- my $tb = Test::Builder->new;
-
- # we check for test passage - a leak may be a part of a TODO
- if ($leaks_found and !$tb->is_passing) {
- $tb->diag(sprintf
- "\n\n%s\n%s\n\nInstall Devel::FindRef and re-run the test with set "
- . '$ENV{TEST_VERBOSE} (prove -v) to see a more detailed leak-report'
- . "\n\n%s\n%s\n\n", ('#' x 16) x 4
- ) if ( !$ENV{TEST_VERBOSE} or !$INC{'Devel/FindRef.pm'} );
+ confess "Result objects supplied on both ends of a relationship"
+ if $rowobj_cnt == 2;
- }
- else {
- $tb->note("Auto checked $refs_traced references for leaks - none detected");
- }
- }
+ $args;
}
1;