X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F74mssql.t;h=172c78dfbb329b2fd76a69a319c58285c561fb24;hb=8a0720e2d4a3914c84ac2ce1dd5913d5931709a1;hp=1e86e2d08c3bab58c860f13d1ba7cf7231ec7cf2;hpb=3ff5b74063e6bb299d8a7443df0e864254ea44b9;p=dbsrgits%2FDBIx-Class.git diff --git a/t/74mssql.t b/t/74mssql.t index 1e86e2d..172c78d 100644 --- a/t/74mssql.t +++ b/t/74mssql.t @@ -1,61 +1,166 @@ use strict; use warnings; +# use this if you keep a copy of DBD::Sybase linked to FreeTDS somewhere else +BEGIN { + if (my $lib_dirs = $ENV{DBICTEST_MSSQL_PERL5LIB}) { + unshift @INC, $_ for split /:/, $lib_dirs; + } +} + use Test::More; +use Test::Exception; use lib qw(t/lib); use DBICTest; my ($dsn, $user, $pass) = @ENV{map { "DBICTEST_MSSQL_${_}" } qw/DSN USER PASS/}; -#warn "$dsn $user $pass"; - plan skip_all => 'Set $ENV{DBICTEST_MSSQL_DSN}, _USER and _PASS to run this test' unless ($dsn); -plan tests => 4; +my $TESTS = 15; + +plan tests => $TESTS * 2; + +my @storage_types = ( + 'DBI::Sybase::Microsoft_SQL_Server', + 'DBI::Sybase::Microsoft_SQL_Server::NoBindVars', +); +my $storage_idx = -1; +my $schema; + +for my $storage_type (@storage_types) { + $storage_idx++; + + $schema = DBICTest::Schema->clone; -my $storage_type = '::DBI::MSSQL'; -$storage_type = '::DBI::Sybase::MSSQL' if $dsn =~ /^dbi:Sybase:/; -# Add more for others in the future when they exist (ODBC? ADO? JDBC?) + if ($storage_idx != 0) { # autodetect + $schema->storage_type("::$storage_type"); + } -my $schema = DBICTest::Schema->clone; -$schema->storage_type($storage_type); -$schema->connection($dsn, $user, $pass); + $schema->connection($dsn, $user, $pass); -my $dbh = $schema->storage->dbh; + $schema->storage->ensure_connected; -$dbh->do("IF OBJECT_ID('artist', 'U') IS NOT NULL - DROP TABLE artist"); + if ($storage_idx == 0 && ref($schema->storage) =~ /NoBindVars\z/) { + my $tb = Test::More->builder; + $tb->skip('no placeholders') for 1..$TESTS; + next; + } -$dbh->do("CREATE TABLE artist (artistid INT IDENTITY PRIMARY KEY, name VARCHAR(255));"); + isa_ok($schema->storage, "DBIx::Class::Storage::$storage_type"); +# start disconnected to test reconnection + $schema->storage->_dbh->disconnect; + + my $dbh; + lives_ok (sub { + $dbh = $schema->storage->dbh; + }, 'reconnect works'); + + $dbh->do("IF OBJECT_ID('artist', 'U') IS NOT NULL + DROP TABLE artist"); + $dbh->do("IF OBJECT_ID('cd', 'U') IS NOT NULL + DROP TABLE cd"); + + $dbh->do("CREATE TABLE artist (artistid INT IDENTITY PRIMARY KEY, name VARCHAR(100), rank INT DEFAULT '13', charfield CHAR(10) NULL);"); + $dbh->do("CREATE TABLE cd (cdid INT IDENTITY PRIMARY KEY, artist INT, title VARCHAR(100), year VARCHAR(100), genreid INT NULL, single_track INT NULL);"); # Just to test compat shim, Auto is in Core -$schema->class('Artist')->load_components('PK::Auto::MSSQL'); + $schema->class('Artist')->load_components('PK::Auto::MSSQL'); # Test PK -my $new = $schema->resultset('Artist')->create( { name => 'foo' } ); -ok($new->artistid, "Auto-PK worked"); + my $new = $schema->resultset('Artist')->create( { name => 'foo' } ); + ok($new->artistid, "Auto-PK worked"); # Test LIMIT -for (1..6) { - $schema->resultset('Artist')->create( { name => 'Artist ' . $_ } ); -} - -my $it = $schema->resultset('Artist')->search( { }, - { rows => 3, - offset => 2, - order_by => 'artistid' + for (1..6) { + $schema->resultset('Artist')->create( { name => 'Artist ' . $_, rank => $_ } ); + } + + my $it = $schema->resultset('Artist')->search( { }, + { rows => 3, + offset => 2, + order_by => 'artistid' + } + ); + +# Test ? in data don't get treated as placeholders + my $cd = $schema->resultset('CD')->create( { + artist => 1, + title => 'Does this break things?', + year => 2007, + } ); + ok($cd->id, 'Not treating ? in data as placeholders'); + + is( $it->count, 3, "LIMIT count ok" ); + ok( $it->next->name, "iterator->next ok" ); + $it->next; + $it->next; + is( $it->next, undef, "next past end of resultset ok" ); + +# test MONEY column support + $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 $rs = $schema->resultset('Money'); + + my $row; + lives_ok { + $row = $rs->create({ amount => 100 }); + } 'inserted a money value'; + + cmp_ok $rs->find($row->id)->amount, '==', 100, 'money value round-trip'; + + lives_ok { + $row->update({ amount => 200 }); + } 'updated a money value'; + + cmp_ok $rs->find($row->id)->amount, '==', 200, + 'updated money value round-trip'; + + lives_ok { + $row->update({ amount => undef }); + } 'updated a money value to NULL'; + + is $rs->find($row->id)->amount, + undef, 'updated money value to NULL round-trip'; + + $rs->create({ amount => 300 }) for (1..3); + + # test multiple active statements + lives_ok { + my $artist_rs = $schema->resultset('Artist'); + while (my $row = $rs->next) { + my $artist = $artist_rs->next; } -); - -is( $it->count, 3, "LIMIT count ok" ); -ok( $it->next->name, "iterator->next ok" ); -$it->next; -$it->next; -is( $it->next, undef, "next past end of resultset ok" ); + $rs->reset; + } 'multiple active statements'; + + # test multiple active statements in a transaction + TODO: { + local $TODO = 'needs similar FreeTDS fixes to the ones in Sybase.pm'; + lives_ok { + $schema->txn_do(sub { + $rs->create({ amount => 400 }); + }); + } 'simple transaction'; + } +} # clean up our mess END { - $dbh->do("IF OBJECT_ID('artist', 'U') IS NOT NULL DROP TABLE artist") - if $dbh; + if (my $dbh = eval { $schema->storage->dbh }) { + $dbh->do("IF OBJECT_ID('artist', 'U') IS NOT NULL DROP TABLE artist"); + $dbh->do("IF OBJECT_ID('cd', 'U') IS NOT NULL DROP TABLE cd"); + $dbh->do("IF OBJECT_ID('money_test', 'U') IS NOT NULL DROP TABLE money_test"); + } }