t/resultset/update_delete.t

   1 use strict;
   2 use warnings;
   3
   4 use lib qw(t/lib);
   5 use Test::More;
   6 use Test::Exception;
   7
   8 # MASSIVE FIXME - there is a hole in ::RSC / as_subselect_rs
   9 # losing the order. Needs a rework/extract of the realiaser,
  10 # and that's a whole another bag of dicks
  11 BEGIN { $ENV{DBIC_SHUFFLE_UNORDERED_RESULTSETS} = 0 }
  12
  13 use DBICTest::Schema::CD;
  14 BEGIN {
  15   # the default scalarref table name will not work well for this test
  16   DBICTest::Schema::CD->table('cd');
  17 }
  18
  19 use DBICTest;
  20
  21 my $schema = DBICTest->init_schema;
  22
  23 my $tkfks = $schema->resultset('FourKeys_to_TwoKeys');
  24
  25 my ($fa, $fb, $fc) = $tkfks->related_resultset ('fourkeys')->populate ([
  26   [qw/foo bar hello goodbye sensors read_count/],
  27   [qw/1   1   1     1       a       10         /],
  28   [qw/2   2   2     2       b       20         /],
  29   [qw/1   1   1     2       c       30         /],
  30 ]);
  31
  32 # This is already provided by DBICTest
  33 #my ($ta, $tb) = $tkfk->related_resultset ('twokeys')->populate ([
  34 #  [qw/artist  cd /],
  35 #  [qw/1       1  /],
  36 #  [qw/2       2  /],
  37 #]);
  38 my ($ta, $tb) = $schema->resultset ('TwoKeys')
  39                   ->search ( [ { artist => 1, cd => 1 }, { artist => 2, cd => 2 } ], { order_by => 'artist' })
  40                     ->all;
  41
  42 my $tkfk_cnt = $tkfks->count;
  43
  44 my $non_void_ctx = $tkfks->populate ([
  45   { autopilot => 'a', fourkeys =>  $fa, twokeys => $ta, pilot_sequence => 10 },
  46   { autopilot => 'b', fourkeys =>  $fb, twokeys => $tb, pilot_sequence => 20 },
  47   { autopilot => 'x', fourkeys =>  $fa, twokeys => $tb, pilot_sequence => 30 },
  48   { autopilot => 'y', fourkeys =>  $fb, twokeys => $ta, pilot_sequence => 40 },
  49 ]);
  50 is ($tkfks->count, $tkfk_cnt += 4, 'FourKeys_to_TwoKeys populated succesfully');
  51
  52 #
  53 # Make sure the forced group by works (i.e. the joining does not cause double-updates)
  54 #
  55
  56 # create a resultset matching $fa and $fb only
  57 my $fks = $schema->resultset ('FourKeys')->search (
  58   {
  59     sensors => { '!=', 'c' },
  60     ( map { $_ => [1, 2] } qw/foo bar hello goodbye/ ),
  61   }, { join => { fourkeys_to_twokeys => 'twokeys' }}
  62 );
  63
  64 is ($fks->count, 4, 'Joined FourKey count correct (2x2)');
  65 $schema->is_executed_sql_bind( sub {
  66   $fks->update ({ read_count => \ 'read_count + 1' })
  67 }, [[
  68   'UPDATE fourkeys
  69    SET read_count = read_count + 1
  70    WHERE ( ( ( bar = ? OR bar = ? ) AND ( foo = ? OR foo = ? ) AND ( goodbye = ? OR goodbye = ? ) AND ( hello = ? OR hello = ? ) AND sensors != ? ) )
  71   ',
  72   (1, 2) x 4,
  73   'c',
  74 ]], 'Correct update-SQL with multijoin with pruning' );
  75
  76 is ($fa->discard_changes->read_count, 11, 'Update ran only once on discard-join resultset');
  77 is ($fb->discard_changes->read_count, 21, 'Update ran only once on discard-join resultset');
  78 is ($fc->discard_changes->read_count, 30, 'Update did not touch outlier');
  79
  80 # make the multi-join stick
  81 my $fks_multi = $fks->search(
  82   { 'fourkeys_to_twokeys.pilot_sequence' => { '!=' => 666 } },
  83   { order_by => [ $fks->result_source->primary_columns ] },
  84 );
  85 $schema->is_executed_sql_bind( sub {
  86   $fks_multi->update ({ read_count => \ 'read_count + 1' })
  87 }, [
  88   [ 'BEGIN' ],
  89   [
  90     'SELECT me.foo, me.bar, me.hello, me.goodbye
  91       FROM fourkeys me
  92       LEFT JOIN fourkeys_to_twokeys fourkeys_to_twokeys
  93         ON fourkeys_to_twokeys.f_bar = me.bar AND fourkeys_to_twokeys.f_foo = me.foo AND fourkeys_to_twokeys.f_goodbye = me.goodbye AND fourkeys_to_twokeys.f_hello = me.hello
  94       WHERE ( bar = ? OR bar = ? ) AND ( foo = ? OR foo = ? ) AND fourkeys_to_twokeys.pilot_sequence != ? AND ( goodbye = ? OR goodbye = ? ) AND ( hello = ? OR hello = ? ) AND sensors != ?
  95       GROUP BY me.foo, me.bar, me.hello, me.goodbye
  96       ORDER BY foo, bar, hello, goodbye
  97     ',
  98     (1, 2) x 2,
  99     666,
 100     (1, 2) x 2,
 101     'c',
 102   ],
 103   [
 104     'UPDATE fourkeys
 105      SET read_count = read_count + 1
 106      WHERE ( bar = ? AND foo = ? AND goodbye = ? AND hello = ? ) OR ( bar = ? AND foo = ? AND goodbye = ? AND hello = ? )
 107     ',
 108     ( (1) x 4, (2) x 4 ),
 109   ],
 110   [ 'COMMIT' ],
 111 ], 'Correct update-SQL with multijoin without pruning' );
 112
 113 is ($fa->discard_changes->read_count, 12, 'Update ran only once on joined resultset');
 114 is ($fb->discard_changes->read_count, 22, 'Update ran only once on joined resultset');
 115 is ($fc->discard_changes->read_count, 30, 'Update did not touch outlier');
 116
 117 $schema->is_executed_sql_bind( sub {
 118   my $res = $fks_multi->search (\' "blah" = "bleh" ')->delete;
 119   ok ($res, 'operation is true');
 120   cmp_ok ($res, '==', 0, 'zero rows affected');
 121 }, [
 122   [ 'BEGIN' ],
 123   [
 124     'SELECT me.foo, me.bar, me.hello, me.goodbye
 125       FROM fourkeys me
 126       LEFT JOIN fourkeys_to_twokeys fourkeys_to_twokeys
 127         ON fourkeys_to_twokeys.f_bar = me.bar AND fourkeys_to_twokeys.f_foo = me.foo AND fourkeys_to_twokeys.f_goodbye = me.goodbye AND fourkeys_to_twokeys.f_hello = me.hello
 128       WHERE "blah" = "bleh" AND ( bar = ? OR bar = ? ) AND ( foo = ? OR foo = ? ) AND fourkeys_to_twokeys.pilot_sequence != ? AND ( goodbye = ? OR goodbye = ? ) AND ( hello = ? OR hello = ? ) AND sensors != ?
 129       GROUP BY me.foo, me.bar, me.hello, me.goodbye
 130       ORDER BY foo, bar, hello, goodbye
 131     ',
 132     (1, 2) x 2,
 133     666,
 134     (1, 2) x 2,
 135     'c',
 136   ],
 137   [ 'COMMIT' ],
 138 ], 'Correct null-delete-SQL with multijoin without pruning' );
 139
 140
 141 # try the same sql with forced multicolumn in
 142 $schema->is_executed_sql_bind( sub {
 143
 144   my $orig_umi = $schema->storage->_use_multicolumn_in;
 145   my $sg = Scope::Guard->new(sub {
 146     $schema->storage->_use_multicolumn_in($orig_umi);
 147   });
 148
 149   $schema->storage->_use_multicolumn_in(1);
 150
 151   # this can't actually execute on sqlite
 152   eval { $fks_multi->update ({ read_count => \ 'read_count + 1' }) };
 153 }, [[
 154   'UPDATE fourkeys
 155     SET read_count = read_count + 1
 156     WHERE (
 157       (foo, bar, hello, goodbye) IN (
 158         SELECT me.foo, me.bar, me.hello, me.goodbye
 159           FROM fourkeys me
 160           LEFT JOIN fourkeys_to_twokeys fourkeys_to_twokeys ON
 161                 fourkeys_to_twokeys.f_bar = me.bar
 162             AND fourkeys_to_twokeys.f_foo = me.foo
 163             AND fourkeys_to_twokeys.f_goodbye = me.goodbye
 164             AND fourkeys_to_twokeys.f_hello = me.hello
 165         WHERE ( bar = ? OR bar = ? ) AND ( foo = ? OR foo = ? ) AND fourkeys_to_twokeys.pilot_sequence != ? AND ( goodbye = ? OR goodbye = ? ) AND ( hello = ? OR hello = ? ) AND sensors != ?
 166         ORDER BY foo, bar, hello, goodbye
 167       )
 168     )
 169   ',
 170   ( 1, 2) x 2,
 171   666,
 172   ( 1, 2) x 2,
 173   'c',
 174 ]], 'Correct update-SQL with multicolumn in support' );
 175
 176 $schema->is_executed_sql_bind( sub {
 177   $fks->search({ 'twokeys.artist' => { '!=' => 666 } })->update({ read_count => \ 'read_count + 1' });
 178 }, [
 179   [ 'BEGIN' ],
 180   [
 181     'SELECT me.foo, me.bar, me.hello, me.goodbye
 182       FROM fourkeys me
 183       LEFT JOIN fourkeys_to_twokeys fourkeys_to_twokeys
 184         ON fourkeys_to_twokeys.f_bar = me.bar AND fourkeys_to_twokeys.f_foo = me.foo AND fourkeys_to_twokeys.f_goodbye = me.goodbye AND fourkeys_to_twokeys.f_hello = me.hello
 185       LEFT JOIN twokeys twokeys
 186         ON twokeys.artist = fourkeys_to_twokeys.t_artist AND twokeys.cd = fourkeys_to_twokeys.t_cd
 187       WHERE ( bar = ? OR bar = ? ) AND ( foo = ? OR foo = ? ) AND ( goodbye = ? OR goodbye = ? ) AND ( hello = ? OR hello = ? ) AND sensors != ? AND twokeys.artist != ?
 188       GROUP BY me.foo, me.bar, me.hello, me.goodbye
 189     ',
 190     (1, 2) x 4,
 191     'c',
 192     666,
 193   ],
 194   [
 195     'UPDATE fourkeys
 196      SET read_count = read_count + 1
 197      WHERE ( bar = ? AND foo = ? AND goodbye = ? AND hello = ? ) OR ( bar = ? AND foo = ? AND goodbye = ? AND hello = ? )
 198     ',
 199     ( (1) x 4, (2) x 4 ),
 200   ],
 201   [ 'COMMIT' ],
 202 ], 'Correct update-SQL with premultiplied restricting join without pruning' );
 203
 204 is ($fa->discard_changes->read_count, 13, 'Update ran only once on joined resultset');
 205 is ($fb->discard_changes->read_count, 23, 'Update ran only once on joined resultset');
 206 is ($fc->discard_changes->read_count, 30, 'Update did not touch outlier');
 207
 208 #
 209 # Make sure multicolumn in or the equivalent functions correctly
 210 #
 211
 212 my $sub_rs = $tkfks->search (
 213   [
 214     { map { $_ => 1 } qw/artist.artistid cd.cdid fourkeys.foo fourkeys.bar fourkeys.hello fourkeys.goodbye/ },
 215     { map { $_ => 2 } qw/artist.artistid cd.cdid fourkeys.foo fourkeys.bar fourkeys.hello fourkeys.goodbye/ },
 216   ],
 217   {
 218     join => [ 'fourkeys', { twokeys => [qw/artist cd/] } ],
 219   },
 220 );
 221
 222 is ($sub_rs->count, 2, 'Only two rows from fourkeys match');
 223
 224 # attempts to delete a grouped rs should fail miserably
 225 throws_ok (
 226   sub { $sub_rs->search ({}, { distinct => 1 })->delete },
 227   qr/attempted a delete operation on a resultset which does group_by/,
 228   'Grouped rs update/delete not allowed',
 229 );
 230
 231 # grouping on PKs only should pass
 232 $sub_rs->search (
 233   {},
 234   {
 235     group_by => [ reverse $sub_rs->result_source->primary_columns ],     # reverse to make sure the PK-list comparison works
 236   },
 237 )->update ({ pilot_sequence => \ 'pilot_sequence + 1' });
 238
 239 is_deeply (
 240   [ $tkfks->search ({ autopilot => [qw/a b x y/]}, { order_by => 'autopilot' })
 241             ->get_column ('pilot_sequence')->all
 242   ],
 243   [qw/11 21 30 40/],
 244   'Only two rows incremented',
 245 );
 246
 247 # also make sure weird scalarref usage works (RT#51409)
 248 $tkfks->search (
 249   \ 'pilot_sequence BETWEEN 11 AND 21',
 250 )->update ({ pilot_sequence => \ 'pilot_sequence + 1' });
 251
 252 is_deeply (
 253   [ $tkfks->search ({ autopilot => [qw/a b x y/]}, { order_by => 'autopilot' })
 254             ->get_column ('pilot_sequence')->all
 255   ],
 256   [qw/12 22 30 40/],
 257   'Only two rows incremented (where => scalarref works)',
 258 );
 259
 260 {
 261   my $rs = $schema->resultset('FourKeys_to_TwoKeys')->search (
 262     {
 263       -or => [
 264         { 'me.pilot_sequence' => 12 },
 265         { 'me.autopilot'      => 'b' },
 266       ],
 267     }
 268   );
 269   lives_ok { $rs->update({ autopilot => 'z' }) }
 270     'Update with table name qualifier in -or conditions lives';
 271   is_deeply (
 272     [ $tkfks->search ({ pilot_sequence => [12, 22]})
 273               ->get_column ('autopilot')->all
 274     ],
 275     [qw/z z/],
 276     '... and yields the right data',
 277   );
 278 }
 279
 280
 281 $sub_rs->delete;
 282 is ($tkfks->count, $tkfk_cnt -= 2, 'Only two rows deleted');
 283
 284 # make sure limit-only deletion works
 285 cmp_ok ($tkfk_cnt, '>', 1, 'More than 1 row left');
 286 $tkfks->search ({}, { rows => 1 })->delete;
 287 is ($tkfks->count, $tkfk_cnt -= 1, 'Only one row deleted');
 288
 289
 290 # check with sql-equality, as sqlite will accept most bad sql just fine
 291 {
 292   my $rs = $schema->resultset('CD')->search(
 293     { 'me.year' => { '!=' => 2010 } },
 294   );
 295
 296   $schema->is_executed_sql_bind( sub {
 297     $rs->search({}, { join => 'liner_notes' })->delete;
 298   }, [[
 299     'DELETE FROM cd WHERE ( year != ? )',
 300     2010,
 301   ]], 'Non-restricting multijoins properly thrown out' );
 302
 303   $schema->is_executed_sql_bind( sub {
 304     $rs->search({}, { prefetch => 'liner_notes' })->delete;
 305   }, [[
 306     'DELETE FROM cd WHERE ( year != ? )',
 307     2010,
 308   ]], 'Non-restricting multiprefetch thrown out' );
 309
 310   $schema->is_executed_sql_bind( sub {
 311     $rs->search({}, { prefetch => 'artist' })->delete;
 312   }, [[
 313     'DELETE FROM cd WHERE ( cdid IN ( SELECT me.cdid FROM cd me JOIN artist artist ON artist.artistid = me.artist WHERE ( me.year != ? ) ) )',
 314     2010,
 315   ]], 'Restricting prefetch left in, selector thrown out');
 316
 317 ### switch artist and cd to fully qualified table names
 318 ### make sure nothing is stripped out
 319   my $cd_rsrc = $schema->source('CD');
 320   $cd_rsrc->name('main.cd');
 321   $cd_rsrc->relationship_info($_)->{attrs}{cascade_delete} = 0
 322     for $cd_rsrc->relationships;
 323
 324   my $art_rsrc = $schema->source('Artist');
 325   $art_rsrc->name(\'main.artist');
 326   $art_rsrc->relationship_info($_)->{attrs}{cascade_delete} = 0
 327     for $art_rsrc->relationships;
 328
 329   $schema->is_executed_sql_bind( sub {
 330     $rs->delete
 331   }, [[
 332     'DELETE FROM main.cd WHERE year != ?',
 333     2010,
 334   ]], 'delete with fully qualified table name' );
 335
 336   $rs->create({ title => 'foo', artist => 1, year => 2000 });
 337   $schema->is_executed_sql_bind( sub {
 338     $rs->delete_all
 339   }, [
 340     [ 'BEGIN' ],
 341     [
 342       'SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track FROM main.cd me WHERE me.year != ?',
 343       2010,
 344     ],
 345     [
 346       'DELETE FROM main.cd WHERE ( cdid = ? )',
 347       1,
 348     ],
 349     [ 'COMMIT' ],
 350   ], 'delete_all with fully qualified table name' );
 351
 352   $rs->create({ cdid => 42, title => 'foo', artist => 2, year => 2000 });
 353   my $cd42 = $rs->find(42);
 354
 355   $schema->is_executed_sql_bind( sub {
 356     $cd42->delete
 357   }, [[
 358     'DELETE FROM main.cd WHERE cdid = ?',
 359     42,
 360   ]], 'delete of object from table with fully qualified name' );
 361
 362   $schema->is_executed_sql_bind( sub {
 363     $cd42->related_resultset('artist')->delete
 364   }, [[
 365     'DELETE FROM main.artist WHERE ( artistid IN ( SELECT me.artistid FROM main.artist me WHERE ( me.artistid = ? ) ) )',
 366     2,
 367   ]], 'delete of related object from scalarref fully qualified named table' );
 368
 369   my $art3 = $schema->resultset('Artist')->find(3);
 370
 371   $schema->is_executed_sql_bind( sub {
 372     $art3->related_resultset('cds')->delete;
 373   }, [[
 374     'DELETE FROM main.cd WHERE ( artist = ? )',
 375     3,
 376   ]], 'delete of related object from fully qualified named table' );
 377
 378   $schema->is_executed_sql_bind( sub {
 379     $art3->cds_unordered->delete;
 380   }, [[
 381     'DELETE FROM main.cd WHERE ( artist = ? )',
 382     3,
 383   ]], 'delete of related object from fully qualified named table via relaccessor' );
 384
 385   $schema->is_executed_sql_bind( sub {
 386     $rs->search({}, { prefetch => 'artist' })->delete;
 387   }, [[
 388     'DELETE FROM main.cd WHERE ( cdid IN ( SELECT me.cdid FROM main.cd me JOIN main.artist artist ON artist.artistid = me.artist WHERE ( me.year != ? ) ) )',
 389     2010,
 390   ]], 'delete with fully qualified table name and subquery correct' );
 391
 392   # check that as_subselect_rs works ok
 393   # inner query is untouched, then a selector
 394   # and an IN condition
 395   $schema->is_executed_sql_bind( sub {
 396     $schema->resultset('CD')->search({
 397       'me.cdid' => 1,
 398       'artist.name' => 'partytimecity',
 399     }, {
 400       join => 'artist',
 401     })->as_subselect_rs->delete;
 402   }, [[
 403     '
 404       DELETE FROM main.cd
 405       WHERE (
 406         cdid IN (
 407           SELECT me.cdid
 408             FROM (
 409               SELECT me.cdid, me.artist, me.title, me.year, me.genreid, me.single_track
 410                 FROM main.cd me
 411                 JOIN main.artist artist ON artist.artistid = me.artist
 412               WHERE artist.name = ? AND me.cdid = ?
 413             ) me
 414         )
 415       )
 416     ',
 417     'partytimecity',
 418     1,
 419   ]], 'Delete from as_subselect_rs works correctly' );
 420 }
 421
 422 done_testing;