Add support for NULLS FIRST/LAST in ORDER BY

[dbsrgits/SQL-Abstract.git] / t / 06order_by.t

diff --git a/t/06order_by.t b/t/06order_by.t
index 440517f..2b3b13f 100644 (file)
--- a/t/06order_by.t
+++ b/t/06order_by.t
@@ -3,6 +3,7 @@
 use strict;
 use warnings;
 use Test::More;
+use Test::Exception;
 
 use SQL::Abstract;
 
@@ -12,7 +13,7 @@ my @cases =
    {
     given => \'colA DESC',
     expects => ' ORDER BY colA DESC',
-    expects_quoted => ' ORDER BY colA DESC',
+    expects_quoted => ' ORDER BY `colA` DESC',
    },
    {
     given => 'colA',
@@ -30,14 +31,14 @@ my @cases =
     expects_quoted => ' ORDER BY `colA`, `colB`',
    },
    {  # it may look odd, but this is the desired behaviour (mst)
-    given => ['colA ASC', 'colB DESC'],
-    expects => ' ORDER BY colA ASC, colB DESC',
-    expects_quoted => ' ORDER BY `colA ASC`, `colB DESC`',
+    given => ['colA', 'colB DESC'],
+    expects => ' ORDER BY colA, colB DESC',
+    expects_quoted => ' ORDER BY `colA`, `colB DESC`',
    },
    {
     given => {-asc => 'colA'},
-    expects => ' ORDER BY colA ASC',
-    expects_quoted => ' ORDER BY `colA` ASC',
+    expects => ' ORDER BY colA',
+    expects_quoted => ' ORDER BY `colA`',
    },
    {
     given => {-desc => 'colB'},
@@ -46,8 +47,8 @@ my @cases =
    },
    {
     given => [{-asc => 'colA'}, {-desc => 'colB'}],
-    expects => ' ORDER BY colA ASC, colB DESC',
-    expects_quoted => ' ORDER BY `colA` ASC, `colB` DESC',
+    expects => ' ORDER BY colA, colB DESC',
+    expects_quoted => ' ORDER BY `colA`, `colB` DESC',
    },
    {
     given => ['colA', {-desc => 'colB'}],
@@ -59,15 +60,113 @@ my @cases =
     expects => '',
     expects_quoted => '',
    },
+
+   {
+    given => [{-desc => [ qw/colA colB/ ] }],
+    expects => ' ORDER BY colA DESC, colB DESC',
+    expects_quoted => ' ORDER BY `colA` DESC, `colB` DESC',
+   },
+   {
+    given => [{-desc => [ qw/colA colB/ ] }, {-asc => 'colC'}],
+    expects => ' ORDER BY colA DESC, colB DESC, colC',
+    expects_quoted => ' ORDER BY `colA` DESC, `colB` DESC, `colC`',
+   },
+   {
+    given => [{-desc => [ qw/colA colB/ ] }, {-asc => [ qw/colC colD/ ] }],
+    expects => ' ORDER BY colA DESC, colB DESC, colC, colD',
+    expects_quoted => ' ORDER BY `colA` DESC, `colB` DESC, `colC`, `colD`',
+   },
+   {
+    given => [{-desc => [ qw/colA colB/ ] }, {-desc => 'colC' }],
+    expects => ' ORDER BY colA DESC, colB DESC, colC DESC',
+    expects_quoted => ' ORDER BY `colA` DESC, `colB` DESC, `colC` DESC',
+   },
+   {
+    given => [{ -asc => 'colA' }, { -desc => [qw/colB/] }, { -asc => [qw/colC colD/] }],
+    expects => ' ORDER BY colA, colB DESC, colC, colD',
+    expects_quoted => ' ORDER BY `colA`, `colB` DESC, `colC`, `colD`',
+   },
+   {
+    given => { -desc => \['colA LIKE ?', 'test'] },
+    expects => ' ORDER BY colA LIKE ? DESC',
+    expects_quoted => ' ORDER BY colA LIKE ? DESC',
+    bind => ['test'],
+   },
+   {
+    given => \['colA LIKE ? DESC', 'test'],
+    expects => ' ORDER BY colA LIKE ? DESC',
+    expects_quoted => ' ORDER BY colA LIKE ? DESC',
+    bind => ['test'],
+   },
+   {
+    given => [ { -asc => \['colA'] }, { -desc => \['colB LIKE ?', 'test'] }, { -asc => \['colC LIKE ?', 'tost'] }],
+    expects => ' ORDER BY colA, colB LIKE ? DESC, colC LIKE ?',
+    expects_quoted => ' ORDER BY colA, colB LIKE ? DESC, colC LIKE ?',
+    bind => [qw/test tost/],
+   },
+   {
+    given => [ { -asc => 'colA', -nulls => 'first' }, { -desc => 'colB', -nulls => 'last' } ],
+    expects => ' ORDER BY colA NULLS FIRST, colB DESC NULLS LAST',
+    expects_quoted => ' ORDER BY `colA` NULLS FIRST, `colB` DESC NULLS LAST',
+   },
+   {
+    given => [ { -asc => 'colA', -nulls => 'first' }, { -desc => 'colB', -nulls => 'last' } ],
+    expects => ' ORDER BY colA NULLS FIRST, colB DESC NULLS LAST',
+    expects_quoted => ' ORDER BY `colA` NULLS FIRST, `colB` DESC NULLS LAST',
+   },
+   {
+    given => { -asc => [qw/colA colB/], -nulls => 'first' } ,
+    expects => ' ORDER BY colA NULLS FIRST, colB NULLS FIRST',
+    expects_quoted => ' ORDER BY `colA` NULLS FIRST, `colB` NULLS FIRST',
+   },
   );
 
 
-plan tests => (scalar(@cases) * 2);
+plan tests => (scalar(@cases) * 2) + 4;
 
 my $sql  = SQL::Abstract->new;
 my $sqlq = SQL::Abstract->new({quote_char => '`'});
 
-for my $case( @cases){
-  is($sql->_order_by($case->{given}), $case->{expects});
-  is($sqlq->_order_by($case->{given}), $case->{expects_quoted});
+for my $case( @cases) {
+  my ($stat, @bind);
+
+  ($stat, @bind) = $sql->_order_by($case->{given});
+  is_same_sql_bind (
+    $stat,
+    \@bind,
+    $case->{expects},
+    $case->{bind} || [],
+  );
+
+  ($stat, @bind) = $sqlq->_order_by($case->{given});
+  is_same_sql_bind (
+    $stat,
+    \@bind,
+    $case->{expects_quoted},
+    $case->{bind} || [],
+  );
 }
+
+throws_ok (
+  sub { $sql->_order_by({-desc => 'colA', -asc => 'colB' }) },
+  qr/hash passed .+ must have exactly one of/,
+  'Undeterministic order exception',
+);
+
+throws_ok (
+  sub { $sql->_order_by({-desc => [ qw/colA colB/ ], -asc => [ qw/colC colD/ ] }) },
+  qr/hash passed .+ must have exactly one of/,
+  'Undeterministic order exception',
+);
+
+throws_ok(
+  sub { $sql->_order_by({-wibble => "fleem" }) },
+  qr/invalid key in hash/,
+  'Invalid order exception',
+);
+
+throws_ok(
+  sub { $sql->_order_by({-nulls => "fleem" }) },
+  qr/invalid value for -nulls/,
+  'Invalid nulls exception',
+);