pact_broker 2.89.1 → 2.93.0

data/spec/features/update_matrix_spec.rb

@@ -1,146 +0,0 @@
-=begin
-
-Things that should update the matrix (ignoring tags):
-* pact created
-* pact content updated
-* pact deleted
-* verification created
-* verification deleted (not yet implemented)
-
-It's easier to update the matrix at the resource level, so we actually update the matrix when:
-* pacticipant deleted
-* version deleted
-* pact deleted
-
-Things that should update the head matrix
-* All of the above
-* tag created
-* tag deleted
-
-=end
-
-describe "Deleting a resource that affects the matrix" do
-
-  let(:td) { TestDataBuilder.new }
-  let(:response_body_json) { JSON.parse(subject.body) }
-
-  subject { delete path; last_response  }
-
-  before do
-    td.create_pact_with_hierarchy("Foo", "1", "Bar")
-      .create_verification(provider_version: "2")
-  end
-
-  context "deleting a pact" do
-    let(:path) { "/pacts/provider/Bar/consumer/Foo/version/1" }
-
-    it "deletes the relevant lines from the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(-1)
-    end
-
-    it "deletes the relevant lines from the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(-1)
-    end
-  end
-
-  context "deleting a pacticipant" do
-    let(:path) { "/pacticipants/Bar" }
-
-    it "deletes the relevant lines from the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(-1)
-    end
-
-    it "deletes the relevant lines from the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(-1)
-    end
-  end
-
-  context "deleting a version" do
-    let(:path) { "/pacticipants/Foo/versions/1" }
-
-    it "deletes the relevant lines from the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(-1)
-    end
-
-    it "deletes the relevant lines from the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(-1)
-    end
-  end
-
-  context "deleting a tag" do
-    before do
-      td.create_consumer_version_tag("prod")
-    end
-
-    let(:path) { "/pacticipants/Foo/versions/1/tags/prod" }
-
-    it "does not delete any lines from the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(0)
-    end
-
-    it "deletes the relevant lines from the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(-1)
-    end
-  end
-end
-
-describe "Creating a resource that affects the matrix" do
-
-  let(:td) { TestDataBuilder.new }
-  let(:response_body_json) { JSON.parse(subject.body) }
-
-  subject { put(path, nil, {"CONTENT_TYPE" => "application/json"}); last_response }
-
-  context "creating a tag" do
-    before do
-      td.create_pact_with_hierarchy("Foo", "1", "Bar")
-    end
-
-    let(:path) { "/pacticipants/Foo/versions/1/tags/prod" }
-
-    it "adds the relevant lines to the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(1)
-    end
-
-    it "does not add any lines to the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(0)
-    end
-  end
-
-  context "creating a pact" do
-    let(:pact_content) { load_fixture("foo-bar.json") }
-    let(:path) { "/pacts/provider/Bar/consumer/Foo/versions/1.2.3" }
-
-    subject { put path, pact_content, {"CONTENT_TYPE" => "application/json"}; last_response }
-
-    it "adds the relevant lines to the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.count }.by(1)
-    end
-
-    it "adds the relevant lines to the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.count }.by(1)
-    end
-  end
-
-  context "creating a verification" do
-    let(:td) { TestDataBuilder.new }
-    let(:path) { "/pacts/provider/Bar/consumer/Foo/pact-version/#{pact.pact_version_sha}/verification-results" }
-    let(:verification_content) { load_fixture("verification.json") }
-    let(:parsed_response_body) { JSON.parse(subject.body) }
-    let(:pact) { td.pact }
-
-    subject { post path, verification_content, {"CONTENT_TYPE" => "application/json" }; last_response  }
-
-    before do
-      td.create_pact_with_hierarchy("Foo", "1", "Bar")
-    end
-
-    it "updates the relevant lines in the matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::Row.first.provider_version_number }.from(nil).to("4.5.6")
-    end
-
-    it "updates the relevant lines in the head matrix" do
-      expect{ subject }.to change{ PactBroker::Matrix::HeadRow.first.provider_version_number }.from(nil).to("4.5.6")
-    end
-  end
-end

data/spec/features/update_pacticipant_spec.rb

@@ -1,98 +0,0 @@
-describe "Update a pacticipant" do
-
-  let(:request_body) { {"repositoryUrl" => "http://foo"} }
-  let(:path) { "/pacticipants/Some%20Consumer" }
-  let(:response_body_hash) { JSON.parse(subject.body, symbolize_names: true) }
-
-  context "with a PUT" do
-    let(:request_body_hash) do
-      {
-        repositoryUrl: "http://foo",
-        repositoryName: "name",
-        repositoryNamespace: "org",
-        mainBranch: "main"
-      }
-    end
-
-    subject { put(path, request_body_hash.to_json, { "CONTENT_TYPE" => "application/json" }) }
-
-    context "when the pacticipant exists" do
-      before do
-        td.create_pacticipant("Some Consumer")
-      end
-
-      it { is_expected.to be_a_hal_json_success_response }
-
-      it "updates the properties" do
-        expect(response_body_hash).to include(request_body_hash)
-      end
-
-      context "with only some of the properties set" do
-        let(:request_body_hash) do
-          {
-            repositoryUrl: "http://foo",
-          }
-        end
-
-        it "blanks out the missing ones" do
-          expect(response_body_hash[:repositoryName]).to be nil
-        end
-      end
-    end
-
-    context "when the pacticipant does not exist" do
-      it { is_expected.to be_a_404_response }
-    end
-
-    context "with application/merge-patch+json" do
-      subject { put(path, request_body.to_json, {"CONTENT_TYPE" => "application/merge-patch+json" })  }
-
-      its(:status) { is_expected.to eq 415 }
-    end
-
-  end
-
-  context "with a PATCH" do
-    subject { patch(path, request_body.to_json, {"CONTENT_TYPE" => "application/json" })  }
-
-    context "when the pacticipant exists" do
-      before do
-        td.create_pacticipant("Some Consumer", repository_name: "existing")
-      end
-
-      it "returns a 200 OK" do
-        puts subject.body unless subject.status == 200
-        expect(subject.status).to be 200
-      end
-
-      it "leaves any existing properties that were not defined" do
-        expect(response_body_hash[:repositoryName]).to eq "existing"
-      end
-
-      it "returns a json body with the updated pacticipant" do
-        expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
-      end
-    end
-
-    context "with application/merge-patch+json" do
-      before do
-        td.create_pacticipant("Some Consumer", repository_name: "existing")
-      end
-
-      subject { patch(path, request_body.to_json, {"CONTENT_TYPE" => "application/merge-patch+json" })  }
-
-      it "returns a 200 OK" do
-        puts subject.body unless subject.status == 200
-        expect(subject.status).to be 200
-      end
-
-      it "leaves any existing properties that were not defined" do
-        expect(response_body_hash[:repositoryName]).to eq "existing"
-      end
-
-      it "returns a json body with the updated pacticipant" do
-        expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
-      end
-    end
-  end
-end

data/spec/features/update_version_spec.rb

@@ -1,100 +0,0 @@
-describe "Updating a pacticipant version" do
-  let(:path) { "/pacticipants/Foo/versions/1234" }
-  let(:headers) { { "CONTENT_TYPE" => content_type } }
-  let(:content_type) { "application/merge-patch+json" }
-  let(:response_body) { JSON.parse(subject.body, symbolize_names: true)}
-  let(:version_hash) do
-    {
-      buildUrl: "http://build"
-    }
-  end
-
-  context "with a PATCH" do
-    subject { patch(path, version_hash.to_json, headers) }
-
-    context "when the version already exists" do
-      before do
-        td.subtract_day
-          .create_consumer("Foo")
-          .create_consumer_version("1234", branch: original_branch, build_url: "original-build-url")
-          .create_consumer_version_tag("dev")
-      end
-      let(:original_branch) { "original-branch" }
-
-      let(:version_hash) { { buildUrl: "new-build-url" } }
-
-      it "returns a 200" do
-        expect(subject.status).to be 200
-      end
-
-      it "does not overwrites any properties that weren't specified" do
-        expect(response_body[:buildUrl]).to eq "new-build-url"
-      end
-
-      context "when no tags are specified" do
-        it "does not change the tags" do
-          expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").tags }
-        end
-      end
-
-      context "when tags are specified" do
-        let(:version_hash) { { tags: [ { name: "main" }] } }
-
-        it "overwrites the tags" do
-          expect(response_body[:_embedded][:tags].size).to eq 1
-          expect(response_body[:_embedded][:tags].first[:name]).to eq "main"
-        end
-      end
-
-      it "does not change the created date" do
-        expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").created_at }
-      end
-    end
-
-    context "when using a PATCH with application/json" do
-      let(:content_type) { "application/json" }
-
-      its(:status) { is_expected.to eq 415 }
-    end
-
-    context "when using a PUT with application/merge-patch+json" do
-      subject { put(path, version_hash.to_json, headers) }
-
-      its(:status) { is_expected.to eq 415 }
-    end
-  end
-
-  context "with a PUT" do
-    let(:content_type) { "application/json" }
-
-    subject { put(path, version_hash.to_json, headers) }
-
-    context "when the version already exists" do
-      before do
-        td.subtract_day
-          .create_consumer("Foo")
-          .create_consumer_version("1234", branch: "original-branch", build_url: "original-build-url")
-          .create_consumer_version_tag("dev")
-      end
-
-      context "when no tags are specified" do
-        it "does not change the tags" do
-          expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").tags }
-        end
-      end
-
-      context "when tags are specified" do
-        let(:version_hash) { { branch: "original-branch", tags: [ { name: "main" }] } }
-
-        it "overwrites the tags" do
-          expect(response_body[:_embedded][:tags].size).to eq 1
-          expect(response_body[:_embedded][:tags].first[:name]).to eq "main"
-        end
-      end
-
-      it "does not change the created date" do
-        expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").created_at }
-      end
-    end
-  end
-end

data/spec/features/update_webhook_spec.rb

@@ -1,47 +0,0 @@
-require "support/test_data_builder"
-
-describe "Updating a webhook" do
-  let(:webhook) do
-    TestDataBuilder.new
-      .create_pact_with_hierarchy("Some Consumer", "1", "Some Provider")
-      .create_global_webhook
-      .and_return(:webhook)
-  end
-
-  let(:headers) { {"CONTENT_TYPE" => "application/json"} }
-  let(:response_body) { JSON.parse(last_response.body, symbolize_names: true)}
-  let(:webhook_json) { webhook_hash.to_json }
-  let(:webhook_hash) do
-    {
-      description: "trigger build",
-      consumer: {
-        name: "Some Consumer"
-      },
-      enabled: false,
-      events: [{
-        name: "contract_published"
-      }],
-      request: {
-        method: "POST",
-        url: "https://example.org",
-        headers: {
-          :"Content-Type" => "application/json"
-        },
-        body: {
-          a: "body"
-        }
-      }
-    }
-  end
-
-  subject { put(path, webhook_json, headers) }
-
-  let(:path) { "/webhooks/#{webhook.uuid}" }
-
-  context "with valid attributes" do
-    it "returns the newly created webhook" do
-      subject
-      expect(response_body).to include description: "trigger build"
-    end
-  end
-end