Separate the oracle core and hq tests
Peter Rabbitson [Sun, 28 Nov 2010 03:39:48 +0000 (04:39 +0100)]
t/73oracle.t
t/73oracle_hq.t [new file with mode: 0644]

index cde22cd..34b3174 100644 (file)
@@ -44,9 +44,7 @@ my ($dsn,  $user,  $pass)  = @ENV{map { "DBICTEST_ORA_${_}" }  qw/DSN USER PASS/
 # optional:
 my ($dsn2, $user2, $pass2) = @ENV{map { "DBICTEST_ORA_EXTRAUSER_${_}" } qw/DSN USER PASS/};
 
-plan skip_all => 'Set $ENV{DBICTEST_ORA_DSN}, _USER and _PASS to run this test. ' .
-  'Warning: This test drops and creates tables called \'artist\', \'cd\', \'track\' and \'sequence_test\''.
-  ' as well as following sequences: \'pkid1_seq\', \'pkid2_seq\' and \'nonpkid_seq\''
+plan skip_all => 'Set $ENV{DBICTEST_ORA_DSN}, _USER and _PASS to run this test.'
   unless ($dsn && $user && $pass);
 
 DBICTest::Schema->load_classes('ArtistFQN');
@@ -58,23 +56,6 @@ my $dbh = $schema->storage->dbh;
 
 do_creates($dbh);
 
-{
-    # Swiped from t/bindtype_columns.t to avoid creating my own Resultset.
-
-    local $SIG{__WARN__} = sub {};
-    eval { $dbh->do('DROP TABLE bindtype_test') };
-
-    $dbh->do(qq[
-        CREATE TABLE bindtype_test
-        (
-            id              integer      NOT NULL   PRIMARY KEY,
-            bytea           integer      NULL,
-            blob            blob         NULL,
-            clob            clob         NULL
-        )
-    ],{ RaiseError => 1, PrintError => 1 });
-}
-
 # This is in Core now, but it's here just to test that it doesn't break
 $schema->class('Artist')->load_components('PK::Auto');
 # These are compat shims for PK::Auto...
@@ -247,6 +228,7 @@ for (1..5) {
 my $st = $schema->resultset('SequenceTest')->create({ name => 'foo', pkid1 => 55 });
 is($st->pkid1, 55, "Oracle Auto-PK without trigger: First primary key set manually");
 
+# test BLOBs
 SKIP: {
   my %binstr = ( 'small' => join('', map { chr($_) } ( 1 .. 127 )) );
   $binstr{'large'} = $binstr{'small'} x 1024;
@@ -284,440 +266,14 @@ SKIP: {
   $schema->storage->debug ($orig_debug);
 }
 
-
-### test hierarchical queries
-if ( $schema->storage->isa('DBIx::Class::Storage::DBI::Oracle::Generic') ) {
-    my $source = $schema->source('Artist');
-
-    $source->add_column( 'parentid' );
-
-    $source->add_relationship('children', 'DBICTest::Schema::Artist',
-        { 'foreign.parentid' => 'self.artistid' },
-        {
-            accessor => 'multi',
-            join_type => 'LEFT',
-            cascade_delete => 1,
-            cascade_copy => 1,
-        } );
-    $source->add_relationship('parent', 'DBICTest::Schema::Artist',
-        { 'foreign.artistid' => 'self.parentid' },
-        { accessor => 'single' } );
-    DBICTest::Schema::Artist->add_column( 'parentid' );
-    DBICTest::Schema::Artist->has_many(
-        children => 'DBICTest::Schema::Artist',
-        { 'foreign.parentid' => 'self.artistid' }
-    );
-    DBICTest::Schema::Artist->belongs_to(
-        parent => 'DBICTest::Schema::Artist',
-        { 'foreign.artistid' => 'self.parentid' }
-    );
-
-    $schema->resultset('Artist')->create ({
-        name => 'root',
-        rank => 1,
-        cds => [],
-        children => [
-            {
-                name => 'child1',
-                rank => 2,
-                children => [
-                    {
-                        name => 'grandchild',
-                        rank => 3,
-                        cds => [
-                            {
-                                title => "grandchilds's cd" ,
-                                year => '2008',
-                                tracks => [
-                                    {
-                                        position => 1,
-                                        title => 'Track 1 grandchild',
-                                    }
-                                ],
-                            }
-                        ],
-                        children => [
-                            {
-                                name => 'greatgrandchild',
-                                rank => 3,
-                            }
-                        ],
-                    }
-                ],
-            },
-            {
-                name => 'child2',
-                rank => 3,
-            },
-        ],
-    });
-
-    $schema->resultset('Artist')->create(
-        {
-            name     => 'cycle-root',
-            children => [
-                {
-                    name     => 'cycle-child1',
-                    children => [ { name => 'cycle-grandchild' } ],
-                },
-                { name => 'cycle-child2' },
-            ],
-        }
-    );
-
-    $schema->resultset('Artist')->find({ name => 'cycle-root' })
-      ->update({ parentid => { -ident => 'artistid' } });
-
-    # select the whole tree
-    {
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
-            FROM artist me
-          START WITH name = ?
-          CONNECT BY parentid = PRIOR artistid 
-        )',
-        [ [ name => 'root'] ],
-      );
-      is_deeply (
-        [ $rs->get_column ('name')->all ],
-        [ qw/root child1 grandchild greatgrandchild child2/ ],
-        'got artist tree',
-      );
-
-
-      is_same_sql_bind (
-        $rs->count_rs->as_query,
-        '(
-          SELECT COUNT( * )
-            FROM artist me
-          START WITH name = ?
-          CONNECT BY parentid = PRIOR artistid 
-        )',
-        [ [ name => 'root'] ],
-      );
-
-      is( $rs->count, 5, 'Connect By count ok' );
-    }
-
-    # use order siblings by statement
-    SKIP: {
-      # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/state21b.htm#2066123
-      skip q{Oracle8i doesn't support ORDER SIBLINGS BY}, 1
-        if $schema->storage->_server_info->{normalized_dbms_version} < 9;
-
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident =>  'artistid' } } },
-        order_siblings_by => { -desc => 'name' },
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
-            FROM artist me
-          START WITH name = ?
-          CONNECT BY parentid = PRIOR artistid 
-          ORDER SIBLINGS BY name DESC
-        )',
-        [ [ name => 'root'] ],
-      );
-
-      is_deeply (
-        [ $rs->get_column ('name')->all ],
-        [ qw/root child2 child1 grandchild greatgrandchild/ ],
-        'Order Siblings By ok',
-      );
-    }
-
-    # get the root node
-    {
-      my $rs = $schema->resultset('Artist')->search({ parentid => undef }, {
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
-            FROM artist me
-          WHERE ( parentid IS NULL )
-          START WITH name = ?
-          CONNECT BY parentid = PRIOR artistid 
-        )',
-        [ [ name => 'root'] ],
-      );
-
-      is_deeply(
-        [ $rs->get_column('name')->all ],
-        [ 'root' ],
-        'found root node',
-      );
-    }
-
-    # combine a connect by with a join
-    SKIP: {
-      # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/state21b.htm#2066123
-      skip q{Oracle8i doesn't support connect by with join}, 1
-        if $schema->storage->_server_info->{normalized_dbms_version} < 9;
-
-      my $rs = $schema->resultset('Artist')->search(
-        {'cds.title' => { -like => '%cd'} },
-        {
-          join => 'cds',
-          start_with => { 'me.name' => 'root' },
-          connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-        }
-      );
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
-            FROM artist me
-            LEFT JOIN cd cds ON cds.artist = me.artistid
-          WHERE ( cds.title LIKE ? )
-          START WITH me.name = ?
-          CONNECT BY parentid = PRIOR artistid 
-        )',
-        [ [ 'cds.title' => '%cd' ], [ 'me.name' => 'root' ] ],
-      );
-
-      is_deeply(
-        [ $rs->get_column('name')->all ],
-        [ 'grandchild' ],
-        'Connect By with a join result name ok'
-      );
-
-
-      is_same_sql_bind (
-        $rs->count_rs->as_query,
-        '(
-          SELECT COUNT( * )
-            FROM artist me
-            LEFT JOIN cd cds ON cds.artist = me.artistid
-          WHERE ( cds.title LIKE ? )
-          START WITH me.name = ?
-          CONNECT BY parentid = PRIOR artistid 
-        )',
-        [ [ 'cds.title' => '%cd' ], [ 'me.name' => 'root' ] ],
-      );
-
-      is( $rs->count, 1, 'Connect By with a join; count ok' );
-    }
-
-    # combine a connect by with order_by
-    {
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-        order_by => { -asc => [ 'LEVEL', 'name' ] },
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
-          FROM artist me
-          START WITH name = ?
-          CONNECT BY parentid = PRIOR artistid 
-          ORDER BY LEVEL ASC, name ASC
-        )',
-        [ [ name => 'root' ] ],
-      );
-
-
-      # Don't use "$rs->get_column ('name')->all" they build a query arround the $rs.
-      #   If $rs has a order by, the order by is in the subquery and this doesn't work with Oracle 8i.
-      # TODO: write extra test and fix order by handling on Oracle 8i
-      is_deeply (
-        [ map { $_->[1] } $rs->cursor->all ],
-        [ qw/root child1 child2 grandchild greatgrandchild/ ],
-        'Connect By with a order_by - result name ok (without get_column)'
-      );
-
-      SKIP: {
-          skip q{Connect By with a order_by - result name ok (with get_column), Oracle8i doesn't support order by in a subquery},1
-            if $schema->storage->_server_info->{normalized_dbms_version} < 9;
-          is_deeply (
-            [  $rs->get_column ('name')->all ],
-            [ qw/root child1 child2 grandchild greatgrandchild/ ],
-            'Connect By with a order_by - result name ok (with get_column)'
-          );
-      }
-    }
-
-
-    # limit a connect by
-    SKIP: {
-      skip q{Oracle8i doesn't support order by in a subquery}, 1
-        if $schema->storage->_server_info->{normalized_dbms_version} < 9;
-
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-        order_by => { -asc => 'name' },
-        rows => 2,
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-            SELECT artistid, name, rank, charfield, parentid FROM (
-              SELECT
-                  me.artistid,
-                  me.name,
-                  me.rank,
-                  me.charfield,
-                  me.parentid
-                FROM artist me
-              START WITH name = ?
-              CONNECT BY parentid = PRIOR artistid
-              ORDER BY name ASC 
-            ) me
-            WHERE ROWNUM <= 2
-        )',
-        [ [ name => 'root' ] ],
-      );
-
-      is_deeply (
-        [ $rs->get_column ('name')->all ],
-        [qw/child1 child2/],
-        'LIMIT a Connect By query - correct names'
-      );
-
-      is_same_sql_bind (
-        $rs->count_rs->as_query,
-        '(
-          SELECT COUNT( * ) FROM (
-            SELECT artistid
-              FROM (
-                SELECT
-                  me.artistid
-                FROM artist me 
-                START WITH name = ? 
-                CONNECT BY parentid = PRIOR artistid
-              ) me
-            WHERE ROWNUM <= 2
-          ) me
-        )',
-        [ [ name => 'root' ] ],
-      );
-
-      is( $rs->count, 2, 'Connect By; LIMIT count ok' );
-    }
-
-    # combine a connect_by with group_by and having
-    {
-      my $rs = $schema->resultset('Artist')->search({}, {
-        select => { count => 'rank', -as => 'cnt' },
-        start_with => { name => 'root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-        group_by => ['rank'],
-        having => \[ 'count(rank) < ?', [ cnt => 2 ] ],
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-            SELECT COUNT(rank) AS cnt
-            FROM artist me
-            START WITH name = ?
-            CONNECT BY parentid = PRIOR artistid
-            GROUP BY rank HAVING count(rank) < ?
-        )',
-        [ [ name => 'root' ], [ cnt => 2 ] ],
-      );
-
-      is_deeply (
-        [ $rs->get_column ('cnt')->all ],
-        [1, 1],
-        'Group By a Connect By query - correct values'
-      );
-    }
-
-
-    # select the whole cycle tree without nocylce
-    {
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'cycle-root' },
-        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
-      });
-      eval { $rs->get_column ('name')->all };
-      if ( $@ =~ /ORA-01436/ ){ # ORA-01436:  CONNECT BY loop in user data
-        pass "connect by initify loop detection without nocycle";
-      }else{
-        fail "connect by initify loop detection without nocycle, not detected by oracle";
-      }
-    }
-
-    # select the whole cycle tree with nocylce
-    SKIP: {
-      # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/expressi.htm#1023748
-      skip q{Oracle8i doesn't support connect by nocycle}, 1
-        if $schema->storage->_server_info->{normalized_dbms_version} < 9;
-
-      my $rs = $schema->resultset('Artist')->search({}, {
-        start_with => { name => 'cycle-root' },
-        '+select'  => \ 'CONNECT_BY_ISCYCLE',
-        '+as'      => [ 'connector' ],
-        connect_by_nocycle => { parentid => { -prior => { -ident => 'artistid' } } },
-      });
-
-      is_same_sql_bind (
-        $rs->as_query,
-        '(
-          SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid, CONNECT_BY_ISCYCLE
-            FROM artist me
-          START WITH name = ?
-          CONNECT BY NOCYCLE parentid = PRIOR artistid 
-        )',
-        [ [ name => 'cycle-root'] ],
-      );
-      is_deeply (
-        [ $rs->get_column ('name')->all ],
-        [ qw/cycle-root cycle-child1 cycle-grandchild cycle-child2/ ],
-        'got artist tree with nocycle (name)',
-      );
-      is_deeply (
-        [ $rs->get_column ('connector')->all ],
-        [ qw/1 0 0 0/ ],
-        'got artist tree with nocycle (CONNECT_BY_ISCYCLE)',
-      );
-
-
-      is_same_sql_bind (
-        $rs->count_rs->as_query,
-        '(
-          SELECT COUNT( * )
-            FROM artist me
-          START WITH name = ?
-          CONNECT BY NOCYCLE parentid = PRIOR artistid 
-        )',
-        [ [ name => 'cycle-root'] ],
-      );
-
-      is( $rs->count, 4, 'Connect By Nocycle count ok' );
-    }
-}
-
-my $schema2;
-
 # test sequence detection from a different schema
+my $schema2;
 SKIP: {
 TODO: {
   skip ((join '',
-'Set DBICTEST_ORA_EXTRAUSER_DSN, _USER and _PASS to a *DIFFERENT* Oracle user',
-' to run the cross-schema autoincrement test.'),
-    1) unless $dsn2 && $user2 && $user2 ne $user;
+    'Set DBICTEST_ORA_EXTRAUSER_DSN, _USER and _PASS to a *DIFFERENT* Oracle user',
+    ' to run the cross-schema autoincrement test.'
+  ), 1) unless $dsn2 && $user2 && $user2 ne $user;
 
   # Oracle8i Reference Release 2 (8.1.6) 
   #   http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a76961/ch294.htm#993
@@ -797,7 +353,7 @@ sub do_creates {
   $dbh->do("CREATE SEQUENCE pkid2_seq START WITH 10 MAXVALUE 999999 MINVALUE 0");
   $dbh->do("CREATE SEQUENCE nonpkid_seq START WITH 20 MAXVALUE 999999 MINVALUE 0");
 
-  $dbh->do("CREATE TABLE artist (artistid NUMBER(12), parentid NUMBER(12), name VARCHAR(255), autoinc_col NUMBER(12), rank NUMBER(38), charfield VARCHAR2(10))");
+  $dbh->do("CREATE TABLE artist (artistid NUMBER(12), name VARCHAR(255), autoinc_col NUMBER(12), rank NUMBER(38), charfield VARCHAR2(10))");
   $dbh->do("ALTER TABLE artist ADD (CONSTRAINT artist_pk PRIMARY KEY (artistid))");
 
   $dbh->do("CREATE TABLE sequence_test (pkid1 NUMBER(12), pkid2 NUMBER(12), nonpkid NUMBER(12), name VARCHAR(255))");
@@ -809,6 +365,8 @@ sub do_creates {
   $dbh->do("CREATE TABLE track (trackid NUMBER(12), cd NUMBER(12) REFERENCES cd(cdid) DEFERRABLE, position NUMBER(12), title VARCHAR(255), last_updated_on DATE, last_updated_at DATE, small_dt DATE)");
   $dbh->do("ALTER TABLE track ADD (CONSTRAINT track_pk PRIMARY KEY (trackid))");
 
+  $dbh->do("CREATE TABLE bindtype_test (id integer NOT NULL PRIMARY KEY, bytea integer NULL, blob blob NULL, clob clob NULL)");
+
   $dbh->do(qq{
     CREATE OR REPLACE TRIGGER artist_insert_trg_auto
     BEFORE INSERT ON artist
diff --git a/t/73oracle_hq.t b/t/73oracle_hq.t
new file mode 100644 (file)
index 0000000..5e4c0b2
--- /dev/null
@@ -0,0 +1,513 @@
+use strict;
+use warnings;
+
+use Test::Exception;
+use Test::More;
+
+use lib qw(t/lib);
+use DBIC::SqlMakerTest;
+
+my ($dsn,  $user,  $pass)  = @ENV{map { "DBICTEST_ORA_${_}" }  qw/DSN USER PASS/};
+
+plan skip_all => 'Set $ENV{DBICTEST_ORA_DSN}, _USER and _PASS to run this test.'
+ unless ($dsn && $user && $pass);
+
+use DBICTest::Schema::Artist;
+BEGIN {
+  DBICTest::Schema::Artist->add_column('parentid');
+
+  DBICTest::Schema::Artist->has_many(
+    children => 'DBICTest::Schema::Artist',
+    { 'foreign.parentid' => 'self.artistid' }
+  );
+
+  DBICTest::Schema::Artist->belongs_to(
+    parent => 'DBICTest::Schema::Artist',
+    { 'foreign.artistid' => 'self.parentid' }
+  );
+}
+
+use DBICTest::Schema;
+
+my $schema = DBICTest::Schema->connect($dsn, $user, $pass);
+
+note "Oracle Version: " . $schema->storage->_server_info->{dbms_version};
+
+my $dbh = $schema->storage->dbh;
+do_creates($dbh);
+
+### test hierarchical queries
+{
+  $schema->resultset('Artist')->create ({
+    name => 'root',
+    rank => 1,
+    cds => [],
+    children => [
+      {
+        name => 'child1',
+        rank => 2,
+        children => [
+          {
+            name => 'grandchild',
+            rank => 3,
+            cds => [
+              {
+                title => "grandchilds's cd" ,
+                year => '2008',
+                tracks => [
+                  {
+                    position => 1,
+                    title => 'Track 1 grandchild',
+                  }
+                ],
+              }
+            ],
+            children => [
+              {
+                name => 'greatgrandchild',
+                rank => 3,
+              }
+            ],
+          }
+        ],
+      },
+      {
+        name => 'child2',
+        rank => 3,
+      },
+    ],
+  });
+
+  $schema->resultset('Artist')->create({
+    name => 'cycle-root',
+    children => [
+      {
+        name => 'cycle-child1',
+        children => [ { name => 'cycle-grandchild' } ],
+      },
+      {
+        name => 'cycle-child2'
+      },
+    ],
+  });
+
+  $schema->resultset('Artist')->find({ name => 'cycle-root' })
+    ->update({ parentid => { -ident => 'artistid' } });
+
+  # select the whole tree
+  {
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid 
+      )',
+      [ [ name => 'root'] ],
+    );
+    is_deeply (
+      [ $rs->get_column ('name')->all ],
+      [ qw/root child1 grandchild greatgrandchild child2/ ],
+      'got artist tree',
+    );
+
+    is_same_sql_bind (
+      $rs->count_rs->as_query,
+      '(
+        SELECT COUNT( * )
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid 
+      )',
+      [ [ name => 'root'] ],
+    );
+
+    is( $rs->count, 5, 'Connect By count ok' );
+  }
+
+  # use order siblings by statement
+  SKIP: {
+    # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/state21b.htm#2066123
+    skip q{Oracle8i doesn't support ORDER SIBLINGS BY}, 1
+      if $schema->storage->_server_info->{normalized_dbms_version} < 9;
+
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident =>  'artistid' } } },
+      order_siblings_by => { -desc => 'name' },
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid 
+        ORDER SIBLINGS BY name DESC
+      )',
+      [ [ name => 'root'] ],
+    );
+
+    is_deeply (
+      [ $rs->get_column ('name')->all ],
+      [ qw/root child2 child1 grandchild greatgrandchild/ ],
+      'Order Siblings By ok',
+    );
+  }
+
+  # get the root node
+  {
+    my $rs = $schema->resultset('Artist')->search({ parentid => undef }, {
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+          FROM artist me
+        WHERE ( parentid IS NULL )
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid 
+      )',
+      [ [ name => 'root'] ],
+    );
+
+    is_deeply(
+      [ $rs->get_column('name')->all ],
+      [ 'root' ],
+      'found root node',
+    );
+  }
+
+  # combine a connect by with a join
+  SKIP: {
+    # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/state21b.htm#2066123
+    skip q{Oracle8i doesn't support connect by with join}, 1
+      if $schema->storage->_server_info->{normalized_dbms_version} < 9;
+
+    my $rs = $schema->resultset('Artist')->search(
+      {'cds.title' => { -like => '%cd'} },
+      {
+        join => 'cds',
+        start_with => { 'me.name' => 'root' },
+        connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+      }
+    );
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+          FROM artist me
+          LEFT JOIN cd cds ON cds.artist = me.artistid
+        WHERE ( cds.title LIKE ? )
+        START WITH me.name = ?
+        CONNECT BY parentid = PRIOR artistid 
+      )',
+      [ [ 'cds.title' => '%cd' ], [ 'me.name' => 'root' ] ],
+    );
+
+    is_deeply(
+      [ $rs->get_column('name')->all ],
+      [ 'grandchild' ],
+      'Connect By with a join result name ok'
+    );
+
+    is_same_sql_bind (
+      $rs->count_rs->as_query,
+      '(
+        SELECT COUNT( * )
+          FROM artist me
+          LEFT JOIN cd cds ON cds.artist = me.artistid
+        WHERE ( cds.title LIKE ? )
+        START WITH me.name = ?
+        CONNECT BY parentid = PRIOR artistid 
+      )',
+      [ [ 'cds.title' => '%cd' ], [ 'me.name' => 'root' ] ],
+    );
+
+    is( $rs->count, 1, 'Connect By with a join; count ok' );
+  }
+
+  # combine a connect by with order_by
+  {
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+      order_by => { -asc => [ 'LEVEL', 'name' ] },
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid 
+        ORDER BY LEVEL ASC, name ASC
+      )',
+      [ [ name => 'root' ] ],
+    );
+
+
+    # Don't use "$rs->get_column ('name')->all" they build a query arround the $rs.
+    #   If $rs has a order by, the order by is in the subquery and this doesn't work with Oracle 8i.
+    # TODO: write extra test and fix order by handling on Oracle 8i
+    is_deeply (
+      [ map { $_->[1] } $rs->cursor->all ],
+      [ qw/root child1 child2 grandchild greatgrandchild/ ],
+      'Connect By with a order_by - result name ok (without get_column)'
+    );
+
+    SKIP: {
+      skip q{Connect By with a order_by - result name ok (with get_column), Oracle8i doesn't support order by in a subquery},1
+        if $schema->storage->_server_info->{normalized_dbms_version} < 9;
+      is_deeply (
+        [  $rs->get_column ('name')->all ],
+        [ qw/root child1 child2 grandchild greatgrandchild/ ],
+        'Connect By with a order_by - result name ok (with get_column)'
+      );
+    }
+  }
+
+
+  # limit a connect by
+  SKIP: {
+    skip q{Oracle8i doesn't support order by in a subquery}, 1
+      if $schema->storage->_server_info->{normalized_dbms_version} < 9;
+
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+      order_by => { -asc => 'name' },
+      rows => 2,
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT artistid, name, rank, charfield, parentid
+          FROM (
+            SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid
+              FROM artist me
+            START WITH name = ?
+            CONNECT BY parentid = PRIOR artistid
+            ORDER BY name ASC
+          ) me
+        WHERE ROWNUM <= 2
+      )',
+      [ [ name => 'root' ] ],
+    );
+
+    is_deeply (
+      [ $rs->get_column ('name')->all ],
+      [qw/child1 child2/],
+      'LIMIT a Connect By query - correct names'
+    );
+
+    is_same_sql_bind (
+      $rs->count_rs->as_query,
+      '(
+        SELECT COUNT( * )
+          FROM (
+            SELECT artistid
+              FROM (
+                SELECT me.artistid
+                  FROM artist me
+                START WITH name = ?
+                CONNECT BY parentid = PRIOR artistid
+              ) me
+            WHERE ROWNUM <= 2
+          ) me
+      )',
+      [ [ name => 'root' ] ],
+    );
+
+    is( $rs->count, 2, 'Connect By; LIMIT count ok' );
+  }
+
+  # combine a connect_by with group_by and having
+  {
+    my $rs = $schema->resultset('Artist')->search({}, {
+      select => { count => 'rank', -as => 'cnt' },
+      start_with => { name => 'root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+      group_by => ['rank'],
+      having => \[ 'count(rank) < ?', [ cnt => 2 ] ],
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT COUNT(rank) AS cnt
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY parentid = PRIOR artistid
+        GROUP BY rank HAVING count(rank) < ?
+      )',
+      [ [ name => 'root' ], [ cnt => 2 ] ],
+    );
+
+    is_deeply (
+      [ $rs->get_column ('cnt')->all ],
+      [1, 1],
+      'Group By a Connect By query - correct values'
+    );
+  }
+
+  # select the whole cycle tree without nocylce
+  {
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'cycle-root' },
+      connect_by => { parentid => { -prior => { -ident => 'artistid' } } },
+    });
+
+    # ORA-01436:  CONNECT BY loop in user data
+    throws_ok { $rs->get_column ('name')->all } qr/ORA-01436/,
+      "connect by initify loop detection without nocycle";
+  }
+
+  # select the whole cycle tree with nocylce
+  SKIP: {
+    # http://download.oracle.com/docs/cd/A87860_01/doc/server.817/a85397/expressi.htm#1023748
+    skip q{Oracle8i doesn't support connect by nocycle}, 1
+      if $schema->storage->_server_info->{normalized_dbms_version} < 9;
+
+    my $rs = $schema->resultset('Artist')->search({}, {
+      start_with => { name => 'cycle-root' },
+      '+select'  => \ 'CONNECT_BY_ISCYCLE',
+      '+as'      => [ 'connector' ],
+      connect_by_nocycle => { parentid => { -prior => { -ident => 'artistid' } } },
+    });
+
+    is_same_sql_bind (
+      $rs->as_query,
+      '(
+        SELECT me.artistid, me.name, me.rank, me.charfield, me.parentid, CONNECT_BY_ISCYCLE
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY NOCYCLE parentid = PRIOR artistid 
+      )',
+      [ [ name => 'cycle-root'] ],
+    );
+    is_deeply (
+      [ $rs->get_column ('name')->all ],
+      [ qw/cycle-root cycle-child1 cycle-grandchild cycle-child2/ ],
+      'got artist tree with nocycle (name)',
+    );
+    is_deeply (
+      [ $rs->get_column ('connector')->all ],
+      [ qw/1 0 0 0/ ],
+      'got artist tree with nocycle (CONNECT_BY_ISCYCLE)',
+    );
+
+    is_same_sql_bind (
+      $rs->count_rs->as_query,
+      '(
+        SELECT COUNT( * )
+          FROM artist me
+        START WITH name = ?
+        CONNECT BY NOCYCLE parentid = PRIOR artistid 
+      )',
+      [ [ name => 'cycle-root'] ],
+    );
+
+    is( $rs->count, 4, 'Connect By Nocycle count ok' );
+  }
+}
+
+done_testing;
+
+sub do_creates {
+  my $dbh = shift;
+
+  eval {
+    $dbh->do("DROP SEQUENCE artist_autoinc_seq");
+    $dbh->do("DROP SEQUENCE artist_pk_seq");
+    $dbh->do("DROP SEQUENCE cd_seq");
+    $dbh->do("DROP SEQUENCE track_seq");
+    $dbh->do("DROP TABLE artist");
+    $dbh->do("DROP TABLE track");
+    $dbh->do("DROP TABLE cd");
+  };
+
+  $dbh->do("CREATE SEQUENCE artist_pk_seq START WITH 1 MAXVALUE 999999 MINVALUE 0");
+  $dbh->do("CREATE SEQUENCE cd_seq START WITH 1 MAXVALUE 999999 MINVALUE 0");
+  $dbh->do("CREATE SEQUENCE track_seq START WITH 1 MAXVALUE 999999 MINVALUE 0");
+
+  $dbh->do("CREATE TABLE artist (artistid NUMBER(12), parentid NUMBER(12), name VARCHAR(255), autoinc_col NUMBER(12), rank NUMBER(38), charfield VARCHAR2(10))");
+  $dbh->do("ALTER TABLE artist ADD (CONSTRAINT artist_pk PRIMARY KEY (artistid))");
+
+  $dbh->do("CREATE TABLE cd (cdid NUMBER(12), artist NUMBER(12), title VARCHAR(255), year VARCHAR(4), genreid NUMBER(12), single_track NUMBER(12))");
+  $dbh->do("ALTER TABLE cd ADD (CONSTRAINT cd_pk PRIMARY KEY (cdid))");
+
+  $dbh->do("CREATE TABLE track (trackid NUMBER(12), cd NUMBER(12) REFERENCES cd(cdid) DEFERRABLE, position NUMBER(12), title VARCHAR(255), last_updated_on DATE, last_updated_at DATE, small_dt DATE)");
+  $dbh->do("ALTER TABLE track ADD (CONSTRAINT track_pk PRIMARY KEY (trackid))");
+
+  $dbh->do(qq{
+    CREATE OR REPLACE TRIGGER artist_insert_trg_pk
+    BEFORE INSERT ON artist
+    FOR EACH ROW
+      BEGIN
+        IF :new.artistid IS NULL THEN
+          SELECT artist_pk_seq.nextval
+          INTO :new.artistid
+          FROM DUAL;
+        END IF;
+      END;
+  });
+  $dbh->do(qq{
+    CREATE OR REPLACE TRIGGER cd_insert_trg
+    BEFORE INSERT OR UPDATE ON cd
+    FOR EACH ROW
+
+      DECLARE
+      tmpVar NUMBER;
+
+      BEGIN
+        tmpVar := 0;
+
+        IF :new.cdid IS NULL THEN
+          SELECT cd_seq.nextval
+          INTO tmpVar
+          FROM dual;
+
+          :new.cdid := tmpVar;
+        END IF;
+      END;
+  });
+  $dbh->do(qq{
+    CREATE OR REPLACE TRIGGER track_insert_trg
+    BEFORE INSERT ON track
+    FOR EACH ROW
+      BEGIN
+        IF :new.trackid IS NULL THEN
+          SELECT track_seq.nextval
+          INTO :new.trackid
+          FROM DUAL;
+        END IF;
+      END;
+  });
+}
+
+# clean up our mess
+END {
+  eval {
+    my $dbh = $schema->storage->dbh;
+    $dbh->do("DROP SEQUENCE artist_pk_seq");
+    $dbh->do("DROP SEQUENCE cd_seq");
+    $dbh->do("DROP SEQUENCE track_seq");
+    $dbh->do("DROP TABLE artist");
+    $dbh->do("DROP TABLE track");
+    $dbh->do("DROP TABLE cd");
+  };
+}