X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=rt%2Ft%2Fticket%2Fcfsort-freeform-single.t;h=262f84a229de35fcd0464169485773f0052b33cb;hb=940da07a1b4a80bab5b2b9c9b893bfaa350bc32d;hp=f1f506bea0986cffd527a72d186f2b02b5fcb5af;hpb=63a268637b2d51a8766412617724b9436439deb6;p=freeside.git

diff --git a/rt/t/ticket/cfsort-freeform-single.t b/rt/t/ticket/cfsort-freeform-single.t
index f1f506bea..262f84a22 100644
--- a/rt/t/ticket/cfsort-freeform-single.t
+++ b/rt/t/ticket/cfsort-freeform-single.t
@@ -1,191 +1,169 @@
-#!/usr/bin/perl
-
-use RT::Test tests => 57;
 
 use strict;
 use warnings;
 
-use RT::Tickets;
-use RT::Queue;
-use RT::CustomField;
-
-# Test Sorting by FreeformSingle custom field.
+use RT::Test nodata => 1, tests => undef;
 
-diag "Create a queue to test with." if $ENV{TEST_VERBOSE};
-my $queue_name = "CFSortQueue-$$";
-my $queue;
-{
-    $queue = RT::Queue->new( $RT::SystemUser );
-    my ($ret, $msg) = $queue->Create(
-        Name => $queue_name,
-        Description => 'queue for custom field sort testing'
-    );
-    ok($ret, "$queue test queue creation. $msg");
-}
+my $queue = RT::Test->load_or_create_queue( Name => "sorting" );
+ok $queue && $queue->id, "Created queue";
+my $queue_name = $queue->Name;
 
 # CFs for testing, later we create another one
-my %CF;
-my $cf_name;
-
-diag "create a CF\n" if $ENV{TEST_VERBOSE};
+my $cf;
+my $cf_name = "ordering";
+diag "create a CF";
 {
-    $cf_name = $CF{'CF'}{'name'} = "Order$$";
-    $CF{'CF'}{'obj'} = RT::CustomField->new( $RT::SystemUser );
-    my ($ret, $msg) = $CF{'CF'}{'obj'}->Create(
-        Name  => $CF{'CF'}{'name'},
+    $cf = RT::CustomField->new( RT->SystemUser );
+    my ($ret, $msg) = $cf->Create(
+        Name  => $cf_name,
         Queue => $queue->id,
         Type  => 'FreeformSingle',
     );
-    ok($ret, "Custom Field $CF{'CF'}{'name'} created");
+    ok($ret, "Custom Field created");
 }
 
-my ($total, @data, @tickets, @test) = (0, ());
-
-sub add_tix_from_data {
-    my @res = ();
-    @data = sort { rand(100) <=> rand(100) } @data;
-    while (@data) {
-        my $t = RT::Ticket->new($RT::SystemUser);
-        my %args = %{ shift(@data) };
-
-        my $subject = '-';
-        foreach my $e ( grep exists $CF{$_} && defined $CF{$_}, keys %args ) {
-            my @values = ();
-            if ( ref $args{ $e } ) {
-                @values = @{ delete $args{ $e } };
-            } else {
-                @values = (delete $args{ $e });
-            }
-            $args{ 'CustomField-'. $CF{ $e }{'obj'}->id } = \@values
-                if @values;
-            $subject = join(",", sort @values) || '-'
-                if $e eq 'CF';
-        }
-
-        my ( $id, undef $msg ) = $t->Create(
-            %args,
-            Queue => $queue->id,
-            Subject => $subject,
-        );
-        ok( $id, "ticket created" ) or diag("error: $msg");
-        push @res, $t;
-        $total++;
-    }
-    return @res;
-}
+run_tests(
+    [
+        { Subject => '-' },
+        { Subject => 'aa', 'CustomField-' . $cf->id => 'aa' },
+        { Subject => 'bb', 'CustomField-' . $cf->id => 'bb' },
+        { Subject => 'cc', 'CustomField-' . $cf->id => 'cc' },
+    ],
+    {                                    Count => 4, Order => "CF.{$cf_name}"             },
+    {                                    Count => 4, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_name} LIKE 'a'", Count => 1, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_name} LIKE 'a'", Count => 1, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_name} != 'cc'",  Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_name} != 'cc'",  Count => 3, Order => "CF.$queue_name.{$cf_name}" },
+);
 
-sub run_tests {
-    my $query_prefix = join ' OR ', map 'id = '. $_->id, @tickets;
-    foreach my $test ( @test ) {
-        my $query = join " AND ", map "( $_ )", grep defined && length,
-            $query_prefix, $test->{'Query'};
-
-        foreach my $order (qw(ASC DESC)) {
-            my $error = 0;
-            my $tix = RT::Tickets->new( $RT::SystemUser );
-            $tix->FromSQL( $query );
-            $tix->OrderBy( FIELD => $test->{'Order'}, ORDER => $order );
-
-            ok($tix->Count, "found ticket(s)")
-                or $error = 1;
-
-            my ($order_ok, $last) = (1, $order eq 'ASC'? '-': 'zzzzzz');
-            my $last_id = $tix->Last->id;
-            while ( my $t = $tix->Next ) {
-                my $tmp;
-                next if $t->id == $last_id and $t->Subject eq "-"; # Nulls are allowed to come last, in Pg
-
-                if ( $order eq 'ASC' ) {
-                    $tmp = ((split( /,/, $last))[0] cmp (split( /,/, $t->Subject))[0]);
-                } else {
-                    $tmp = -((split( /,/, $last))[-1] cmp (split( /,/, $t->Subject))[-1]);
-                }
-                if ( $tmp > 0 ) {
-                    $order_ok = 0; last;
-                }
-                $last = $t->Subject;
-            }
 
-            ok( $order_ok, "$order order of tickets is good" )
-                or $error = 1;
 
-            if ( $error ) {
-                diag "Wrong SQL query:". $tix->BuildSelectQuery;
-                $tix->GotoFirstItem;
-                while ( my $t = $tix->Next ) {
-                    diag sprintf "%02d - %s", $t->id, $t->Subject;
-                }
-            }
-        }
-    }
+my $other_cf;
+my $other_name = "othercf";
+diag "create another CF";
+{
+    $other_cf = RT::CustomField->new( RT->SystemUser );
+    my ($ret, $msg) = $other_cf->Create(
+        Name  => $other_name,
+        Queue => $queue->id,
+        Type  => 'FreeformSingle',
+    );
+    ok($ret, "Other Custom Field created");
 }
 
-@data = (
-    { },
-    { CF => 'a' },
-    { CF => 'b' },
-);
-@tickets = add_tix_from_data();
-@test = (
-    { Order => "CF.{$cf_name}" },
-    { Order => "CF.$queue_name.{$cf_name}" },
-);
-run_tests();
-
-@data = (
-    { },
-    { CF => 'aa' },
-    { CF => 'ab' },
-);
-@tickets = add_tix_from_data();
-@test = (
-    { Query => "CF.{$cf_name} LIKE 'a'", Order => "CF.{$cf_name}" },
-    { Query => "CF.{$cf_name} LIKE 'a'", Order => "CF.$queue_name.{$cf_name}" },
-);
-run_tests();
-
-@data = (
-    { Subject => '-', },
-    { Subject => 'a', CF => 'a' },
-    { Subject => 'b', CF => 'b' },
-    { Subject => 'c', CF => 'c' },
-);
-@tickets = add_tix_from_data();
-@test = (
-    { Query => "CF.{$cf_name} != 'c'", Order => "CF.{$cf_name}" },
-    { Query => "CF.{$cf_name} != 'c'", Order => "CF.$queue_name.{$cf_name}" },
+# Test that order is not affected by other CFs
+run_tests(
+    [
+        { Subject => '-', },
+        { Subject => 'aa', "CustomField-" . $cf->id => 'aa', "CustomField-" . $other_cf->id => 'za' },
+        { Subject => 'bb', "CustomField-" . $cf->id => 'bb', "CustomField-" . $other_cf->id => 'ya' },
+        { Subject => 'cc', "CustomField-" . $cf->id => 'cc', "CustomField-" . $other_cf->id => 'xa' },
+    ],
+    {                                      Count => 4, Order => "CF.{$cf_name}"             },
+    {                                      Count => 4, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_name} LIKE 'a'",   Count => 1, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_name} LIKE 'a'",   Count => 1, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_name} != 'cc'",    Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_name} != 'cc'",    Count => 3, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$other_name} != 'za'", Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$other_name} != 'za'", Count => 3, Order => "CF.$queue_name.{$cf_name}" },
 );
-run_tests();
-
-
 
-diag "create another CF\n" if $ENV{TEST_VERBOSE};
+# And then add a CF with a duplicate name, on a different queue
 {
-    $CF{'AnotherCF'}{'name'} = "OrderAnother$$";
-    $CF{'AnotherCF'}{'obj'} = RT::CustomField->new( $RT::SystemUser );
-    my ($ret, $msg) = $CF{'AnotherCF'}{'obj'}->Create(
-        Name  => $CF{'AnotherCF'}{'name'},
-        Queue => $queue->id,
+    my $other_queue = RT::Test->load_or_create_queue( Name => "other_queue" );
+    ok $other_queue && $other_queue->id, "Created queue";
+
+    my $dup = RT::CustomField->new( RT->SystemUser );
+    my ($ret, $msg) = $dup->Create(
+        Name  => $cf_name,
+        Queue => $other_queue->id,
         Type  => 'FreeformSingle',
     );
-    ok($ret, "Custom Field $CF{'AnotherCF'}{'name'} created");
+    ok($ret, "Custom Field created");
 }
 
-# test that order is not affect by other fields (had such problem)
-@data = (
-    { Subject => '-', },
-    { Subject => 'a', CF => 'a', AnotherCF => 'za' },
-    { Subject => 'b', CF => 'b', AnotherCF => 'ya' },
-    { Subject => 'c', CF => 'c', AnotherCF => 'xa' },
-);
-@tickets = add_tix_from_data();
-@test = (
-    { Order => "CF.{$cf_name}" },
-    { Order => "CF.$queue_name.{$cf_name}" },
-    { Query => "CF.{$cf_name} != 'c'", Order => "CF.{$cf_name}" },
-    { Query => "CF.{$cf_name} != 'c'", Order => "CF.$queue_name.{$cf_name}" },
+my $cf_id = $cf->id;
+run_tests(
+    [
+        { Subject => '-', },
+        { Subject => 'aa', "CustomField-" . $cf->id => 'aa', "CustomField-" . $other_cf->id => 'za' },
+        { Subject => 'bb', "CustomField-" . $cf->id => 'bb', "CustomField-" . $other_cf->id => 'ya' },
+        { Subject => 'cc', "CustomField-" . $cf->id => 'cc', "CustomField-" . $other_cf->id => 'xa' },
+    ],
+    {                                                Count => 4, Order => "CF.{$cf_name}"             },
+    {                                                Count => 4, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_id} LIKE 'a'",               Count => 1, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_id} LIKE 'a'",               Count => 1, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$cf_id} != 'cc'",                Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$cf_id} != 'cc'",                Count => 3, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.$queue_name.{$cf_name} LIKE 'a'", Count => 1, Order => "CF.{$cf_name}"             },
+    { Query => "CF.$queue_name.{$cf_name} LIKE 'a'", Count => 1, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.$queue_name.{$cf_name} != 'cc'",  Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.$queue_name.{$cf_name} != 'cc'",  Count => 3, Order => "CF.$queue_name.{$cf_name}" },
+    { Query => "CF.{$other_name} != 'za'",           Count => 3, Order => "CF.{$cf_name}"             },
+    { Query => "CF.{$other_name} != 'za'",           Count => 3, Order => "CF.$queue_name.{$cf_name}" },
+
+    { Query => "CF.{$cf_id} != 'cc'",                Count => 3, Order => "CF.{$cf_id}"               },
+    { Query => "CF.{$cf_id} != 'cc'",                Count => 3, Order => "CF.$queue_name.{$cf_id}"   },
+    { Query => "CF.$queue_name.{$cf_name} != 'cc'",  Count => 3, Order => "CF.{$cf_id}"               },
+    { Query => "CF.$queue_name.{$cf_name} != 'cc'",  Count => 3, Order => "CF.$queue_name.{$cf_id}"   },
+    { Query => "CF.{$other_name} != 'za'",           Count => 3, Order => "CF.{$cf_id}"               },
+    { Query => "CF.{$other_name} != 'za'",           Count => 3, Order => "CF.$queue_name.{$cf_id}"   },
 );
-run_tests();
 
+sub run_tests {
+    my $tickets = shift;
+    my @tickets = RT::Test->create_tickets( { Queue => $queue->id, RandomOrder => 1 }, @{ $tickets });
+    my $base_query = join(" OR ", map {"id = ".$_->id} @tickets) || "id > 0";
+
+    my @tests = @_;
+    for my $test ( @tests ) {
+        $test->{'Query'} ||= "id > 0";
+        my $query = "( $base_query ) AND " . $test->{'Query'};
+        for my $order (qw(ASC DESC)) {
+            subtest $test->{'Query'} . " ORDER BY ".$test->{'Order'}. " $order" => sub {
+                my $error = 0;
+                my $tix = RT::Tickets->new( RT->SystemUser );
+                $tix->FromSQL( $query );
+                $tix->OrderBy( FIELD => $test->{'Order'}, ORDER => $order );
+
+                is($tix->Count, $test->{'Count'}, "found right number of tickets (".$test->{Count}.")")
+                    or $error = 1;
+
+                my ($order_ok, $last) = (1, $order eq 'ASC'? '-': 'zzzzzz');
+                if ($tix->Count) {
+                    my $last_id = $tix->Last->id;
+                    while ( my $t = $tix->Next ) {
+                        my $tmp;
+                        next if $t->id == $last_id and $t->Subject eq "-"; # Nulls are allowed to come last, in Pg
+
+                        if ( $order eq 'ASC' ) {
+                            $tmp = ((split( /,/, $last))[0] cmp (split( /,/, $t->Subject))[0]);
+                        } else {
+                            $tmp = -((split( /,/, $last))[-1] cmp (split( /,/, $t->Subject))[-1]);
+                        }
+                        if ( $tmp > 0 ) {
+                            $order_ok = 0; last;
+                        }
+                        $last = $t->Subject;
+                    }
+                }
+
+                ok( $order_ok, "$order order of tickets is good" )
+                    or $error = 1;
 
+                if ( $error ) {
+                    diag "Wrong SQL query:". $tix->BuildSelectQuery;
+                    $tix->GotoFirstItem;
+                    while ( my $t = $tix->Next ) {
+                        diag sprintf "%02d - %s", $t->id, $t->Subject;
+                    }
+                }
+            };
+        }
+    }
+}
 
+done_testing;