pact_broker-client 1.25.0 → 1.27.2

data/spec/service_providers/pact_broker_client_matrix_spec.rb

@@ -19,7 +19,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.3&q[][pacticipant]=Bar&q[][version]=4.5.6&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Bversion%5D=4.5.6&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -42,7 +42,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo%20Thing&q[][version]=1.2.3&q[][pacticipant]=Bar&q[][version]=4.5.6&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo%20Thing&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Bversion%5D=4.5.6&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -67,7 +67,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.3&latestby=cvp&latest=true"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&latestby=cvp&latest=true"
             ).
             will_respond_with(
               status: 200,
@@ -92,7 +92,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.3&q[][pacticipant]=Bar&q[][version]=9.9.9&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Bversion%5D=9.9.9&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -119,7 +119,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Wiffle&q[][version]=1.2.3&q[][pacticipant]=Meep&q[][version]=9.9.9&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Wiffle&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Meep&q%5B%5D%5Bversion%5D=9.9.9&latestby=cvpv"
             ).
             will_respond_with(
               status: 400,
@@ -147,7 +147,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][pacticipant]=Bar&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bpacticipant%5D=Bar&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -174,7 +174,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][pacticipant]=Bar&latestby=cvpv&success[]=true"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bpacticipant%5D=Bar&latestby=cvpv&success%5B%5D=true"
             ).
             will_respond_with(
               status: 200,
@@ -200,7 +200,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.3&q[][pacticipant]=Bar&q[][latest]=true&q[][tag]=prod&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Blatest%5D=true&q%5B%5D%5Btag%5D=prod&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -226,7 +226,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.4&q[][pacticipant]=Bar&q[][latest]=true&latestby=cvpv"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.4&q%5B%5D%5Bpacticipant%5D=Bar&q%5B%5D%5Blatest%5D=true&latestby=cvpv"
             ).
             will_respond_with(
               status: 200,
@@ -252,7 +252,7 @@ module PactBroker::Client
             with(
               method: :get,
               path: "/matrix",
-              query: "q[][pacticipant]=Foo&q[][version]=1.2.3&latestby=cvp&latest=true&tag=prod"
+              query: "q%5B%5D%5Bpacticipant%5D=Foo&q%5B%5D%5Bversion%5D=1.2.3&latestby=cvp&latest=true&tag=prod"
             ).
             will_respond_with(
               status: 200,

data/spec/service_providers/pact_helper.rb

@@ -58,12 +58,41 @@ module PactBrokerPactHelperMethods
             _links: {
               :'pb:webhooks' => {
                 href: placeholder_url_term('pb:webhooks')
+              },
+              :'pb:pacticipants' => {
+                href: placeholder_url_term('pb:pacticipants')
+              },
+              :'pb:pacticipant' => {
+                href: placeholder_url_term('pb:pacticipant', ['pacticipant'])
               }
             }
           }
         )
   end
 
+  def mock_pact_broker_index_with_relations(context, links, provider_state)
+    _links = links.each_with_object({}) do | (key, value), new_links |
+      new_links[key] = {
+        href: value
+      }
+    end
+
+    pact_broker
+      .given(provider_state)
+      .upon_receiving("a request for the index resource")
+      .with(
+          method: :get,
+          path: '/',
+          headers: context.get_request_headers).
+        will_respond_with(
+          status: 200,
+          headers: context.pact_broker_response_headers,
+          body: {
+            _links: _links
+          }
+        )
+  end
+
   def mock_pact_broker_index_with_webhook_relation(context)
     pact_broker
       .upon_receiving("a request for the index resource with the webhook relation")

data/spec/service_providers/pacticipants_create_spec.rb

@@ -0,0 +1,118 @@
+require_relative 'pact_helper'
+require 'pact_broker/client/pacticipants/create'
+
+RSpec.describe "creating or updating a pacticipant", pact: true do
+  include_context "pact broker"
+  include PactBrokerPactHelperMethods
+
+  before do
+    index_links = {
+      'pb:pacticipants' => Pact.term('http://localhost:1234/pacticipants', %r{http://.*}),
+      'pb:pacticipant' => Pact.term('http://localhost:1234/pacticipants/{pacticipant}', %r{http://.*\{pacticipant\}}),
+    }
+    mock_pact_broker_index_with_relations(self, index_links, "the pacticipant relations are present")
+  end
+
+  let(:params) do
+    {
+      name: "Foo",
+      repository_url: "http://foo"
+    }
+  end
+
+  let(:request_body) { { name: "Foo", repositoryUrl: "http://foo" } }
+
+
+  let(:response_status) { 201 }
+  let(:create_success_response) do
+    {
+      status: response_status,
+      headers: pact_broker_response_headers,
+      body: {
+        name: "Foo",
+        repositoryUrl: "http://foo",
+        _links: {
+          self: {
+            href: Pact.term('http://localhost:1234/pacticipants/Foo', %r{http://.*}),
+          }
+        }
+      }
+    }
+  end
+
+  let(:get_success_response) do
+    {
+      status: 200,
+      headers: pact_broker_response_headers,
+      body: {
+        _links: {
+          self: {
+            href: Pact.term('http://localhost:1234/pacticipants/Foo', %r{http://.*}),
+          }
+        }
+      }
+    }
+  end
+
+  let(:pact_broker_client_options) { {} }
+
+  subject { PactBroker::Client::Pacticipants2::Create.call(params, broker_base_url, pact_broker_client_options) }
+
+  context "when the pacticipant does not already exist" do
+    before do
+      pact_broker
+        .upon_receiving("a request to retrieve a pacticipant")
+        .with(
+            method: :get,
+            path: '/pacticipants/Foo',
+            headers: get_request_headers)
+        .will_respond_with(status: 404)
+
+      pact_broker
+        .upon_receiving("a request to create a pacticipant")
+        .with(
+            method: :post,
+            path: '/pacticipants',
+            headers: post_request_headers,
+            body: request_body)
+        .will_respond_with(create_success_response)
+    end
+
+    it "returns a CommandResult with success = true" do
+      expect(subject).to be_a PactBroker::Client::CommandResult
+      expect(subject.success).to be true
+      expect(subject.message).to eq "Pacticipant \"Foo\" created"
+    end
+  end
+
+  context "when the pacticipant does already exist" do
+    before do
+      pact_broker
+        .given("a pacticipant with name Foo exists")
+        .upon_receiving("a request to retrieve a pacticipant")
+        .with(
+          method: :get,
+          path: '/pacticipants/Foo',
+          headers: get_request_headers)
+        .will_respond_with(get_success_response)
+
+      pact_broker
+        .given("a pacticipant with name Foo exists")
+        .upon_receiving("a request to update a pacticipant")
+        .with(
+          method: :patch,
+          path: '/pacticipants/Foo',
+          headers: post_request_headers,
+          body: request_body)
+        .will_respond_with(create_success_response)
+    end
+
+    let(:response_status) { 200 }
+
+    it "returns a CommandResult with success = true" do
+      expect(subject).to be_a PactBroker::Client::CommandResult
+      expect(subject.success).to be true
+      expect(subject.message).to eq "Pacticipant \"Foo\" updated"
+    end
+  end
+end

data/spec/service_providers/webhooks_create_spec.rb

@@ -47,11 +47,13 @@ RSpec.describe "creating a webhook", pact: true do
     }
   end
 
+  let(:response_status) { 201 }
   let(:success_response) do
     {
-      status: 201,
+      status: response_status,
       headers: pact_broker_response_headers,
       body: {
+        description: Pact.like("a webhook"),
         _links: {
           self: {
             href: Pact.term('http://localhost:1234/some-url', %r{http://.*}),
@@ -82,7 +84,7 @@ RSpec.describe "creating a webhook", pact: true do
     it "returns a CommandResult with success = true" do
       expect(subject).to be_a PactBroker::Client::CommandResult
       expect(subject.success).to be true
-      expect(subject.message).to eq "Webhook \"A title\" created"
+      expect(subject.message).to eq "Webhook \"a webhook\" created"
     end
   end
 
@@ -201,7 +203,7 @@ RSpec.describe "creating a webhook", pact: true do
 
     it "returns a CommandResult with success = true" do
       expect(subject.success).to be true
-      expect(subject.message).to eq "Webhook \"A title\" created"
+      expect(subject.message).to eq "Webhook \"a webhook\" created"
     end
   end
 
@@ -282,23 +284,50 @@ RSpec.describe "creating a webhook", pact: true do
       request_body["provider"] = { "name" => "Pricing Service" }
       request_body["consumer"] = { "name" => "Condor" }
       mock_pact_broker_index_with_webhook_relation(self)
-
-      pact_broker
-        .upon_receiving("a request to create a webhook with a JSON body and a uuid")
-        .given("the 'Pricing Service' and 'Condor' already exist in the pact-broker")
-        .with(
-            method: :put,
-            path: "/webhooks/#{uuid}",
-            headers: put_request_headers,
-            body: request_body)
-        .will_respond_with(success_response)
     end
+
     let(:uuid) { '696c5f93-1b7f-44bc-8d03-59440fcaa9a0' }
 
-    it "returns a CommandResult with success = true" do
-      expect(subject).to be_a PactBroker::Client::CommandResult
-      expect(subject.success).to be true
-      expect(subject.message).to eq "Webhook \"A title\" created"
+    context "when the webhook does not already exist" do
+      before do
+        pact_broker
+          .upon_receiving("a request to create a webhook with a JSON body and a uuid")
+          .given("the 'Pricing Service' and 'Condor' already exist in the pact-broker")
+          .with(
+              method: :put,
+              path: "/webhooks/#{uuid}",
+              headers: put_request_headers,
+              body: request_body)
+          .will_respond_with(success_response)
+      end
+
+      it "returns a CommandResult with success = true" do
+        expect(subject).to be_a PactBroker::Client::CommandResult
+        expect(subject.success).to be true
+        expect(subject.message).to eq "Webhook \"a webhook\" created"
+      end
+    end
+
+    context "when the webhook does exist" do
+      before do
+        pact_broker
+          .upon_receiving("a request to update a webhook")
+          .given("a webhook with the uuid #{uuid} exists")
+          .with(
+              method: :put,
+              path: "/webhooks/#{uuid}",
+              headers: put_request_headers,
+              body: request_body)
+          .will_respond_with(success_response)
+      end
+
+      let(:response_status) { 200 }
+
+      it "returns a CommandResult with success = true" do
+        expect(subject).to be_a PactBroker::Client::CommandResult
+        expect(subject.success).to be true
+        expect(subject.message).to eq "Webhook \"a webhook\" updated"
+      end
     end
   end
 end

metadata

@@ -1,57 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker-client
 version: !ruby/object:Gem::Version
-  version: 1.25.0
+  version: 1.27.2
 platform: ruby
 authors:
 - Beth Skurrie
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-17 00:00:00.000000000 Z
+date: 2020-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-- !ruby/object:Gem::Dependency
-  name: json
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.18'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.18'
 - !ruby/object:Gem::Dependency
   name: term-ansicolor
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: table_print
   requirement: !ruby/object:Gem::Requirement
@@ -84,16 +70,16 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: fakefs
   requirement: !ruby/object:Gem::Requirement
@@ -173,6 +159,8 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/trigger_pact_cli_release.yml"
+- ".github/workflows/trigger_pact_docs_update.yml"
 - ".gitignore"
 - ".rspec"
 - ".travis.yml"
@@ -213,6 +201,7 @@ files:
 - lib/pact_broker/client/hal/entry_point.rb
 - lib/pact_broker/client/hal/http_client.rb
 - lib/pact_broker/client/hal/link.rb
+- lib/pact_broker/client/hal_client_methods.rb
 - lib/pact_broker/client/matrix.rb
 - lib/pact_broker/client/matrix/formatter.rb
 - lib/pact_broker/client/matrix/json_formatter.rb
@@ -223,7 +212,9 @@ files:
 - lib/pact_broker/client/pact_file.rb
 - lib/pact_broker/client/pact_hash.rb
 - lib/pact_broker/client/pacticipants.rb
+- lib/pact_broker/client/pacticipants/create.rb
 - lib/pact_broker/client/pacts.rb
+- lib/pact_broker/client/pacts/list_latest_versions.rb
 - lib/pact_broker/client/publish_pacts.rb
 - lib/pact_broker/client/retry.rb
 - lib/pact_broker/client/tasks.rb
@@ -235,6 +226,7 @@ files:
 - lib/pact_broker/client/versions/json_formatter.rb
 - lib/pact_broker/client/versions/text_formatter.rb
 - lib/pact_broker/client/webhooks/create.rb
+- lib/pact_broker/client/webhooks/test.rb
 - lib/pact_broker_client.rb
 - pact-broker-client.gemspec
 - script/generate-cli-usage.sh
@@ -261,6 +253,7 @@ files:
 - spec/lib/pact_broker/client/matrix_spec.rb
 - spec/lib/pact_broker/client/merge_pacts_spec.rb
 - spec/lib/pact_broker/client/pact_broker_client_spec.rb
+- spec/lib/pact_broker/client/pacticipants/create_spec.rb
 - spec/lib/pact_broker/client/pacticipants_spec.rb
 - spec/lib/pact_broker/client/publish_pacts_spec.rb
 - spec/lib/pact_broker/client/retry_spec.rb
@@ -279,6 +272,7 @@ files:
 - spec/service_providers/pact_broker_client_tags_spec.rb
 - spec/service_providers/pact_broker_client_versions_spec.rb
 - spec/service_providers/pact_helper.rb
+- spec/service_providers/pacticipants_create_spec.rb
 - spec/service_providers/webhooks_create_spec.rb
 - spec/spec_helper.rb
 - spec/support/cli_test_pacts/bar.json
@@ -311,8 +305,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: See description
@@ -338,6 +331,7 @@ test_files:
 - spec/lib/pact_broker/client/matrix_spec.rb
 - spec/lib/pact_broker/client/merge_pacts_spec.rb
 - spec/lib/pact_broker/client/pact_broker_client_spec.rb
+- spec/lib/pact_broker/client/pacticipants/create_spec.rb
 - spec/lib/pact_broker/client/pacticipants_spec.rb
 - spec/lib/pact_broker/client/publish_pacts_spec.rb
 - spec/lib/pact_broker/client/retry_spec.rb
@@ -356,6 +350,7 @@ test_files:
 - spec/service_providers/pact_broker_client_tags_spec.rb
 - spec/service_providers/pact_broker_client_versions_spec.rb
 - spec/service_providers/pact_helper.rb
+- spec/service_providers/pacticipants_create_spec.rb
 - spec/service_providers/webhooks_create_spec.rb
 - spec/spec_helper.rb
 - spec/support/cli_test_pacts/bar.json