[Mezuro] Remove project only if it exists on service

Carlos Morais + Paulo Meirelles · Carlos Morais
1 parent 2058c115
Showing 2 changed files with 12 additions and 2 deletions Show diff stats
plugins/mezuro/lib/kalibro/client/project_client.rb
plugins/mezuro/test/unit/kalibro/client/project_client_test.rb
@@ -10,7 +10,10 @@ class Kalibro::Client::ProjectClient
   end
   def self.remove(project_name)
-    new.remove(project_name)
+    instance = new
+    if (instance.project_names.include?(project_name))
+      instance.remove(project_name)
+    end
   end
   def self.create_project (project_content)
@@ -24,11 +24,18 @@ class ProjectClientTest &lt; ActiveSupport::TestCase
     Kalibro::Client::ProjectClient.save(@project_content)
   end
-  should 'remove project by name' do
+  should 'remove existent project from service' do
+    @port.expects(:request).with(:get_project_names).returns({:project_name => @project.name})
     @port.expects(:request).with(:remove_project, {:project_name => @project.name})
     Kalibro::Client::ProjectClient.remove(@project.name)
   end
+  should 'not try to remove inexistent project from service' do
+    @port.expects(:request).with(:get_project_names).returns({:project_name => 'Different project'})
+    @port.expects(:request).with(:remove_project, {:project_name => @project.name}).never
+    Kalibro::Client::ProjectClient.remove(@project.name)
+  end
+
   private
   def create_project_content_mock