X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F746mssql.t;h=dafee694d5176c102bba8a8c3f224d34c9c0c078;hb=e81b50f4f429e1119fc6efb369122225fa7557d8;hp=f9142f70229dd472aa360b8ca32d27f4fef0c474;hpb=b8c333730df3277fee493df7ba6601ce1bb5f6ec;p=dbsrgits%2FDBIx-Class.git diff --git a/t/746mssql.t b/t/746mssql.t index f9142f7..dafee69 100644 --- a/t/746mssql.t +++ b/t/746mssql.t @@ -3,15 +3,29 @@ use warnings; use Test::More; use Test::Exception; +use Try::Tiny; +use DBIx::Class::SQLMaker::LimitDialects; +use DBIx::Class::Optional::Dependencies (); use lib qw(t/lib); use DBICTest; use DBIC::SqlMakerTest; +plan skip_all => 'Test needs ' . DBIx::Class::Optional::Dependencies->req_missing_for ('test_rdbms_mssql_odbc') + unless DBIx::Class::Optional::Dependencies->req_ok_for ('test_rdbms_mssql_odbc'); + +my $OFFSET = DBIx::Class::SQLMaker::LimitDialects->__offset_bindtype; +my $TOTAL = DBIx::Class::SQLMaker::LimitDialects->__total_bindtype; + my ($dsn, $user, $pass) = @ENV{map { "DBICTEST_MSSQL_ODBC_${_}" } qw/DSN USER PASS/}; plan skip_all => 'Set $ENV{DBICTEST_MSSQL_ODBC_DSN}, _USER and _PASS to run this test' unless ($dsn && $user); +{ + my $srv_ver = DBICTest::Schema->connect($dsn, $user, $pass)->storage->_server_info->{dbms_version}; + ok ($srv_ver, 'Got a test server version on fresh schema: ' . ($srv_ver||'???') ); +} + DBICTest::Schema->load_classes('ArtistGUID'); my $schema = DBICTest::Schema->connect($dsn, $user, $pass); @@ -29,158 +43,118 @@ my $schema = DBICTest::Schema->connect($dsn, $user, $pass); isa_ok( $schema->storage, 'DBIx::Class::Storage::DBI::ODBC::Microsoft_SQL_Server' ); { - my $schema2 = $schema->connect ($schema->storage->connect_info); + my $schema2 = $schema->connect (@{$schema->storage->connect_info}); ok (! $schema2->storage->connected, 'a re-connected cloned schema starts unconnected'); } +$schema->storage->_dbh->disconnect; -$schema->storage->dbh_do (sub { - my ($storage, $dbh) = @_; - eval { $dbh->do("DROP TABLE artist") }; - $dbh->do(<<'SQL'); -CREATE TABLE artist ( - artistid INT IDENTITY NOT NULL, - name VARCHAR(100), - rank INT NOT NULL DEFAULT '13', - charfield CHAR(10) NULL, - primary key(artistid) -) -SQL -}); - -my %seen_id; - -my @opts = ( - { on_connect_call => 'use_dynamic_cursors' }, - {}, +lives_ok { + $schema->storage->dbh_do(sub { $_[1]->do('select 1') }) +} '_ping works'; + +my %opts = ( + use_mars => + { opts => { on_connect_call => 'use_mars' } }, + use_dynamic_cursors => + { opts => { on_connect_call => 'use_dynamic_cursors' }, + required => $schema->storage->_using_freetds ? 0 : 1, + }, + use_server_cursors => + { opts => { on_connect_call => 'use_server_cursors' } }, + plain => + { opts => {}, required => 1 }, ); -my $new; -# test Auto-PK with different options -for my $opts (@opts) { +for my $opts_name (keys %opts) { SKIP: { + my $opts = $opts{$opts_name}{opts}; $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opts); - eval { + try { $schema->storage->ensure_connected - }; - if ($@ =~ /dynamic cursors/) { - skip -'Dynamic Cursors not functional, tds_version 8.0 or greater required if using'. -' FreeTDS', 1; } + catch { + if ($opts{$opts_name}{required}) { + die "on_connect_call option '$opts_name' is not functional: $_"; + } + else { + skip + "on_connect_call option '$opts_name' not functional in this configuration: $_", + 1 + ; + } + }; - $schema->resultset('Artist')->search({ name => 'foo' })->delete; - - $new = $schema->resultset('Artist')->create({ name => 'foo' }); - - ok($new->artistid > 0, "Auto-PK worked"); - } -} - -$seen_id{$new->artistid}++; - -# test LIMIT support -for (1..6) { - $new = $schema->resultset('Artist')->create({ name => 'Artist ' . $_ }); - is ( $seen_id{$new->artistid}, undef, "id for Artist $_ is unique" ); - $seen_id{$new->artistid}++; -} - -my $it = $schema->resultset('Artist')->search( {}, { - rows => 3, - order_by => 'artistid', -}); - -is( $it->count, 3, "LIMIT count ok" ); -is( $it->next->name, "foo", "iterator->next ok" ); -$it->next; -is( $it->next->name, "Artist 2", "iterator->next ok" ); -is( $it->next, undef, "next past end of resultset ok" ); - -# test GUID columns - -$schema->storage->dbh_do (sub { - my ($storage, $dbh) = @_; - eval { $dbh->do("DROP TABLE artist") }; - $dbh->do(<<'SQL'); + $schema->storage->dbh_do (sub { + my ($storage, $dbh) = @_; + eval { $dbh->do("DROP TABLE artist") }; + $dbh->do(<<'SQL'); CREATE TABLE artist ( - artistid UNIQUEIDENTIFIER NOT NULL, + artistid INT IDENTITY NOT NULL, name VARCHAR(100), rank INT NOT NULL DEFAULT '13', charfield CHAR(10) NULL, - a_guid UNIQUEIDENTIFIER, primary key(artistid) ) SQL -}); + }); -# start disconnected to make sure insert works on an un-reblessed storage -$schema = DBICTest::Schema->connect($dsn, $user, $pass); +# test Auto-PK + $schema->resultset('Artist')->search({ name => 'foo' })->delete; -my $row; -lives_ok { - $row = $schema->resultset('ArtistGUID')->create({ name => 'mtfnpy' }) -} 'created a row with a GUID'; + my $new = $schema->resultset('Artist')->create({ name => 'foo' }); -ok( - eval { $row->artistid }, - 'row has GUID PK col populated', -); -diag $@ if $@; + ok(($new->artistid||0) > 0, "Auto-PK worked for $opts_name"); -ok( - eval { $row->a_guid }, - 'row has a GUID col with auto_nextval populated', -); -diag $@ if $@; +# Test multiple active statements + SKIP: { + skip 'not a multiple active statements configuration', 1 + if $opts_name eq 'plain'; -my $row_from_db = $schema->resultset('ArtistGUID') - ->search({ name => 'mtfnpy' })->first; + $schema->storage->ensure_connected; -is $row_from_db->artistid, $row->artistid, - 'PK GUID round trip'; + lives_ok { -is $row_from_db->a_guid, $row->a_guid, - 'NON-PK GUID round trip'; + no warnings 'redefine'; + local *DBI::connect = sub { die "NO RECONNECTS!!!" }; -# test MONEY type -$schema->storage->dbh_do (sub { - my ($storage, $dbh) = @_; - eval { $dbh->do("DROP TABLE money_test") }; - $dbh->do(<<'SQL'); -CREATE TABLE money_test ( - id INT IDENTITY PRIMARY KEY, - amount MONEY NULL -) -SQL -}); + my $artist_rs = $schema->resultset('Artist'); -my $rs = $schema->resultset('Money'); + $artist_rs->delete; -lives_ok { - $row = $rs->create({ amount => 100 }); -} 'inserted a money value'; + $artist_rs->create({ name => "Artist$_" }) for (1..3); -cmp_ok $rs->find($row->id)->amount, '==', 100, 'money value round-trip'; + my $forward = $artist_rs->search({}, + { order_by => { -asc => 'artistid' } }); + my $backward = $artist_rs->search({}, + { order_by => { -desc => 'artistid' } }); -lives_ok { - $row->update({ amount => 200 }); -} 'updated a money value'; + my @map = ( + [qw/Artist1 Artist3/], [qw/Artist2 Artist2/], [qw/Artist3 Artist1/] + ); + my @result; -cmp_ok $rs->find($row->id)->amount, '==', 200, - 'updated money value round-trip'; + while (my $forward_row = $forward->next) { + my $backward_row = $backward->next; + push @result, [$forward_row->name, $backward_row->name]; + } -lives_ok { - $row->update({ amount => undef }); -} 'updated a money value to NULL'; + is_deeply \@result, \@map, "multiple active statements in $opts_name"; -is $rs->find($row->id)->amount, undef,'updated money value to NULL round-trip'; + $artist_rs->delete; -$schema->storage->dbh_do (sub { - my ($storage, $dbh) = @_; - eval { $dbh->do("DROP TABLE owners") }; - eval { $dbh->do("DROP TABLE books") }; - $dbh->do(<<'SQL'); + is($artist_rs->count, 0, '$dbh still viable'); + } "Multiple active statements survive $opts_name"; + } + +# Test populate + + { + $schema->storage->dbh_do (sub { + my ($storage, $dbh) = @_; + eval { $dbh->do("DROP TABLE owners") }; + eval { $dbh->do("DROP TABLE books") }; + $dbh->do(<<'SQL'); CREATE TABLE books ( id INT IDENTITY (1, 1) NOT NULL, source VARCHAR(100), @@ -194,301 +168,397 @@ CREATE TABLE owners ( name VARCHAR(100), ) SQL + }); + + lives_ok ( sub { + # start a new connection, make sure rebless works + my $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opts); + $schema->populate ('Owners', [ + [qw/id name /], + [qw/1 wiggle/], + [qw/2 woggle/], + [qw/3 boggle/], + [qw/4 fRIOUX/], + [qw/5 fRUE/], + [qw/6 fREW/], + [qw/7 fROOH/], + [qw/8 fISMBoC/], + [qw/9 station/], + [qw/10 mirror/], + [qw/11 dimly/], + [qw/12 face_to_face/], + [qw/13 icarus/], + [qw/14 dream/], + [qw/15 dyrstyggyr/], + ]); + }, 'populate with PKs supplied ok' ); + + + lives_ok (sub { + # start a new connection, make sure rebless works + # test an insert with a supplied identity, followed by one without + my $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opts); + for (2, 1) { + my $id = $_ * 20 ; + $schema->resultset ('Owners')->create ({ id => $id, name => "troglodoogle $id" }); + $schema->resultset ('Owners')->create ({ name => "troglodoogle " . ($id + 1) }); + } + }, 'create with/without PKs ok' ); + + is ($schema->resultset ('Owners')->count, 19, 'owner rows really in db' ); + + lives_ok ( sub { + # start a new connection, make sure rebless works + my $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opts); + $schema->populate ('BooksInLibrary', [ + [qw/source owner title /], + [qw/Library 1 secrets0/], + [qw/Library 1 secrets1/], + [qw/Eatery 1 secrets2/], + [qw/Library 2 secrets3/], + [qw/Library 3 secrets4/], + [qw/Eatery 3 secrets5/], + [qw/Library 4 secrets6/], + [qw/Library 5 secrets7/], + [qw/Eatery 5 secrets8/], + [qw/Library 6 secrets9/], + [qw/Library 7 secrets10/], + [qw/Eatery 7 secrets11/], + [qw/Library 8 secrets12/], + ]); + }, 'populate without PKs supplied ok' ); + } -}); - -lives_ok ( sub { - # start a new connection, make sure rebless works - my $schema = DBICTest::Schema->connect($dsn, $user, $pass); - $schema->populate ('Owners', [ - [qw/id name /], - [qw/1 wiggle/], - [qw/2 woggle/], - [qw/3 boggle/], - [qw/4 fRIOUX/], - [qw/5 fRUE/], - [qw/6 fREW/], - [qw/7 fROOH/], - [qw/8 fISMBoC/], - [qw/9 station/], - [qw/10 mirror/], - [qw/11 dimly/], - [qw/12 face_to_face/], - [qw/13 icarus/], - [qw/14 dream/], - [qw/15 dyrstyggyr/], - ]); -}, 'populate with PKs supplied ok' ); - -lives_ok (sub { - # start a new connection, make sure rebless works - # test an insert with a supplied identity, followed by one without - my $schema = DBICTest::Schema->connect($dsn, $user, $pass); - for (2, 1) { - my $id = $_ * 20 ; - $schema->resultset ('Owners')->create ({ id => $id, name => "troglodoogle $id" }); - $schema->resultset ('Owners')->create ({ name => "troglodoogle " . ($id + 1) }); - } -}, 'create with/without PKs ok' ); - -is ($schema->resultset ('Owners')->count, 19, 'owner rows really in db' ); - -lives_ok ( sub { - # start a new connection, make sure rebless works - my $schema = DBICTest::Schema->connect($dsn, $user, $pass); - $schema->populate ('BooksInLibrary', [ - [qw/source owner title /], - [qw/Library 1 secrets0/], - [qw/Library 1 secrets1/], - [qw/Eatery 1 secrets2/], - [qw/Library 2 secrets3/], - [qw/Library 3 secrets4/], - [qw/Eatery 3 secrets5/], - [qw/Library 4 secrets6/], - [qw/Library 5 secrets7/], - [qw/Eatery 5 secrets8/], - [qw/Library 6 secrets9/], - [qw/Library 7 secrets10/], - [qw/Eatery 7 secrets11/], - [qw/Library 8 secrets12/], - ]); -}, 'populate without PKs supplied ok' ); - -# make sure ordered subselects work -{ - my $owners = $schema->resultset ('Owners')->search ({}, { order_by => 'name' }); - my $al = $owners->current_source_alias; - my $sealed_owners = $owners->result_source->resultset->search ( - {}, - { - alias => $al, - from => [{ - -alias => $al, - -source_handle => $owners->result_source->handle, - $al => $owners->as_query, - }], - }, - ); +# test simple, complex LIMIT and limited prefetch support, with both dialects and quote combinations (if possible) + for my $dialect ( + 'Top', + ($schema->storage->_server_info->{normalized_dbms_version} || 0 ) >= 9 + ? ('RowNumberOver') + : () + , + ) { + for my $quoted (0, 1) { + + $schema = DBICTest::Schema->connect($dsn, $user, $pass, { + limit_dialect => $dialect, + %$opts, + $quoted + ? ( quote_char => [ qw/ [ ] / ], name_sep => '.' ) + : () + , + }); + + my $test_type = "Dialect:$dialect Quoted:$quoted"; + + # basic limit support + { + my $art_rs = $schema->resultset ('Artist'); + $art_rs->delete; + $art_rs->create({ name => 'Artist ' . $_ }) for (1..6); + + my $it = $schema->resultset('Artist')->search( {}, { + rows => 4, + offset => 3, + order_by => 'artistid', + }); + + is( $it->count, 3, "$test_type: LIMIT count ok" ); + + local $TODO = "Top-limit does not work when your limit ends up past the resultset" + if $dialect eq 'Top'; + + is( $it->next->name, 'Artist 4', "$test_type: iterator->next ok" ); + $it->next; + is( $it->next->name, 'Artist 6', "$test_type: iterator->next ok" ); + is( $it->next, undef, "$test_type: next past end of resultset ok" ); + } + + # plain ordered subqueries throw + throws_ok (sub { + $schema->resultset('Owners')->search ({}, { order_by => 'name' })->as_query + }, qr/ordered subselect encountered/, "$test_type: Ordered Subselect detection throws ok"); + + # make sure ordered subselects *somewhat* work + { + my $owners = $schema->resultset ('Owners')->search ({}, { order_by => 'name', offset => 2, rows => 3, unsafe_subselect_ok => 1 }); + my $sealed_owners = $owners->as_subselect_rs; + + is_deeply ( + [ map { $_->name } ($sealed_owners->all) ], + [ map { $_->name } ($owners->all) ], + "$test_type: Sort preserved from within a subquery", + ); + } + + # still even with lost order of IN, we should be getting correct + # sets + { + my $owners = $schema->resultset ('Owners')->search ({}, { order_by => 'name', offset => 2, rows => 3, unsafe_subselect_ok => 1 }); + my $corelated_owners = $owners->result_source->resultset->search ( + { + id => { -in => $owners->get_column('id')->as_query }, + }, + { + order_by => 'name' #reorder because of what is shown above + }, + ); + + is ( + join ("\x00", map { $_->name } ($corelated_owners->all) ), + join ("\x00", map { $_->name } ($owners->all) ), + "$test_type: With an outer order_by, everything still matches", + ); + } + + # make sure right-join-side single-prefetch ordering limit works + { + my $rs = $schema->resultset ('BooksInLibrary')->search ( + { + 'owner.name' => { '!=', 'woggle' }, + }, + { + prefetch => 'owner', + order_by => 'owner.name', + } + ); + # this is the order in which they should come from the above query + my @owner_names = qw/boggle fISMBoC fREW fRIOUX fROOH fRUE wiggle wiggle/; + + is ($rs->all, 8, "$test_type: Correct amount of objects from right-sorted joined resultset"); + is_deeply ( + [map { $_->owner->name } ($rs->all) ], + \@owner_names, + "$test_type: Prefetched rows were properly ordered" + ); + + my $limited_rs = $rs->search ({}, {rows => 6, offset => 2, unsafe_subselect_ok => 1}); + is ($limited_rs->count, 6, "$test_type: Correct count of limited right-sorted joined resultset"); + is ($limited_rs->count_rs->next, 6, "$test_type: Correct count_rs of limited right-sorted joined resultset"); + + my $queries; + my $orig_debug = $schema->storage->debug; + $schema->storage->debugcb(sub { $queries++; }); + $schema->storage->debug(1); + + is_deeply ( + [map { $_->owner->name } ($limited_rs->all) ], + [@owner_names[2 .. 7]], + "$test_type: Prefetch-limited rows were properly ordered" + ); + is ($queries, 1, "$test_type: Only one query with prefetch"); + + $schema->storage->debugcb(undef); + $schema->storage->debug($orig_debug); + + is_deeply ( + [map { $_->name } ($limited_rs->search_related ('owner')->all) ], + [@owner_names[2 .. 7]], + "$test_type: Rows are still properly ordered after search_related", + ); + } + + # try a ->has_many direction with duplicates + my $owners = $schema->resultset ('Owners')->search ( + { + 'books.id' => { '!=', undef }, + 'me.name' => { '!=', 'somebogusstring' }, + }, + { + prefetch => 'books', + order_by => [ { -asc => \['name + ?', [ test => 'xxx' ]] }, 'me.id' ], # test bindvar propagation + group_by => [ map { "me.$_" } $schema->source('Owners')->columns ], # the literal order_by requires an explicit group_by + rows => 3, # 8 results total + unsafe_subselect_ok => 1, + }, + ); + + my ($sql, @bind) = @${$owners->page(3)->as_query}; + # not testing the SQL as it is quite different between top/rno + is_same_bind ( + \@bind, + [ + [ { dbic_colname => 'test' } + => 'xxx' ], + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.name' } + => 'somebogusstring' ], + + ($dialect eq 'Top' + ? [ { dbic_colname => 'test' } => 'xxx' ] # the extra re-order bind + : ([ $OFFSET => 7 ], [ $TOTAL => 9 ]) # parameterised RNO + ), + + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'me.name' } + => 'somebogusstring' ], + [ { dbic_colname => 'test' } + => 'xxx' ], + ], + ); + + is ($owners->page(1)->all, 3, "$test_type: has_many prefetch returns correct number of rows"); + is ($owners->page(1)->count, 3, "$test_type: has-many prefetch returns correct count"); + + is ($owners->page(3)->count, 2, "$test_type: has-many prefetch returns correct count"); + { + local $TODO = "Top-limit does not work when your limit ends up past the resultset" + if $dialect eq 'Top'; + is ($owners->page(3)->all, 2, "$test_type: has_many prefetch returns correct number of rows"); + is ($owners->page(3)->count_rs->next, 2, "$test_type: has-many prefetch returns correct count_rs"); + } + + + # try a ->belongs_to direction (no select collapse, group_by should work) + my $books = $schema->resultset ('BooksInLibrary')->search ( + { + 'owner.name' => [qw/wiggle woggle/], + }, + { + distinct => 1, + having => \['1 = ?', [ test => 1 ] ], #test having propagation + prefetch => 'owner', + rows => 2, # 3 results total + order_by => [{ -desc => 'me.owner' }, 'me.id'], + unsafe_subselect_ok => 1, + }, + ); + + ($sql, @bind) = @${$books->page(3)->as_query}; + # not testing the SQL as it is quite different between top/rno + is_same_bind ( + \@bind, + [ + # inner + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' } + => 'wiggle' ], + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' } + => 'woggle' ], + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'source' } + => 'Library' ], + [ { dbic_colname => 'test' } + => '1' ], + + # rno(?) + $dialect ne 'Top' ? ( [ $OFFSET => 5 ], [ $TOTAL => 6 ] ) : (), + # outer + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' } + => 'wiggle' ], + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' } + => 'woggle' ], + [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'source' } + => 'Library' ], + ], + ); + + is ($books->page(1)->all, 2, "$test_type: Prefetched grouped search returns correct number of rows"); + is ($books->page(1)->count, 2, "$test_type: Prefetched grouped search returns correct count"); + + is ($books->page(2)->count, 1, "$test_type: Prefetched grouped search returns correct count"); + { + local $TODO = "Top-limit does not work when your limit ends up past the resultset" + if $dialect eq 'Top'; + is ($books->page(2)->all, 1, "$test_type: Prefetched grouped search returns correct number of rows"); + is ($books->page(2)->count_rs->next, 1, "$test_type: Prefetched grouped search returns correct count_rs"); + } + } + } - $schema->storage->debug (1); - diag "\n"; - my @names = map { $_->name } ($owners->all); - my @subq_names = map { $_->name } ($sealed_owners->all); - is_deeply ( - \@names, - \@subq_names, - 'Sort preserved from within a subquery' - ) || diag do { require Data::Dumper::Concise; Data::Dumper::Concise::Dumper (\@names, \@subq_names) }; - $schema->storage->debug (0); +# test GUID columns + { + $schema->storage->dbh_do (sub { + my ($storage, $dbh) = @_; + eval { $dbh->do("DROP TABLE artist_guid") }; + $dbh->do(<<'SQL'); +CREATE TABLE artist_guid ( + artistid UNIQUEIDENTIFIER NOT NULL, + name VARCHAR(100), + rank INT NOT NULL DEFAULT '13', + charfield CHAR(10) NULL, + a_guid UNIQUEIDENTIFIER, + primary key(artistid) +) +SQL + }); + # start disconnected to make sure insert works on an un-reblessed storage + $schema = DBICTest::Schema->connect($dsn, $user, $pass, $opts); + my $row; + lives_ok { + $row = $schema->resultset('ArtistGUID')->create({ name => 'mtfnpy' }) + } 'created a row with a GUID'; - my $book_owner_ids = $schema->resultset ('BooksInLibrary') - ->search ({}, { join => 'owner', distinct => 1, order_by => 'owner.name' }) - ->get_column ('owner'); + ok( + eval { $row->artistid }, + 'row has GUID PK col populated', + ); + diag $@ if $@; - my $book_owners = $schema->resultset ('Owners')->search ({ - id => { -in => $book_owner_ids->as_query } - }); + ok( + eval { $row->a_guid }, + 'row has a GUID col with auto_nextval populated', + ); + diag $@ if $@; - is ($book_owners->count, 8, 'Correct amount of book owners'); + my $row_from_db = $schema->resultset('ArtistGUID') + ->search({ name => 'mtfnpy' })->first; - is_deeply ( - [ map { $_->id } ($book_owners->all) ], - [ $book_owner_ids->all ], - 'Sort is preserved across IN subqueries', - ); -} + is $row_from_db->artistid, $row->artistid, + 'PK GUID round trip'; -# make sure right-join-side single-prefetch ordering limit works -{ - my $rs = $schema->resultset ('BooksInLibrary')->search ( - { - 'owner.name' => { '!=', 'woggle' }, - }, - { - prefetch => 'owner', - order_by => 'owner.name', + is $row_from_db->a_guid, $row->a_guid, + 'NON-PK GUID round trip'; } - ); - # this is the order in which they should come from the above query - my @owner_names = qw/boggle fISMBoC fREW fRIOUX fROOH fRUE wiggle wiggle/; - - is ($rs->all, 8, 'Correct amount of objects from right-sorted joined resultset'); - is_deeply ( - [map { $_->owner->name } ($rs->all) ], - \@owner_names, - 'Rows were properly ordered' - ); - - my $limited_rs = $rs->search ({}, {rows => 7, offset => 2}); - is ($limited_rs->count, 6, 'Correct count of limited right-sorted joined resultset'); - is ($limited_rs->count_rs->next, 6, 'Correct count_rs of limited right-sorted joined resultset'); - - my $queries; - $schema->storage->debugcb(sub { $queries++; }); - $schema->storage->debug(1); - - is_deeply ( - [map { $_->owner->name } ($limited_rs->all) ], - [@owner_names[2 .. 7]], - 'Limited rows were properly ordered' - ); - is ($queries, 1, 'Only one query with prefetch'); - - $schema->storage->debugcb(undef); - $schema->storage->debug(0); - - - is_deeply ( - [map { $_->name } ($limited_rs->search_related ('owner')->all) ], - [@owner_names[2 .. 7]], - 'Rows are still properly ordered after search_related' - ); -} - - -# -# try a prefetch on tables with identically named columns -# -# set quote char - make sure things work while quoted -$schema->storage->_sql_maker->{quote_char} = [qw/[ ]/]; -$schema->storage->_sql_maker->{name_sep} = '.'; - -{ - # try a ->has_many direction - my $owners = $schema->resultset ('Owners')->search ( - { - 'books.id' => { '!=', undef }, - 'me.name' => { '!=', 'somebogusstring' }, - }, +# test MONEY type { - prefetch => 'books', - order_by => { -asc => \['name + ?', [ test => 'xxx' ]] }, # test bindvar propagation - rows => 3, # 8 results total - }, - ); + $schema->storage->dbh_do (sub { + my ($storage, $dbh) = @_; + eval { $dbh->do("DROP TABLE money_test") }; + $dbh->do(<<'SQL'); +CREATE TABLE money_test ( + id INT IDENTITY PRIMARY KEY, + amount MONEY NULL +) +SQL + }); - my ($sql, @bind) = @${$owners->page(3)->as_query}; - is_deeply ( - \@bind, - [ ([ 'me.name' => 'somebogusstring' ], [ test => 'xxx' ]) x 2 ], # double because of the prefetch subq - ); + { + my $freetds_and_dynamic_cursors = 1 + if $opts_name eq 'use_dynamic_cursors' && + $schema->storage->_using_freetds; - is ($owners->page(1)->all, 3, 'has_many prefetch returns correct number of rows'); - is ($owners->page(1)->count, 3, 'has-many prefetch returns correct count'); + local $TODO = +'these tests fail on freetds with dynamic cursors for some reason' + if $freetds_and_dynamic_cursors; + local $ENV{DBIC_NULLABLE_KEY_NOWARN} = 1 + if $freetds_and_dynamic_cursors; - is ($owners->page(3)->all, 2, 'has_many prefetch returns correct number of rows'); - is ($owners->page(3)->count, 2, 'has-many prefetch returns correct count'); - is ($owners->page(3)->count_rs->next, 2, 'has-many prefetch returns correct count_rs'); + my $rs = $schema->resultset('Money'); + my $row; + lives_ok { + $row = $rs->create({ amount => 100 }); + } 'inserted a money value'; - # try a ->belongs_to direction (no select collapse, group_by should work) - my $books = $schema->resultset ('BooksInLibrary')->search ( - { - 'owner.name' => [qw/wiggle woggle/], - }, - { - distinct => 1, - having => \['1 = ?', [ test => 1 ] ], #test having propagation - prefetch => 'owner', - rows => 2, # 3 results total - order_by => { -desc => 'owner' }, - }, - ); - - ($sql, @bind) = @${$books->page(3)->as_query}; - is_deeply ( - \@bind, - [ - # inner - [ 'owner.name' => 'wiggle' ], [ 'owner.name' => 'woggle' ], [ source => 'Library' ], [ test => '1' ], - # outer - [ 'owner.name' => 'wiggle' ], [ 'owner.name' => 'woggle' ], [ source => 'Library' ], - ], - ); - - is ($books->page(1)->all, 2, 'Prefetched grouped search returns correct number of rows'); - is ($books->page(1)->count, 2, 'Prefetched grouped search returns correct count'); - - is ($books->page(2)->all, 1, 'Prefetched grouped search returns correct number of rows'); - is ($books->page(2)->count, 1, 'Prefetched grouped search returns correct count'); - is ($books->page(2)->count_rs->next, 1, 'Prefetched grouped search returns correct count_rs'); -} + cmp_ok ((try { $rs->find($row->id)->amount })||0, '==', 100, + 'money value round-trip'); + lives_ok { + $row->update({ amount => 200 }); + } 'updated a money value'; + cmp_ok ((try { $rs->find($row->id)->amount })||0, '==', 200, + 'updated money value round-trip'); -# Just to aid bug-hunting, delete block before merging -{ + lives_ok { + $row->update({ amount => undef }); + } 'updated a money value to NULL'; - my $limited_rs = $schema->resultset ('BooksInLibrary')->search ( - { - 'owner.name' => { '!=', 'woggle' }, - }, - { - prefetch => 'owner', - order_by => 'owner.name', - rows => 7, - offset => 2, + is try { $rs->find($row->id)->amount }, undef, + 'updated money value to NULL round-trip'; + } } - ); - - -=begin - -Alan's SQL: - - SELECT me.id, me.surveyor_id, me.survey_site_id, me.year, surveyor.id, surveyor.name, surveyor.email, surveyor.phone, surveyor.login, surveyor.password, surveyor.is_active, surveyor.is_verifier, surveyor.arm_length, surveyor.eye_height, surveyor.year_joined - FROM ( - SELECT * - FROM ( - SELECT orig_query.*, ROW_NUMBER() OVER( ORDER BY (SELECT(1)) ) AS rno__row__index - FROM ( - SELECT me.id, me.surveyor_id, me.survey_site_id, me.year - FROM ( - SELECT TOP 100 PERCENT me.id, me.surveyor_id, me.survey_site_id, me.year - FROM surveyors_survey_sites me - JOIN surveyors surveyor ON surveyor.id = me.surveyor_id - ORDER BY surveyor.name - ) me - ) orig_query - ) rno_subq - WHERE rno__row__index BETWEEN 136 AND 150 - ) me - JOIN surveyors surveyor ON surveyor.id = me.surveyor_id - ORDER BY surveyor.name -=cut - - is_same_sql_bind ( - $limited_rs->as_query, - '( - SELECT TOP 100 PERCENT [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price], [owner].[id], [owner].[name] - FROM ( - SELECT * - FROM ( - SELECT [me].*, ROW_NUMBER() OVER( ORDER BY (SELECT(1)) ) AS rno__row__index - FROM ( - SELECT [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price] - FROM ( - SELECT TOP 100 PERCENT [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price] - FROM [books] [me] - JOIN [owners] [owner] ON [owner].[id] = [me].[owner] - WHERE ( ( [owner].[name] != ? AND [source] = ? ) ) - ORDER BY [owner].[name] - ) [me] - ) [me] - ) rno_subq - WHERE rno__row__index BETWEEN 3 AND 9 - ) [me] - JOIN [owners] [owner] ON [owner].[id] = [me].[owner] - WHERE ( ( [owner].[name] != ? AND [source] = ? ) ) - ORDER BY [owner].[name] - )', - [ ([ 'owner.name' => 'woggle' ], [ source => 'Library' ]) x 2 ], - 'Expected SQL executed', - ); + } } done_testing; @@ -497,7 +567,8 @@ done_testing; END { if (my $dbh = eval { $schema->storage->_dbh }) { eval { $dbh->do("DROP TABLE $_") } - for qw/artist money_test books owners/; + for qw/artist artist_guid money_test books owners/; } + undef $schema; } # vim:sw=2 sts=2