test suite add_biblios: don't use zebraqueue_daemon
authorGalen Charlton <galen.charlton@liblime.com>
Tue, 1 Jul 2008 17:36:29 +0000 (12:36 -0500)
committerJoshua Ferraro <jmf@liblime.com>
Wed, 2 Jul 2008 17:26:03 +0000 (12:26 -0500)
The add_biblios() routine now reindexes all bibs
in batch instead of waiting for zebraqueue_daemon - this
is moderately faster.  A separate set of test
cases for zebraqueue_deamon will be witten.

No documentation changes.

Signed-off-by: Andrew Moore <andrew.moore@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
t/lib/KohaTest.pm

index 8c857c9..8ff9fd0 100644 (file)
@@ -435,27 +435,15 @@ sub add_biblios {
         }
         push @{$self->{'biblios'}}, $biblionumber;
     }
-    
+   
+    $self->reindex_marc(); 
     my $query = 'Finn Test';
-
-    # XXX we're going to repeatedly try to fetch the marc records that
-    # we inserted above. It may take a while before they all show
-    # up. why?
-    my $tries = 30;
-    DELAY: foreach my $trial ( 1..$tries ) {
-        diag "waiting for zebra indexing. Trial: $trial of $tries";
-        my ( $error, $results ) = SimpleSearch( $query );
-        if ( $param{'count'} <= scalar( @$results ) ) {
-            ok( $tries, "found all $param{'count'} titles after $trial tries" );
-            last DELAY;
-        }
-        sleep( 3 );
-    } continue {
-        if ( $trial == $tries ) {
-            fail( "we never found all $param{'count'} titles even after $tries tries." );
-        }
+    my ( $error, $results ) = SimpleSearch( $query );
+    if ( $param{'count'} <= scalar( @$results ) ) {
+        pass( "found all $param{'count'} titles" );
+    } else {
+        fail( "we never found all $param{'count'} titles" );
     }
-
     
 }