Changeset 18858

Timestamp:

Aug 1, 2008, 1:55:26 PM (18 years ago)

Author:

jhoblitt

Message:

fix ->delete() still work if the neb implies a volume on which there is no instance

Location:

trunk/Nebulous

Files:

• Changes (modified) (1 diff)
• lib/Nebulous/Client.pm (modified) (2 diffs)
• t/59_client_delete.t (modified) (3 diffs)

trunk/Nebulous/Changes

@@ -2 +2 @@
 
 0.10
+    - fix ->delete() still work if the neb implies a volume on which there is
+      no instance
     - retry all calls to open()
 

trunk/Nebulous/lib/Nebulous/Client.pm

@@ -1 +1 @@
 # Copyright (c) 2004-2008  Joshua Hoblitt
 #
-# $Id: Client.pm,v 1.57 2008-07-10 23:21:27 jhoblitt Exp $
+# $Id: Client.pm,v 1.58 2008-08-01 23:55:26 jhoblitt Exp $
 
 package Nebulous::Client;
@@ -783 +783 @@
     $log->debug( "entered - @_" );
 
-    my $locations = $self->find_instances( $key );
+    my $locations = $self->find_instances( $key, 'any' );
 
     return undef unless $locations;

trunk/Nebulous/t/59_client_delete.t

@@ -3 +3 @@
 # Copryight (C) 2004-2005  Joshua Hoblitt
 #
-# $Id: 59_client_delete.t,v 1.1 2005-12-03 02:52:31 jhoblitt Exp $
+# $Id: 59_client_delete.t,v 1.2 2008-08-01 23:55:26 jhoblitt Exp $
 
 use strict;
@@ -10 +10 @@
 use Apache::Test qw( -withtestmore );
 
-plan tests => 7;
+plan tests => 9;
 
 use lib qw( ./t ./lib );
@@ -57 +57 @@
         proxy => "http://$hostport/nebulous",
     );
+    $neb->create( "neb://node01/foo" );
+
+    ok( $neb->delete( "neb://node02/foo" ), "delete object" );
+
+    my $locations = $neb->find_instances( "foo" );
+
+    is( $locations, undef, "no instances" );
+}
+
+Test::Nebulous->setup;
+
+{
+    my $neb = Nebulous::Client->new(
+        proxy => "http://$hostport/nebulous",
+    );
 
     is( $neb->delete( "foo" ), undef, "delete non-existant object" );