fix(basic auth): add missing basic auth credentials for retriving pac…

…ts by URL Closes: #42
pact-foundation · Nov 10, 2019 · e84f9b8 · e84f9b8
1 parent 72b0541
commit e84f9b8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/pact/provider_verifier/aggregate_pact_configs.rb b/lib/pact/provider_verifier/aggregate_pact_configs.rb
@@ -28,7 +28,7 @@ def call
       attr_reader :pact_urls, :provider_name, :consumer_version_tags, :provider_version_tags, :pact_broker_base_url, :http_client_options
 
       def specified_pact_uris
-        pact_urls.collect{ | url | Pact::PactBroker.build_pact_uri(url) }
+        pact_urls.collect{ | url | Pact::PactBroker.build_pact_uri(url, http_client_options) }
       end
 
       def pacts_urls_from_broker

diff --git a/spec/lib/pact/provider_verifier/aggregate_pact_configs_spec.rb b/spec/lib/pact/provider_verifier/aggregate_pact_configs_spec.rb
@@ -37,6 +37,11 @@ module ProviderVerifier
           it "returns the hardcoded urls" do
             expect(subject).to eq [OpenStruct.new(uri: "http://pact-1")]
           end
+
+          it "uses the basic auth credentials to retrieve the pact" do
+            expect(pact_broker_api).to receive(:build_pact_uri).with(pact_urls.first, http_client_options)
+            subject
+          end
         end
 
         context "with broker config" do