Cope with $record not set at all (though IIRC this should never

[irspy-moved-to-github.git] / lib / ZOOM / IRSpy / Test / Record / Fetch.pm

diff --git a/lib/ZOOM/IRSpy/Test/Record/Fetch.pm b/lib/ZOOM/IRSpy/Test/Record/Fetch.pm
index 049e169..320f9ca 100644 (file)
--- a/lib/ZOOM/IRSpy/Test/Record/Fetch.pm
+++ b/lib/ZOOM/IRSpy/Test/Record/Fetch.pm
@@ -1,4 +1,4 @@
-# $Id: Fetch.pm,v 1.19 2007-01-29 17:24:51 mike Exp $
+# $Id: Fetch.pm,v 1.26 2007-02-24 01:27:20 mike Exp $
 
 # See the "Main" test package for documentation
 
@@ -29,8 +29,8 @@ sub start {
     # managed to search for in the Search/Bib1 or Search/Dan1 tests.
     # But how?  So far we search for title: 1=4
     $conn->irspy_search_pqf($queries[0], { queryindex => 0 }, {},
-                           ZOOM::Event::RECV_SEARCH, \&completed_search,
-                           exception => \&search_error);
+                           ZOOM::Event::ZEND, \&completed_search,
+                           exception => \&completed_search);
 }
 
 
@@ -38,19 +38,20 @@ sub completed_search {
     my($conn, $task, $udata, $event) = @_;
 
     my $n = $task->{rs}->size();
-    $conn->log("irspy_test", "Fetch test search found $n records");
+    $conn->log("irspy_test", "Fetch test search (", $task->{query}, ") ",
+              ref $event && $event->isa("ZOOM::Exception") ?
+              "failed: $event" : "found $n records (event=$event)");
     if ($n == 0) {
-       my $n = $udata->{queryindex}+1;
-       my $q = $queries[$n];
-       if (defined $q) {
-           $conn->log("irspy_test", "Trying another search ...");
-           $conn->irspy_search_pqf($queries[$n], { queryindex => $n }, {},
-                                   ZOOM::Event::RECV_SEARCH, \&completed_search,
-                                   exception => \&search_error);
-           return ZOOM::IRSpy::Status::TASK_DONE;
-       } else {
-           return ZOOM::IRSpy::Status::TEST_SKIPPED;
-       }
+       my $qindex = $udata->{queryindex}+1;
+       my $q = $queries[$qindex];
+       return ZOOM::IRSpy::Status::TEST_SKIPPED
+           if !defined $q;
+
+       $conn->log("irspy_test", "Trying another search ...");
+       $conn->irspy_search_pqf($queries[$qindex], { queryindex => $qindex }, {},
+                               ZOOM::Event::ZEND, \&completed_search,
+                               exception => \&completed_search);
+       return ZOOM::IRSpy::Status::TASK_DONE;
     }
 
     my @syntax = (
@@ -63,7 +64,7 @@ sub completed_search {
                    'librismarc',
                    'mab',
                    'normarc',
-                   'opac',
+#                   'opac',
                    'picamarc',
                    'rusmarc',
                    'summary',
@@ -82,7 +83,7 @@ sub completed_search {
                                 last => ($i == $#syntax) },
                               { start => 0, count => 1,
                                 preferredRecordSyntax => $syntax },
-                                ZOOM::Event::RECV_RECORD, \&record,
+                                ZOOM::Event::ZEND, \&record,
                                exception => \&fetch_error);
     }
 
@@ -97,9 +98,9 @@ sub record {
 
     my $record = _fetch_record($rs, 0, $syn);
     my $ok = 0;
-    if ($record->error()) {
+    if (!$record || $record->error()) {
        $conn->log("irspy_test", "retrieval of $syn record failed: ",
-                  $record->exception());
+                  defined $record ? $record->exception() : "???");
     } else {
        $ok = 1;
        my $text = $record->render();
@@ -133,7 +134,7 @@ sub _fetch_record {
 }
 
 
-sub search_error {
+sub __UNUSED_search_error {
     my($conn, $task, $test_args, $exception) = @_;
 
     $conn->log("irspy_test", "Initial search failed: ", $exception);