pact_broker 2.27.5 → 2.27.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9df632a63f0e302e71f2d85b1ad246dc882da7b5
-  data.tar.gz: 0a3e61d43a525092db4417186cf4196369f33397
+  metadata.gz: 0e90caf2f7b3f95a72409e71d887025c4af961ca
+  data.tar.gz: c7e7ffdac02f644178308f80420500dc5729e77a
 SHA512:
-  metadata.gz: 12719772681d2fc8f601f1fcf691479e2a14c38b70e6c6c9955e8f5ac6d077f8add8a9cb25c64f3f916c1a0d76a17114f37b90e029e4b8ee41e27e6a9c4ca9af
-  data.tar.gz: 1ae584eaf3a67581b46a2a88349d3cc1d5b80e92d9e3cab0d7d29a7a8e22e55ec9b76632601f12f4c507ffec46b891f766207c6d2e3b371f3394d01fdb08650b
+  metadata.gz: 334d5d897be6dd5886cdf761d4d6cbad10ab46f23acca55e05736b91e9ba651c40347b99e051a04451380135572c868bf829afd099362e30a066c72fc8ca17d3
+  data.tar.gz: e92686f872dc74ab1ded642e4a76c63eb39520bd77549d543416c8e3dce7e84ba7703beacf52f798ab306d7f5ce95cca7f965f9e7999df2f9aedf67372db6776

data/CHANGELOG.md

@@ -1,3 +1,12 @@
+<a name="v2.27.6"></a>
+### v2.27.6 (2018-11-23)
+
+
+#### Bug Fixes
+
+* URL encode consumer version number in pact URL	 ([b58f4fbf](/../../commit/b58f4fbf))
+
+
 <a name="v2.27.5"></a>
 ### v2.27.5 (2018-11-15)
 

data/lib/pact_broker/api/pact_broker_urls.rb

@@ -47,7 +47,7 @@ module PactBroker
       end
 
       def pact_url base_url, pact
-        "#{pactigration_base_url(base_url, pact)}/version/#{pact.consumer_version_number}"
+        "#{pactigration_base_url(base_url, pact)}/version/#{url_encode(pact.consumer_version_number)}"
       end
 
       def pact_version_url pact, base_url = ''

data/lib/pact_broker/version.rb

@@ -1,3 +1,3 @@
 module PactBroker
-  VERSION = '2.27.5'
+  VERSION = '2.27.6'
 end

data/spec/lib/pact_broker/api/pact_broker_urls_spec.rb

@@ -8,39 +8,47 @@ module PactBroker
       # \/\{[^\}\s\[\(\.]+\}
 
       let(:base_url) { "http://example.org" }
-      let(:pact) { double('pact', consumer: consumer, provider: provider, consumer_version_number: "123", pact_version_sha: "5hbfu") }
-      let(:consumer) { double('pacticipant', name: "Foo") }
-      let(:provider) { double('pacticipant', name: "Bar") }
+      let(:consumer_name) { "Foo/Foo" }
+      let(:provider_name) { "Bar/Bar" }
+      let(:pact) { double('pact', consumer: consumer, provider: provider, consumer_version_number: "123/456", pact_version_sha: "5hbfu") }
+      let(:consumer) { double('pacticipant', name: consumer_name) }
+      let(:provider) { double('pacticipant', name: provider_name) }
       let(:verification) do
         instance_double(PactBroker::Domain::Verification,
-          consumer_name: "Foo",
-          provider_name: "Bar",
+          consumer_name: consumer_name,
+          provider_name: provider_name,
           pact_version_sha: "1234",
           number: "1")
       end
 
+      describe "pact_url" do
+        subject { PactBrokerUrls.pact_url(base_url, pact) }
+
+        it { is_expected.to eq "http://example.org/pacts/provider/Bar%2FBar/consumer/Foo%2FFoo/version/123%2F456" }
+      end
+
       describe "templated_tag_url_for_pacticipant" do
-        subject { PactBrokerUrls.templated_tag_url_for_pacticipant("Bar", base_url) }
+        subject { PactBrokerUrls.templated_tag_url_for_pacticipant(provider_name, base_url) }
 
-        it { is_expected.to eq "http://example.org/pacticipants/Bar/versions/{version}/tags/{tag}" }
+        it { is_expected.to eq "http://example.org/pacticipants/Bar%2FBar/versions/{version}/tags/{tag}" }
       end
 
       describe "pact_triggered_webhooks_url" do
         subject { PactBrokerUrls.pact_triggered_webhooks_url(pact, base_url) }
 
-        it { is_expected.to eq "http://example.org/pacts/provider/Bar/consumer/Foo/version/123/triggered-webhooks" }
+        it { is_expected.to eq "http://example.org/pacts/provider/Bar%2FBar/consumer/Foo%2FFoo/version/123%2F456/triggered-webhooks" }
       end
 
       describe "verification_triggered_webhooks_url" do
         subject { PactBrokerUrls.verification_triggered_webhooks_url(verification, base_url) }
 
-        it { is_expected.to eq "http://example.org/pacts/provider/Bar/consumer/Foo/pact-version/1234/verification-results/1/triggered-webhooks" }
+        it { is_expected.to eq "http://example.org/pacts/provider/Bar%2FBar/consumer/Foo%2FFoo/pact-version/1234/verification-results/1/triggered-webhooks" }
       end
 
       describe "templated_diff_url" do
         subject { PactBrokerUrls.templated_diff_url(pact, base_url) }
 
-        it { is_expected.to eq "http://example.org/pacts/provider/Bar/consumer/Foo/pact-version/5hbfu/diff/pact-version/{pactVersion}" }
+        it { is_expected.to eq "http://example.org/pacts/provider/Bar%2FBar/consumer/Foo%2FFoo/pact-version/5hbfu/diff/pact-version/{pactVersion}" }
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker
 version: !ruby/object:Gem::Version
-  version: 2.27.5
+  version: 2.27.6
 platform: ruby
 authors:
 - Bethany Skurrie
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-15 00:00:00.000000000 Z
+date: 2018-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty