Skip to content

Commit 344f761

Browse files
committed
Promote leaky rscond todo to a smoke-fail - try to establish a pattern
1 parent 1a08c5e commit 344f761

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

t/52leaks.t

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,9 @@ my @compose_ns_classes;
262262

263263
# this needs to fire, even if it can't find anything
264264
# see FIXME below
265-
$rs_bind_circref->next;
265+
# we run this only on smokers - trying to establish a pattern
266+
$rs_bind_circref->next
267+
if ( ($ENV{TRAVIS}||'') ne 'true' and DBICTest::RunMode->is_smoker);
266268

267269
require Storable;
268270
%$base_collection = (
@@ -413,10 +415,8 @@ for my $moniker ( keys %{DBICTest::Schema->source_registrations || {}} ) {
413415
# ^ /
414416
# \-------- bound value on prepared/cached STH <-----------/
415417
#
416-
TODO: {
417-
local $TODO = 'Not sure how to fix this yet, an entanglment could be an option';
418-
my $r = $weak_registry->{'basic leaky_resultset_cond'}{weakref};
419-
ok(! defined $r, 'We no longer leak!')
418+
if ( my $r = $weak_registry->{'basic leaky_resultset_cond'}{weakref} ) {
419+
ok(! defined $r, 'Self-referential RS conditions no longer leak!')
420420
or $r->result_source(undef);
421421
}
422422

0 commit comments

Comments
 (0)