pact_broker 2.77.0 → 2.80.0

data/spec/fixtures/approvals/publish_contract_verification_already_exists.approved.json

@@ -0,0 +1,149 @@
+{
+  "request": {
+    "path": "/contracts/publish",
+    "headers": {
+      "Content-Type": "application/json",
+      "Accept": "application/hal+json"
+    },
+    "body": {
+      "pacticipantName": "Foo",
+      "pacticipantVersionNumber": "1",
+      "branch": "main",
+      "tags": [
+        "a",
+        "b"
+      ],
+      "buildUrl": "http://ci/builds/1234",
+      "contracts": [
+        {
+          "consumerName": "Foo",
+          "providerName": "Bar",
+          "specification": "pact",
+          "contentType": "application/json",
+          "content": "eyJjb25zdW1lciI6eyJuYW1lIjoiRm9vIn0sInByb3ZpZGVyIjp7Im5hbWUiOiJCYXIifSwiaW50ZXJhY3Rpb25zIjpbXX0=",
+          "onConflict": "overwrite"
+        }
+      ]
+    }
+  },
+  "response": {
+    "status": 200,
+    "headers": {
+      "Content-Type": "application/hal+json;charset=utf-8",
+      "Content-Length": "2933"
+    },
+    "body": {
+      "logs": [
+        {
+          "level": "debug",
+          "message": "Updated Foo version 1 with branch main and tags a, b",
+          "deprecationWarning": "Replaced by notices"
+        },
+        {
+          "level": "warning",
+          "message": "Pact with changed content published over existing content for Foo version 1 and provider Bar. This is not recommended in normal cicumstances and may indicate that you have not configured your Pact pipeline correctly. For more information see https://docs.pact.io/go/versioning",
+          "deprecationWarning": "Replaced by notices"
+        },
+        {
+          "level": "debug",
+          "message": "  View the published pact at http://example.org/pacts/provider/Bar/consumer/Foo/version/1",
+          "deprecationWarning": "Replaced by notices"
+        },
+        {
+          "level": "debug",
+          "message": "  Events detected: contract_published, contract_content_changed (pact content modified since previous publication for Foo version 1)",
+          "deprecationWarning": "Replaced by notices"
+        },
+        {
+          "level": "warning",
+          "message": "  Next steps:",
+          "deprecationWarning": "Replaced by notices"
+        },
+        {
+          "level": "warning",
+          "message": "    * Configure separate Bar pact verification build and webhook to trigger it when the pact content changes. See https://docs.pact.io/go/webhooks",
+          "deprecationWarning": "Replaced by notices"
+        }
+      ],
+      "notices": [
+        {
+          "type": "debug",
+          "text": "Updated Foo version 1 with branch main and tags a, b"
+        },
+        {
+          "type": "warning",
+          "text": "Pact with changed content published over existing content for Foo version 1 and provider Bar. This is not recommended in normal cicumstances and may indicate that you have not configured your Pact pipeline correctly. For more information see https://docs.pact.io/go/versioning"
+        },
+        {
+          "type": "debug",
+          "text": "  View the published pact at http://example.org/pacts/provider/Bar/consumer/Foo/version/1"
+        },
+        {
+          "type": "debug",
+          "text": "  Events detected: contract_published, contract_content_changed (pact content modified since previous publication for Foo version 1)"
+        },
+        {
+          "type": "warning",
+          "text": "  Next steps:"
+        },
+        {
+          "type": "warning",
+          "text": "    * Configure separate Bar pact verification build and webhook to trigger it when the pact content changes. See https://docs.pact.io/go/webhooks"
+        }
+      ],
+      "_embedded": {
+        "pacticipant": {
+          "name": "Foo",
+          "_links": {
+            "self": {
+              "href": "http://example.org/pacticipants/Foo"
+            }
+          }
+        },
+        "version": {
+          "number": "1",
+          "branch": "main",
+          "buildUrl": "http://ci/builds/1234",
+          "_links": {
+            "self": {
+              "title": "Version",
+              "name": "1",
+              "href": "http://example.org/pacticipants/Foo/versions/1"
+            }
+          }
+        }
+      },
+      "_links": {
+        "pb:pacticipant": {
+          "title": "Pacticipant",
+          "name": "Foo",
+          "href": "http://example.org/pacticipants/Foo"
+        },
+        "pb:pacticipant-version": {
+          "title": "Pacticipant version",
+          "name": "1",
+          "href": "http://example.org/pacticipants/Foo/versions/1"
+        },
+        "pb:pacticipant-version-tags": [
+          {
+            "title": "Tag",
+            "name": "a",
+            "href": "http://example.org/pacticipants/Foo/versions/1/tags/a"
+          },
+          {
+            "title": "Tag",
+            "name": "b",
+            "href": "http://example.org/pacticipants/Foo/versions/1/tags/b"
+          }
+        ],
+        "pb:contracts": [
+          {
+            "title": "Pact",
+            "name": "Pact between Foo (1) and Bar",
+            "href": "http://example.org/pacts/provider/Bar/consumer/Foo/version/1"
+          }
+        ]
+      }
+    }
+  }
+}

data/spec/fixtures/approvals/publish_contract_with_validation_error.approved.json

@@ -0,0 +1,42 @@
+{
+  "request": {
+    "path": "/contracts/publish",
+    "headers": {
+      "Content-Type": "application/json",
+      "Accept": "application/hal+json"
+    },
+    "body": {
+      "pacticipantName": "Foo",
+      "branch": "main",
+      "tags": [
+        "a",
+        "b"
+      ],
+      "buildUrl": "http://ci/builds/1234",
+      "contracts": [
+        {
+          "consumerName": "Foo",
+          "providerName": "Bar",
+          "specification": "pact",
+          "contentType": "application/json",
+          "content": "eyJjb25zdW1lciI6eyJuYW1lIjoiRm9vIn0sInByb3ZpZGVyIjp7Im5hbWUiOiJCYXIifSwiaW50ZXJhY3Rpb25zIjpbXX0=",
+          "onConflict": "overwrite"
+        }
+      ]
+    }
+  },
+  "response": {
+    "status": 400,
+    "headers": {
+      "Content-Type": "application/hal+json;charset=utf-8",
+      "Content-Length": "79"
+    },
+    "body": {
+      "errors": {
+        "pacticipantVersionNumber": [
+          "pacticipantVersionNumber is missing"
+        ]
+      }
+    }
+  }
+}

data/spec/fixtures/approvals/publish_contracts_results_decorator.approved.json

@@ -0,0 +1,61 @@
+{
+  "logs": [
+    {
+      "level": "warning",
+      "message": "foo",
+      "deprecationWarning": "Replaced by notices"
+    }
+  ],
+  "notices": [
+    {
+      "type": "warning",
+      "text": "foo"
+    }
+  ],
+  "_embedded": {
+    "pacticipant": {
+      "name": "Foo",
+      "_links": {
+        "self": {
+          "href": "http://example.org/pacticipants/Foo"
+        }
+      }
+    },
+    "version": {
+      "number": "1",
+      "_links": {
+        "self": {
+          "title": "Version",
+          "name": "1",
+          "href": "http://example.org/pacticipants/Foo/versions/1"
+        }
+      }
+    }
+  },
+  "_links": {
+    "pb:pacticipant": {
+      "title": "Pacticipant",
+      "name": "Foo",
+      "href": "pacticipant_url"
+    },
+    "pb:pacticipant-version": {
+      "title": "Pacticipant version",
+      "name": "1",
+      "href": "version_url"
+    },
+    "pb:pacticipant-version-tags": [
+      {
+        "title": "Tag",
+        "name": "main",
+        "href": "tag_url"
+      }
+    ],
+    "pb:contracts": [
+      {
+        "title": "Pact",
+        "name": "pact name",
+        "href": "pact_url"
+      }
+    ]
+  }
+}

data/spec/integration/app_spec.rb

@@ -1,15 +1,15 @@
-require 'spec_helper'
-require 'pact_broker/app'
-require 'db'
+require "spec_helper"
+require "pact_broker/app"
+require "db"
 
 module PactBroker
   describe App do
     before do
-      TestDataBuilder.new.create_pact_with_hierarchy('Some Consumer', '1.0', 'Some Provider').and_return(:pact)
+      TestDataBuilder.new.create_pact_with_hierarchy("Some Consumer", "1.0", "Some Provider").and_return(:pact)
     end
 
     let(:hal_browser_enabled) { true }
-    let(:group_url) { '/groups/Some%20Consumer' }
+    let(:group_url) { "/groups/Some%20Consumer" }
     let(:enable_diagnostic_endpoints) { false }
 
     let(:app) do
@@ -23,43 +23,43 @@ module PactBroker
 
     it "exposes a favicon.ico file" do
       get "/favicon.ico"
-      expect(last_response.headers['Content-Type']).to eq "image/x-icon"
+      expect(last_response.headers["Content-Type"]).to eq "image/x-icon"
     end
 
     context "when Accept includes text/html" do
-      let(:env) { {'HTTP_ACCEPT' => 'text/html'} }
+      let(:env) { {"HTTP_ACCEPT" => "text/html"} }
 
-      subject { get(path, '', env) }
+      subject { get(path, "", env) }
 
       describe "a request for root" do
-        let(:path) { '/' }
+        let(:path) { "/" }
 
         it "returns the relationships page" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/html'
-          expect(subject.body).to include 'Pacts'
+          expect(subject.headers["Content-Type"]).to include "text/html"
+          expect(subject.body).to include "Pacts"
         end
 
       end
 
       describe "a request for /ui/relationships" do
-        let(:path) { '/ui/relationships/' }
+        let(:path) { "/ui/relationships/" }
 
         it "returns the relationships page" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/html'
-          expect(subject.body).to include 'Pacts'
+          expect(subject.headers["Content-Type"]).to include "text/html"
+          expect(subject.body).to include "Pacts"
         end
       end
 
       describe "a request for /groups/:pacticipant_name" do
-        let(:path) { '/groups/Some%20Consumer' }
+        let(:path) { "/groups/Some%20Consumer" }
 
         it "returns the group page" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/html'
-          expect(subject.body).to include 'Network graph'
-          expect(subject.body).to include 'Some Consumer'
+          expect(subject.headers["Content-Type"]).to include "text/html"
+          expect(subject.body).to include "Network graph"
+          expect(subject.body).to include "Some Consumer"
         end
 
         it "does not screw up the HTML escaping, haven't you heard of semantic versioning padrino?? Seriously." do
@@ -69,12 +69,12 @@ module PactBroker
       end
 
       describe "a request for /doc/:rel_name" do
-        let(:path) { '/doc/webhooks' }
+        let(:path) { "/doc/webhooks" }
 
         it "returns the HAL docs for the given rel" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/html'
-          expect(subject.body).to include 'Webhooks'
+          expect(subject.headers["Content-Type"]).to include "text/html"
+          expect(subject.body).to include "Webhooks"
         end
       end
 
@@ -83,28 +83,28 @@ module PactBroker
 
         it "returns the HTML representation of the group" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/html'
-          expect(subject.body).to include 'Network graph'
+          expect(subject.headers["Content-Type"]).to include "text/html"
+          expect(subject.body).to include "Network graph"
         end
       end
 
       describe "a request for a path that does not exist" do
-        let(:path) { '/foo' }
+        let(:path) { "/foo" }
 
         it "returns a 404" do
           expect(subject.status).to eq 404
-          expect(subject.headers['Content-Type']).to include 'text/html'
+          expect(subject.headers["Content-Type"]).to include "text/html"
         end
       end
 
       context "when the HALBrowser is enabled" do
         context "when application/hal+json is also specified as an Accept" do
-          let(:env) { {'HTTP_ACCEPT' => 'text/html;application/hal+json'} }
-          let(:path) { '/something' }
+          let(:env) { {"HTTP_ACCEPT" => "text/html;application/hal+json"} }
+          let(:path) { "/something" }
 
           it "redirects to the HAL Browser" do
             expect(subject.status).to eq 303
-            expect(subject.headers['Location']).to eq '/hal-browser/browser.html#/something'
+            expect(subject.headers["Location"]).to eq "/hal-browser/browser.html#/something"
           end
         end
       end
@@ -112,8 +112,8 @@ module PactBroker
       context "when the HALBrowser is not enabled" do
         context "when application/hal+json is also specified as an Accept" do
           let(:hal_browser_enabled) { false }
-          let(:env) { {'HTTP_ACCEPT' => 'text/html;application/hal+json'} }
-          let(:path) { '/something' }
+          let(:env) { {"HTTP_ACCEPT" => "text/html;application/hal+json"} }
+          let(:path) { "/something" }
 
           it "does not redirect to the HAL Browser" do
             expect(subject.status).to_not eq 303
@@ -122,27 +122,27 @@ module PactBroker
       end
 
       context "when a .csv extension is specified" do
-        let(:path) { '/groups/Some%20Consumer.csv' }
+        let(:path) { "/groups/Some%20Consumer.csv" }
 
         it "returns the CSV Content-Type" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to eq "text/csv;charset=utf-8"
+          expect(subject.headers["Content-Type"]).to eq "text/csv;charset=utf-8"
         end
       end
 
     end
 
     context "when the Accept header is */* (default curl)" do
-      let(:env) { { 'HTTP_ACCEPT' => '*/*' } }
+      let(:env) { { "HTTP_ACCEPT" => "*/*" } }
 
-      subject { get path, '', env; last_response }
+      subject { get path, "", env; last_response }
 
       describe "a request for root" do
         let(:path) { "/" }
 
         it "returns an API response" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'application/hal+json'
+          expect(subject.headers["Content-Type"]).to include "application/hal+json"
         end
       end
     end
@@ -150,14 +150,14 @@ module PactBroker
     context "when no Accept header is specified" do
       let(:env) { {} }
 
-      subject { get path, '', env; last_response }
+      subject { get path, "", env; last_response }
 
       describe "a request for root" do
         let(:path) { "/" }
 
         it "returns an API response" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to_not include 'html'
+          expect(subject.headers["Content-Type"]).to_not include "html"
         end
       end
 
@@ -166,16 +166,16 @@ module PactBroker
 
         it "returns the API representation of the group" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to_not include 'html'
+          expect(subject.headers["Content-Type"]).to_not include "html"
         end
       end
 
       describe "a request for a stylesheet" do
-        let(:path) { '/stylesheets/pact.css' }
+        let(:path) { "/stylesheets/pact.css" }
 
         it "returns the stylesheet" do
           expect(subject.status).to eq 200
-          expect(subject.headers['Content-Type']).to include 'text/css'
+          expect(subject.headers["Content-Type"]).to include "text/css"
         end
       end
     end
@@ -199,7 +199,7 @@ module PactBroker
     describe "when a resource identifier contains a slash" do
       let(:path) { "/pacticipants/Foo/versions/1.2.3/tags/feat%2Fbar" }
 
-      subject { put path, nil, {'CONTENT_TYPE' => 'application/json'}; last_response }
+      subject { put path, nil, {"CONTENT_TYPE" => "application/json"}; last_response }
 
       it "returns a success status" do
         expect(subject.status).to eq 201