Commit | Line | Data |
c0329273 |
1 | BEGIN { do "./t/lib/ANFANG.pm" or die ( $@ || $! ) } |
2 | |
8d005ad9 |
3 | use strict; |
4 | use warnings; |
5 | use Test::More; |
6 | use Test::Warn; |
e466c62b |
7 | use Test::Exception; |
8d005ad9 |
8 | |
c0329273 |
9 | |
a5a7bb73 |
10 | use DBICTest ':DiffSQL'; |
8fc4291e |
11 | use DBIx::Class::_Util qw( UNRESOLVABLE_CONDITION dump_value ); |
497d0451 |
12 | use DBIx::Class::SQLMaker::Util qw( normalize_sqla_condition extract_equality_conditions ); |
8d005ad9 |
13 | |
ff7d03e6 |
14 | BEGIN { |
15 | if ( eval { require Test::Differences } ) { |
16 | no warnings 'redefine'; |
17 | *is_deeply = \&Test::Differences::eq_or_diff; |
18 | } |
19 | } |
8d005ad9 |
20 | |
497d0451 |
21 | my $sm = DBICTest->init_schema( no_deploy => 1)->storage->sql_maker; |
8d005ad9 |
22 | |
7638636b |
23 | { |
24 | package # hideee |
25 | DBICTest::SillyInt; |
26 | |
27 | use overload |
28 | fallback => 1, |
29 | '0+' => sub { ${$_[0]} }, |
30 | ; |
31 | } |
32 | my $num = bless( \do { my $foo = 69 }, 'DBICTest::SillyInt' ); |
33 | |
34 | is($num, 69, 'test overloaded object is "sane"'); |
35 | is("$num", 69, 'test overloaded object is "sane"'); |
36 | |
e466c62b |
37 | my @tests = ( |
8d005ad9 |
38 | { |
39 | where => { artistid => 1, charfield => undef }, |
497d0451 |
40 | normalized => { artistid => 1, charfield => undef }, |
8d005ad9 |
41 | sql => 'WHERE artistid = ? AND charfield IS NULL', |
497d0451 |
42 | equality_extract => { artistid => 1 }, |
43 | equality_considering_nulls_extract => { artistid => 1, charfield => undef }, |
8d005ad9 |
44 | }, |
45 | { |
46 | where => { -and => [ artistid => 1, charfield => undef, { rank => 13 } ] }, |
497d0451 |
47 | normalized => { artistid => 1, charfield => undef, rank => 13 }, |
8d005ad9 |
48 | sql => 'WHERE artistid = ? AND charfield IS NULL AND rank = ?', |
497d0451 |
49 | equality_extract => { artistid => 1, rank => 13 }, |
50 | equality_considering_nulls_extract => { artistid => 1, charfield => undef, rank => 13 }, |
8d005ad9 |
51 | }, |
52 | { |
53 | where => { -and => [ { artistid => 1, charfield => undef}, { rank => 13 } ] }, |
497d0451 |
54 | normalized => { artistid => 1, charfield => undef, rank => 13 }, |
8d005ad9 |
55 | sql => 'WHERE artistid = ? AND charfield IS NULL AND rank = ?', |
497d0451 |
56 | equality_extract => { artistid => 1, rank => 13 }, |
57 | equality_considering_nulls_extract => { artistid => 1, charfield => undef, rank => 13 }, |
8d005ad9 |
58 | }, |
59 | { |
60 | where => { -and => [ -or => { name => 'Caterwauler McCrae' }, 'rank' ] }, |
497d0451 |
61 | normalized => { name => 'Caterwauler McCrae', rank => undef }, |
8d005ad9 |
62 | sql => 'WHERE name = ? AND rank IS NULL', |
497d0451 |
63 | equality_extract => { name => 'Caterwauler McCrae' }, |
64 | equality_considering_nulls_extract => { name => 'Caterwauler McCrae', rank => undef }, |
8d005ad9 |
65 | }, |
66 | { |
67 | where => { -and => [ [ [ artist => {'=' => \'foo' } ] ], { name => \[ '= ?', 'bar' ] } ] }, |
497d0451 |
68 | normalized => { artist => {'=' => \'foo' }, name => \[ '= ?', 'bar' ] }, |
8d005ad9 |
69 | sql => 'WHERE artist = foo AND name = ?', |
497d0451 |
70 | equality_extract => { artist => \'foo' }, |
8d005ad9 |
71 | }, |
72 | { |
73 | where => { -and => [ -or => { name => 'Caterwauler McCrae', artistid => 2 } ] }, |
497d0451 |
74 | normalized => { -or => [ artistid => 2, name => 'Caterwauler McCrae' ] }, |
8d005ad9 |
75 | sql => 'WHERE artistid = ? OR name = ?', |
497d0451 |
76 | equality_extract => {}, |
8e40a627 |
77 | }, |
78 | { |
79 | where => { -or => { name => 'Caterwauler McCrae', artistid => 2 } }, |
497d0451 |
80 | normalized => { -or => [ artistid => 2, name => 'Caterwauler McCrae' ] }, |
8e40a627 |
81 | sql => 'WHERE artistid = ? OR name = ?', |
497d0451 |
82 | equality_extract => {}, |
8d005ad9 |
83 | }, |
84 | { |
7638636b |
85 | where => { -and => [ \'foo=bar', [ { artistid => { '=', $num } } ], { name => 'Caterwauler McCrae'} ] }, |
497d0451 |
86 | normalized => { -and => [ \'foo=bar' ], name => 'Caterwauler McCrae', artistid => $num }, |
8d005ad9 |
87 | sql => 'WHERE foo=bar AND artistid = ? AND name = ?', |
497d0451 |
88 | equality_extract => { name => 'Caterwauler McCrae', artistid => $num }, |
8d005ad9 |
89 | }, |
90 | { |
07add744 |
91 | where => { -and => [ \'foo=bar', [ { artistid => { '=', $num } } ], { name => 'Caterwauler McCrae'}, \'buzz=bozz' ] }, |
497d0451 |
92 | normalized => { -and => [ \'foo=bar', \'buzz=bozz' ], name => 'Caterwauler McCrae', artistid => $num }, |
93 | sql => 'WHERE foo=bar AND artistid = ? AND name = ? AND buzz=bozz', |
94 | normalized_sql => 'WHERE foo=bar AND buzz=bozz AND artistid = ? AND name = ?', |
95 | equality_extract => { name => 'Caterwauler McCrae', artistid => $num }, |
07add744 |
96 | }, |
97 | { |
7638636b |
98 | where => { artistid => [ $num ], rank => [ 13, 2, 3 ], charfield => [ undef ] }, |
497d0451 |
99 | normalized => { artistid => $num, charfield => undef, rank => [13, 2, 3] }, |
8d005ad9 |
100 | sql => 'WHERE artistid = ? AND charfield IS NULL AND ( rank = ? OR rank = ? OR rank = ? )', |
497d0451 |
101 | equality_extract => { artistid => $num }, |
102 | equality_considering_nulls_extract => { artistid => $num, charfield => undef }, |
8d005ad9 |
103 | }, |
104 | { |
105 | where => { artistid => { '=' => 1 }, rank => { '>' => 12 }, charfield => { '=' => undef } }, |
497d0451 |
106 | normalized => { artistid => 1, charfield => undef, rank => { '>' => 12 } }, |
8d005ad9 |
107 | sql => 'WHERE artistid = ? AND charfield IS NULL AND rank > ?', |
497d0451 |
108 | equality_extract => { artistid => 1 }, |
109 | equality_considering_nulls_extract => { artistid => 1, charfield => undef }, |
8d005ad9 |
110 | }, |
111 | { |
953d5b7d |
112 | where => { artistid => { '=' => [ 1 ], }, charfield => { '=' => [ -AND => \'1', \['?',2] ] }, rank => { '=' => [ -OR => $num, $num ] } }, |
497d0451 |
113 | normalized => { artistid => 1, charfield => [-and => { '=' => \['?',2] }, { '=' => \'1' } ], rank => { '=' => [$num, $num] } }, |
114 | sql => 'WHERE artistid = ? AND charfield = 1 AND charfield = ? AND ( rank = ? OR rank = ? )', |
115 | normalized_sql => 'WHERE artistid = ? AND charfield = ? AND charfield = 1 AND ( rank = ? OR rank = ? )', |
116 | equality_extract => { artistid => 1, charfield => UNRESOLVABLE_CONDITION }, |
8d005ad9 |
117 | }, |
118 | { |
119 | where => { -and => [ artistid => 1, artistid => 2 ], name => [ -and => { '!=', 1 }, 2 ], charfield => [ -or => { '=', 2 } ], rank => [-and => undef, { '=', undef }, { '!=', 2 } ] }, |
497d0451 |
120 | normalized => { artistid => [ -and => 1, 2 ], name => [ -and => { '!=', 1 }, 2 ], charfield => 2, rank => [ -and => { '!=', 2 }, undef ] }, |
121 | sql => 'WHERE artistid = ? AND artistid = ? AND charfield = ? AND name != ? AND name = ? AND rank IS NULL AND rank IS NULL AND rank != ?', |
122 | normalized_sql => 'WHERE artistid = ? AND artistid = ? AND charfield = ? AND name != ? AND name = ? AND rank != ? AND rank IS NULL', |
123 | equality_extract => { |
8e40a627 |
124 | artistid => UNRESOLVABLE_CONDITION, |
125 | name => 2, |
126 | charfield => 2, |
127 | }, |
497d0451 |
128 | equality_considering_nulls_extract => { |
8e40a627 |
129 | artistid => UNRESOLVABLE_CONDITION, |
130 | name => 2, |
131 | charfield => 2, |
132 | rank => undef, |
133 | }, |
8d005ad9 |
134 | }, |
135ac69d |
135 | (map { { |
136 | where => $_, |
497d0451 |
137 | sql => 'WHERE (rank = 13 OR charfield IS NULL OR artistid = ?) AND (artistid = ? OR charfield IS NULL OR rank != 42)', |
138 | normalized_sql => 'WHERE (artistid = ? OR charfield IS NULL OR rank = 13) AND (artistid = ? OR charfield IS NULL OR rank != 42)', |
139 | normalized => { -and => [ |
135ac69d |
140 | { -or => [ artistid => 1, charfield => undef, rank => { '=' => \13 } ] }, |
141 | { -or => [ artistid => 1, charfield => undef, rank => { '!=' => \42 } ] }, |
142 | ] }, |
497d0451 |
143 | equality_extract => {}, |
144 | equality_considering_nulls_extract => {}, |
135ac69d |
145 | } } ( |
ff7d03e6 |
146 | |
135ac69d |
147 | { -and => [ |
148 | -or => [ rank => { '=' => \13 }, charfield => { '=' => undef }, artistid => 1 ], |
149 | -or => { artistid => { '=' => 1 }, charfield => undef, rank => { '!=' => \42 } }, |
150 | ] }, |
151 | |
152 | { |
153 | -OR => [ rank => { '=' => \13 }, charfield => { '=' => undef }, artistid => 1 ], |
154 | -or => { artistid => { '=' => 1 }, charfield => undef, rank => { '!=' => \42 } }, |
155 | }, |
ff7d03e6 |
156 | |
135ac69d |
157 | ) ), |
158 | { |
c1f3f2e8 |
159 | where => { -or => [ |
160 | -and => [ foo => { '!=', { -value => undef } }, bar => { -in => [ 69, 42 ] } ], |
161 | foo => { '=', { -value => undef } }, |
162 | baz => { '!=' => { -ident => 'bozz' } }, |
163 | baz => { -ident => 'buzz' }, |
164 | ] }, |
497d0451 |
165 | sql => 'WHERE ( foo IS NOT NULL AND bar IN ( ?, ? ) ) OR foo IS NULL OR baz != bozz OR baz = buzz', |
166 | normalized_sql => 'WHERE baz != bozz OR baz = buzz OR foo IS NULL OR ( bar IN ( ?, ? ) AND foo IS NOT NULL )', |
167 | normalized => { -or => [ |
c1f3f2e8 |
168 | baz => { '!=' => { -ident => 'bozz' } }, |
169 | baz => { '=' => { -ident => 'buzz' } }, |
95da0f23 |
170 | foo => undef, |
171 | { bar => { -in => [ 69, 42 ] }, foo => { '!=', undef } } |
172 | ] }, |
497d0451 |
173 | equality_extract => {}, |
95da0f23 |
174 | }, |
175 | { |
135ac69d |
176 | where => { -or => [ rank => { '=' => \13 }, charfield => { '=' => undef }, artistid => { '=' => 1 }, genreid => { '=' => \['?', 2] } ] }, |
497d0451 |
177 | sql => 'WHERE rank = 13 OR charfield IS NULL OR artistid = ? OR genreid = ?', |
178 | normalized_sql => 'WHERE artistid = ? OR charfield IS NULL OR genreid = ? OR rank = 13', |
179 | normalized => { -or => [ artistid => 1, charfield => undef, genreid => { '=' => \['?', 2] }, rank => { '=' => \13 } ] }, |
180 | equality_extract => {}, |
181 | equality_considering_nulls_extract => {}, |
135ac69d |
182 | }, |
183 | { |
184 | where => { -and => [ |
185 | -or => [ rank => { '=' => \13 }, charfield => { '=' => undef }, artistid => 1 ], |
186 | -or => { artistid => { '=' => 1 }, charfield => undef, rank => { '=' => \13 } }, |
187 | ] }, |
497d0451 |
188 | normalized => { -and => [ |
135ac69d |
189 | { -or => [ artistid => 1, charfield => undef, rank => { '=' => \13 } ] }, |
190 | { -or => [ artistid => 1, charfield => undef, rank => { '=' => \13 } ] }, |
191 | ] }, |
497d0451 |
192 | sql => 'WHERE (rank = 13 OR charfield IS NULL OR artistid = ?) AND (artistid = ? OR charfield IS NULL OR rank = 13)', |
193 | normalized_sql => 'WHERE (artistid = ? OR charfield IS NULL OR rank = 13) AND (artistid = ? OR charfield IS NULL OR rank = 13)', |
194 | equality_extract => {}, |
195 | equality_considering_nulls_extract => {}, |
135ac69d |
196 | }, |
197 | { |
198 | where => { -and => [ |
199 | -or => [ rank => { '=' => \13 }, charfield => { '=' => undef }, artistid => 1 ], |
200 | -or => { artistid => { '=' => 1 }, charfield => undef, rank => { '!=' => \42 } }, |
201 | -and => [ foo => { '=' => \1 }, bar => 2 ], |
202 | -and => [ foo => 3, bar => { '=' => \4 } ], |
203 | -exists => \'(SELECT 1)', |
204 | -exists => \'(SELECT 2)', |
205 | -not => { foo => 69 }, |
206 | -not => { foo => 42 }, |
207 | ]}, |
208 | sql => 'WHERE |
209 | ( rank = 13 OR charfield IS NULL OR artistid = ? ) |
210 | AND ( artistid = ? OR charfield IS NULL OR rank != 42 ) |
211 | AND foo = 1 |
212 | AND bar = ? |
213 | AND foo = ? |
214 | AND bar = 4 |
215 | AND (EXISTS (SELECT 1)) |
216 | AND (EXISTS (SELECT 2)) |
217 | AND NOT foo = ? |
218 | AND NOT foo = ? |
219 | ', |
497d0451 |
220 | normalized_sql => 'WHERE |
135ac69d |
221 | ( artistid = ? OR charfield IS NULL OR rank = 13 ) |
222 | AND ( artistid = ? OR charfield IS NULL OR rank != 42 ) |
223 | AND (EXISTS (SELECT 1)) |
224 | AND (EXISTS (SELECT 2)) |
225 | AND NOT foo = ? |
226 | AND NOT foo = ? |
227 | AND bar = 4 |
228 | AND bar = ? |
229 | AND foo = 1 |
230 | AND foo = ? |
231 | ', |
497d0451 |
232 | normalized => { |
135ac69d |
233 | -and => [ |
234 | { -or => [ artistid => 1, charfield => undef, rank => { '=' => \13 } ] }, |
235 | { -or => [ artistid => 1, charfield => undef, rank => { '!=' => \42 } ] }, |
236 | { -exists => \'(SELECT 1)' }, |
237 | { -exists => \'(SELECT 2)' }, |
238 | { -not => { foo => 69 } }, |
239 | { -not => { foo => 42 } }, |
240 | ], |
241 | foo => [ -and => { '=' => \1 }, 3 ], |
242 | bar => [ -and => { '=' => \4 }, 2 ], |
243 | }, |
497d0451 |
244 | equality_extract => { |
135ac69d |
245 | foo => UNRESOLVABLE_CONDITION, |
246 | bar => UNRESOLVABLE_CONDITION, |
247 | }, |
497d0451 |
248 | equality_considering_nulls_extract => { |
135ac69d |
249 | foo => UNRESOLVABLE_CONDITION, |
250 | bar => UNRESOLVABLE_CONDITION, |
251 | }, |
252 | }, |
8d005ad9 |
253 | { |
254 | where => { -and => [ |
255 | [ '_macro.to' => { -like => '%correct%' }, '_wc_macros.to' => { -like => '%correct%' } ], |
256 | { -and => [ { 'group.is_active' => 1 }, { 'me.is_active' => 1 } ] } |
257 | ] }, |
497d0451 |
258 | normalized => { |
8d005ad9 |
259 | 'group.is_active' => 1, |
260 | 'me.is_active' => 1, |
261 | -or => [ |
262 | '_macro.to' => { -like => '%correct%' }, |
263 | '_wc_macros.to' => { -like => '%correct%' }, |
264 | ], |
265 | }, |
266 | sql => 'WHERE ( _macro.to LIKE ? OR _wc_macros.to LIKE ? ) AND group.is_active = ? AND me.is_active = ?', |
497d0451 |
267 | equality_extract => { 'group.is_active' => 1, 'me.is_active' => 1 }, |
8d005ad9 |
268 | }, |
5f35ba0f |
269 | |
b5ce6748 |
270 | { |
5f35ba0f |
271 | where => { -and => [ |
272 | artistid => { -value => [1] }, |
273 | charfield => { -ident => 'foo' }, |
274 | name => { '=' => { -value => undef } }, |
275 | rank => { '=' => { -ident => 'bar' } }, |
aa072cab |
276 | arrayfield => { '>' => { -value => [3,1] } }, |
5f35ba0f |
277 | ] }, |
497d0451 |
278 | normalized => { |
5f35ba0f |
279 | artistid => { -value => [1] }, |
280 | name => undef, |
281 | charfield => { '=', { -ident => 'foo' } }, |
282 | rank => { '=' => { -ident => 'bar' } }, |
aa072cab |
283 | arrayfield => { '>' => { -value => [3,1] } }, |
5f35ba0f |
284 | }, |
aa072cab |
285 | sql => 'WHERE artistid = ? AND charfield = foo AND name IS NULL AND rank = bar AND arrayfield > ?', |
286 | normalized_sql => 'WHERE arrayfield > ? AND artistid = ? AND charfield = foo AND name IS NULL AND rank = bar', |
497d0451 |
287 | equality_extract => { |
5f35ba0f |
288 | artistid => [1], |
289 | charfield => { -ident => 'foo' }, |
290 | rank => { -ident => 'bar' }, |
291 | }, |
497d0451 |
292 | equality_considering_nulls_extract => { |
5f35ba0f |
293 | artistid => [1], |
294 | name => undef, |
295 | charfield => { -ident => 'foo' }, |
296 | rank => { -ident => 'bar' }, |
297 | }, |
b5ce6748 |
298 | }, |
5f35ba0f |
299 | |
8d005ad9 |
300 | { |
301 | where => { artistid => [] }, |
497d0451 |
302 | normalized => { artistid => [] }, |
303 | equality_extract => {}, |
8d005ad9 |
304 | }, |
305 | (map { |
306 | { |
307 | where => { -and => $_ }, |
497d0451 |
308 | normalized => undef, |
309 | equality_extract => {}, |
8d005ad9 |
310 | sql => '', |
311 | }, |
312 | { |
313 | where => { -or => $_ }, |
497d0451 |
314 | normalized => undef, |
315 | equality_extract => {}, |
8d005ad9 |
316 | sql => '', |
317 | }, |
22485a7e |
318 | { |
319 | where => { -or => [ foo => 1, $_ ] }, |
497d0451 |
320 | normalized => { foo => 1 }, |
321 | equality_extract => { foo => 1 }, |
22485a7e |
322 | sql => 'WHERE foo = ?', |
323 | }, |
324 | { |
325 | where => { -or => [ $_, foo => 1 ] }, |
497d0451 |
326 | normalized => { foo => 1 }, |
327 | equality_extract => { foo => 1 }, |
22485a7e |
328 | sql => 'WHERE foo = ?', |
329 | }, |
330 | { |
331 | where => { -and => [ fuu => 2, $_, foo => 1 ] }, |
497d0451 |
332 | sql => 'WHERE fuu = ? AND foo = ?', |
333 | normalized_sql => 'WHERE foo = ? AND fuu = ?', |
334 | normalized => { foo => 1, fuu => 2 }, |
335 | equality_extract => { foo => 1, fuu => 2 }, |
22485a7e |
336 | }, |
8d005ad9 |
337 | } ( |
338 | # bare |
339 | [], {}, |
340 | # singles |
341 | [ {} ], [ [] ], |
342 | # doubles |
343 | [ [], [] ], [ {}, {} ], [ [], {} ], [ {}, [] ], |
344 | # tripples |
345 | [ {}, [], {} ], [ [], {}, [] ] |
346 | )), |
347 | |
348 | # FIXME legacy compat crap, possibly worth undef/dieing in SQLMaker |
497d0451 |
349 | { where => { artistid => {} }, sql => '', normalized => undef, equality_extract => {}, equality_considering_nulls_extract => {} }, |
8d005ad9 |
350 | |
351 | # batshit insanity, just to be thorough |
352 | { |
353 | where => { -and => [ [ 'artistid' ], [ -and => [ artistid => { '!=', 69 }, artistid => undef, artistid => { '=' => 200 } ]], artistid => [], { -or => [] }, { -and => [] }, [ 'charfield' ], { name => [] }, 'rank' ] }, |
497d0451 |
354 | normalized => { artistid => [ -and => [], { '!=', 69 }, undef, 200 ], charfield => undef, name => [], rank => undef }, |
355 | sql => 'WHERE artistid IS NULL AND artistid != ? AND artistid IS NULL AND artistid = ? AND 0=1 AND charfield IS NULL AND 0=1 AND rank IS NULL', |
356 | normalized_sql => 'WHERE 0=1 AND artistid != ? AND artistid IS NULL AND artistid = ? AND charfield IS NULL AND 0=1 AND rank IS NULL', |
357 | equality_extract => { artistid => UNRESOLVABLE_CONDITION }, |
358 | equality_considering_nulls_extract => { artistid => UNRESOLVABLE_CONDITION, charfield => undef, rank => undef }, |
8d005ad9 |
359 | }, |
8acda57c |
360 | |
361 | # original test from RT#93244 |
362 | { |
363 | where => { |
364 | -and => [ |
365 | \[ |
366 | "LOWER(me.title) LIKE ?", |
367 | '%spoon%', |
368 | ], |
369 | [ { 'me.title' => 'Spoonful of bees' } ], |
370 | ]}, |
497d0451 |
371 | normalized => { |
07add744 |
372 | -and => [ \[ |
8acda57c |
373 | "LOWER(me.title) LIKE ?", |
374 | '%spoon%', |
07add744 |
375 | ]], |
8acda57c |
376 | 'me.title' => 'Spoonful of bees', |
377 | }, |
378 | sql => 'WHERE LOWER(me.title) LIKE ? AND me.title = ?', |
497d0451 |
379 | equality_extract => { 'me.title' => 'Spoonful of bees' }, |
07add744 |
380 | }, |
381 | |
382 | # crazy literals |
383 | { |
384 | where => { |
385 | -or => [ |
386 | \'foo = bar', |
387 | ], |
388 | }, |
389 | sql => 'WHERE foo = bar', |
497d0451 |
390 | normalized => { |
07add744 |
391 | -and => [ |
392 | \'foo = bar', |
393 | ], |
394 | }, |
497d0451 |
395 | equality_extract => {}, |
07add744 |
396 | }, |
397 | { |
398 | where => { |
399 | -or => [ |
400 | \'foo = bar', |
401 | \'baz = ber', |
402 | ], |
403 | }, |
497d0451 |
404 | sql => 'WHERE foo = bar OR baz = ber', |
405 | normalized_sql => 'WHERE baz = ber OR foo = bar', |
406 | normalized => { |
07add744 |
407 | -or => [ |
408 | \'baz = ber', |
409 | \'foo = bar', |
410 | ], |
411 | }, |
497d0451 |
412 | equality_extract => {}, |
07add744 |
413 | }, |
414 | { |
415 | where => { |
416 | -and => [ |
417 | \'foo = bar', |
418 | \'baz = ber', |
419 | ], |
420 | }, |
421 | sql => 'WHERE foo = bar AND baz = ber', |
497d0451 |
422 | normalized => { |
07add744 |
423 | -and => [ |
424 | \'foo = bar', |
425 | \'baz = ber', |
426 | ], |
427 | }, |
497d0451 |
428 | equality_extract => {}, |
07add744 |
429 | }, |
430 | { |
431 | where => { |
432 | -and => [ |
433 | \'foo = bar', |
434 | \'baz = ber', |
435 | x => { -ident => 'y' }, |
436 | ], |
437 | }, |
438 | sql => 'WHERE foo = bar AND baz = ber AND x = y', |
497d0451 |
439 | normalized => { |
07add744 |
440 | -and => [ |
441 | \'foo = bar', |
442 | \'baz = ber', |
443 | ], |
444 | x => { '=' => { -ident => 'y' } } |
445 | }, |
497d0451 |
446 | equality_extract => { x => { -ident => 'y' } }, |
07add744 |
447 | }, |
e466c62b |
448 | ); |
449 | |
450 | # these die as of SQLA 1.80 - make sure we do not transform them |
451 | # into something usable instead |
452 | for my $lhs (undef, '', { -ident => 'foo' }, { -value => 'foo' } ) { |
453 | no warnings 'uninitialized'; |
454 | |
455 | for my $w ( |
456 | ( map { { -or => $_ }, (ref $lhs ? () : { @$_ } ) } |
457 | [ $lhs => "foo" ], |
458 | [ $lhs => { "=" => "bozz" } ], |
459 | [ $lhs => { "=" => \"bozz" } ], |
460 | [ $lhs => { -max => \"bizz" } ], |
461 | ), |
462 | |
463 | (ref $lhs) ? () : ( |
464 | { -or => [ -and => { $lhs => "baz" }, bizz => "buzz" ] }, |
465 | { -or => [ foo => "bar", { $lhs => "baz" }, bizz => "buzz" ] }, |
466 | { foo => "bar", -or => { $lhs => "baz" } }, |
467 | { foo => "bar", -or => { $lhs => \"baz" }, bizz => "buzz" }, |
468 | ), |
469 | |
470 | { foo => "bar", -and => [ $lhs => \"baz" ], bizz => "buzz" }, |
471 | { foo => "bar", -or => [ $lhs => \"baz" ], bizz => "buzz" }, |
472 | |
473 | { -or => [ foo => "bar", [ $lhs => \"baz" ], bizz => "buzz" ] }, |
474 | { -or => [ foo => "bar", $lhs => \"baz", bizz => "buzz" ] }, |
475 | { -or => [ foo => "bar", $lhs => \["baz"], bizz => "buzz" ] }, |
476 | { -or => [ $lhs => \"baz" ] }, |
477 | { -or => [ $lhs => \["baz"] ] }, |
478 | |
479 | ) { |
480 | push @tests, { |
481 | where => $w, |
482 | throw => qr/ |
483 | \QSupplying an empty left hand side argument is not supported in \E(?:array|hash)-pairs |
484 | | |
485 | \QIllegal use of top-level '-\E(?:value|ident)' |
486 | /x, |
487 | } |
488 | } |
489 | } |
490 | |
491 | # these are deprecated as of SQLA 1.79 - make sure we do not transform |
492 | # them without losing the warning |
493 | for my $lhs (undef, '') { |
494 | for my $rhs ( \"baz", \[ "baz" ] ) { |
495 | no warnings 'uninitialized'; |
496 | |
497 | my $expected_warning = qr/\QHash-pairs consisting of an empty string with a literal are deprecated/; |
498 | |
499 | push @tests, { |
500 | where => { $lhs => $rhs }, |
497d0451 |
501 | normalized => { -and => [ $rhs ] }, |
502 | equality_extract => {}, |
e466c62b |
503 | sql => 'WHERE baz', |
504 | warn => $expected_warning, |
505 | }; |
506 | |
507 | for my $w ( |
508 | { foo => "bar", -and => { $lhs => $rhs }, bizz => "buzz" }, |
509 | { foo => "bar", $lhs => $rhs, bizz => "buzz" }, |
510 | ) { |
511 | push @tests, { |
512 | where => $w, |
497d0451 |
513 | normalized => { |
e466c62b |
514 | -and => [ $rhs ], |
515 | bizz => "buzz", |
516 | foo => "bar", |
517 | }, |
497d0451 |
518 | equality_extract => { |
e466c62b |
519 | foo => "bar", |
520 | bizz => "buzz", |
521 | }, |
522 | sql => 'WHERE baz AND bizz = ? AND foo = ?', |
523 | warn => $expected_warning, |
524 | }; |
525 | } |
526 | } |
527 | } |
528 | |
529 | # lots of extra silly tests with a false column |
530 | for my $eq ( |
531 | \"= baz", |
532 | \[ "= baz" ], |
533 | { '=' => { -ident => 'baz' } }, |
534 | { '=' => \'baz' }, |
8d005ad9 |
535 | ) { |
e466c62b |
536 | for my $where ( |
537 | { foo => "bar", -and => [ 0 => $eq ], bizz => "buzz" }, |
538 | { foo => "bar", -or => [ 0 => $eq ], bizz => "buzz" }, |
539 | { foo => "bar", -and => { 0 => $eq }, bizz => "buzz" }, |
540 | { foo => "bar", -or => { 0 => $eq }, bizz => "buzz" }, |
541 | { foo => "bar", 0 => $eq, bizz => "buzz" }, |
542 | ) { |
543 | push @tests, { |
544 | where => $where, |
497d0451 |
545 | normalized => { |
e466c62b |
546 | 0 => $eq, |
547 | foo => 'bar', |
548 | bizz => 'buzz', |
549 | }, |
497d0451 |
550 | equality_extract => { |
e466c62b |
551 | foo => 'bar', |
552 | bizz => 'buzz', |
553 | ( ref $eq eq 'HASH' ? ( 0 => $eq->{'='} ) : () ), |
554 | }, |
555 | sql => 'WHERE 0 = baz AND bizz = ? AND foo = ?', |
556 | }; |
8d005ad9 |
557 | |
e466c62b |
558 | push @tests, { |
559 | where => { -or => $where }, |
497d0451 |
560 | normalized => { -or => [ |
e466c62b |
561 | "0" => $eq, |
562 | bizz => 'buzz', |
563 | foo => 'bar', |
564 | ]}, |
497d0451 |
565 | equality_extract => {}, |
e466c62b |
566 | sql => 'WHERE 0 = baz OR bizz = ? OR foo = ?', |
567 | } |
568 | |
569 | } |
570 | |
571 | for my $where ( |
572 | [ foo => "bar", -and => [ 0 => $eq ], bizz => "buzz" ], |
573 | [ foo => "bar", -or => [ 0 => $eq ], bizz => "buzz" ], |
574 | [ foo => "bar", -and => { 0 => $eq }, bizz => "buzz" ], |
575 | [ foo => "bar", -or => { 0 => $eq }, bizz => "buzz" ], |
576 | [ foo => "bar", 0 => $eq, bizz => "buzz" ], |
577 | ) { |
578 | push @tests, { |
579 | where => { -or => $where }, |
497d0451 |
580 | normalized => { -or => [ |
e466c62b |
581 | "0" => $eq, |
582 | bizz => 'buzz', |
583 | foo => 'bar', |
584 | ]}, |
497d0451 |
585 | equality_extract => {}, |
586 | sql => 'WHERE foo = ? OR 0 = baz OR bizz = ?', |
587 | normalized_sql => 'WHERE 0 = baz OR bizz = ? OR foo = ?', |
e466c62b |
588 | } |
589 | } |
590 | |
591 | for my $where ( |
592 | [ {foo => "bar"}, -and => { 0 => "baz" }, bizz => "buzz" ], |
593 | [ -or => [ foo => "bar", -or => { 0 => "baz" }, bizz => "buzz" ] ], |
594 | ) { |
595 | push @tests, { |
596 | where => { -or => $where }, |
497d0451 |
597 | normalized => { -or => [ |
e466c62b |
598 | "0" => 'baz', |
599 | bizz => 'buzz', |
600 | foo => 'bar', |
601 | ]}, |
497d0451 |
602 | equality_extract => {}, |
603 | sql => 'WHERE foo = ? OR 0 = ? OR bizz = ?', |
604 | normalized_sql => 'WHERE 0 = ? OR bizz = ? OR foo = ?', |
e466c62b |
605 | }; |
606 | } |
607 | |
608 | }; |
609 | |
610 | for my $t (@tests) { |
8d005ad9 |
611 | for my $w ( |
612 | $t->{where}, |
953d5b7d |
613 | $t->{where}, # do it twice, make sure we didn't destory the condition |
8d005ad9 |
614 | [ -and => $t->{where} ], |
953d5b7d |
615 | [ -AND => $t->{where} ], |
616 | { -OR => [ -AND => $t->{where} ] }, |
e466c62b |
617 | ( ( keys %{$t->{where}} == 1 and length( (keys %{$t->{where}})[0] ) ) |
618 | ? [ %{$t->{where}} ] |
619 | : () |
620 | ), |
8e40a627 |
621 | ( (keys %{$t->{where}} == 1 and $t->{where}{-or}) |
622 | ? ( ref $t->{where}{-or} eq 'HASH' |
623 | ? [ map { $_ => $t->{where}{-or}{$_} } sort keys %{$t->{where}{-or}} ] |
624 | : $t->{where}{-or} |
625 | ) |
626 | : () |
627 | ), |
8d005ad9 |
628 | ) { |
e466c62b |
629 | die unless Test::Builder->new->is_passing; |
630 | |
8fc4291e |
631 | my $name = do { local $Data::Dumper::Indent = 0; dump_value $w }; |
8d005ad9 |
632 | |
497d0451 |
633 | my ($normalized_cond, $normalized_cond_as_sql); |
8d005ad9 |
634 | |
e466c62b |
635 | if ($t->{throw}) { |
636 | throws_ok { |
497d0451 |
637 | $sm->where( normalize_sqla_condition($w) ); |
e466c62b |
638 | } $t->{throw}, "Exception on attempted collapse/render of $name" |
639 | and |
640 | next; |
641 | } |
8d005ad9 |
642 | |
e466c62b |
643 | warnings_exist { |
497d0451 |
644 | $normalized_cond = normalize_sqla_condition($w); |
645 | ($normalized_cond_as_sql) = $sm->where($normalized_cond); |
e466c62b |
646 | } $t->{warn} || [], "Expected warning when collapsing/rendering $name"; |
07add744 |
647 | |
8d005ad9 |
648 | is_deeply( |
497d0451 |
649 | $normalized_cond, |
650 | $t->{normalized}, |
8d005ad9 |
651 | "Expected collapsed condition produced on $name", |
652 | ); |
653 | |
e466c62b |
654 | my ($original_sql) = do { |
655 | local $SIG{__WARN__} = sub {}; |
656 | $sm->where($w); |
657 | }; |
658 | |
659 | is_same_sql ( $original_sql, $t->{sql}, "Expected original SQL from $name" ) |
660 | if exists $t->{sql}; |
661 | |
662 | is_same_sql( |
497d0451 |
663 | $normalized_cond_as_sql, |
664 | ( $t->{normalized_sql} || $t->{sql} || $original_sql ), |
665 | "Normalization did not alter *the semantics* of the final SQL based on $name", |
e466c62b |
666 | ); |
667 | |
8d005ad9 |
668 | is_deeply( |
497d0451 |
669 | extract_equality_conditions($normalized_cond), |
670 | $t->{equality_extract}, |
671 | "Expected equality_conditions produced on $name", |
8d005ad9 |
672 | ); |
8e40a627 |
673 | |
674 | is_deeply( |
497d0451 |
675 | extract_equality_conditions($normalized_cond, 'consider_nulls'), |
676 | ( $t->{equality_considering_nulls_extract} || $t->{equality_extract} ), |
677 | "Expected equality_conditions including NULLs produced on $name", |
678 | ); |
ff7d03e6 |
679 | |
e466c62b |
680 | is_deeply( |
497d0451 |
681 | $normalized_cond, |
682 | $t->{normalized}, |
683 | "Collapsed condition result unaltered by equality conditions extractor", |
e466c62b |
684 | ); |
8d005ad9 |
685 | } |
686 | } |
687 | |
aa072cab |
688 | # test separately |
689 | is_deeply( |
690 | normalize_sqla_condition( UNRESOLVABLE_CONDITION ), |
691 | { -and => [ UNRESOLVABLE_CONDITION ] }, |
692 | ); |
693 | |
8d005ad9 |
694 | done_testing; |