Fixes and tests for next/previous _sibling.

[dbsrgits/DBIx-Class.git] / t / run / 26positioned.tl

diff --git a/t/run/26positioned.tl b/t/run/26positioned.tl
index ad6a919..c73e257 100644 (file)
--- a/t/run/26positioned.tl
+++ b/t/run/26positioned.tl
@@ -2,48 +2,58 @@
 
 sub run_tests {
 
-    plan tests => 96;
+    plan tests => 56;
     my $schema = shift;
 
     my $employees = $schema->resultset('Employee::Positioned');
+    $employees->delete();
 
-    if ($employees->result_class->can('position_column')) {
-
-        $employees->delete();
-        foreach (1..5) {
-            $employees->create({ name=>'temp' });
-        }
+    foreach (1..5) {
+        $employees->create({ name=>'temp' });
+    }
+    $employees = $employees->search(undef,{order_by=>'position'});
+    ok( check_positions($employees), "$class: intial positions" );
 
-        $employees = $employees->search(undef,{order_by=>'position'});
-        ok( check_positions($employees), "$class: intial positions" );
+    my $employee;
 
-        my $employee;
+    foreach my $position (1..$employees->count()) {
 
-        foreach my $position (1..$employees->count()) {
+        $employee = $employees->find({ position=>$position });
+        $employee->move_previous();
+        ok( check_positions($employees), "$class: move_previous( $position )" );
 
-            $employee = $employees->find({ position=>$position });
-            $employee->move_previous();
-            ok( check_positions($employees), "$class: move_previous( $position )" );
+        $employee = $employees->find({ position=>$position });
+        $employee->move_next();
+        ok( check_positions($employees), "$class: move_next( $position )" );
 
-            $employee = $employees->find({ position=>$position });
-            $employee->move_next();
-            ok( check_positions($employees), "$class: move_next( $position )" );
+        $employee = $employees->find({ position=>$position });
+        $employee->move_first();
+        ok( check_positions($employees), "$class: move_first( $position )" );
 
-            $employee = $employees->find({ position=>$position });
-            $employee->move_first();
-            ok( check_positions($employees), "$class: move_first( $position )" );
+        $employee = $employees->find({ position=>$position });
+        $employee->move_last();
+        ok( check_positions($employees), "$class: move_last( $position )" );
 
+        foreach my $to_position (1..$employees->count()) {
             $employee = $employees->find({ position=>$position });
-            $employee->move_last();
-            ok( check_positions($employees), "$class: move_last( $position )" );
-
-            foreach my $to_position (1..$employees->count()) {
-                $employee = $employees->find({ position=>$position });
-                $employee->move_to($to_position);
-                ok( check_positions($employees), "$class: move_to( $position => $to_position )" );
-            }
+            $employee->move_to($to_position);
+            ok( check_positions($employees), "$class: move_to( $position => $to_position )" );
+        }
 
+        $employee = $employees->find({ position=>$position });
+        if ($position==1) {
+            ok( !$employee->previous_sibling(), 'no previous sibling' );
+            ok( $employee->next_sibling(), 'next sibling' );
+        }
+        elsif ($position==$employees->count()) {
+            ok( $employee->previous_sibling(), 'previous sibling' );
+            ok( !$employee->next_sibling(), 'no next sibling' );
         }
+        else {
+            ok( $employee->previous_sibling(), 'previous sibling' );
+            ok( $employee->next_sibling(), 'next sibling' );
+        }
+
     }
 }