pact_broker 2.0.5 → 2.1.0

data/spec/features/delete_version_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Delete version" do
 
@@ -8,7 +8,7 @@ describe "Delete version" do
   subject { delete path; last_response }
 
   before do
-    ProviderStateBuilder.new
+    TestDataBuilder.new
       .create_consumer("Another Consumer")
       .create_consumer("Consumer")
       .create_consumer_version("1.2.3")

data/spec/features/delete_webhook_spec.rb

@@ -1,9 +1,9 @@
-require 'support/provider_state_builder'
+require 'support/test_data_builder'
 
 describe "Delete a webhook" do
 
   let!(:webhook) do
-    ProviderStateBuilder.new
+    TestDataBuilder.new
       .create_consumer("Some Consumer")
       .create_consumer_version("Some Provider")
       .create_provider

data/spec/features/get_diff_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get diff between versions" do
 
@@ -17,7 +17,7 @@ describe "Get diff between versions" do
   let(:pact_content_version_3) { pact_content_version_2 }
 
   before do
-    ProviderStateBuilder.new
+    TestDataBuilder.new
       .create_consumer("Consumer")
       .create_provider("Provider")
       .create_consumer_version("1")

data/spec/features/get_latest_tagged_pact_spec.rb

@@ -0,0 +1,22 @@
+describe "retrieving the latest tagged pact" do
+
+  let(:path) { "/pacts/provider/Provider/consumer/Consumer/latest/prod"}
+
+  subject { get path; last_response  }
+  let(:json_response_body) { JSON.parse(subject.body, symbolize_names: true) }
+
+  before do
+    TestDataBuilder.new
+      .create_consumer("Consumer")
+      .create_provider("Provider")
+      .create_consumer_version("1.2.3")
+      .create_consumer_version_tag("prod")
+      .create_pact
+      .create_consumer_version("4.5.6")
+      .create_pact
+  end
+
+  it "returns the latest tagged pact version" do
+    expect(json_response_body[:_links][:self][:href]).to end_with("1.2.3")
+  end
+end

data/spec/features/get_latest_untagged_pact_spec.rb

@@ -0,0 +1,22 @@
+describe "retrieving the latest untagged pact" do
+
+  let(:path) { "/pacts/provider/Provider/consumer/Consumer/latest-untagged"}
+
+  subject { get path; last_response  }
+  let(:json_response_body) { JSON.parse(subject.body, symbolize_names: true) }
+
+  before do
+    TestDataBuilder.new
+      .create_consumer("Consumer")
+      .create_provider("Provider")
+      .create_consumer_version("1.2.3")
+      .create_pact
+      .create_consumer_version("4.5.6")
+      .create_consumer_version_tag('prod')
+      .create_pact
+  end
+
+  it "returns the latest untagged pact version" do
+    expect(json_response_body[:_links][:self][:href]).to end_with("1.2.3")
+  end
+end

data/spec/features/get_pact_spec.rb

@@ -7,7 +7,7 @@ describe "retrieving a pact" do
     let(:path) { "/pacts/provider/a%20provider/consumer/a%20consumer/version/1.2.3A" }
 
     before do
-      ProviderStateBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3a", "A Provider"
+      TestDataBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3a", "A Provider"
     end
 
     context "when case sensitivity is turned on" do
@@ -35,7 +35,7 @@ describe "retrieving a pact" do
     let(:path) { "/pacts/provider/a%20provider/consumer/a%20consumer/latest/PROD" }
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_consumer("A Consumer")
         .create_consumer_version("1.2.3")
         .create_consumer_version_tag("prod")

data/spec/features/get_pact_versions_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get pact versions" do
 
@@ -10,7 +10,7 @@ describe "Get pact versions" do
   context "when the pacts exist" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_provider("Provider")
         .create_consumer("Consumer")
         .create_consumer_version("1.0.0")

data/spec/features/get_previous_distinct_version.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get previous distinct version of pact" do
 
@@ -16,7 +16,7 @@ describe "Get previous distinct version of pact" do
   let(:pact_content_version_3) { pact_content_version_2 }
 
   before do
-    ProviderStateBuilder.new
+    TestDataBuilder.new
       .create_consumer("Consumer")
       .create_provider("Provider")
       .create_consumer_version("1")

data/spec/features/get_provider_pacts_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get provider pacts" do
 
@@ -10,7 +10,7 @@ describe "Get provider pacts" do
   context "when the provider exists" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_provider("Provider")
         .create_consumer("Consumer")
         .create_consumer_version("1.0.0")

data/spec/features/get_verifications_for_consumer_version_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get verifications for consumer version" do
 
@@ -10,7 +10,7 @@ describe "Get verifications for consumer version" do
   context "when the consumer exists" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_provider("Provider")
         .create_consumer("Consumer")
         .create_consumer_version("1.2.3")

data/spec/features/get_version_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get version" do
 
@@ -10,7 +10,7 @@ describe "Get version" do
   context "when the version exists" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_consumer("Another Consumer")
         .create_consumer("Consumer")
         .create_consumer_version("1.2.3")
@@ -32,7 +32,7 @@ describe "Get version" do
   context "when the version does not exist" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_consumer("Consumer")
         .create_version("1.2.4")
     end

data/spec/features/get_versions_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Get versions" do
 
@@ -10,7 +10,7 @@ describe "Get versions" do
   context "when the pacticipant exists" do
 
     before do
-      ProviderStateBuilder.new
+      TestDataBuilder.new
         .create_consumer("Consumer")
         .create_consumer_version("1.0.0")
         .create_consumer_version("1.0.1")

data/spec/features/merge_pact_spec.rb

@@ -1,4 +1,4 @@
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 
 describe "Merging a pact" do
 
@@ -29,7 +29,7 @@ describe "Merging a pact" do
     let(:merged_pact_content) { load_fixture('a_consumer-a_provider-merged.json') }
 
     before do
-      ProviderStateBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3", "A Provider", existing_pact_content
+      TestDataBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3", "A Provider", existing_pact_content
     end
 
     it "returns a 200 Success" do

data/spec/features/publish_pact_spec.rb

@@ -23,7 +23,7 @@ describe "Publishing a pact" do
   context "when a pact for this consumer version does exist" do
 
     before do
-      ProviderStateBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3", "A Provider"
+      TestDataBuilder.new.create_pact_with_hierarchy "A Consumer", "1.2.3", "A Provider"
     end
 
     it "returns a 200 Success" do

data/spec/features/publish_verification_spec.rb

@@ -9,7 +9,7 @@ describe "Recording a pact verification" do
   subject { post path, verification_content, {'CONTENT_TYPE' => 'application/json' }; last_response  }
 
   let(:pact) do
-    ProviderStateBuilder.new
+    TestDataBuilder.new
       .create_provider("Provider")
       .create_consumer("Consumer")
       .create_consumer_version("1.2.3")

data/spec/integration/app_spec.rb

@@ -7,7 +7,7 @@ module PactBroker
   describe App do
 
     before do
-      ProviderStateBuilder.new.create_pact_with_hierarchy 'Some Consumer', '1.0', 'Some Provider'
+      TestDataBuilder.new.create_pact_with_hierarchy 'Some Consumer', '1.0', 'Some Provider'
     end
 
     let(:hal_browser_enabled) { true }

data/spec/integration/endpoints/group.rb

@@ -6,7 +6,7 @@ describe "/groups/{pacticipant-name}" do
 
   describe "GET" do
     before do
-      ProviderStateBuilder.new.create_pact_with_hierarchy "Consumer", "1.2.3", "Provider"
+      TestDataBuilder.new.create_pact_with_hierarchy "Consumer", "1.2.3", "Provider"
       get "/groups/Consumer"
     end
 

data/spec/lib/pact_broker/api/decorators/embedded_tag_decorator_spec.rb

@@ -1,6 +1,6 @@
 require 'pact_broker/api/decorators/embedded_tag_decorator'
 require 'pact_broker/tags/repository'
-require 'support/provider_state_builder'
+require 'support/test_data_builder'
 
 module PactBroker
 
@@ -11,7 +11,7 @@ module PactBroker
       describe EmbeddedTagDecorator do
 
         let(:tag) do
-          ProviderStateBuilder.new
+          TestDataBuilder.new
             .create_consumer("Consumer")
             .create_version("1.2.3")
             .create_tag("prod")

data/spec/lib/pact_broker/api/decorators/embedded_version_decorator_spec.rb

@@ -7,7 +7,7 @@ module PactBroker
       describe EmbeddedVersionDecorator do
 
         let(:version) do
-          ProviderStateBuilder.new
+          TestDataBuilder.new
             .create_consumer("Consumer")
             .create_version("1.2.3")
           PactBroker::Versions::Repository.new.find_by_pacticipant_name_and_number "Consumer", "1.2.3"

data/spec/lib/pact_broker/api/decorators/latest_pact_decorator_spec.rb

@@ -9,7 +9,7 @@ module PactBroker
 
       describe LatestPactDecorator do
 
-        let(:pact) { RepresentablePact.new(ProviderStateBuilder.new.create_pact_with_hierarchy 'Consumer', '1.2.3', 'Provider') }
+        let(:pact) { RepresentablePact.new(TestDataBuilder.new.create_pact_with_hierarchy 'Consumer', '1.2.3', 'Provider') }
         let(:base_url) { 'http://example.org' }
 
         subject { JSON.parse LatestPactDecorator.new(pact).to_json(user_options: { base_url: base_url }), symbolize_names: true}

data/spec/lib/pact_broker/api/decorators/relationships_csv_decorator_spec.rb

@@ -9,7 +9,7 @@ module PactBroker
 
       describe RelationshipsCsvDecorator do
 
-        let(:pact) { ProviderStateBuilder.new.create_pact_with_hierarchy "My Consumer", "1.0", "My Provider"}
+        let(:pact) { TestDataBuilder.new.create_pact_with_hierarchy "My Consumer", "1.0", "My Provider"}
         let(:pacts) { [pact]}
 
 

data/spec/lib/pact_broker/api/decorators/representable_pact_spec.rb

@@ -1,10 +1,10 @@
 require 'spec_helper'
-require 'spec/support/provider_state_builder'
+require 'spec/support/test_data_builder'
 require 'pact_broker/api/decorators/representable_pact'
 
 module PactBroker::Api::Decorators
   describe RepresentablePact do
-    let(:pact) { ProviderStateBuilder.new.create_pact_with_hierarchy 'Consumer', '1.2.3', 'Provider' }
+    let(:pact) { TestDataBuilder.new.create_pact_with_hierarchy 'Consumer', '1.2.3', 'Provider' }
 
     subject { RepresentablePact.new(pact) }
 

data/spec/lib/pact_broker/api/decorators/tag_decorator_spec.rb

@@ -1,7 +1,7 @@
 require 'pact_broker/api/decorators/tag_decorator'
 require 'pact_broker/tags/repository'
 
-require 'support/provider_state_builder'
+require 'support/test_data_builder'
 
 module PactBroker
 
@@ -12,7 +12,7 @@ module PactBroker
       describe TagDecorator do
 
         let(:tag) do
-          ProviderStateBuilder.new
+          TestDataBuilder.new
             .create_consumer("Consumer")
             .create_version("1.2.3")
             .create_tag("prod")

data/spec/lib/pact_broker/api/decorators/version_decorator_spec.rb

@@ -7,7 +7,7 @@ module PactBroker
       describe VersionDecorator do
 
         let(:version) do
-          ProviderStateBuilder.new
+          TestDataBuilder.new
             .create_consumer("Consumer")
             .create_consumer_version("1.2.3")
             .create_consumer_version_tag("prod")

data/spec/lib/pact_broker/api/decorators/versions_decorator_spec.rb

@@ -24,7 +24,7 @@ module PactBroker
 
         context "with versions" do
           let(:version) do
-            ProviderStateBuilder.new
+            TestDataBuilder.new
               .create_consumer("Consumer")
               .create_consumer_version("1.2.3")
               .create_consumer_version_tag("prod")

data/spec/lib/pact_broker/app_spec.rb

@@ -60,6 +60,134 @@ module PactBroker
       end
     end
 
+    describe "authenticate" do
+      before do
+        PactBroker.configuration.authenticate do | resource, authorization_header, options |
+          authorization_header == 'letmein'
+        end
+      end
+
+      context "with an invalid Authorization header" do
+        it "returns a 401" do
+          get "/", {}, {'HTTP_AUTHORIZATION' => 'wrong'}
+          expect(last_response.status).to eq 401
+        end
+      end
+
+      context "with valid Authorization header" do
+        it "returns a 200" do
+          get "/", {}, {'HTTP_AUTHORIZATION' => 'letmein'}
+          expect(last_response.status).to eq 200
+        end
+      end
+    end
+
+    describe "authenticate_with_basic_auth" do
+      before do
+        PactBroker.configuration.authenticate_with_basic_auth do | resource, username, password, options |
+          username == 'username' && password == 'password'
+        end
+      end
+
+      context "with a request for the API with incorrect username or password" do
+        it "returns a 401" do
+          basic_authorize 'foo', 'password'
+          get "/"
+          expect(last_response.status).to eq 401
+        end
+      end
+
+      context "with a request for the UI with incorrect username or password" do
+        it "returns a 401" do
+          basic_authorize 'foo', 'password'
+          get "/", nil, {'HTTP_ACCEPT' => 'text/html'}
+          expect(last_response.status).to eq 401
+        end
+      end
+
+      context "with a request for diagnostics with incorrect username or password" do
+        it "returns a 401" do
+          basic_authorize 'foo', 'password'
+          get "/diagnostic/status/heartbeat"
+          expect(last_response.status).to eq 401
+        end
+      end
+
+      context "with a request for the API with correct username and password" do
+        it "returns a 200" do
+          basic_authorize 'username', 'password'
+          get "/"
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for the UI with correct username or password" do
+        it "returns a 200" do
+          basic_authorize 'username', 'password'
+          get "/", nil, {'HTTP_ACCEPT' => 'text/html'}
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for diagnostics with correct username or password" do
+        it "returns a 200" do
+          basic_authorize 'username', 'password'
+          get "/diagnostic/status/heartbeat"
+          expect(last_response.status).to eq 200
+        end
+      end
+    end
+
+    describe "authorize" do
+      before do
+        PactBroker.configuration.authorize do | resource, options |
+          resource.request.headers['Role'] == 'important'
+        end
+      end
+
+      context "with a request for the API with an authorized request" do
+        it "returns a 200" do
+          get "/", nil, {'HTTP_ROLE' => 'important'}
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for the UI with an authorized request" do
+        it "returns a 200" do
+          get "/", nil, {'HTTP_ACCEPT' => 'text/html', 'HTTP_ROLE' => 'important'}
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for diagnostics with an authorized request" do
+        it "returns a 200" do
+          get "/diagnostic/status/heartbeat", nil, {'HTTP_ROLE' => 'important'}
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for the API with an unauthorized request" do
+        it "returns a 403" do
+          get "/"
+          expect(last_response.status).to eq 403
+        end
+      end
+
+      context "with a request for the UI with an unauthorized request" do
+        it "returns a 200 because there's no point doing authorization on the UI at the moment" do
+          get "/", nil, {'HTTP_ACCEPT' => 'text/html'}
+          expect(last_response.status).to eq 200
+        end
+      end
+
+      context "with a request for diagnostics with an unauthorized request" do
+        it "returns a 403" do
+          get "/diagnostic/status/heartbeat"
+          expect(last_response.status).to eq 403
+        end
+      end
+    end
+
     describe "transactions", no_db_clean: true do
       let(:pact_content) { load_fixture('a_consumer-a_provider.json') }
       let(:path) { "/pacts/provider/A%20Provider/consumer/A%20Consumer/versions/1.2.3" }