Commit | Line | Data |
a917fb06 |
1 | use strict; |
2 | use warnings; |
226d1c35 |
3 | |
f05edfd1 |
4 | # Do the override as early as possible so that CORE::bless doesn't get compiled away |
5 | # We will replace $bless_override only if we are in author mode |
6 | my $bless_override; |
7 | BEGIN { |
8 | $bless_override = sub { |
9 | CORE::bless( $_[0], (@_ > 1) ? $_[1] : caller() ); |
10 | }; |
11 | *CORE::GLOBAL::bless = sub { goto $bless_override }; |
12 | } |
13 | |
a917fb06 |
14 | use Test::More; |
d12d8272 |
15 | BEGIN { |
16 | plan skip_all => '5.13.6 leaks like a sieve (fixed in blead/cefd5c7c)' |
17 | if $] == '5.013006'; |
18 | } |
19 | |
a8c2c746 |
20 | use Scalar::Util qw/refaddr reftype weaken/; |
21 | use Carp qw/longmess/; |
22 | use Try::Tiny; |
a917fb06 |
23 | |
24 | use lib qw(t/lib); |
a8c2c746 |
25 | use DBICTest::RunMode; |
a917fb06 |
26 | |
a8c2c746 |
27 | my $have_test_cycle; |
a917fb06 |
28 | BEGIN { |
226d1c35 |
29 | require DBIx::Class::Optional::Dependencies; |
a8c2c746 |
30 | $have_test_cycle = DBIx::Class::Optional::Dependencies->req_ok_for ('test_leaks') |
31 | and import Test::Memory::Cycle; |
a917fb06 |
32 | } |
33 | |
a8c2c746 |
34 | # this is what holds all weakened refs to be checked for leakage |
35 | my $weak_registry = {}; |
36 | |
37 | # Skip the heavy-duty leak tracing when just doing an install |
38 | unless (DBICTest::RunMode->is_plain) { |
f05edfd1 |
39 | |
40 | # Some modules are known to install singletons on-load |
41 | # Load them before we swap out $bless_override |
42 | require DBI; |
43 | require DBD::SQLite; |
44 | require Errno; |
45 | require Class::Struct; |
46 | require FileHandle; |
307ab4c5 |
47 | require Hash::Merge; |
f05edfd1 |
48 | |
a8c2c746 |
49 | no warnings qw/redefine once/; |
50 | no strict qw/refs/; |
51 | |
f05edfd1 |
52 | # redefine the bless override so that we can catch each and every object created |
53 | $bless_override = sub { |
54 | |
a8c2c746 |
55 | my $obj = CORE::bless( |
56 | $_[0], (@_ > 1) ? $_[1] : do { |
57 | my ($class, $fn, $line) = caller(); |
58 | fail ("bless() of $_[0] into $class without explicit class specification at $fn line $line") |
59 | if $class =~ /^ (?: DBIx\:\:Class | DBICTest ) /x; |
60 | $class; |
61 | } |
62 | ); |
63 | |
64 | my $slot = (sprintf '%s=%s(0x%x)', # so we don't trigger stringification |
65 | ref $obj, |
66 | reftype $obj, |
67 | refaddr $obj, |
68 | ); |
69 | |
70 | # weaken immediately to avoid weird side effects |
71 | $weak_registry->{$slot} = { weakref => $obj, strace => longmess() }; |
72 | weaken $weak_registry->{$slot}{weakref}; |
73 | |
74 | return $obj; |
75 | }; |
76 | |
77 | for my $func (qw/try catch finally/) { |
78 | my $orig = \&{"Try::Tiny::$func"}; |
79 | *{"Try::Tiny::$func"} = sub (&;@) { |
80 | |
81 | my $slot = sprintf ('CODE(0x%x)', refaddr $_[0]); |
82 | |
83 | $weak_registry->{$slot} = { weakref => $_[0], strace => longmess() }; |
84 | weaken $weak_registry->{$slot}{weakref}; |
85 | |
86 | goto $orig; |
87 | } |
88 | } |
89 | } |
90 | |
91 | { |
92 | require DBICTest; |
a917fb06 |
93 | |
a8c2c746 |
94 | my $schema = DBICTest->init_schema; |
95 | my $rs = $schema->resultset ('Artist'); |
96 | my $storage = $schema->storage; |
a917fb06 |
97 | |
a8c2c746 |
98 | ok ($storage->connected, 'we are connected'); |
a917fb06 |
99 | |
a8c2c746 |
100 | my $row_obj = $rs->next; |
101 | ok ($row_obj, 'row from db'); |
102 | |
103 | my ($mc_row_obj, $pager, $pager_explicit_count) = $schema->txn_do (sub { |
104 | |
105 | my $artist = $rs->create ({ |
106 | name => 'foo artist', |
107 | cds => [{ |
108 | title => 'foo cd', |
109 | year => 1984, |
110 | }], |
111 | }); |
112 | |
113 | my $pg = $rs->search({}, { rows => 1})->page(2)->pager; |
114 | |
115 | my $pg_wcount = $rs->page(4)->pager->total_entries (66); |
116 | |
117 | return ($artist, $pg, $pg_wcount); |
118 | }); |
119 | |
120 | is ($pager->next_page, 3, 'There is one more page available'); |
121 | |
122 | # based on 66 per 10 pages |
123 | is ($pager_explicit_count->last_page, 7, 'Correct last page'); |
551e711a |
124 | |
a8c2c746 |
125 | my $base_collection = { |
126 | schema => $schema, |
127 | storage => $storage, |
dc35bbe6 |
128 | |
a8c2c746 |
129 | resultset => $rs, |
307ab4c5 |
130 | |
a8c2c746 |
131 | row_object => $row_obj, |
551e711a |
132 | |
a8c2c746 |
133 | result_source => $rs->result_source, |
551e711a |
134 | |
a8c2c746 |
135 | fresh_pager => $rs->page(5)->pager, |
136 | pager => $pager, |
137 | pager_explicit_count => $pager_explicit_count, |
551e711a |
138 | |
a8c2c746 |
139 | sql_maker => $storage->sql_maker, |
140 | dbh => $storage->_dbh |
141 | }; |
574d9b69 |
142 | |
a8c2c746 |
143 | memory_cycle_ok ($base_collection, 'No cycles in the object collection') |
144 | if $have_test_cycle; |
574d9b69 |
145 | |
a8c2c746 |
146 | for (keys %$base_collection) { |
147 | $weak_registry->{"basic $_"} = { weakref => $base_collection->{$_} }; |
148 | weaken $weak_registry->{"basic $_"}{weakref}; |
574d9b69 |
149 | } |
150 | |
551e711a |
151 | } |
152 | |
a8c2c746 |
153 | memory_cycle_ok($weak_registry, 'No cycles in the weakened object collection') |
154 | if $have_test_cycle; |
155 | |
307ab4c5 |
156 | # Naturally we have some exceptions |
157 | my $cleared; |
158 | for my $slot (keys %$weak_registry) { |
d12d8272 |
159 | if ($slot =~ /^\QSQL::Translator/) { |
307ab4c5 |
160 | # SQLT is a piece of shit, leaks all over |
161 | delete $weak_registry->{$slot}; |
162 | } |
d12d8272 |
163 | elsif ($slot =~ /^\QHash::Merge/) { |
307ab4c5 |
164 | # only clear one object - more would indicate trouble |
165 | delete $weak_registry->{$slot} |
166 | unless $cleared->{hash_merge_singleton}{$weak_registry->{$slot}{weakref}{behavior}}++; |
167 | } |
1f870d5a |
168 | elsif ($slot =~ /^__TxnScopeGuard__FIXUP__/) { |
169 | die 'The $@ debacle should have been fixed by now!!!' if $] >= 5.013008; |
170 | delete $weak_registry->{$slot}; |
171 | } |
307ab4c5 |
172 | } |
173 | |
174 | |
a8c2c746 |
175 | # FIXME |
176 | # For reasons I can not yet fully understand the table() god-method (located in |
177 | # ::ResultSourceProxy::Table) attaches an actual source instance to each class |
178 | # as virtually *immortal* class-data. |
179 | # For now just blow away these instances manually but there got to be a saner way |
180 | $_->result_source_instance(undef) for ( |
181 | 'DBICTest::BaseResult', |
182 | map { DBICTest::Schema->class ($_) } DBICTest::Schema->sources |
183 | ); |
184 | |
185 | # FIXME |
186 | # same problem goes for the schema - its classdata contains live result source |
187 | # objects, which to add insult to the injury are *different* instances from the |
188 | # ones we destroyed above |
189 | DBICTest::Schema->source_registrations(undef); |
190 | |
191 | my $tb = Test::More->builder; |
192 | for my $slot (keys %$weak_registry) { |
a8c2c746 |
193 | |
194 | ok (! defined $weak_registry->{$slot}{weakref}, "No leaks of $slot") or do { |
195 | my $diag = ''; |
196 | |
197 | $diag .= Devel::FindRef::track ($weak_registry->{$slot}{weakref}, 20) . "\n" |
198 | if ( $ENV{TEST_VERBOSE} && try { require Devel::FindRef }); |
199 | |
200 | if (my $stack = $weak_registry->{$slot}{strace}) { |
201 | $diag .= " Reference first seen$stack"; |
202 | } |
203 | |
204 | diag $diag if $diag; |
205 | }; |
551e711a |
206 | } |
207 | |
208 | done_testing; |