X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F02where.t;h=644eca5c8b1ecde7290bec4bcc97f117f993d943;hb=541af9148987e73b124229ee4f6b80d7c8d30e63;hp=42e19b56fdb5a22dbbe3ab39b54294f96c8dab80;hpb=a5e0ac94f214621462dd718cda341d4e65425a75;p=scpubgit%2FQ-Branch.git

diff --git a/t/02where.t b/t/02where.t
index 42e19b5..644eca5 100644
--- a/t/02where.t
+++ b/t/02where.t
@@ -1,7 +1,9 @@
 use strict;
 use warnings;
 use Test::More;
-use SQL::Abstract::Test import => [qw(is_same_sql_bind diag_where) ];
+use Test::Warn;
+use Test::Exception;
+use SQL::Abstract::Test import => [qw(is_same_sql_bind diag_where dumper) ];
 
 use SQL::Abstract;
 
@@ -9,6 +11,12 @@ my $not_stringifiable = bless {}, 'SQLA::NotStringifiable';
 
 my @handle_tests = (
     {
+        where => 'foo',
+        order => [],
+        stmt => ' WHERE foo',
+        bind => [],
+    },
+    {
         where => {
             requestor => 'inna',
             worker => ['nwiger', 'rcwe', 'sfz'],
@@ -132,6 +140,7 @@ my @handle_tests = (
         order => \'requestor, ticket',
         stmt => " WHERE ( ( priority BETWEEN ? AND ? ) AND requestor IS NULL ) ORDER BY requestor, ticket",
         bind => [qw/1 3/],
+        warns => qr/Supplying an undefined argument to 'LIKE' is deprecated/,
     },
 
 
@@ -157,6 +166,7 @@ my @handle_tests = (
                  },
         stmt => " WHERE ( ( ( foo NOT LIKE ? ) OR ( foo NOT LIKE ? ) OR ( foo NOT LIKE ? ) ) AND ( ( fum LIKE ? ) OR ( fum LIKE ? ) ) AND ( nix BETWEEN ? AND ? ) AND ( nox NOT BETWEEN ? AND ? ) AND wix IN ( ?, ? ) AND wux NOT IN ( ?, ? ) )",
         bind => [7,8,9,'a','b',100,200,150,160,'zz','yy','30','40'],
+        warns => qr/\QA multi-element arrayref as an argument to the inequality op 'NOT LIKE' is technically equivalent to an always-true 1=1/,
     },
 
     {
@@ -383,13 +393,29 @@ my @handle_tests = (
         stmt  => " WHERE ( (NOT ( c AND (NOT ( (NOT a = ?) AND (NOT b) )) )) ) ",
         bind => [ 1 ],
     },
+    {
+        where => \"0",
+        stmt  => " WHERE ( 0 ) ",
+        bind => [ ],
+    },
+    {
+        where => { artistid => {} },
+        stmt => '',
+        bind => [ ],
+    }
 );
 
 for my $case (@handle_tests) {
     my $sql = SQL::Abstract->new;
-    my ($stmt, @bind) = $sql->where($case->{where}, $case->{order});
+    my ($stmt, @bind);
+    lives_ok {
+      warnings_exist {
+        ($stmt, @bind) = $sql->where($case->{where}, $case->{order});
+      } $case->{warns} || [];
+    };
+
     is_same_sql_bind($stmt, \@bind, $case->{stmt}, $case->{bind})
-      || diag_where ( $case->{where} );
+      || do { diag_where ( $case->{where} ); diag dumper($sql->_expand_expr($case->{where})) };
 }
 
 done_testing;