Merge pull request #585 from liggitt/bug_1090912_handle_global_team_d…

…elete_error

Merged by openshift-bot

lib/rhc/rest/mock.rb

@@ -698,6 +698,7 @@ def initialize(client, name, id="123")
     end
 
     def destroy
+      raise RHC::OperationNotSupportedException.new("The server does not support deleting this resource.") unless supports? 'DELETE'
       client.teams.delete_if { |t| t.name == @name }
     end
 

lib/rhc/rest/team.rb

@@ -20,6 +20,7 @@ def to_s
 
       def destroy(force=false)
         debug "Deleting team #{name} (#{id})"
+        raise RHC::OperationNotSupportedException.new("The server does not support deleting this resource.") unless supports? 'DELETE'
         rest_method "DELETE"
       end
       alias :delete :destroy

spec/rhc/commands/team_spec.rb

@@ -162,6 +162,20 @@
       it { run_output.should match("Team with name deleteme not found") }
     end
 
+    context 'when the server does not allow deleting' do
+      before do
+        t = rest_client.add_team("dontdelete")
+        t.links.delete 'DELETE'
+        t.id = "123"
+      end
+      let(:arguments) { ['team','delete','--team-id','123'] }
+      it "should error out" do
+        expect { run }.to exit_with_code(1)
+        rest_client.teams[0].name.should == 'dontdelete'
+      end
+      it { run_output.should match(/not support/) }
+    end
+
   end
 
   describe 'help' do