--- trunk/t/5_Node.t	2006/01/26 15:29:20	92
+++ trunk/t/5_Node.t	2006/01/28 17:38:00	94
@@ -126,19 +126,22 @@
 # user doesn't exist
 ok(! $node->set_user('foobar', 1), 'set_user');
 
-ok(my $node1 = new Search::Estraier::Node( "http://localhost:1978/node/$test2_node" ), "new $test2_node");
-ok($node1->set_auth('admin','admin'), "set_auth $test2_node");
+ok(my $node2 = new Search::Estraier::Node( "http://localhost:1978/node/$test2_node" ), "new $test2_node");
+ok($node2->set_auth('admin','admin'), "set_auth $test2_node");
 
 # croak_on_error
 
-ok(my $node2 = new Search::Estraier::Node( url => "http://localhost:1978/non-existant", croak_on_error => 1 ), "new non-existant");
-throws_ok { $node2->name } qr/404/, 'croak on error';
+ok($node = new Search::Estraier::Node( url => "http://localhost:1978/non-existant", croak_on_error => 1 ), "new non-existant");
+throws_ok { $node->name } qr/404/, 'croak on error';
+
+# support for undef and 0 values
+#ok($node = new Search::Estraier::Node( url => "http://localhost:1978/$test1_node", croak_on_error => 1 ), "new $test1_node");
 
 SKIP: {
-	skip "no $test2_node in Hyper Estraier, skipping set_link", 2 unless (my $test1_label = $node1->label);
+	skip "no $test2_node in Hyper Estraier, skipping set_link", 2 unless (my $test2_label = $node2->label);
 
-	ok($node->set_link("http://localhost:1978/node/$test2_node", $test1_label, 42), "set_link $test2_node ($test1_label) 42");
-	ok($node->set_link("http://localhost:1978/node/$test2_node", $test1_label, 0), "set_link $test2_node ($test1_label) delete");
+	ok($node->set_link("http://localhost:1978/node/$test2_node", $test2_label, 42), "set_link $test2_node ($test2_label) 42");
+	ok($node->set_link("http://localhost:1978/node/$test2_node", $test2_label, 0), "set_link $test2_node ($test2_label) delete");
 }	# SKIP 2
 
 }	# SKIP 1