fix: ensure the presence of basic auth credentials does not cause an error when displaying the path of a pact on the local filesystem

Author: bethesque

lib/pact/provider/pact_uri.rb

@@ -29,10 +29,10 @@ def password
       end
 
       def to_s
-        if(basic_auth?)
+        if basic_auth? && uri.start_with?('http://', 'https://')
           URI(@uri).tap { |x| x.userinfo="#{username}:*****"}.to_s
         else
-          @uri
+          uri
         end
       end
     end

spec/lib/pact/provider/pact_uri_spec.rb

@@ -30,6 +30,14 @@
       it 'should include user name and password' do
         expect(pact_uri.to_s).to eq('http://pact:*****@uri')
       end
+
+      context 'when basic auth credentials have been set for a local file (eg. via environment variables, unintentionally)' do
+        let(:uri) { '/some/file thing.json' }
+
+        it 'does not blow up' do
+          expect(pact_uri.to_s).to eq uri
+        end
+      end
     end
 
     context 'without userinfo' do