Fix fetched? check in PerformTaskJob

.rubocop.yml

@@ -56,6 +56,9 @@ Style/DoubleNegation:
 Style/Next:
   Enabled: false
 
+Style/EmptyMethod:
+  Enabled: false
+
 Metrics/LineLength:
   Exclude:
     - config/routes.rb

app/jobs/shipit/perform_task_job.rb

@@ -60,7 +60,7 @@ def perform_task
     end
 
     def checkout_repository
-      unless @commands.fetched?(@task.until_commit)
+      unless @commands.fetched?(@task.until_commit).tap(&:run).success?
         @task.acquire_git_cache_lock do
           capture! @commands.fetch
         end

test/jobs/perform_task_job_test.rb

@@ -2,6 +2,15 @@
 
 module Shipit
   class PerformTaskJobTest < ActiveSupport::TestCase
+    class FakeSuccessfulCommand
+      def run
+      end
+
+      def success?
+        true
+      end
+    end
+
     setup do
       @job = PerformTaskJob.new
       @deploy = shipit_deploys(:shipit_pending)
@@ -14,7 +23,7 @@ class PerformTaskJobTest < ActiveSupport::TestCase
       @commands = stub(:commands)
       Commands.expects(:for).with(@deploy).returns(@commands)
 
-      @commands.expects(:fetched?).once.returns(true)
+      @commands.expects(:fetched?).once.returns(FakeSuccessfulCommand.new)
       @commands.expects(:clone).returns([]).once
       @commands.expects(:checkout).with(@deploy.until_commit).once
       @commands.expects(:install_dependencies).returns([]).once