pact_broker 2.91.0 → 2.92.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1634d0a3519aae8c805e0c8f13e2e132e86b623c92e019d37285b98ecd7574d
-  data.tar.gz: 2652bd80a29b04e222bad6ffb0513c95618f16185feb98cffc9c7bd283858b79
+  metadata.gz: 24079bee3bfb14bfb86f5c9bc12d6f1110a19eb3e4098b8a1543d93a309a562c
+  data.tar.gz: 59ef52df7edeb89738512914f8dbaca0a500d27ce84700e2c5e3bd06f3ce1421
 SHA512:
-  metadata.gz: 8e3bc701ea3fe67d6ffe24330faf61b5def23d7293e2c59bfa1d4c5b8874dd3ef8b622f8adbef202bdfd784804f67a5c9efd39c79c1bba54bb52a8fe2abc7661
-  data.tar.gz: ab7a4c68f71251bbf36e3f58bb18267b28cc1bd72df5f3959caedd8fe1b776439b6ba25fd972f2a819ba5f0872d9861d22284da501a5d921f255aea368bedfab
+  metadata.gz: 954b6742ba7f76c9af9153c7ba05205e51993df3b97eba56219cbb7a5027df3808cabcb3a4611d72bb2bff7376a2bf3e5b7876eadccc0bda8f23e0affdaad28f
+  data.tar.gz: 8485529fb6980f2925892871d382376d0860b9c7ee9e72afeb3235a99dff18cb2afb08993666095fe6b084373ed79ce5396a16a22a3c70ea5a656246a0335310

data/CHANGELOG.md

@@ -1,3 +1,18 @@
+<a name="v2.92.0"></a>
+### v2.92.0 (2021-11-27)
+
+#### Features
+
+* allow SQL caller logging to be enabled	 ([861d8f21](/../../commit/861d8f21))
+* use a separate table to track the successful verifications of pact versions for each provider version tag (feature toggled with "new_wip_calculation")	 ([df0acfa3](/../../commit/df0acfa3))
+
+#### Bug Fixes
+
+* **cleanup**
+  * Improve delete orphans SQL query (#527)	 ([853354ea](/../../commit/853354ea))
+
+* cast PACT_BROKER_DATABASE_CONNECTION_VALIDATION_TIMEOUT to an integer	 ([8816c61f](/../../commit/8816c61f))
+
 <a name="v2.91.0"></a>
 ### v2.91.0 (2021-11-15)
 

data/db/migrations/20210608_add_uuid_to_webhook.rb

@@ -8,6 +8,7 @@ Sequel.migration do
 
   down do
     alter_table(:triggered_webhooks) do
+      drop_index([:uuid], name: "triggered_webhooks_uuid", unique: true)
       drop_column(:uuid)
     end
   end

data/db/migrations/20211120_create_pact_version_provider_tag_successful_verifications.rb

@@ -0,0 +1,23 @@
+Sequel.migration do
+  up do
+    create_table(:pact_version_provider_tag_successful_verifications, charset: "utf8") do
+      primary_key :id
+      foreign_key :pact_version_id, :pact_versions, null: false, on_delete: :cascade, foreign_key_constraint_name: "pact_version_provider_tag_successful_verifications_pv_id_fk"
+      String :provider_version_tag_name, null: false
+      Boolean :wip, null: false
+      Integer :verification_id
+      DateTime :execution_date, null: false
+      index([:pact_version_id, :provider_version_tag_name, :wip], unique: true, name: "pact_version_provider_tag_verifications_pv_pvtn_wip_unique")
+      # The implication of the on_delete: :set_null for verification_id is
+      # that even if the verification result is deleted from the broker,
+      # the wip/pending status stays the same.
+      # We may or may not want this. Will have to wait and see.
+      # Have made the foreign key a separate declaration so it can more easily be remade.
+      foreign_key([:verification_id], :verifications, on_delete: :set_null, name: "pact_version_provider_tag_successful_verifications_v_id_fk")
+    end
+  end
+
+  down do
+    drop_table(:pact_version_provider_tag_successful_verifications)
+  end
+end

data/db/migrations/20211121_migrate_pact_version_provider_tag_successful_verifications_data.rb

@@ -0,0 +1,11 @@
+require "pact_broker/db/data_migrations/migrate_pact_version_provider_tag_successful_verifications"
+
+Sequel.migration do
+  up do
+    PactBroker::DB::DataMigrations::MigratePactVersionProviderTagSuccessfulVerifications.call(self)
+  end
+
+  down do
+
+  end
+end

data/lib/pact_broker/api/resources/deployed_versions_for_version_and_environment.rb

@@ -40,11 +40,11 @@ module PactBroker
         end
 
         def policy_name
-          :'versions::version'
+          :'versions::deployed_version'
         end
 
         def policy_record
-          version
+          environment
         end
 
         private

data/lib/pact_broker/api/resources/released_versions_for_version_and_environment.rb

@@ -45,7 +45,11 @@ module PactBroker
         end
 
         def policy_name
-          :'versions::versions'
+          :'versions::released_version'
+        end
+
+        def policy_record
+          environment
         end
 
         def finish_request

data/lib/pact_broker/config/runtime_configuration_database_methods.rb

@@ -18,6 +18,7 @@ module PactBroker
             database_sslmode: nil,
             sql_log_level: :debug,
             sql_log_warn_duration: 5,
+            sql_enable_caller_logging: false,
             database_max_connections: nil,
             database_pool_timeout: 5,
             database_connect_max_retries: 0,
@@ -36,6 +37,7 @@ module PactBroker
                 encoding: "utf8",
                 sslmode: database_sslmode,
                 sql_log_level: sql_log_level,
+                enable_caller_logging: sql_enable_caller_logging,
                 log_warn_duration: sql_log_warn_duration,
                 max_connections: database_max_connections,
                 pool_timeout: database_pool_timeout,
@@ -65,6 +67,10 @@ module PactBroker
             super(metrics_sql_statement_timeout&.to_i)
           end
 
+          def database_connection_validation_timeout= database_connection_validation_timeout
+            super(database_connection_validation_timeout&.to_i)
+          end
+
           def postgres?
             database_credentials[:adapter] == "postgres"
           end

data/lib/pact_broker/db/clean_incremental.rb

@@ -92,8 +92,23 @@ module PactBroker
       end
 
       def delete_orphan_pact_versions
-        referenced_pact_version_ids = db[:pact_publications].select(:pact_version_id).union(db[:verifications].select(:pact_version_id))
-        db[:pact_versions].where(id: referenced_pact_version_ids).invert.delete
+        db[:pact_versions].where(id: orphan_pact_versions).delete
+      rescue Sequel::DatabaseError => e
+        raise unless e.cause.class.name == "Mysql2::Error"
+
+        ids = orphan_pact_versions.map { |row| row[:id] }
+        db[:pact_versions].where(id: ids).delete
+      end
+
+      def orphan_pact_versions
+        db[:pact_versions]
+          .left_join(:pact_publications, pact_version_id: :id)
+          .left_join(:verifications, pact_version_id: :id)
+          .select(Sequel[:pact_versions][:id])
+          .where(
+            Sequel[:pact_publications][:id] => nil,
+            Sequel[:verifications][:id] => nil
+          )
       end
 
       def version_info(version)

data/lib/pact_broker/db/data_migrations/migrate_pact_version_provider_tag_successful_verifications.rb

@@ -0,0 +1,38 @@
+require "pact_broker/db/data_migrations/helpers"
+
+module PactBroker
+  module DB
+    module DataMigrations
+      class MigratePactVersionProviderTagSuccessfulVerifications
+        extend Helpers
+
+        def self.call(connection)
+          successful_verifications_join = {
+            Sequel[:sv][:pact_version_id] => Sequel[:verifications][:pact_version_id],
+            Sequel[:sv][:provider_version_tag_name] => Sequel[:tags][:name],
+            Sequel[:sv][:wip] => Sequel[:verifications][:wip]
+          }
+
+          missing_verifications = connection
+                                    .select(
+                                      Sequel[:verifications][:pact_version_id],
+                                      Sequel[:tags][:name],
+                                      Sequel[:verifications][:wip],
+                                      Sequel[:verifications][:id],
+                                      Sequel[:verifications][:execution_date]
+                                    )
+                                    .order(Sequel[:verifications][:execution_date], Sequel[:verifications][:id])
+                                    .from(:verifications)
+                                    .join(:tags, { Sequel[:verifications][:provider_version_id] => Sequel[:tags][:version_id] })
+                                    .left_outer_join(:pact_version_provider_tag_successful_verifications, successful_verifications_join, { table_alias: :sv })
+                                    .where(Sequel[:sv][:pact_version_id] => nil)
+                                    .where(Sequel[:verifications][:success] => true)
+
+          connection[:pact_version_provider_tag_successful_verifications]
+            .insert_ignore
+            .insert([:pact_version_id, :provider_version_tag_name, :wip, :verification_id, :execution_date], missing_verifications)
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/db/migrate_data.rb

@@ -28,6 +28,7 @@ module PactBroker
         DataMigrations::SetExtraColumnsForTags.call(database_connection)
         DataMigrations::CreateBranches.call(database_connection)
         DataMigrations::MigrateIntegrations.call(database_connection)
+        DataMigrations::MigratePactVersionProviderTagSuccessfulVerifications.call(database_connection)
       end
     end
   end

data/lib/pact_broker/deployments/deployed_version.rb

@@ -88,6 +88,15 @@ module PactBroker
         !!currently_deployed_version_id
       end
 
+      # target has been renamed to applicationInstance in the API.
+      def application_instance
+        target
+      end
+
+      def application_instance= application_instance
+        self.target = application_instance
+      end
+
       def version_number
         version.number
       end
@@ -100,7 +109,6 @@ module PactBroker
         self.class.where(id: id).record_undeployed
         self.refresh
       end
-
     end
   end
 end

data/lib/pact_broker/initializers/database_connection.rb

@@ -9,7 +9,8 @@ module PactBroker
     sequel_config = config.dup
     max_retries = sequel_config.delete(:connect_max_retries) || 0
     connection_validation_timeout = config.delete(:connection_validation_timeout)
-    configure_logger(sequel_config)
+    enable_caller_logging = config.delete(:enable_caller_logging)
+    configure_logger(sequel_config, logger)
     create_sqlite_database_dir(config)
 
     connection = with_retries(max_retries, logger) do
@@ -18,7 +19,7 @@ module PactBroker
 
     logger&.info "Connected to database #{sequel_config[:database]}"
 
-    configure_connection(connection, connection_validation_timeout)
+    configure_connection(connection, connection_validation_timeout, enable_caller_logging)
   end
 
   private_class_method def self.with_retries(max_retries, logger)
@@ -46,7 +47,7 @@ module PactBroker
     end
   end
 
-  private_class_method def self.configure_logger(sequel_config)
+  private_class_method def self.configure_logger(sequel_config, logger)
     if sequel_config[:sql_log_level] == :none
       sequel_config.delete(:sql_log_level)
     elsif logger
@@ -72,8 +73,9 @@ module PactBroker
   # when databases are restarted and connections are killed.  This has a performance
   # penalty, so consider increasing this timeout if building a frequently accessed service.
 
-  private_class_method def self.configure_connection(connection, connection_validation_timeout)
+  private_class_method def self.configure_connection(connection, connection_validation_timeout, enable_caller_logging)
     connection.extension(:connection_validator)
+    connection.extension(:caller_logging) if enable_caller_logging
     connection.pool.connection_validation_timeout = connection_validation_timeout if connection_validation_timeout
     connection
   end

data/lib/pact_broker/pacts/pact_publication_dataset_module.rb

@@ -140,65 +140,62 @@ module PactBroker
       end
 
       # The latest pact publication for each tag
+      # This uses the old logic of "the latest pact for a version that has a tag" (which always returns a pact)
+      # rather than "the pact for the latest version with a tag"
+      # Need to see about updating this.
       def latest_by_consumer_tag
         tags_join = {
-          Sequel[:pact_publications][:consumer_version_id] => Sequel[:tags][:version_id]
+          Sequel[:pact_publications][:consumer_version_id] => Sequel[:tags][:version_id],
         }
 
-        base_query = join(:tags, tags_join)
+        max_orders = join(:tags, tags_join)
+                      .select_group(:consumer_id, :provider_id, Sequel[:tags][:name].as(:tag_name))
+                      .select_append{ max(consumer_version_order).as(latest_consumer_version_order) }
+
+        max_join = {
+          Sequel[:max_orders][:consumer_id] => Sequel[:pact_publications][:consumer_id],
+          Sequel[:max_orders][:provider_id] => Sequel[:pact_publications][:provider_id],
+          Sequel[:max_orders][:latest_consumer_version_order] => Sequel[:pact_publications][:consumer_version_order]
+        }
 
+        base_query = self
         if no_columns_selected?
-          base_query = base_query.select_all_qualified.select_append(Sequel[:tags][:name].as(:tag_name))
+          base_query = base_query.select_all_qualified.select_append(Sequel[:max_orders][:tag_name])
         end
 
-        joined_query = base_query.select(
-          Sequel[:pact_publications][:consumer_id],
-          Sequel[:tags][:version_order],
-          Sequel[:tags][:name].as(:tag_name)
-        )
-
-        self_join = {
-          Sequel[:pact_publications][:consumer_id] => Sequel[:pp2][:consumer_id],
-          Sequel[:tags][:name] => Sequel[:pp2][:tag_name]
-        }
-        base_query.left_join(joined_query, self_join, { table_alias: :pp2 } ) do
-          Sequel[:pp2][:version_order] > Sequel[:tags][:version_order]
-        end
-        .where(Sequel[:pp2][:version_order] => nil)
-        .remove_overridden_revisions_from_complete_query
+        base_query
+          .join(max_orders, max_join, { table_alias: :max_orders })
+          .remove_overridden_revisions_from_complete_query
       end
 
+      # This uses the old logic of "the latest pact for a version that has a tag" (which always returns a pact)
+      # rather than "the pact for the latest version with a tag"
+      # Need to see about updating this.
       def latest_for_consumer_tag(tag_name)
         tags_join = {
           Sequel[:pact_publications][:consumer_version_id] => Sequel[:tags][:version_id],
           Sequel[:tags][:name] => tag_name
         }
 
-        base_query = self
-        if no_columns_selected?
-          base_query = base_query.select_all_qualified.select_append(Sequel[:tags][:name].as(:tag_name))
-        end
-
-        base_query = base_query
-          .join(:tags, tags_join)
-          .where(Sequel[:tags][:name] => tag_name)
+        max_orders = join(:tags, tags_join)
+                      .select_group(:consumer_id, :provider_id, Sequel[:tags][:name].as(:tag_name))
+                      .select_append{ max(consumer_version_order).as(latest_consumer_version_order) }
 
-        joined_query = base_query.select(
-          Sequel[:pact_publications][:consumer_id],
-          Sequel[:tags][:name].as(:tag_name),
-          Sequel[:tags][:version_order]
-        )
 
-        self_join = {
-          Sequel[:pact_publications][:consumer_id] => Sequel[:pp2][:consumer_id],
-          Sequel[:tags][:name] => Sequel[:pp2][:tag_name]
+        max_join = {
+          Sequel[:max_orders][:consumer_id] => Sequel[:pact_publications][:consumer_id],
+          Sequel[:max_orders][:provider_id] => Sequel[:pact_publications][:provider_id],
+          Sequel[:max_orders][:latest_consumer_version_order] => Sequel[:pact_publications][:consumer_version_order]
         }
 
-        base_query.left_join(joined_query, self_join, { table_alias: :pp2 } ) do
-          Sequel[:pp2][:version_order] > Sequel[:tags][:version_order]
+        base_query = self
+        if no_columns_selected?
+          base_query = base_query.select_all_qualified.select_append(Sequel[:max_orders][:tag_name])
         end
-        .where(Sequel[:pp2][:version_order] => nil)
-        .remove_overridden_revisions_from_complete_query
+
+        base_query
+          .join(max_orders, max_join, { table_alias: :max_orders })
+          .remove_overridden_revisions_from_complete_query
       end
 
       # The pacts for the latest versions with the specified tag (new logic)

data/lib/pact_broker/pacts/pact_publication_wip_dataset_module.rb

@@ -33,6 +33,8 @@ module PactBroker
       end
 
       def successfully_verified_by_provider_tag_when_not_wip(provider_id, provider_tag)
+        return new_successfully_verified_by_provider_tag_when_not_wip(provider_id, provider_tag) if PactBroker.feature_enabled?(:new_wip_calculation)
+
         from_self(alias: :pp)
           .select(Sequel[:pp].*)
           .where(Sequel[:pp][:provider_id] => provider_id)
@@ -41,7 +43,23 @@ module PactBroker
           .distinct
       end
 
+      def new_successfully_verified_by_provider_tag_when_not_wip(_provider_id, provider_tag)
+        pact_version_provider_tag_verifications_join = {
+          Sequel[:sv][:pact_version_id] => Sequel[:pp][:pact_version_id],
+          Sequel[:sv][:provider_version_tag_name] => provider_tag,
+          Sequel[:sv][:wip] => false
+        }
+
+        from_self(alias: :pp)
+          .select(Sequel[:pp].*)
+          .join(:pact_version_provider_tag_successful_verifications, pact_version_provider_tag_verifications_join, { table_alias: :sv })
+          .distinct
+
+      end
+
       def successfully_verified_by_provider_another_tag_before_this_tag_first_created(provider_id, provider_tag)
+        return new_successfully_verified_by_provider_another_tag_before_this_tag_first_created(provider_id, provider_tag) if PactBroker.feature_enabled?(:new_wip_calculation)
+
         first_tag_with_name = PactBroker::Domain::Tag.where(pacticipant_id: provider_id, name: provider_tag).order(:created_at).first
         from_self(alias: :pp)
           .select(Sequel[:pp].*)
@@ -54,6 +72,32 @@ module PactBroker
           .distinct
       end
 
+      def new_successfully_verified_by_provider_another_tag_before_this_tag_first_created(provider_id, provider_tag)
+        first_tag_with_name = PactBroker::Domain::Tag.where(pacticipant_id: provider_id, name: provider_tag).order(:created_at).first
+
+        pact_version_provider_tag_verifications_join = {
+          Sequel[:sv][:pact_version_id] => Sequel[:pp][:pact_version_id],
+          Sequel[:sv][:wip] => false
+        }
+
+        created_at_criteria = if first_tag_with_name
+                                Sequel.lit("sv.execution_date < ?", first_tag_with_name.created_at)
+                              else
+                                Sequel.lit("1 = 1")
+                              end
+
+        from_self(alias: :pp)
+          .select(Sequel[:pp].*)
+          .where(Sequel[:pp][:provider_id] => provider_id)
+          .join(:pact_version_provider_tag_successful_verifications, pact_version_provider_tag_verifications_join, { table_alias: :sv }) do
+            Sequel.&(
+              Sequel.lit("sv.provider_version_tag_name NOT IN (?)", provider_tag),
+              created_at_criteria
+            )
+          end
+          .distinct
+      end
+
       protected
 
       def verified_before_date(date)

data/lib/pact_broker/test/http_test_data_builder.rb

@@ -273,6 +273,7 @@ module PactBroker
               "eventName" => "${pactbroker.eventName}",
               "consumerVersionNumber" => "${pactbroker.consumerVersionNumber}",
               "consumerVersionTags" => "${pactbroker.consumerVersionTags}",
+              "consumerVersionBranch" => "${pactbroker.consumerVersionBranch}",
               "githubVerificationStatus" => "${pactbroker.githubVerificationStatus}",
               "providerVersionNumber" => "${pactbroker.providerVersionNumber}",
               "providerVersionTags" => "${pactbroker.providerVersionTags}",

data/lib/pact_broker/test/test_data_builder.rb

@@ -31,6 +31,8 @@ require "pact_broker/deployments/deployed_version_service"
 require "pact_broker/deployments/released_version_service"
 require "pact_broker/versions/branch_version_repository"
 require "pact_broker/integrations/repository"
+require "pact_broker/contracts/service"
+
 require "ostruct"
 
 module PactBroker
@@ -219,6 +221,27 @@ module PactBroker
         self
       end
 
+      def publish_pact(consumer_name:, provider_name:, consumer_version_number: , tags: nil, branch: nil, build_url: nil, json_content: nil)
+        json_content = json_content || random_json_content(consumer_name, provider_name)
+        contracts = [
+          PactBroker::Contracts::ContractToPublish.from_hash(consumer_name: consumer_name, provider_name: provider_name, decoded_content: json_content, content_type: "application/json", specification: "pact")
+        ]
+        contracts_to_publish = PactBroker::Contracts::ContractsToPublish.from_hash(
+          pacticipant_name: consumer_name,
+          pacticipant_version_number: consumer_version_number,
+          tags: tags,
+          branch: branch,
+          build_url: build_url,
+          contracts: contracts
+        )
+        PactBroker::Contracts::Service.publish(contracts_to_publish, base_url: "http://example.org")
+        @consumer = find_pacticipant(consumer_name)
+        @consumer_version = find_version(consumer_name, consumer_version_number)
+        @provider = find_pacticipant(provider_name)
+        @pact = PactBroker::Pacts::PactPublication.last.to_domain
+        self
+      end
+
       def create_pact params = {}
         params.delete(:comment)
         json_content = params[:json_content] || default_json_content
@@ -363,6 +386,7 @@ module PactBroker
       end
 
       def create_verification parameters = {}
+        # This should use the verification service. what a mess
         parameters.delete(:comment)
         branch = parameters.delete(:branch)
         tag_names = [parameters.delete(:tag_names), parameters.delete(:tag_name)].flatten.compact
@@ -373,20 +397,23 @@ module PactBroker
         parameters.delete(:provider_version)
         verification = PactBroker::Domain::Verification.new(parameters)
         pact_version = PactBroker::Pacts::Repository.new.find_pact_version(@consumer, @provider, pact.pact_version_sha)
-        @verification = PactBroker::Verifications::Repository.new.create(verification, provider_version_number, pact_version)
-        @provider_version = PactBroker::Domain::Version.where(pacticipant_id: @provider.id, number: provider_version_number).single_record
+        @provider_version = version_repository.find_by_pacticipant_id_and_number_or_create(provider.id, provider_version_number)
         PactBroker::Versions::BranchVersionRepository.new.add_branch(@provider_version, branch) if branch
 
-        set_created_at_if_set(parameters[:created_at], :verifications, id: @verification.id)
-        set_created_at_if_set(parameters[:created_at], :versions, id: @provider_version.id)
-        set_created_at_if_set(parameters[:created_at], :latest_verification_id_for_pact_version_and_provider_version, pact_version_id: pact_version_id, provider_version_id: @provider_version.id)
-
         if tag_names.any?
           tag_names.each do | tag_name |
-            PactBroker::Domain::Tag.new(name: tag_name, version: @provider_version).insert_ignore
+            PactBroker::Domain::Tag.new(name: tag_name, version: @provider_version, version_order: @provider_version.order).insert_ignore
             set_created_at_if_set(parameters[:created_at], :tags, version_id: @provider_version.id, name: tag_name)
           end
         end
+
+        @verification = PactBroker::Verifications::Repository.new.create(verification, provider_version_number, pact_version)
+
+        set_created_at_if_set(parameters[:created_at], :verifications, id: @verification.id)
+        set_created_at_if_set(parameters[:created_at], :versions, id: @provider_version.id)
+        set_created_at_if_set(parameters[:created_at], :latest_verification_id_for_pact_version_and_provider_version, pact_version_id: pact_version_id, provider_version_id: @provider_version.id)
+        set_created_at_if_set(parameters[:created_at], :pact_version_provider_tag_successful_verifications, { verification_id: @verification.id }, :execution_date)
+
         self
       end
 
@@ -594,10 +621,10 @@ module PactBroker
         PactBroker::Pacts::Content.from_json(json_content).with_ids(false).to_json
       end
 
-      def set_created_at_if_set created_at, table_name, selector
+      def set_created_at_if_set created_at, table_name, selector, date_column_name = :created_at
         date_to_set = created_at || @now
         if date_to_set
-          Sequel::Model.db[table_name].where(selector).update(created_at: date_to_set)
+          Sequel::Model.db[table_name].where(selector).update(date_column_name => date_to_set)
           if Sequel::Model.db.schema(table_name).any?{ |col| col.first == :updated_at }
             Sequel::Model.db[table_name].where(selector.keys.first => selector.values.first).update(updated_at: date_to_set)
           end

data/lib/pact_broker/verifications/pact_version_provider_tag_successful_verification.rb

@@ -0,0 +1,11 @@
+require "pact_broker/domain/verification"
+
+# Represents a non WIP, successful verification for a provider version with a tag.
+
+module PactBroker
+  module Verifications
+    class PactVersionProviderTagSuccessfulVerification < Sequel::Model
+      plugin :insert_ignore, identifying_columns: [:pact_version_id, :provider_version_tag_name, :wip]
+    end
+  end
+end

data/lib/pact_broker/verifications/repository.rb

@@ -2,6 +2,7 @@ require "sequel"
 require "pact_broker/domain/verification"
 require "pact_broker/verifications/sequence"
 require "pact_broker/verifications/latest_verification_id_for_pact_version_and_provider_version"
+require "pact_broker/verifications/pact_version_provider_tag_successful_verification"
 
 module PactBroker
   module Verifications
@@ -27,6 +28,7 @@ module PactBroker
         verification.tag_names = version.tag_names # TODO pass this in from contracts service
         verification.save
         update_latest_verification_id(verification)
+        update_pact_version_provider_tag_verifications(verification, version.tag_names)
         verification
       end
 
@@ -46,6 +48,20 @@ module PactBroker
         LatestVerificationIdForPactVersionAndProviderVersion.new(params).upsert
       end
 
+      def update_pact_version_provider_tag_verifications(verification, tag_names)
+        if verification.success
+          tag_names&.each do | tag_name |
+            PactVersionProviderTagSuccessfulVerification.new(
+              pact_version_id: verification.pact_version_id,
+              provider_version_tag_name: tag_name,
+              wip: verification.wip,
+              verification_id: verification.id,
+              execution_date: verification.execution_date
+            ).insert_ignore
+          end
+        end
+      end
+
       def find consumer_name, provider_name, pact_version_sha, verification_number
         PactBroker::Domain::Verification
           .select_all_qualified

data/lib/pact_broker/version.rb

@@ -1,3 +1,3 @@
 module PactBroker
-  VERSION = "2.91.0"
+  VERSION = "2.92.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker
 version: !ruby/object:Gem::Version
-  version: 2.91.0
+  version: 2.92.0
 platform: ruby
 authors:
 - Bethany Skurrie
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-11-15 00:00:00.000000000 Z
+date: 2021-11-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -587,6 +587,8 @@ files:
 - db/migrations/20211102_create_table_temp_integrations.rb
 - db/migrations/20211103_migrate_integrations.rb
 - db/migrations/20211104_switch_integrations_and_temp_integrations.rb
+- db/migrations/20211120_create_pact_version_provider_tag_successful_verifications.rb
+- db/migrations/20211121_migrate_pact_version_provider_tag_successful_verifications_data.rb
 - db/migrations/migration_helper.rb
 - docs/CONFIGURATION.md
 - docs/api/WEBHOOKS.md
@@ -797,6 +799,7 @@ files:
 - lib/pact_broker/db/data_migrations/delete_deprecated_webhook_executions.rb
 - lib/pact_broker/db/data_migrations/helpers.rb
 - lib/pact_broker/db/data_migrations/migrate_integrations.rb
+- lib/pact_broker/db/data_migrations/migrate_pact_version_provider_tag_successful_verifications.rb
 - lib/pact_broker/db/data_migrations/migrate_webhook_headers.rb
 - lib/pact_broker/db/data_migrations/set_consumer_ids_for_pact_publications.rb
 - lib/pact_broker/db/data_migrations/set_consumer_version_order_for_pact_publications.rb
@@ -1028,6 +1031,7 @@ files:
 - lib/pact_broker/verifications/latest_verification_for_consumer_and_provider.rb
 - lib/pact_broker/verifications/latest_verification_for_consumer_version_tag.rb
 - lib/pact_broker/verifications/latest_verification_id_for_pact_version_and_provider_version.rb
+- lib/pact_broker/verifications/pact_version_provider_tag_successful_verification.rb
 - lib/pact_broker/verifications/placeholder_verification.rb
 - lib/pact_broker/verifications/pseudo_branch_status.rb
 - lib/pact_broker/verifications/repository.rb
@@ -1220,7 +1224,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.31
+rubygems_version: 3.2.32
 signing_key: 
 specification_version: 4
 summary: See description