Commit | Line | Data |
4c2b30d6 |
1 | use strict; |
2 | use warnings; |
3 | |
4 | use Test::More; |
52864fbd |
5 | use Test::Deep; |
4c2b30d6 |
6 | use lib qw(t/lib); |
7 | use DBICTest; |
8 | use DBIC::SqlMakerTest; |
9 | |
10 | my $schema = DBICTest->init_schema(); |
11 | my $orig_debug = $schema->storage->debug; |
12 | |
13 | my $cdrs = $schema->resultset('CD')->search({ 'me.artist' => { '!=', 2 }}); |
14 | |
15 | my $cd_data = { map { |
16 | $_->cdid => { |
17 | siblings => $cdrs->search ({ artist => $_->get_column('artist') })->count - 1, |
18 | track_titles => [ map { $_->title } ($_->tracks->all) ], |
19 | }, |
20 | } ( $cdrs->all ) }; |
21 | |
4c2b30d6 |
22 | my $c_rs = $cdrs->search ({}, { |
23 | prefetch => 'tracks', |
24 | '+columns' => { sibling_count => $cdrs->search( |
25 | { |
26 | 'siblings.artist' => { -ident => 'me.artist' }, |
0e773352 |
27 | 'siblings.cdid' => { '!=' => ['-and', { -ident => 'me.cdid' }, 23414] }, |
4c2b30d6 |
28 | }, { alias => 'siblings' }, |
29 | )->count_rs->as_query, |
30 | }, |
31 | }); |
32 | |
33 | is_same_sql_bind( |
34 | $c_rs->as_query, |
35 | '( |
36 | SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track, |
37 | (SELECT COUNT( * ) |
38 | FROM cd siblings |
39 | WHERE siblings.artist = me.artist |
40 | AND siblings.cdid != me.cdid |
41 | AND siblings.cdid != ? |
42 | AND me.artist != ? |
43 | ), |
44 | tracks.trackid, tracks.cd, tracks.position, tracks.title, tracks.last_updated_on, tracks.last_updated_at |
45 | FROM cd me |
46 | LEFT JOIN track tracks |
47 | ON tracks.cd = me.cdid |
48 | WHERE me.artist != ? |
4c2b30d6 |
49 | )', |
50 | [ |
36fd7f07 |
51 | |
52 | # subselect |
0e773352 |
53 | [ { sqlt_datatype => 'integer', dbic_colname => 'siblings.cdid' } |
54 | => 23414 ], |
55 | |
56 | [ { sqlt_datatype => 'integer', dbic_colname => 'me.artist' } |
57 | => 2 ], |
36fd7f07 |
58 | |
59 | # outher WHERE |
0e773352 |
60 | [ { sqlt_datatype => 'integer', dbic_colname => 'me.artist' } |
61 | => 2 ], |
4c2b30d6 |
62 | ], |
63 | 'Expected SQL on correlated realiased subquery' |
64 | ); |
65 | |
36fd7f07 |
66 | my $queries = 0; |
67 | $schema->storage->debugcb(sub { $queries++; }); |
68 | $schema->storage->debug(1); |
69 | |
52864fbd |
70 | cmp_deeply ( |
4c2b30d6 |
71 | { map |
72 | { $_->cdid => { |
73 | track_titles => [ map { $_->title } ($_->tracks->all) ], |
74 | siblings => $_->get_column ('sibling_count'), |
75 | } } |
76 | $c_rs->all |
77 | }, |
78 | $cd_data, |
79 | 'Proper information retrieved from correlated subquery' |
80 | ); |
81 | |
82 | is ($queries, 1, 'Only 1 query fired to retrieve everything'); |
83 | |
84 | $schema->storage->debug($orig_debug); |
85 | $schema->storage->debugcb(undef); |
86 | |
36fd7f07 |
87 | # now add an unbalanced select/as pair |
88 | $c_rs = $c_rs->search ({}, { |
89 | '+select' => $cdrs->search( |
90 | { 'siblings.artist' => { -ident => 'me.artist' } }, |
91 | { alias => 'siblings', columns => [ |
92 | { first_year => { min => 'year' }}, |
93 | { last_year => { max => 'year' }}, |
94 | ]}, |
95 | )->as_query, |
96 | '+as' => [qw/active_from active_to/], |
97 | }); |
98 | |
36fd7f07 |
99 | is_same_sql_bind( |
100 | $c_rs->as_query, |
101 | '( |
102 | SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track, |
103 | (SELECT COUNT( * ) |
104 | FROM cd siblings |
105 | WHERE siblings.artist = me.artist |
106 | AND siblings.cdid != me.cdid |
107 | AND siblings.cdid != ? |
108 | AND me.artist != ? |
109 | ), |
110 | (SELECT MIN( year ), MAX( year ) |
111 | FROM cd siblings |
112 | WHERE siblings.artist = me.artist |
113 | AND me.artist != ? |
114 | ), |
f7f53a89 |
115 | tracks.trackid, tracks.cd, tracks.position, tracks.title, tracks.last_updated_on, tracks.last_updated_at |
36fd7f07 |
116 | FROM cd me |
117 | LEFT JOIN track tracks |
118 | ON tracks.cd = me.cdid |
119 | WHERE me.artist != ? |
36fd7f07 |
120 | )', |
121 | [ |
122 | |
123 | # first subselect |
0e773352 |
124 | [ { sqlt_datatype => 'integer', dbic_colname => 'siblings.cdid' } |
125 | => 23414 ], |
126 | |
127 | [ { sqlt_datatype => 'integer', dbic_colname => 'me.artist' } |
128 | => 2 ], |
36fd7f07 |
129 | |
130 | # second subselect |
0e773352 |
131 | [ { sqlt_datatype => 'integer', dbic_colname => 'me.artist' } |
132 | => 2 ], |
36fd7f07 |
133 | |
36fd7f07 |
134 | # outher WHERE |
0e773352 |
135 | [ { sqlt_datatype => 'integer', dbic_colname => 'me.artist' } |
136 | => 2 ], |
36fd7f07 |
137 | ], |
138 | 'Expected SQL on correlated realiased subquery' |
139 | ); |
140 | |
4c2b30d6 |
141 | done_testing; |