pact_broker 2.65.0 → 2.66.0

data/spec/lib/pact_broker/feature_toggle_spec.rb

@@ -7,7 +7,9 @@ module PactBroker
         allow(ENV).to receive(:[]).and_call_original
       end
 
-      subject { FeatureToggle.enabled?(:foo) }
+      let(:ignore_env) { false }
+
+      subject { FeatureToggle.enabled?(:foo, ignore_env) }
 
       context "when RACK_ENV is not production" do
         before do
@@ -28,6 +30,12 @@ module PactBroker
           end
 
           it { is_expected.to be true }
+
+          context "when ignore env is set" do
+            let(:ignore_env) { true }
+
+            it { is_expected.to be false }
+          end
         end
       end
 

data/spec/lib/pact_broker/pacticipants/repository_spec.rb

@@ -154,8 +154,15 @@ module PactBroker
         it "returns all the application versions for the given consumer" do
           expect(subject.collect(&:number)).to eq ["4.5.6", "1.2.3"]
         end
-      end
 
+        context "with pagination options" do
+          subject { Repository.new.find_all_pacticipant_versions_in_reverse_order "Foo", page_number: 1, page_size: 1 }
+
+          it "paginates the query" do
+            expect(subject.collect(&:number)).to eq ["4.5.6"]
+          end
+        end
+      end
     end
   end
 end

data/spec/lib/pact_broker/pacts/latest_tagged_pact_publications_spec.rb

@@ -0,0 +1,99 @@
+require 'pact_broker/pacts/latest_tagged_pact_publications'
+
+module PactBroker
+  module Pacts
+    describe LatestTaggedPactPublications do
+      subject { LatestTaggedPactPublications.for_selector(selector).order(:consumer_name, :consumer_version_order) }
+
+      context "for latest for a specified tag" do
+        before do
+          td.create_pact_with_hierarchy("Foo", "1", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "2", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "3", "Bar")
+            .create_consumer_version_tag("prod")
+        end
+
+        let(:selector) { PactBroker::Matrix::UnresolvedSelector.new(tag: "dev", latest: true) }
+
+        it "returns matching rows" do
+          expect(subject.count).to eq 1
+          expect(subject.first.consumer_version_number).to eq "2"
+        end
+      end
+
+      context "for latest for a specified tag and consumer" do
+        before do
+          td.create_pact_with_hierarchy("Foo", "1", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "2", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo2", "3", "Bar")
+            .create_consumer_version_tag("dev")
+        end
+
+        let(:selector) { PactBroker::Matrix::UnresolvedSelector.new(tag: "dev", latest: true, pacticipant_name: "Foo") }
+
+        it "returns matching rows" do
+          expect(subject.count).to eq 1
+          expect(subject.first.consumer_version_number).to eq "2"
+        end
+      end
+
+      context "for all for a specified tag" do
+        before do
+          td.create_pact_with_hierarchy("Foo", "1", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "2", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "3", "Bar")
+            .create_consumer_version_tag("prod")
+        end
+
+        let(:selector) { PactBroker::Matrix::UnresolvedSelector.new(tag: "dev") }
+
+        it "returns matching rows (which are only the latest)" do
+          expect(subject.count).to eq 1
+          expect(subject.first.consumer_version_number).to eq "2"
+        end
+      end
+
+      context "for latest for any tag" do
+        before do
+          td.create_pact_with_hierarchy("Foo", "1", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "2", "Bar")
+            .create_consumer_version_tag("dev")
+            .create_pact_with_hierarchy("Foo", "3", "Bar")
+            .create_consumer_version_tag("prod")
+        end
+
+        let(:selector) { PactBroker::Matrix::UnresolvedSelector.new(tag: true, latest: true) }
+
+        it "returns matching rows" do
+          expect(subject.count).to eq 2
+          expect(subject.first.consumer_version_number).to eq "2"
+        end
+      end
+
+      context "for latest for any tag with a max age" do
+        before do
+          td.subtract_days(7)
+            .create_pact_with_hierarchy("Foo", "1", "Bar")
+            .create_consumer_version_tag("prod")
+            .add_days(4)
+            .create_pact_with_hierarchy("Foo", "2", "Bar")
+            .create_consumer_version_tag("dev")
+        end
+
+        let(:selector) { PactBroker::Matrix::UnresolvedSelector.new(tag: true, latest: true, max_age: 3) }
+
+        it "returns matching rows" do
+          expect(subject.count).to eq 1
+          expect(subject.first.consumer_version_number).to eq "2"
+        end
+      end
+    end
+  end
+end

data/spec/lib/pact_broker/webhooks/repository_spec.rb

@@ -563,6 +563,26 @@ module PactBroker
           expect(subject.collect(&:trigger_uuid).sort).to eq ['332', '638']
         end
 
+        context "when a webhook has been triggered by different events" do
+          before do
+            td.create_pact_with_hierarchy("Foo2", "1.0.0", "Bar2")
+              .create_webhook
+              .create_triggered_webhook(trigger_uuid: '333', event_name: 'foo')
+              .create_triggered_webhook(trigger_uuid: '555', event_name: 'foo')
+              .create_webhook_execution
+              .create_triggered_webhook(trigger_uuid: '444', event_name: 'bar')
+              .create_triggered_webhook(trigger_uuid: '777', event_name: 'bar')
+              .create_webhook_execution
+              .create_triggered_webhook(trigger_uuid: '111', event_name: nil)
+              .create_triggered_webhook(trigger_uuid: '888', event_name: nil)
+              .create_webhook_execution
+          end
+
+          it "returns one for each event" do
+            expect(subject.collect(&:trigger_uuid).sort).to eq ['555', '777', '888']
+          end
+        end
+
         context "when there are two 'latest' triggered webhooks at the same time" do
           before do
             td.create_triggered_webhook(trigger_uuid: '888', created_at: DateTime.new(2018))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker
 version: !ruby/object:Gem::Version
-  version: 2.65.0
+  version: 2.66.0
 platform: ruby
 authors:
 - Bethany Skurrie
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-09-25 00:00:00.000000000 Z
+date: 2020-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -402,6 +402,7 @@ files:
 - db/ddl_statements/latest_pact_consumer_version_orders.rb
 - db/ddl_statements/latest_pact_publications_by_consumer_versions.rb
 - db/ddl_statements/latest_tagged_pact_consumer_version_orders.rb
+- db/ddl_statements/latest_triggered_webhooks.rb
 - db/ddl_statements/latest_verification_ids_for_consumer_version_tags.rb
 - db/ddl_statements/latest_verification_ids_for_pact_versions.rb
 - db/migrations/000001_create_pacticipant_table.rb
@@ -518,6 +519,7 @@ files:
 - db/migrations/20200318_add_created_at_to_latest_pact_publications.rb
 - db/migrations/20200319_add_created_at_to_latest_verifications.rb
 - db/migrations/20200922_add_event_to_triggered_webhook.rb
+- db/migrations/20200930_update_latest_triggered_webhooks.rb
 - db/migrations/migration_helper.rb
 - db/test/backwards_compatibility/.rspec
 - db/test/backwards_compatibility/Appraisals
@@ -612,6 +614,7 @@ files:
 - lib/pact_broker/api/decorators/pact_webhooks_status_decorator.rb
 - lib/pact_broker/api/decorators/pacticipant_collection_decorator.rb
 - lib/pact_broker/api/decorators/pacticipant_decorator.rb
+- lib/pact_broker/api/decorators/pagination_links.rb
 - lib/pact_broker/api/decorators/provider_pacts_decorator.rb
 - lib/pact_broker/api/decorators/reason_decorator.rb
 - lib/pact_broker/api/decorators/relationships_csv_decorator.rb
@@ -764,6 +767,7 @@ files:
 - lib/pact_broker/doc/views/pact/tag-prod-version.markdown
 - lib/pact_broker/doc/views/pact/tag-version.markdown
 - lib/pact_broker/doc/views/pacticipant/can-i-deploy-badge.markdown
+- lib/pact_broker/doc/views/pacticipant/versions.markdown
 - lib/pact_broker/doc/views/pending-provider-pacts.markdown
 - lib/pact_broker/doc/views/provider-pacts-for-verification.markdown
 - lib/pact_broker/doc/views/provider.markdown
@@ -1259,6 +1263,7 @@ files:
 - spec/lib/pact_broker/pacts/diff_spec.rb
 - spec/lib/pact_broker/pacts/generate_interaction_sha_spec.rb
 - spec/lib/pact_broker/pacts/generate_sha_spec.rb
+- spec/lib/pact_broker/pacts/latest_tagged_pact_publications_spec.rb
 - spec/lib/pact_broker/pacts/merger_spec.rb
 - spec/lib/pact_broker/pacts/pact_params_spec.rb
 - spec/lib/pact_broker/pacts/pact_publication_spec.rb
@@ -1649,6 +1654,7 @@ test_files:
 - spec/lib/pact_broker/pacts/diff_spec.rb
 - spec/lib/pact_broker/pacts/generate_interaction_sha_spec.rb
 - spec/lib/pact_broker/pacts/generate_sha_spec.rb
+- spec/lib/pact_broker/pacts/latest_tagged_pact_publications_spec.rb
 - spec/lib/pact_broker/pacts/merger_spec.rb
 - spec/lib/pact_broker/pacts/pact_params_spec.rb
 - spec/lib/pact_broker/pacts/pact_publication_spec.rb