projects / dbsrgits/DBIx-Class.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw
more has_relationship_loaded tests + fix for the failing tests
[dbsrgits/DBIx-Class.git] / t / 746mssql.t
diff --git a/t/746mssql.t b/t/746mssql.t
index b822138..57c44fb 100644 (file)
--- a/t/746mssql.t
+++ b/t/746mssql.t
@@ -4,14 +4,15 @@ use warnings;
 use Test::More;
 use Test::Exception;
 use Try::Tiny;
-use DBIx::Class::SQLMaker::LimitDialects;
+
 use DBIx::Class::Optional::Dependencies ();
+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');
+
 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');
+use DBIx::Class::SQLMaker::LimitDialects;
 
 my $OFFSET = DBIx::Class::SQLMaker::LimitDialects->__offset_bindtype;
 my $TOTAL  = DBIx::Class::SQLMaker::LimitDialects->__total_bindtype;
@@ -75,7 +76,7 @@ for my $opts_name (keys %opts) {
     }
     catch {
       if ($opts{$opts_name}{required}) {
-        BAIL_OUT "on_connect_call option '$opts_name' is not functional: $_";
+        die "on_connect_call option '$opts_name' is not functional: $_";
       }
       else {
         skip
@@ -372,10 +373,59 @@ SQL
           },
         );
 
-        my ($sql, @bind) = @${$owners->page(3)->as_query};
-        # not testing the SQL as it is quite different between top/rno
-        is_same_bind (
-          \@bind,
+        is_same_sql_bind (
+          $owners->page(3)->as_query,
+          $dialect eq 'Top'
+            ? '(
+              SELECT TOP 2147483647 [me].[id], [me].[name],
+                                    [books].[id], [books].[source], [books].[owner], [books].[title], [books].[price]
+                FROM (
+                  SELECT TOP 2147483647 [me].[id], [me].[name]
+                    FROM (
+                      SELECT TOP 3 [me].[id], [me].[name], [ORDER__BY__001]
+                        FROM (
+                          SELECT TOP 9 [me].[id], [me].[name], name + ? AS [ORDER__BY__001]
+                            FROM [owners] [me]
+                            LEFT JOIN [books] [books]
+                              ON [books].[owner] = [me].[id]
+                          WHERE [books].[id] IS NOT NULL AND [me].[name] != ?
+                          GROUP BY [me].[id], [me].[name]
+                          ORDER BY name + ? ASC, [me].[id]
+                        ) [me]
+                      ORDER BY [ORDER__BY__001] DESC, [me].[id] DESC
+                    ) [me]
+                  ORDER BY [ORDER__BY__001] ASC, [me].[id]
+                ) [me]
+                LEFT JOIN [books] [books]
+                  ON [books].[owner] = [me].[id]
+              WHERE [books].[id] IS NOT NULL AND [me].[name] != ?
+              ORDER BY name + ? ASC, [me].[id]
+            )'
+            : '(
+              SELECT TOP 2147483647 [me].[id], [me].[name],
+                                    [books].[id], [books].[source], [books].[owner], [books].[title], [books].[price]
+                FROM (
+                  SELECT TOP 2147483647 [me].[id], [me].[name]
+                    FROM (
+                      SELECT [me].[id], [me].[name],
+                             ROW_NUMBER() OVER( ORDER BY [ORDER__BY__001] ASC, [me].[id] ) AS [rno__row__index]
+                        FROM (
+                          SELECT [me].[id], [me].[name], name + ? AS [ORDER__BY__001]
+                            FROM [owners] [me]
+                            LEFT JOIN [books] [books]
+                              ON [books].[owner] = [me].[id]
+                          WHERE [books].[id] IS NOT NULL AND [me].[name] != ?
+                          GROUP BY [me].[id], [me].[name]
+                        ) [me]
+                    ) [me]
+                  WHERE [rno__row__index] >= ? AND [rno__row__index] <= ?
+                ) [me]
+                LEFT JOIN [books] [books]
+                  ON [books].[owner] = [me].[id]
+              WHERE [books].[id] IS NOT NULL AND [me].[name] != ?
+              ORDER BY name + ? ASC, [me].[id]
+            )'
+          ,
           [
             [ { dbic_colname => 'test' }
               => 'xxx' ],
@@ -392,7 +442,7 @@ SQL
             [ { dbic_colname => 'test' }
               => 'xxx' ],
           ],
-        );
+        ) if $quoted;
 
         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");
@@ -421,10 +471,61 @@ SQL
           },
         );
 
-        ($sql, @bind) = @${$books->page(3)->as_query};
-        # not testing the SQL as it is quite different between top/rno
-        is_same_bind (
-          \@bind,
+        is_same_sql_bind (
+          $books->page(3)->as_query,
+          $dialect eq 'Top'
+            ? '(
+              SELECT TOP 2147483647 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price],
+                                    [owner].[id], [owner].[name]
+                FROM (
+                  SELECT TOP 2147483647 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                    FROM (
+                      SELECT TOP 2 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                        FROM (
+                          SELECT TOP 6 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                            FROM [books] [me]
+                            JOIN [owners] [owner]
+                              ON [owner].[id] = [me].[owner]
+                          WHERE ( [owner].[name] = ? OR [owner].[name] = ? ) AND [source] = ?
+                          GROUP BY [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                          HAVING 1 = ?
+                          ORDER BY [me].[owner] DESC, [me].[id]
+                        ) [me]
+                      ORDER BY [me].[owner] ASC, [me].[id] DESC
+                    ) [me]
+                  ORDER BY [me].[owner] DESC, [me].[id]
+                ) [me]
+                JOIN [owners] [owner]
+                  ON [owner].[id] = [me].[owner]
+              WHERE ( [owner].[name] = ? OR [owner].[name] = ? ) AND [source] = ?
+              ORDER BY [me].[owner] DESC, [me].[id]
+            )'
+            : '(
+              SELECT TOP 2147483647 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price],
+                                    [owner].[id], [owner].[name]
+                FROM (
+                  SELECT TOP 2147483647 [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                    FROM (
+                      SELECT [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price],
+                             ROW_NUMBER() OVER( ORDER BY [me].[owner] DESC, [me].[id] ) AS [rno__row__index]
+                        FROM (
+                          SELECT [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                            FROM [books] [me]
+                            JOIN [owners] [owner]
+                              ON [owner].[id] = [me].[owner]
+                          WHERE ( [owner].[name] = ? OR [owner].[name] = ? ) AND [source] = ?
+                          GROUP BY [me].[id], [me].[source], [me].[owner], [me].[title], [me].[price]
+                          HAVING 1 = ?
+                        ) [me]
+                    ) [me]
+                  WHERE [rno__row__index] >= ? AND [rno__row__index] <= ?
+                ) [me]
+                JOIN [owners] [owner]
+                  ON [owner].[id] = [me].[owner]
+              WHERE ( [owner].[name] = ? OR [owner].[name] = ? ) AND [source] = ?
+              ORDER BY [me].[owner] DESC, [me].[id]
+            )'
+          ,
           [
             # inner
             [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' }
@@ -436,8 +537,12 @@ SQL
             [ { dbic_colname => 'test' }
               => '1' ],
 
-            # rno(?)
-            $dialect ne 'Top' ? ( [ $OFFSET => 5 ], [ $TOTAL => 6 ] ) : (),
+            # top(?)
+            $dialect eq 'Top'
+              ? ()
+              : ( [ $OFFSET => 5 ], [ $TOTAL => 6 ] )
+            ,
+
             # outer
             [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'owner.name' }
               => 'wiggle' ],
@@ -446,7 +551,7 @@ SQL
             [ { sqlt_datatype => 'varchar', sqlt_size => 100, dbic_colname => 'source' }
               => 'Library' ],
           ],
-        );
+        ) if $quoted;
 
         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");
The Perl ORM (dbsrgits@git.shadowcat.co.uk:DBIx-Class.git)