Merge pull request #169 from pact-foundation/feat/do_not_retry_in_dry…

…_run_mode

feat(can-i-deploy): skip --retry-while-unknown if --dry-run true

lib/pact_broker/client/can_i_deploy.rb

@@ -109,7 +109,7 @@ def fetch_matrix
 
       def fetch_matrix_with_retries
         matrix = fetch_matrix
-        if retry_while_unknown?
+        if retry_while_unknown? && !dry_run_or_false
           check_if_retry_while_unknown_supported(matrix)
           if matrix.any_unknown?
             results = matrix.unknown_count == 1 ? "result" : "results"

spec/lib/pact_broker/client/can_i_deploy_spec.rb

@@ -163,9 +163,9 @@ module Client
               expect(subject.success).to be true
             end
 
-            it "returns a failure message" do
+            it "returns a success message" do
               expect(subject.message).to include "[dry-run]"
-              expect(subject.message).to match /does not provide a count/
+              expect(subject.message).to match /Dry run enabled - ignoring any failures/
             end
           end
         end