pact_broker 2.77.0 → 2.80.0

data/script/seed-example-matrix.rb

@@ -2,23 +2,23 @@
 
 raise "Please supply database path" unless ARGV[0]
 
-$LOAD_PATH.unshift './lib'
-$LOAD_PATH.unshift './spec'
-$LOAD_PATH.unshift './tasks'
-ENV['RACK_ENV'] = 'development'
-require 'sequel'
-require 'logger'
-DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => 'utf8'}
+$LOAD_PATH.unshift "./lib"
+$LOAD_PATH.unshift "./spec"
+$LOAD_PATH.unshift "./tasks"
+ENV["RACK_ENV"] = "development"
+require "sequel"
+require "logger"
+DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => "utf8"}
 #DATABASE_CREDENTIALS = {adapter: "postgres", database: "pact_broker", username: 'pact_broker', password: 'pact_broker', :encoding => 'utf8'}
 
 connection = Sequel.connect(DATABASE_CREDENTIALS)
 connection.timezone = :utc
-require 'pact_broker/db'
+require "pact_broker/db"
 PactBroker::DB.connection = connection
-require 'pact_broker'
-require 'support/test_data_builder'
+require "pact_broker"
+require "support/test_data_builder"
 
-require 'database/table_dependency_calculator'
+require "database/table_dependency_calculator"
 PactBroker::Database::TableDependencyCalculator.call(connection).each do | table_name |
   connection[table_name].delete
 end

data/script/seed-for-webhook-test.rb

@@ -2,39 +2,39 @@
 
 raise "Please supply database path" unless ARGV[0]
 
-$LOAD_PATH.unshift './lib'
-$LOAD_PATH.unshift './spec'
-$LOAD_PATH.unshift './tasks'
-ENV['RACK_ENV'] = 'development'
-require 'sequel'
-require 'logger'
-DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => 'utf8'}.tap { |it| puts it }
+$LOAD_PATH.unshift "./lib"
+$LOAD_PATH.unshift "./spec"
+$LOAD_PATH.unshift "./tasks"
+ENV["RACK_ENV"] = "development"
+require "sequel"
+require "logger"
+DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => "utf8"}.tap { |it| puts it }
 #DATABASE_CREDENTIALS = {adapter: "postgres", database: "pact_broker", username: 'pact_broker', password: 'pact_broker', :encoding => 'utf8'}
 
 connection = Sequel.connect(DATABASE_CREDENTIALS)
 connection.timezone = :utc
 # Uncomment these lines to open a pry session for inspecting the database
 
-require 'pact_broker/db'
+require "pact_broker/db"
 PactBroker::DB.connection = connection
-require 'pact_broker'
-require 'support/test_data_builder'
+require "pact_broker"
+require "support/test_data_builder"
 
-require 'database/table_dependency_calculator'
+require "database/table_dependency_calculator"
 PactBroker::Database::TableDependencyCalculator.call(connection).each do | table_name |
   connection[table_name].delete
 end
 
 webhook_body = {
-  'pactUrl' => '${pactbroker.pactUrl}',
-  'verificationResultUrl' => '${pactbroker.verificationResultUrl}',
-  'consumerVersionNumber' => '${pactbroker.consumerVersionNumber}',
-  'providerVersionNumber' => '${pactbroker.providerVersionNumber}',
-  'providerVersionTags' => '${pactbroker.providerVersionTags}',
-  'consumerVersionTags' => '${pactbroker.consumerVersionTags}',
-  'consumerName' => '${pactbroker.consumerName}',
-  'providerName' => '${pactbroker.providerName}',
-  'githubVerificationStatus' => '${pactbroker.githubVerificationStatus}'
+  "pactUrl" => "${pactbroker.pactUrl}",
+  "verificationResultUrl" => "${pactbroker.verificationResultUrl}",
+  "consumerVersionNumber" => "${pactbroker.consumerVersionNumber}",
+  "providerVersionNumber" => "${pactbroker.providerVersionNumber}",
+  "providerVersionTags" => "${pactbroker.providerVersionTags}",
+  "consumerVersionTags" => "${pactbroker.consumerVersionTags}",
+  "consumerName" => "${pactbroker.consumerName}",
+  "providerName" => "${pactbroker.providerName}",
+  "githubVerificationStatus" => "${pactbroker.githubVerificationStatus}"
 }
 
   # .create_global_webhook(
@@ -48,12 +48,12 @@ PactBroker.configuration.base_equality_only_on_content_that_affects_verification
 
 TestDataBuilder.new
     .create_global_contract_published_webhook(
-      method: 'POST',
+      method: "POST",
       url: "http://localhost:9292/pact-changed-webhook",
       body: webhook_body.to_json,
       username: "foo",
       password: "bar")
     .create_global_verification_webhook(
-      method: 'POST',
+      method: "POST",
       url: "http://localhost:9292/verification-published-webhook",
       body: webhook_body.to_json)

data/script/seed-matrix.rb

@@ -2,21 +2,21 @@
 
 raise "Please supply database path" unless ARGV[0]
 
-$LOAD_PATH.unshift './lib'
-$LOAD_PATH.unshift './spec'
-$LOAD_PATH.unshift './tasks'
-ENV['RACK_ENV'] = 'development'
-require 'sequel'
-require 'logger'
-DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => 'utf8'}
+$LOAD_PATH.unshift "./lib"
+$LOAD_PATH.unshift "./spec"
+$LOAD_PATH.unshift "./tasks"
+ENV["RACK_ENV"] = "development"
+require "sequel"
+require "logger"
+DATABASE_CREDENTIALS = {logger: Logger.new($stdout), adapter: "sqlite", database: ARGV[0], :encoding => "utf8"}
 connection = Sequel.connect(DATABASE_CREDENTIALS)
 connection.timezone = :utc
-require 'pact_broker/db'
+require "pact_broker/db"
 PactBroker::DB.connection = connection
-require 'pact_broker'
-require 'support/test_data_builder'
+require "pact_broker"
+require "support/test_data_builder"
 
-require 'database/table_dependency_calculator'
+require "database/table_dependency_calculator"
 PactBroker::Database::TableDependencyCalculator.call(connection).each do | table_name |
   connection[table_name].delete
 end
@@ -41,20 +41,20 @@ A -> B ->  C
 TestDataBuilder.new
   .create_pact_with_hierarchy("A", "1", "B")
   .create_consumer_version_tag("master")
-  .create_verification(provider_version: '1', success: false, execution_date: Date.today - 1 )
-  .create_verification(provider_version: '1', number: 2, success: true)
-  .create_verification(provider_version: '2', number: 3)
-  .create_verification(provider_version: '4', number: 4)
+  .create_verification(provider_version: "1", success: false, execution_date: Date.today - 1 )
+  .create_verification(provider_version: "1", number: 2, success: true)
+  .create_verification(provider_version: "2", number: 3)
+  .create_verification(provider_version: "4", number: 4)
   .create_provider_version("5")
   .use_consumer("B")
   .use_consumer_version("1")
   .create_consumer_version_tag("master")
   .create_provider("C")
   .create_pact
-  .create_verification(provider_version: '1', success: false)
+  .create_verification(provider_version: "1", success: false)
   .use_consumer_version("2")
   .create_pact
-  .create_verification(provider_version: '2', success: true)
+  .create_verification(provider_version: "2", success: true)
   .create_consumer_version("3")
   .create_pact
   .use_consumer("A")
@@ -62,14 +62,14 @@ TestDataBuilder.new
   .create_consumer_version_tag("master")
   .use_provider("B")
   .create_pact
-  .create_verification(provider_version: '5')
+  .create_verification(provider_version: "5")
   .create_pact_with_hierarchy("X", "1", "Z")
   .create_consumer_version_tag("feat-x")
-  .create_verification(provider_version: '6')
+  .create_verification(provider_version: "6")
   .create_consumer_version("2")
   .create_consumer_version_tag("feat-y")
   .create_pact
-  .create_verification(provider_version: '7')
+  .create_verification(provider_version: "7")
   .create_consumer_version("3")
   .create_consumer_version_tag("feat-x")
   .create_consumer_version_tag("feat-y")

data/script/seed.rb

@@ -2,16 +2,16 @@
 
 raise "Please supply database path" unless ARGV[0]
 
-$LOAD_PATH.unshift './lib'
-$LOAD_PATH.unshift './spec'
-$LOAD_PATH.unshift './tasks'
-ENV['RACK_ENV'] = 'development'
-require 'sequel'
-require 'logger'
-require 'stringio'
-logger = Logger.new($stdout)
+$LOAD_PATH.unshift "./lib"
+$LOAD_PATH.unshift "./spec"
+$LOAD_PATH.unshift "./tasks"
+ENV["RACK_ENV"] = "development"
+require "sequel"
+require "logger"
+require "stringio"
+# logger = Logger.new($stdout)
 logger = Logger.new(StringIO.new)
-DATABASE_CREDENTIALS = {logger: logger, adapter: "sqlite", database: ARGV[0], :encoding => 'utf8'}.tap { |it| puts it }
+DATABASE_CREDENTIALS = {logger: logger, adapter: "sqlite", database: ARGV[0], :encoding => "utf8"}.tap { |it| puts it }
 #DATABASE_CREDENTIALS = {adapter: "postgres", database: "pact_broker", username: 'pact_broker', password: 'pact_broker', :encoding => 'utf8'}
 
 connection = Sequel.connect(DATABASE_CREDENTIALS)
@@ -21,29 +21,29 @@ connection.timezone = :utc
 # require 'pry'; pry(binding);
 # exit
 
-require 'pact_broker/db'
+require "pact_broker/db"
 PactBroker::DB.connection = connection
-require 'pact_broker'
-require 'support/test_data_builder'
+require "pact_broker"
+require "support/test_data_builder"
 
-require 'database/table_dependency_calculator'
+require "database/table_dependency_calculator"
 PactBroker::Database::TableDependencyCalculator.call(connection).each do | table_name |
   connection[table_name].delete
 end
 
-  # .create_webhook(method: 'GET', url: 'https://localhost:9393?url=${pactbroker.pactUrl}', body: '${pactbroker.pactUrl}')
+# .create_webhook(method: 'GET', url: 'https://localhost:9393?url=${pactbroker.pactUrl}', body: '${pactbroker.pactUrl}')
 
-webhook_body = {
-  'pactUrl' => '${pactbroker.pactUrl}',
-  'verificationResultUrl' => '${pactbroker.verificationResultUrl}',
-  'consumerVersionNumber' => '${pactbroker.consumerVersionNumber}',
-  'providerVersionNumber' => '${pactbroker.providerVersionNumber}',
-  'providerVersionTags' => '${pactbroker.providerVersionTags}',
-  'consumerVersionTags' => '${pactbroker.consumerVersionTags}',
-  'consumerName' => '${pactbroker.consumerName}',
-  'providerName' => '${pactbroker.providerName}',
-  'githubVerificationStatus' => '${pactbroker.githubVerificationStatus}'
-}
+# webhook_body = {
+#   'pactUrl' => '${pactbroker.pactUrl}',
+#   'verificationResultUrl' => '${pactbroker.verificationResultUrl}',
+#   'consumerVersionNumber' => '${pactbroker.consumerVersionNumber}',
+#   'providerVersionNumber' => '${pactbroker.providerVersionNumber}',
+#   'providerVersionTags' => '${pactbroker.providerVersionTags}',
+#   'consumerVersionTags' => '${pactbroker.consumerVersionTags}',
+#   'consumerName' => '${pactbroker.consumerName}',
+#   'providerName' => '${pactbroker.providerName}',
+#   'githubVerificationStatus' => '${pactbroker.githubVerificationStatus}'
+# }
 
 PactBroker.configuration.base_equality_only_on_content_that_affects_verification_results = false
 

data/script/test/approval-all.sh

@@ -0,0 +1,6 @@
+#!/bin/sh
+
+for file in $(find spec/fixtures/approvals -ipath "*.received.*"); do
+  approved_path=$(echo "$file" | sed 's/received/approved/')
+  mv "$file" "$approved_path"
+done;

data/script/webhook-server.ru

@@ -1,10 +1,10 @@
-require 'rack/utils'
+require "rack/utils"
 STDOUT.sync = true
 puts "Starting webhook server"
 
 run -> (env) {
   status = 200
-  puts Rack::Utils.parse_nested_query(env['QUERY_STRING']) if env['QUERY_STRING'] && env['QUERY_STRING'] != ''
-  puts env['rack.input'].read
+  puts Rack::Utils.parse_nested_query(env["QUERY_STRING"]) if env["QUERY_STRING"] && env["QUERY_STRING"] != ""
+  puts env["rack.input"].read
   [status, {"Content-Type" => "text/plain"}, ["Webhook response.\n"]]
 }

data/spec/features/base_equality_only_on_content_that_affects_verification_results_spec.rb

@@ -1,9 +1,9 @@
 RSpec.describe "base_equality_only_on_content_that_affects_verification_results" do
   let(:td) { TestDataBuilder.new }
-  let(:json_content_1) { load_fixture('foo-bar.json') }
+  let(:json_content_1) { load_fixture("foo-bar.json") }
   let(:json_content_2) do
-    pact_hash = load_json_fixture('foo-bar.json')
-    pact_hash['interactions'] = pact_hash['interactions'].reverse
+    pact_hash = load_json_fixture("foo-bar.json")
+    pact_hash["interactions"] = pact_hash["interactions"].reverse
     pact_hash.to_json
   end
   let(:base_equality_only_on_content_that_affects_verification_results) { true }

data/spec/features/can_i_deploy_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe "can i deploy" do
 
   let(:response_body) { JSON.parse(subject.body, symbolize_names: true) }
 
-  subject { get("/can-i-deploy", query, { 'HTTP_ACCEPT' => 'application/hal+json'}) }
+  subject { get("/can-i-deploy", query, { "HTTP_ACCEPT" => "application/hal+json"}) }
 
   it "returns the matrix response" do
     expect(subject).to be_a_hal_json_success_response

data/spec/features/create_environment_spec.rb

@@ -0,0 +1,47 @@
+require "pact_broker/api/pact_broker_urls"
+
+describe "Creating an environment" do
+  let(:path) { PactBroker::Api::PactBrokerUrls.environments_url }
+  let(:headers) { { "CONTENT_TYPE" => "application/json" } }
+  let(:response_body) { JSON.parse(subject.body, symbolize_names: true)}
+  let(:environment_hash) do
+    {
+      name: "test",
+      displayName: "Test",
+      production: false,
+      contacts: [
+        { name: "Team Awesome", details: { email: "foo@bar.com", arbitraryThing: "thing" } }
+      ]
+    }
+  end
+
+  subject { post(path, environment_hash.to_json, headers) }
+
+  it "returns a 201 response" do
+    expect(subject.status).to be 201
+  end
+
+  it "returns the Location header" do
+    expect(subject.headers["Location"]).to eq PactBroker::Api::PactBrokerUrls.environment_url(PactBroker::Deployments::Environment.order(:id).last, "http://example.org")
+  end
+
+  it "returns a JSON Content Type" do
+    expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
+  end
+
+  it "returns the newly created environment" do
+    expect(response_body).to include environment_hash.merge(name: "test")
+    expect(response_body[:uuid]).to_not be nil
+  end
+
+  context "with invalid params" do
+    before do
+      td.create_environment("test")
+    end
+
+    it "returns a 400 response" do
+      expect(subject.status).to be 400
+      expect(response_body[:errors]).to_not be nil
+    end
+  end
+end

data/spec/features/create_pacticipant_spec.rb

@@ -1,12 +1,18 @@
-require 'support/test_data_builder'
-
 describe "Creating a pacticipant" do
   let(:path) { "/pacticipants" }
-  let(:headers) { {'CONTENT_TYPE' => 'application/json'} }
+  let(:headers) { {"CONTENT_TYPE" => "application/json"} }
   let(:response_body) { JSON.parse(last_response.body, symbolize_names: true)}
-  let(:pacticipant_hash) { { name: 'Foo Thing' } }
+  let(:pacticipant_hash) do
+    {
+      name: "Foo Thing",
+      mainBranch: "main",
+      repositoryUrl: "http://url",
+      repositoryName: "foo-thing",
+      repositoryNamespace: "some-group"
+    }
+  end
 
-  subject { post path, pacticipant_hash.to_json, headers; last_response }
+  subject { post(path, pacticipant_hash.to_json, headers) }
 
   it "returns a 201 response" do
     subject
@@ -15,12 +21,12 @@ describe "Creating a pacticipant" do
 
   it "returns the Location header" do
     subject
-    expect(last_response.headers['Location']).to eq 'http://example.org/pacticpants/Foo%20Thing'
+    expect(last_response.headers["Location"]).to eq "http://example.org/pacticpants/Foo%20Thing"
   end
 
   it "returns a JSON Content Type" do
     subject
-    expect(last_response.headers['Content-Type']).to eq 'application/hal+json;charset=utf-8'
+    expect(last_response.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
   end
 
   it "returns the newly created webhook" do

data/spec/features/create_tag_spec.rb

@@ -0,0 +1,32 @@
+describe "Creating a tag" do
+  let(:path) { "/pacticipants/Foo/versions/1234/tags/foo" }
+  let(:headers) { { "CONTENT_TYPE" => "application/json" } }
+  let(:response_body) { JSON.parse(subject.body, symbolize_names: true)}
+
+  subject { put(path, {}, headers) }
+
+  it "returns a 201 response" do
+    expect(subject.status).to be 201
+  end
+
+  it "returns a HAL JSON Content Type" do
+    expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
+  end
+
+  it "returns the newly created tag" do
+    expect(response_body).to include name: "foo"
+  end
+
+  context "when the tag already exists" do
+    before do
+      td.subtract_day
+        .create_consumer("Foo")
+        .create_consumer_version("1234")
+        .create_consumer_version_tag("foo")
+    end
+
+    it "returns a 200 response" do
+      expect(subject.status).to be 200
+    end
+  end
+end

data/spec/features/create_version_spec.rb

@@ -1,44 +1,58 @@
 describe "Creating a pacticipant version" do
   let(:path) { "/pacticipants/Foo/versions/1234" }
-  let(:headers) { { 'CONTENT_TYPE' => 'application/json' } }
+  let(:headers) { { "CONTENT_TYPE" => content_type } }
+  let(:content_type) { "application/json" }
   let(:response_body) { JSON.parse(subject.body, symbolize_names: true)}
-  let(:version_hash) { { branch: "main", buildUrl: "http://build" } }
+  let(:version_hash) do
+    {
+      branch: "main",
+      buildUrl: "http://build",
+      tags: [{ name: "foo" }, { name: "bar" }]
+    }
+  end
 
-  subject { put(path, version_hash.to_json, headers) }
+  context "with a PUT" do
+    subject { put(path, version_hash.to_json, headers) }
 
-  it "returns a 201 response" do
-    expect(subject.status).to be 201
-  end
+    it "returns a 201 response" do
+      expect(subject.status).to be 201
+    end
 
-  it "returns a HAL JSON Content Type" do
-    expect(subject.headers['Content-Type']).to eq 'application/hal+json;charset=utf-8'
-  end
+    it "returns a HAL JSON Content Type" do
+      expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
+    end
 
-  it "returns the newly created version" do
-    expect(response_body).to include version_hash
-  end
+    it "returns the newly created version" do
+      expect(response_body).to include branch: "main", buildUrl: "http://build"
+      expect(response_body[:_embedded][:tags].size).to eq 2
+    end
 
-  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")
+    it "creates the specified tags" do
+      expect { subject }.to change { PactBroker::Domain::Tag.count }.by(2)
     end
+  end
+
+  context "with a PATCH" do
+    let(:content_type) { "application/merge-patch+json" }
 
-    let(:version_hash) { { branch: "new-branch" } }
+    subject { patch(path, version_hash.to_json, headers) }
 
-    it "overwrites the direct properties" do
-      expect(response_body[:branch]).to eq "new-branch"
-      expect(response_body).to_not have_key(:buildUrl)
+    it "returns a 201 response" do
+      expect(subject.status).to be 201
     end
 
-    it "does not change the tags" do
-      expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").tags }
+    it "returns a HAL JSON Content Type" do
+      expect(subject.headers["Content-Type"]).to eq "application/hal+json;charset=utf-8"
     end
 
-    it "does not change the created date" do
-      expect { subject }.to_not change { PactBroker::Domain::Version.for("Foo", "1234").created_at }
+    it "returns the newly created version" do
+      expect(response_body).to include branch: "main", buildUrl: "http://build"
+      expect(response_body[:_embedded][:tags].size).to eq 2
+    end
+
+    it "creates the specified tags" do
+      expect { subject }.to change { PactBroker::Domain::Tag.count }.by(2)
     end
   end
+
 end