[Rt-commit] rt branch, 5.0/configurable-linked-queue-portlets, repushed

[Blaine Motsinger]

The branch 5.0/configurable-linked-queue-portlets was deleted and repushed:
       was 50a23b198e0d57a492efe183fdab798c90666e36
       now dc43b873c9faf3a57696d201f3915f645e63c739

1: 53f5f99534 = 1: 53f5f99534 Add a configurable queue portlet to ticket display page
2: 6927815148 = 2: 6927815148 Add docs for the new Linked Queue Portlets configs
3: 50a23b198e ! 3: dc43b873c9 Add test for Linked Queue Portlets
    @@ -15,15 +15,25 @@
     +# create ticket in general queue
     +# this ticket will display the portlet for the other queue, with the other ticket in it
     +my $queue_one  = RT::Test->load_or_create_queue( Name => 'General' );
    -+my $ticket_one = RT::Test->create_ticket( Subject => 'test ticket in General queue', Queue => $queue_one->Id );
    ++my $ticket_one = RT::Test->create_ticket(
    ++    Subject => 'test ticket in General queue',
    ++    Queue   => $queue_one->Id
    ++);
     +ok( $ticket_one && $ticket_one->Id, 'created ticket in "General" queue' );
     +
     +# create test queue and test ticket in it
     +my $queue_name = 'test queue';
    -+my $queue_two  = RT::Test->load_or_create_queue( Name => $queue_name, Description => $queue_name );
    ++my $queue_two  = RT::Test->load_or_create_queue(
    ++    Name        => $queue_name,
    ++    Description => $queue_name
    ++);
     +ok( $queue_two && $queue_two->Id, 'created "test queue" queue' );
    -+my $ticket_two = RT::Test->create_ticket( Subject => 'test ticket in "' . $queue_name . '" queue', Queue => $queue_two->Id );
    -+ok( $ticket_two && $ticket_two->Id, 'created ticket in "' . $queue_name . '" queue' );
    ++my $ticket_two = RT::Test->create_ticket(
    ++    Subject => 'test ticket in "' . $queue_name . '" queue',
    ++    Queue   => $queue_two->Id
    ++);
    ++ok( $ticket_two && $ticket_two->Id,
    ++    'created ticket in "' . $queue_name . '" queue' );
     +
     +# change config to load new queue portlet in general
     +# this isn't exercising limiting to a specific link relationship set such as 'HasMember', 'MemberOf', or 'RefersTo'; just 'All'
    @@ -41,18 +51,17 @@
     +
     +my $linked_queue_class = 'linked-queue';
     +$m->get_ok( "/Ticket/Display.html?id=" . $ticket_one->Id );
    -+$m->content_contains( $linked_queue_class, 'ticket in "General" queue contains linked queue portlet for "' . $queue_name . '" queue' );
    ++$m->content_contains( $linked_queue_class,
    ++    'ticket in "General" queue contains linked queue portlet for "' . $queue_name . '" queue' );
     +
     +# link tickets so the ticket shows up in the linked queue portlet
    -+ok( $ticket_one->AddLink( Type => 'RefersTo', Target => $ticket_two->Id ), 'linked ticket in "General" queue to ticket in "' . $queue_name . '" queue' );
    ++ok( $ticket_one->AddLink( Type => 'RefersTo', Target => $ticket_two->Id ),
    ++    'linked ticket in "General" queue to ticket in "' . $queue_name . '" queue' );
     +
     +$m->get_ok( "/Ticket/Display.html?id=" . $ticket_one->Id );
    -+my $ticket_two_link = 'Ticket\/Display\.html\?id=' . $ticket_two->Id;
    -+like(
    -+    $m->dom->at( 'div.ticket-info-links.' . $linked_queue_class )->content,
    -+    qr/$ticket_two_link/,
    -+    'linked queue portlet contains link to ticket in "' . $queue_name . '" queue'
    -+);
    ++is( $m->dom->find(".$linked_queue_class .collection-as-table a")->first->attr('href'),
    ++    '/Ticket/Display.html?id=' . $ticket_two->Id,
    ++    'linked queue portlet contains link to ticket in "' . $queue_name . '" queue' );
     +
     +# TODO:
     +# limit the linked queue configuration to only specific link relationships to ensure only those tickets show up in the portlet