pact_broker-client 1.44.0 → 1.47.1

data/spec/pacts/pact_broker_client-pactflow.json

@@ -0,0 +1,118 @@
+{
+  "consumer": {
+    "name": "Pact Broker Client"
+  },
+  "provider": {
+    "name": "Pactflow"
+  },
+  "interactions": [
+    {
+      "description": "a request for the index resource",
+      "request": {
+        "method": "get",
+        "path": "/",
+        "headers": {
+          "Accept": "application/hal+json"
+        }
+      },
+      "response": {
+        "status": 200,
+        "headers": {
+          "Content-Type": "application/hal+json;charset=utf-8"
+        },
+        "body": {
+          "_links": {
+            "pb:webhooks": {
+              "href": "http://localhost:1235/HAL-REL-PLACEHOLDER-PB-WEBHOOKS"
+            },
+            "pb:pacticipants": {
+              "href": "http://localhost:1235/HAL-REL-PLACEHOLDER-PB-PACTICIPANTS"
+            },
+            "pb:pacticipant": {
+              "href": "http://localhost:1235/HAL-REL-PLACEHOLDER-PB-PACTICIPANT-{pacticipant}"
+            }
+          }
+        },
+        "matchingRules": {
+          "$.body._links.pb:webhooks.href": {
+            "match": "regex",
+            "regex": "http:\\/\\/.*"
+          },
+          "$.body._links.pb:pacticipants.href": {
+            "match": "regex",
+            "regex": "http:\\/\\/.*"
+          },
+          "$.body._links.pb:pacticipant.href": {
+            "match": "regex",
+            "regex": "http:\\/\\/.*{pacticipant}"
+          }
+        }
+      }
+    },
+    {
+      "description": "a request to create a webhook for a team",
+      "providerState": "a team with UUID 2abbc12a-427d-432a-a521-c870af1739d9 exists",
+      "request": {
+        "method": "post",
+        "path": "/HAL-REL-PLACEHOLDER-PB-WEBHOOKS",
+        "headers": {
+          "Content-Type": "application/json",
+          "Accept": "application/hal+json"
+        },
+        "body": {
+          "description": "a webhook",
+          "events": [
+            {
+              "name": "contract_content_changed"
+            }
+          ],
+          "request": {
+            "url": "https://webhook",
+            "method": "POST",
+            "headers": {
+              "Foo": "bar",
+              "Bar": "foo"
+            },
+            "body": {
+              "some": "body"
+            }
+          },
+          "teamUuid": "2abbc12a-427d-432a-a521-c870af1739d9"
+        }
+      },
+      "response": {
+        "status": 201,
+        "headers": {
+          "Content-Type": "application/hal+json;charset=utf-8"
+        },
+        "body": {
+          "description": "a webhook",
+          "teamUuid": "2abbc12a-427d-432a-a521-c870af1739d9",
+          "_links": {
+            "self": {
+              "href": "http://localhost:1234/some-url",
+              "title": "A title"
+            }
+          }
+        },
+        "matchingRules": {
+          "$.body.description": {
+            "match": "type"
+          },
+          "$.body._links.self.href": {
+            "match": "regex",
+            "regex": "http:\\/\\/.*"
+          },
+          "$.body._links.self.title": {
+            "match": "type"
+          }
+        }
+      }
+    }
+  ],
+  "metadata": {
+    "pactSpecification": {
+      "version": "2.0.0"
+    }
+  }
+}

data/spec/service_providers/pact_broker_client_register_repository_spec.rb

@@ -14,7 +14,7 @@ module PactBroker::Client
             with(
                 method: :patch,
                 path: '/pacticipants/Pricing%20Service',
-                headers: patch_request_headers,
+                headers: old_patch_request_headers,
                 body: {repository_url: repository_url} ).
               will_respond_with(
                 status: 201,
@@ -33,7 +33,7 @@ module PactBroker::Client
             with(
               method: :patch,
               path: '/pacticipants/Pricing%20Service',
-              headers: patch_request_headers,
+              headers: old_patch_request_headers,
               body: { repository_url: repository_url }).
             will_respond_with(
               status: 200,

data/spec/service_providers/pact_helper.rb

@@ -8,7 +8,6 @@ Pact.configure do | config |
 end
 
 Pact.service_consumer 'Pact Broker Client' do
-
   has_pact_with "Pact Broker" do
     mock_service :pact_broker do
       port 1234
@@ -16,6 +15,12 @@ Pact.service_consumer 'Pact Broker Client' do
     end
   end
 
+  has_pact_with "Pactflow" do
+    mock_service :pactflow do
+      port 1235
+      pact_specification_version "2.0"
+    end
+  end
 end
 
 module PactBrokerPactHelperMethods
@@ -30,22 +35,22 @@ module PactBrokerPactHelperMethods
     path
   end
 
-  def placeholder_url(relation, params = [])
-    "#{pact_broker.mock_service_base_url}#{placeholder_path(relation, params)}"
+  def placeholder_url(relation, params = [], mock_service = pact_broker)
+    "#{mock_service.mock_service_base_url}#{placeholder_path(relation, params)}"
   end
 
-  def placeholder_url_term(relation, params = [])
+  def placeholder_url_term(relation, params = [], mock_service = pact_broker)
     regexp = "http:\/\/.*"
     if params.any?
       joined_params_for_regexp = params.collect{ |param| "{#{param}}"}.join(".*")
       regexp = "#{regexp}#{joined_params_for_regexp}"
     end
 
-    Pact.term(placeholder_url(relation, params), /#{regexp}/)
+    Pact.term(placeholder_url(relation, params, mock_service), /#{regexp}/)
   end
 
-  def mock_pact_broker_index(context)
-    pact_broker
+  def mock_pact_broker_index(context, mock_service = pact_broker)
+    mock_service
       .upon_receiving("a request for the index resource")
       .with(
           method: :get,
@@ -57,13 +62,13 @@ module PactBrokerPactHelperMethods
           body: {
             _links: {
               :'pb:webhooks' => {
-                href: placeholder_url_term('pb:webhooks')
+                href: placeholder_url_term('pb:webhooks', [], mock_service)
               },
               :'pb:pacticipants' => {
-                href: placeholder_url_term('pb:pacticipants')
+                href: placeholder_url_term('pb:pacticipants', [], mock_service)
               },
               :'pb:pacticipant' => {
-                href: placeholder_url_term('pb:pacticipant', ['pacticipant'])
+                href: placeholder_url_term('pb:pacticipant', ['pacticipant'], mock_service)
               }
             }
           }

data/spec/service_providers/pactflow_webhooks_create_spec.rb

@@ -0,0 +1,86 @@
+require_relative 'pact_helper'
+require 'pact_broker/client/webhooks/create'
+
+RSpec.describe "creating a webhook in Pactflow", pact: true do
+
+  include_context "pact broker"
+  include PactBrokerPactHelperMethods
+
+  let(:params) do
+    {
+      description: "a webhook",
+      events: %w{contract_content_changed},
+      http_method: "POST",
+      url: "https://webhook",
+      headers: { "Foo" => "bar", "Bar" => "foo"},
+      body: body,
+      team_uuid: "2abbc12a-427d-432a-a521-c870af1739d9"
+    }
+  end
+
+  let(:body) { { some: "body" }.to_json }
+
+  let(:request_body) do
+    {
+      "description" => "a webhook",
+      "events" => [
+        "name" => "contract_content_changed"
+      ],
+      "request" => {
+        "url" => "https://webhook",
+        "method" => "POST",
+        "headers" => {
+          "Foo" => "bar",
+          "Bar" => "foo"
+        },
+        "body" => {
+          "some" => "body"
+        },
+      },
+      "teamUuid" => "2abbc12a-427d-432a-a521-c870af1739d9"
+    }
+  end
+
+  let(:response_status) { 201 }
+  let(:success_response) do
+    {
+      status: response_status,
+      headers: pact_broker_response_headers,
+      body: {
+        description: Pact.like("a webhook"),
+        teamUuid: "2abbc12a-427d-432a-a521-c870af1739d9",
+        _links: {
+          self: {
+            href: Pact.term('http://localhost:1234/some-url', %r{http://.*}),
+            title: Pact.like("A title")
+          }
+        }
+      }
+    }
+  end
+
+  let(:pact_broker_client_options) { {} }
+
+  subject { PactBroker::Client::Webhooks::Create.call(params, pactflow.mock_service_base_url, pact_broker_client_options) }
+
+  context "when a valid webhook with a team specified is submitted" do
+    before do
+      mock_pact_broker_index(self, pactflow)
+      pactflow
+        .given("a team with UUID 2abbc12a-427d-432a-a521-c870af1739d9 exists")
+        .upon_receiving("a request to create a webhook for a team")
+        .with(
+            method: :post,
+            path: '/HAL-REL-PLACEHOLDER-PB-WEBHOOKS',
+            headers: post_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
+end

data/spec/service_providers/record_deployment_spec.rb

@@ -1,9 +1,7 @@
 require 'service_providers/pact_helper'
 require 'pact_broker/client/deployments/record_deployment'
 
-deployment_feature_on = ENV.fetch('PACT_BROKER_FEATURES', '').include?("deployments")
-
-RSpec.describe "recording a deployment", pact: true, skip: !deployment_feature_on do
+RSpec.describe "recording a deployment", pact: true do
   include_context "pact broker"
   include PactBrokerPactHelperMethods
 

data/spec/service_providers/record_release_spec.rb

@@ -1,9 +1,7 @@
 require 'service_providers/pact_helper'
 require 'pact_broker/client/deployments/record_deployment'
 
-deployment_feature_on = ENV.fetch('PACT_BROKER_FEATURES', '').include?("deployments")
-
-RSpec.describe "recording a release", pact: true, skip: !deployment_feature_on do
+RSpec.describe "recording a release", pact: true do
   include_context "pact broker"
   include PactBrokerPactHelperMethods
 
@@ -113,10 +111,7 @@ RSpec.describe "recording a release", pact: true, skip: !deployment_feature_on d
       )
       .will_respond_with(
         status: 201,
-        headers: pact_broker_response_headers,
-        body: {
-          target: target
-        }
+        headers: pact_broker_response_headers
       )
   end
 

data/spec/service_providers/record_undeployment_spec.rb

@@ -0,0 +1,164 @@
+require 'service_providers/pact_helper'
+require 'pact_broker/client/deployments/record_undeployment'
+
+RSpec.describe "recording an undeployment", pact: true do
+  include_context "pact broker"
+  include PactBrokerPactHelperMethods
+
+  let(:pacticipant_name) { "Foo" }
+  let(:environment_name) { "test" }
+  let(:output) { "text" }
+  let(:target) { "customer-1" }
+  let(:params) do
+    {
+      pacticipant_name: pacticipant_name,
+      environment_name: environment_name,
+      target: target
+    }
+  end
+  let(:options) do
+    {
+      output: output,
+      verbose: true
+    }
+  end
+  let(:webmock_base_url) { "http://broker" }
+  let(:pact_broker_client_options) { { pact_broker_base_url: webmock_base_url } }
+  let(:test_environment_placeholder_path) { "/HAL-REL-PLACEHOLDER-PB-ENVIRONMENT-16926ef3-590f-4e3f-838e-719717aa88c9" }
+  let(:currently_deployed_versions_placeholder_path) { "/PLACEHOLDER-ENVIRONMENT-CURRENTLY-DEPLOYED-16926ef3-590f-4e3f-838e-719717aa88c9" }
+  let(:deployed_version_placeholder_path) { "/PLACEHOLDER-DEPLOYED-VERSION-ff3adecf-cfc5-4653-a4e3-f1861092f8e0"}
+
+  subject { PactBroker::Client::Deployments::RecordUndeployment.call(params, options, pact_broker_client_options) }
+
+  let(:index_body_hash) do
+    {
+      _links: {
+        :'pb:environments' => {
+          href: "#{webmock_base_url}/environments"
+        }
+      }
+    }
+  end
+
+  let(:environments_hash) do
+    {
+      _links: {
+        :'pb:environments' => [
+          {
+            name: "test",
+            href: pact_broker.mock_service_base_url + test_environment_placeholder_path
+          }
+        ]
+      }
+    }
+  end
+
+  let!(:index_request) do
+    stub_request(:get, "http://broker").to_return(status: 200, body: index_body_hash.to_json, headers: { "Content-Type" => "application/hal+json" }  )
+  end
+
+  let!(:environments_request) do
+    stub_request(:get, "http://broker/environments").to_return(status: 200, body: environments_hash.to_json, headers: { "Content-Type" => "application/hal+json" }  )
+  end
+
+  def mock_test_environment
+    pact_broker
+      .given("an environment with name test and UUID 16926ef3-590f-4e3f-838e-719717aa88c9 exists")
+      .upon_receiving("a request for an environment")
+      .with(
+        method: "GET",
+        path: test_environment_placeholder_path,
+        headers: get_request_headers
+      )
+      .will_respond_with(
+        status: 200,
+        headers: pact_broker_response_headers,
+        body: {
+          _links: {
+            :'pb:currently-deployed-deployed-versions' => {
+              href: Pact.term( pact_broker.mock_service_base_url + currently_deployed_versions_placeholder_path, /^http.*/)
+            }
+          }
+        }
+      )
+  end
+
+  def mock_deployed_versions_search_results
+    pact_broker
+      .given("an version is deployed to environment with UUID 16926ef3-590f-4e3f-838e-719717aa88c9 with target customer-1")
+      .upon_receiving("a request to list the versions deployed to an environment for a pacticipant name and target")
+      .with(
+        method: "GET",
+        path: currently_deployed_versions_placeholder_path,
+        query: { pacticipant: pacticipant_name },
+        headers: get_request_headers
+      )
+      .will_respond_with(
+        status: 200,
+        headers: pact_broker_response_headers,
+        body: {
+          _embedded: {
+            deployedVersions: [
+              {
+                target: target,
+                _links: {
+                  self: {
+                    href: Pact.term(pact_broker.mock_service_base_url + deployed_version_placeholder_path, /^http/)
+                  }
+                }
+              }
+            ]
+          }
+        }
+      )
+  end
+
+  def mock_mark_deployed_version_as_undeployed
+    pact_broker
+      .given("a currently deployed version exists")
+      .upon_receiving("a request to mark a deployed version as not currently deploye")
+      .with(
+        method: "PATCH",
+        path: deployed_version_placeholder_path,
+        body: { currentlyDeployed: false },
+        headers: patch_request_headers
+      )
+      .will_respond_with(
+        status: 200,
+        headers: pact_broker_response_headers,
+        body: deployed_version_hash
+      )
+  end
+
+  let(:deployed_version_hash) do
+    {
+      "currentlyDeployed" => false,
+      "_embedded" => {
+        "version" => {
+          "number" => Pact.like("2")
+        }
+      }
+    }
+  end
+
+  context "when the deployment is recorded successfully" do
+    before do
+      mock_test_environment
+      mock_deployed_versions_search_results
+      mock_mark_deployed_version_as_undeployed
+    end
+
+    it "returns a success message" do
+      expect(subject.success).to be true
+      expect(subject.message).to include "Recorded undeployment of Foo version 2 from test environment (target customer-1) in the Pact Broker"
+    end
+
+    context "when the output is json" do
+      let(:output) { "json" }
+
+      it "returns the JSON payload" do
+        expect(JSON.parse(subject.message)).to eq [Pact::Reification.from_term(deployed_version_hash)]
+      end
+    end
+  end
+end