Correct Red Hat connection-failure checks -- addinfo is empty when errcode==TIMEOUT
authorMike Taylor <mike@miketaylor.org.uk>
Tue, 8 Jun 2010 14:32:33 +0000 (15:32 +0100)
committerMike Taylor <mike@miketaylor.org.uk>
Tue, 8 Jun 2010 14:32:33 +0000 (15:32 +0100)
t/1-Net-Z3950-ZOOM.t
t/2-ZOOM.t

index 27c479e..1e85894 100644 (file)
@@ -20,8 +20,8 @@ my $host = "no.such.host";
 my $conn = Net::Z3950::ZOOM::connection_new($host, 0);
 $errcode = Net::Z3950::ZOOM::connection_error($conn, $errmsg, $addinfo);
 # For some reason, Red Hat signals this as a TIMEOUT rather than a CONNECT
-ok(($errcode == Net::Z3950::ZOOM::ERROR_CONNECT ||
-    $errcode == Net::Z3950::ZOOM::ERROR_TIMEOUT) && $addinfo eq $host,
+ok(($errcode == Net::Z3950::ZOOM::ERROR_CONNECT && $addinfo eq $host) ||
+   ($errcode == Net::Z3950::ZOOM::ERROR_TIMEOUT && $addinfo eq ""),
    "connection to non-existent host '$host' fails: errcode=$errcode, addinfo=$addinfo");
 
 $host = "z3950.indexdata.com/gils";
index aaf867d..ada3ea4 100644 (file)
@@ -19,8 +19,8 @@ my $conn;
 eval { $conn = new ZOOM::Connection($host, 0) };
 # For some reason, Red Hat signals this as a TIMEOUT rather than a CONNECT
 ok($@ && $@->isa("ZOOM::Exception") &&
-   ($@->code() == ZOOM::Error::CONNECT ||
-    $@->code() == ZOOM::Error::TIMEOUT) && $@->addinfo() eq $host,
+   (($@->code() == ZOOM::Error::CONNECT && $@->addinfo() eq $host) ||
+    ($@->code() == ZOOM::Error::TIMEOUT && $@->addinfo() eq "")),
    "connection to non-existent host '$host' fails: \$\@=$@");
 
 $host = "z3950.indexdata.com/gils";