pact_broker 2.76.2 → 2.77.0

data/lib/pact_broker/ui/view_models/index_item.rb

@@ -3,11 +3,16 @@ require 'pact_broker/ui/helpers/url_helper'
 require 'pact_broker/date_helper'
 require 'pact_broker/versions/abbreviate_number'
 require 'pact_broker/configuration'
+require 'forwardable'
 
 module PactBroker
   module UI
     module ViewDomain
       class IndexItem
+        extend Forwardable
+
+        delegate [:consumer_version_branch, :provider_version_branch, :latest_for_branch?] => :relationship
+
 
         include PactBroker::Api::PactBrokerUrls
 
@@ -186,6 +191,10 @@ module PactBroker
         def base_url
           @options[:base_url]
         end
+
+        private
+
+        attr_reader :relationship
       end
     end
   end

data/lib/pact_broker/ui/view_models/matrix_line.rb

@@ -4,6 +4,7 @@ require 'pact_broker/date_helper'
 require 'pact_broker/ui/view_models/matrix_tag'
 require 'pact_broker/versions/abbreviate_number'
 require 'pact_broker/messages'
+require 'forwardable'
 
 module PactBroker
   module UI
@@ -11,6 +12,9 @@ module PactBroker
       class MatrixLine
         include PactBroker::Api::PactBrokerUrls
         include PactBroker::Messages
+        extend Forwardable
+
+        delegate [:consumer_version_branch, :provider_version_branch] => :line
 
         def initialize line, options = {}
           @line = line
@@ -97,27 +101,47 @@ module PactBroker
           end
         end
 
+        def consumer_version_branch_tooltip
+          branch_tooltip(consumer_name, consumer_version_branch, consumer_version_latest_for_branch?)
+        end
+
+        def consumer_version_latest_for_branch?
+          @line.consumer_version.latest_for_branch?
+        end
+
+        def provider_version_branch_tooltip
+          branch_tooltip(provider_name, provider_version_branch, provider_version_latest_for_branch?)
+        end
+
+        def provider_version_latest_for_branch?
+          @line.provider_version.latest_for_branch?
+        end
+
         def latest_consumer_version_tags
           @line.consumer_version_tags
             .select(&:latest)
+            .sort_by(&:created_at)
             .collect{ | tag | MatrixTag.new(tag.to_hash.merge(pacticipant_name: consumer_name, version_number: consumer_version_number)) }
         end
 
         def other_consumer_version_tags
           @line.consumer_version_tags
             .reject(&:latest)
+            .sort_by(&:created_at)
             .collect{ | tag | MatrixTag.new(tag.to_hash.merge(pacticipant_name: consumer_name, version_number: consumer_version_number)) }
         end
 
         def latest_provider_version_tags
           @line.provider_version_tags
             .select(&:latest)
+            .sort_by(&:created_at)
             .collect{ | tag | MatrixTag.new(tag.to_hash.merge(pacticipant_name: provider_name, version_number: provider_version_number)) }
         end
 
         def other_provider_version_tags
           @line.provider_version_tags
             .reject(&:latest)
+            .sort_by(&:created_at)
             .collect{ | tag | MatrixTag.new(tag.to_hash.merge(pacticipant_name: provider_name, version_number: provider_version_number)) }
         end
 
@@ -187,6 +211,18 @@ module PactBroker
         def base_url
           @options[:base_url]
         end
+
+        private
+
+        attr_reader :line
+
+        def branch_tooltip(pacticipant_name, branch, latest)
+          if latest
+            "This is the latest version of #{pacticipant_name} from branch \"#{branch}\"."
+          else
+            "A more recent version of #{pacticipant_name} from branch \"#{branch}\" exists."
+          end
+        end
       end
     end
   end

data/lib/pact_broker/ui/views/index/show-with-tags.haml

@@ -47,6 +47,10 @@
               - if index_item.latest?
                 .tag.badge.badge-success
                   latest
+              - if index_item.consumer_version_branch
+                - branch_class = index_item.latest_for_branch? ? "tag badge badge-dark" : "tag badge badge-secondary"
+                %div{"class": branch_class}
+                  = "[" + index_item.consumer_version_branch + "]"
               - index_item.consumer_version_latest_tag_names.each do | tag_name |
                 .tag.badge.badge-primary
                   = tag_name
@@ -64,6 +68,10 @@
                 - if index_item.provider_version_number
                   %button.clippy.invisible{ title: "Copy to clipboard" }
                     %span.copy-icon
+              - if index_item.provider_version_branch
+                - branch_class = "tag badge badge-dark"
+                %div{"class": branch_class}
+                  = "[" + index_item.provider_version_branch + "]"
               - index_item.provider_version_latest_tag_names.each do | tag_name |
                 .tag.badge.badge-primary
                   = tag_name

data/lib/pact_broker/ui/views/matrix/show.haml

@@ -118,6 +118,11 @@
               - if line.display_consumer_version_number
                 %button.clippy.invisible{ title: "Copy to clipboard" }
                   %span.copy-icon
+            - if line.consumer_version_branch
+              .tag-parent{"title": line.consumer_version_branch_tooltip, "data-toggle": "tooltip", "data-placement": "right"}
+                - branch_class = line.consumer_version_latest_for_branch? ? "tag badge badge-dark" : "tag badge badge-secondary"
+                %div{"class": branch_class}
+                  = "[" + line.consumer_version_branch + "]"
             - line.latest_consumer_version_tags.each do | tag |
               .tag-parent{"title": tag.tooltip, "data-toggle": "tooltip", "data-placement": "right"}
                 %a{href: tag.url}
@@ -145,6 +150,11 @@
               - if line.display_provider_version_number
                 %button.clippy.invisible{ title: "Copy to clipboard" }
                   %span.copy-icon
+            - if line.provider_version_branch
+              .tag-parent{"title": line.provider_version_branch_tooltip, "data-toggle": "tooltip", "data-placement": "right"}
+                - branch_class = line.provider_version_latest_for_branch? ? "tag badge badge-dark" : "tag badge badge-secondary"
+                %div{"class": branch_class}
+                  = "[" + line.provider_version_branch + "]"
             - line.latest_provider_version_tags.each do | tag |
               .tag-parent{"title": tag.tooltip, "data-toggle": "tooltip", "data-placement": "right"}
                 %a{href: tag.url}

data/lib/pact_broker/verifications/latest_verification_id_for_pact_version_and_provider_version.rb

@@ -16,12 +16,14 @@ module PactBroker
 end
 
 # Table: latest_verification_id_for_pact_version_and_provider_version
+# Primary Key: (pact_version_id, provider_version_id)
 # Columns:
-#  consumer_id         | integer | NOT NULL
-#  pact_version_id     | integer | NOT NULL
-#  provider_id         | integer | NOT NULL
-#  provider_version_id | integer | NOT NULL
-#  verification_id     | integer | NOT NULL
+#  consumer_id         | integer                     | NOT NULL
+#  pact_version_id     | integer                     | NOT NULL
+#  provider_id         | integer                     | NOT NULL
+#  provider_version_id | integer                     | NOT NULL
+#  verification_id     | integer                     | NOT NULL
+#  created_at          | timestamp without time zone |
 # Indexes:
 #  latest_v_id_for_pv_and_pv_pv_id_pv_id_unq | UNIQUE btree (pact_version_id, provider_version_id)
 #  latest_v_id_for_pv_and_pv_v_id_unq        | UNIQUE btree (verification_id)

data/lib/pact_broker/version.rb

@@ -1,3 +1,3 @@
 module PactBroker
-  VERSION = '2.76.2'
+  VERSION = '2.77.0'
 end

data/lib/pact_broker/versions/eager_loaders.rb

@@ -0,0 +1,71 @@
+module PactBroker
+  module Versions
+    module EagerLoaders
+      class LatestVersionForBranch
+        def self.call(eo, **other)
+          initialize_association(eo[:rows])
+          populate_associations(eo[:rows])
+        end
+
+        def self.initialize_association(versions)
+          versions.each{|version| version.associations[:latest_version_for_branch] = nil }
+        end
+
+        def self.populate_associations(versions)
+          group_by_pacticipant_id(versions).each do | pacticipant, versions |
+            populate_associations_by_pacticipant(pacticipant, versions)
+          end
+        end
+
+        def self.group_by_pacticipant_id(versions)
+          versions.to_a.group_by(&:pacticipant_id)
+        end
+
+        def self.populate_associations_by_pacticipant(pacticipant_id, versions)
+          latest_versions_for_branches = latest_versions_for_pacticipant_branches(
+            pacticipant_id,
+            versions.collect(&:branch).uniq.compact,
+            versions.first.class
+          )
+          self.populate_versions_with_branches(versions, latest_versions_for_branches)
+        end
+
+        def self.populate_versions_with_branches(versions, latest_versions_for_branches)
+          versions.select(&:branch).each do | version |
+            version.associations[:latest_version_for_branch] = latest_versions_for_branches[[version.pacticipant_id, version.branch]]
+          end
+        end
+
+        def self.latest_versions_for_pacticipant_branches(pacticipant_id, branches, version_class)
+          version_class.latest_versions_for_pacticipant_branches(pacticipant_id, branches).each_with_object({}) do | row, hash |
+            hash[[row.pacticipant_id, row.branch]] = row
+          end
+        end
+      end
+
+      class LatestVersionForPacticipant
+        def self.call(eo, **other)
+          populate_associations(eo[:rows])
+        end
+
+        def self.populate_associations(versions)
+          group_by_pacticipant(versions).each do | pacticipant, versions |
+            populate_associations_by_pacticipant(pacticipant, versions)
+          end
+        end
+
+        def self.group_by_pacticipant(versions)
+          versions.to_a.group_by(&:pacticipant)
+        end
+
+        def self.populate_associations_by_pacticipant(pacticipant, versions)
+          latest_version = versions.first.class.latest_version_for_pacticipant(pacticipant).single_record
+
+          versions.each do | version |
+            version.associations[:latest_version_for_pacticipant] = latest_version
+          end
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/versions/lazy_loaders.rb

@@ -0,0 +1,13 @@
+module PactBroker
+  module Versions
+    module LazyLoaders
+      LATEST_VERSION_FOR_BRANCH = lambda {
+        self.class
+          .where(branch: branch, pacticipant_id: pacticipant_id)
+          .exclude(branch: nil)
+          .order(Sequel.desc(:order))
+          .limit(1)
+      }
+    end
+  end
+end

data/lib/pact_broker/versions/repository.rb

@@ -57,7 +57,16 @@ module PactBroker
           updated_at: Sequel.datetime_class.now
         }
 
-        PactBroker::Domain::Version.new(version_params).insert_ignore
+        PactBroker::Domain::Version.new(version_params).upsert
+      end
+
+      def create_or_update(pacticipant, version_number, version)
+        PactBroker::Domain::Version.new(
+          number: version_number,
+          pacticipant: pacticipant,
+          build_url: version.build_url,
+          branch: version.branch
+        ).upsert
       end
 
       def find_by_pacticipant_id_and_number_or_create pacticipant_id, number
@@ -79,7 +88,7 @@ module PactBroker
         version_ids_to_keep = (version_ids_with_pact_publications + version_ids_with_verifications).uniq
 
         PactBroker::Domain::Version
-          .where(pacticipant_id: [consumer.id, provider.id])
+          .where(Sequel[:versions][:pacticipant_id] => [consumer.id, provider.id])
           .exclude(id: (version_ids_with_pact_publications + version_ids_with_verifications).uniq)
           .delete
       end

data/lib/pact_broker/versions/service.rb

@@ -1,7 +1,6 @@
 require 'pact_broker/repositories'
 
 module PactBroker
-
   module Versions
     class Service
 
@@ -19,6 +18,11 @@ module PactBroker
         version_repository.find_by_pacticipant_name_and_latest_tag(pacticipant_name, tag)
       end
 
+      def self.create_or_update(pacticipant_name, version_number, version)
+        pacticipant = pacticipant_repository.find_by_name_or_create(pacticipant_name)
+        version_repository.create_or_update(pacticipant, version_number, version)
+      end
+
       def self.delete version
         tag_repository.delete_by_version_id version.id
         webhook_repository.delete_triggered_webhooks_by_version_id version.id

data/lib/pact_broker/webhooks/execution.rb

@@ -32,13 +32,14 @@ end
 
 # Table: webhook_executions
 # Columns:
-#  id                   | integer                     | PRIMARY KEY DEFAULT nextval('webhook_executions_id_seq'::regclass)
+#  id                   | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  triggered_webhook_id | integer                     |
 #  success              | boolean                     | NOT NULL
 #  logs                 | text                        |
 #  created_at           | timestamp without time zone | NOT NULL
 # Indexes:
-#  webhook_executions_pkey | PRIMARY KEY btree (id)
+#  webhook_executions_pkey                       | PRIMARY KEY btree (id)
+#  webhook_executions_triggered_webhook_id_index | btree (triggered_webhook_id)
 # Foreign key constraints:
 #  webhook_executions_consumer_id_fkey          | (consumer_id) REFERENCES pacticipants(id)
 #  webhook_executions_pact_publication_id_fkey  | (pact_publication_id) REFERENCES pact_publications(id)

data/lib/pact_broker/webhooks/latest_triggered_webhook.rb

@@ -21,3 +21,5 @@ end
 #  status              | text                        |
 #  created_at          | timestamp without time zone |
 #  updated_at          | timestamp without time zone |
+#  verification_id     | integer                     |
+#  event_name          | text                        |

data/lib/pact_broker/webhooks/triggered_webhook.rb

@@ -110,7 +110,7 @@ end
 
 # Table: triggered_webhooks
 # Columns:
-#  id                  | integer                     | PRIMARY KEY DEFAULT nextval('triggered_webhooks_id_seq'::regclass)
+#  id                  | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  trigger_uuid        | text                        | NOT NULL
 #  trigger_type        | text                        | NOT NULL
 #  pact_publication_id | integer                     | NOT NULL
@@ -122,10 +122,17 @@ end
 #  created_at          | timestamp without time zone | NOT NULL
 #  updated_at          | timestamp without time zone | NOT NULL
 #  verification_id     | integer                     |
+#  event_name          | text                        |
+#  event_context       | text                        |
 # Indexes:
-#  triggered_webhooks_pkey     | PRIMARY KEY btree (id)
-#  uq_triggered_webhook_ppi_wi | UNIQUE btree (pact_publication_id, webhook_id, trigger_uuid)
-#  uq_triggered_webhook_wi     | UNIQUE btree (webhook_id, trigger_uuid)
+#  triggered_webhooks_pkey                      | PRIMARY KEY btree (id)
+#  uq_triggered_webhook_ppi_wi                  | UNIQUE btree (pact_publication_id, webhook_id, trigger_uuid)
+#  uq_triggered_webhook_wi                      | UNIQUE btree (webhook_id, trigger_uuid)
+#  triggered_webhooks_consumer_id_index         | btree (consumer_id)
+#  triggered_webhooks_pact_publication_id_index | btree (pact_publication_id)
+#  triggered_webhooks_provider_id_index         | btree (provider_id)
+#  triggered_webhooks_verification_id_index     | btree (verification_id)
+#  triggered_webhooks_webhook_id_index          | btree (webhook_id)
 # Foreign key constraints:
 #  triggered_webhooks_consumer_id_fkey         | (consumer_id) REFERENCES pacticipants(id)
 #  triggered_webhooks_pact_publication_id_fkey | (pact_publication_id) REFERENCES pact_publications(id)

data/lib/pact_broker/webhooks/webhook.rb

@@ -94,7 +94,7 @@ end
 
 # Table: webhooks
 # Columns:
-#  id                   | integer                     | PRIMARY KEY DEFAULT nextval('webhooks_id_seq'::regclass)
+#  id                   | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  uuid                 | text                        | NOT NULL
 #  method               | text                        | NOT NULL
 #  url                  | text                        | NOT NULL

data/lib/pact_broker/webhooks/webhook_event.rb

@@ -44,7 +44,7 @@ end
 
 # Table: webhook_events
 # Columns:
-#  id         | integer                     | PRIMARY KEY DEFAULT nextval('webhook_events_id_seq'::regclass)
+#  id         | integer                     | PRIMARY KEY GENERATED BY DEFAULT AS IDENTITY
 #  webhook_id | integer                     |
 #  name       | text                        |
 #  created_at | timestamp without time zone | NOT NULL

data/lib/pact_broker/webhooks/webhook_request_logger.rb

@@ -117,8 +117,12 @@ module PactBroker
       end
 
       def success?(response)
-        !response.nil? && response.code.to_i < 300
+        unless response.nil?
+          # Response HTTP Code must be in success list otherwise it is false
+          PactBroker.configuration.webhook_http_code_success.include? response.code.to_i
+        end
       end
+
     end
   end
 end

data/lib/sequel/plugins/upsert.rb

@@ -5,11 +5,12 @@ module Sequel
       def self.configure(model, opts=OPTS)
         model.instance_exec do
           @upsert_plugin_identifying_columns = opts.fetch(:identifying_columns)
+          @upsert_plugin_ignore_columns_on_update = opts[:ignore_columns_on_update]
         end
       end
 
       module ClassMethods
-        attr_reader :upsert_plugin_identifying_columns
+        attr_reader :upsert_plugin_identifying_columns, :upsert_plugin_ignore_columns_on_update
       end
 
       module InstanceMethods
@@ -67,7 +68,8 @@ module Sequel
           query = values.select{ |k, _| self.class.upsert_plugin_identifying_columns.include?(k) }
           existing_record = model.where(query).single_record
           if existing_record
-            existing_record.update(values)
+            existing_record.set(values_to_update)
+            existing_record.save(opts)
           else
             save(opts)
           end
@@ -78,9 +80,9 @@ module Sequel
         def _insert_dataset
           if upsert_plugin_upserting
             if postgres?
-              super.insert_conflict(update: values, target: self.class.upsert_plugin_identifying_columns)
+              super.insert_conflict(update: values_to_update, target: self.class.upsert_plugin_identifying_columns)
             elsif mysql?
-              columns_to_update = values.keys - self.class.upsert_plugin_identifying_columns
+              columns_to_update = values_to_update.keys - self.class.upsert_plugin_identifying_columns
               super.on_duplicate_key_update(*columns_to_update)
             else
               super
@@ -97,6 +99,18 @@ module Sequel
         def postgres?
           model.db.adapter_scheme.to_s == "postgres"
         end
+
+        def values_to_update
+          columns_with_nil_values.merge(values).reject{ |k, _| columns_to_not_update_when_exists.include?(k) }
+        end
+
+        def columns_with_nil_values
+          self.class.columns.each_with_object({}) {|key, hash| hash[key] = nil }
+        end
+
+        def columns_to_not_update_when_exists
+          @columns_to_not_update ||=  self.class.upsert_plugin_ignore_columns_on_update || (self.class.upsert_plugin_identifying_columns + [:created_at, self.class.primary_key]).flatten
+        end
       end
     end
   end