pact_broker 2.0.0.beta.6 → 2.0.0.beta.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ba498f4801e7aa66d4d689a7177a3ddb7dcd2568
-  data.tar.gz: e839365f5a50c7cac6bc676603647f20ad60b80f
+  metadata.gz: a23016da41a3bd4b5a3a71cd21c5ee26f34ebc15
+  data.tar.gz: 45cf807ca83536d65a7a75404dea32cb7407eab1
 SHA512:
-  metadata.gz: b57d197a572c7d3713ee08dad512983c9e632fc4571b69a6c3720e584b8e3509007bb8d19b619a3ab87b1d59fdf828de1c0124b8d3bed5af596320c76b8f0f9c
-  data.tar.gz: d5f88508286307cf7e12dfd4df6d013048cac9816278914df34e7fe065763e0f68af5835d6c85507d7150e7e5d41fc919c7c70ee1cfc88e81b8b81ed0ad5cd76
+  metadata.gz: 2e992dcc04ccf7453864d261f4a87a891ee23c624da4c116cbbb92d7cf3228214da2c5bc80e022d4ba2c523c31b4de0757eb8ae09c5ddcb7c1b34723c7d49998
+  data.tar.gz: aa4ace1b1d4032f7ebde957c9dab32023c378e41f9f950a38fdf8baa58cee6d082e3f4060b339da6032649c181bc9d1f948814eae5892c4d5e8e6250abb5567e

data/CHANGELOG.md

@@ -2,6 +2,10 @@ Do this to generate your change history
 
     $ git log --pretty=format:'  * %h - %s (%an, %ad)' vX.Y.Z..HEAD
 
+#### 2.0.0.beta.7 (2017-05-12)
+* 741bf96 - Include information about missing verifications in the latest verifications resource. Only set success to be true when all pacts have been successfully verified. (Beth Skurrie, Fri May 12 14:59:48 2017 +1000)
+* 64f20c6 - Allow one, two or three "parts" in the application version number. Eg. 12, 3.4 and 1.2.400 are all valid. (Beth Skurrie, Wed May 10 16:19:07 2017 +1000)
+
 #### 2.0.0.beta.6 (2017-05-09)
 * 8f1c911 - Ensure all resources provide application/hal+json. (Beth Skurrie, Tue May 9 18:32:37 2017 +1000)
 
@@ -32,6 +36,11 @@ Do this to generate your change history
 * 77eaf7b - Added pb:latest-verifications link to version resource. (Beth Skurrie, Tue Apr 11 16:25:45 2017 +1000)
 * aaf44d9 - Added endpoint to view the latest verifications for a given consumer version. (Beth Skurrie, Tue Apr 11 11:16:03 2017 +1000)
 
+#### 1.18.0 (2017-05-09)
+* 397060b - Display application versions in reverse order in the Versions resource. (Beth Skurrie, Tue May 9 13:59:54 2017 +1000)
+* 251c878 - Allow application versions to be ordered by creation date where no consistent orderable object can be extracted from the consumer application version. (Beth Skurrie, Tue May 9 13:22:36 2017 +1000)
+* 68bb6d9 - Execute webhooks using sucker punch. (Beth Skurrie, Mon May 8 10:32:45 2017 +1000)
+
 #### 1.17.2 (2017-05-04)
 * b8f45e1 - fix issue with pact document link not displaying #94 (Matt Fellows, Wed May 3 11:23:09 2017 +1000)
 

data/README.md

@@ -1,6 +1,6 @@
 # Pact Broker
 
-[![Build Status](https://travis-ci.org/bethesque/pact_broker.svg?branch=master)](https://travis-ci.org/bethesque/pact_broker) [![Join the chat at https://gitter.im/bethesque/pact_broker](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/bethesque/pact_broker?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
+[![Build Status](https://travis-ci.org/pact-foundation/pact_broker.svg?branch=master)](https://travis-ci.org/pact-foundation/pact_broker) [![Join the chat at https://gitter.im/pact-foundation/pact_broker](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/pact-foundation/pact_broker?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
 The Pact Broker provides a repository for consumer driven contracts created using the pact gem.
 
@@ -23,7 +23,7 @@ Features:
 * Tracks changes between Pact versions so you can tell when a consumer has changed its expectations.
 * [Docker Pact Broker][docker]
 
-Travis CI Status: [![Build Status](https://travis-ci.org/bethesque/pact_broker.svg?branch=master)](https://travis-ci.org/bethesque/pact_broker)
+Travis CI Status: [![Build Status](https://travis-ci.org/pact-foundation/pact_broker.svg?branch=master)](https://travis-ci.org/pact-foundation/pact_broker)
 
 ### How would I use the Pact Broker?
 
@@ -41,47 +41,47 @@ If you don't have a [Pact Broker CI Nerf Gun][nerf], you'll probably want to rea
 
 ## Documentation
 
-See the [Pact Broker Client](https://github.com/bethesque/pact_broker-client) for documentation on how to publish a pact to the Pact Broker, and configure the URLs in the provider project.
+See the [Pact Broker Client](https://github.com/pact-foundation/pact_broker-client) for documentation on how to publish a pact to the Pact Broker, and configure the URLs in the provider project.
 
-See the [wiki](https://github.com/bethesque/pact_broker/wiki) for documentation related to the Pact Broker itself.
+See the [wiki](https://github.com/pact-foundation/pact_broker/wiki) for documentation related to the Pact Broker itself.
 
 ### Screenshots
 
 #### Index
 
 * * *
-<img src="https://raw.githubusercontent.com/wiki/bethesque/pact_broker/images/index.png"/>
+<img src="https://raw.githubusercontent.com/wiki/pact-foundation/pact_broker/images/index.png"/>
 
 #### Autogenerated documentation
 
 Paste the pact URL into a browser to view a HTML version of the pact.
 * * *
-<img src="https://raw.githubusercontent.com/wiki/bethesque/pact_broker/images/autogenerated_documentation.png"/>
+<img src="https://raw.githubusercontent.com/wiki/pact-foundation/pact_broker/images/autogenerated_documentation.png"/>
 
 
 #### Network diagram
 
 * * *
-<img src="https://raw.githubusercontent.com/wiki/bethesque/pact_broker/images/network_diagram.png"/>
+<img src="https://raw.githubusercontent.com/wiki/pact-foundation/pact_broker/images/network_diagram.png"/>
 
 #### HAL browser
 
 Use the embedded HAL browser to navigate the API.
 * * *
-<img src="https://raw.githubusercontent.com/wiki/bethesque/pact_broker/images/hal_browser.png"/>
+<img src="https://raw.githubusercontent.com/wiki/pact-foundation/pact_broker/images/hal_browser.png"/>
 
 #### HAL documentation
 
 Use the HAL browser to view documentation as you browse.
 * * *
-<img src="https://raw.githubusercontent.com/wiki/bethesque/pact_broker/images/hal_documentation.png"/>
+<img src="https://raw.githubusercontent.com/wiki/pact-foundation/pact_broker/images/hal_documentation.png"/>
 
 ## Usage
 
 ### To have a play around on your local machine
 
 * Install ruby 2.2.0 or later and bundler >= 1.12.0
-* Run `git clone git@github.com:bethesque/pact_broker.git && cd pact_broker/example`
+* Run `git clone git@github.com:pact-foundation/pact_broker.git && cd pact_broker/example`
 * Run `bundle`
 * Run `bundle exec rackup -p 8080`
 * Open [http://localhost:8080](http://localhost:8080) and you should see a list containing the pact between the Zoo App and the Animal Service.
@@ -100,7 +100,7 @@ the [Hosted Pact Broker](https://pact.dius.com.au/?utm_source=github&utm_campaig
 You can use the [Pact Broker Docker container][docker] or [Terraform on AWS][terraform] or to roll your own...
 
 * Create a database using a product that is supported by the Sequel gem (listed on this page http://sequel.jeremyevans.net/rdoc/files/README_rdoc.html). The migrations have been tested on MySQL and PostgreSQL - your mileage will vary on other databases.
-* __Note:__ It is recommended to use __PostgreSQL__ as it will support JSON search features that are planned in a future release, however MySQL the other [semi supported](https://github.com/bethesque/pact_broker/issues/33) database.
+* __Note:__ It is recommended to use __PostgreSQL__ as it will support JSON search features that are planned in a future release, however MySQL the other [semi supported](https://github.com/pact-foundation/pact_broker/issues/33) database.
 * Install ruby 2.2.0 or later and bundler >= 1.12.0
 * Copy the [example](/example) directory to the location you want to install the application.
 * Modify the config.ru and Gemfile as desired (eg. choose database driver gem, set your database credentials. Use the "pg" gem if using Postgres.)
@@ -110,7 +110,7 @@ You can use the [Pact Broker Docker container][docker] or [Terraform on AWS][ter
 
 [decouple]: http://techblog.realestate.com.au/enter-the-pact-matrix-or-how-to-decouple-the-release-cycles-of-your-microservices/
 [pact]: https://github.com/realestate-com-au/pact
-[nerf]: https://github.com/bethesque/pact_broker/wiki/pact-broker-ci-nerf-gun
+[nerf]: https://github.com/pact-foundation/pact_broker/wiki/pact-broker-ci-nerf-gun
 [different-teams]: https://github.com/realestate-com-au/pact/wiki/Using-pact-where-the-consumer-team-is-different-from-the-provider-team
 [docker]: https://hub.docker.com/r/dius/pact-broker
 [terraform]: https://github.com/nadnerb/terraform-pact-broker

data/db/migrations/08_create_latest_pact_view.rb

@@ -1,10 +1,10 @@
 Sequel.migration do
   change do
     create_view(:all_pacts,
-      Sequel::Model.db[:pacts].select(:pacts__id, :c__id___consumer_id, :c__name___consumer_name,
-      :cv__number___consumer_version_number, :cv__order___consumer_version_order,
-      :p__id___provider_id, :p__name___provider_name,
-      :pacts__json_content).
+      Sequel::Model.db[:pacts].select(Sequel[:pacts][:id], Sequel[:c][:id].as(:consumer_id), Sequel[:c][:name].as(:consumer_name),
+      Sequel[:cv][:number].as(:consumer_version_number), Sequel[:cv][:order].as(:consumer_version_order),
+      Sequel[:p][:id].as(:provider_id), Sequel[:p][:name].as(:provider_name),
+      Sequel[:pacts][:json_content]).
       join(:versions, {:id => :version_id}, {:table_alias => :cv, implicit_qualifier: :pacts}).
       join(:pacticipants, {:id => :pacticipant_id}, {:table_alias => :c, implicit_qualifier: :cv}).
       join(:pacticipants, {:id => :provider_id}, {:table_alias => :p, implicit_qualifier: :pacts}))

data/db/migrations/14_add_timestamps_to_pact_views.rb

@@ -1,10 +1,11 @@
 Sequel.migration do
   change do
     create_or_replace_view(:all_pacts,
-      Sequel::Model.db[:pacts].select(:pacts__id, :c__id___consumer_id, :c__name___consumer_name,
-      :cv__number___consumer_version_number, :cv__order___consumer_version_order,
-      :p__id___provider_id, :p__name___provider_name,
-      :pacts__json_content, :pacts__created_at, :pacts__updated_at).
+      Sequel::Model.db[:pacts].select(
+      Sequel[:pacts][:id], Sequel[:c][:id].as(:consumer_id), Sequel[:c][:name].as(:consumer_name),
+      Sequel[:cv][:number].as(:consumer_version_number), Sequel[:cv][:order].as(:consumer_version_order),
+      Sequel[:p][:id].as(:provider_id), Sequel[:p][:name].as(:provider_name),
+      Sequel[:pacts][:json_content], Sequel[:pacts][:created_at], Sequel[:pacts][:updated_at]).
       join(:versions, {:id => :version_id}, {:table_alias => :cv, implicit_qualifier: :pacts}).
       join(:pacticipants, {:id => :pacticipant_id}, {:table_alias => :c, implicit_qualifier: :cv}).
       join(:pacticipants, {:id => :provider_id}, {:table_alias => :p, implicit_qualifier: :pacts}))

data/db/migrations/20_add_pact_version_content_sha_to_all_pacts_view.rb

@@ -4,15 +4,17 @@ require_relative 'migration_helper'
 Sequel.migration do
   change do
     create_or_replace_view(:all_pacts,
-      Sequel::Model.db[:pacts].select(:pacts__id,
-      :c__id___consumer_id, :c__name___consumer_name,
-      :cv__id___consumer_version_id, :cv__number___consumer_version_number, :cv__order___consumer_version_order,
-      :p__id___provider_id, :p__name___provider_name,
-      :pvc__sha___pact_version_content_sha, :pacts__created_at, :pacts__updated_at).
-      join(:versions, {:id => :version_id}, {:table_alias => :cv, implicit_qualifier: :pacts}).
-      join(:pacticipants, {:id => :pacticipant_id}, {:table_alias => :c, implicit_qualifier: :cv}).
-      join(:pacticipants, {:id => :provider_id}, {:table_alias => :p, implicit_qualifier: :pacts}).
-      join(:pact_version_contents, {:sha => :pact_version_content_sha}, {:table_alias => :pvc, implicit_qualifier: :pacts})
-    )
+        Sequel::Model.db[:pacts].select(
+        Sequel[:pacts][:id],
+        Sequel[:c][:id].as(:consumer_id), Sequel[:c][:name].as(:consumer_name),
+        Sequel[:cv][:id].as(:consumer_version_id), Sequel[:cv][:number].as(:consumer_version_number), Sequel[:cv][:order].as(:consumer_version_order),
+        Sequel[:p][:id].as(:provider_id), Sequel[:p][:name].as(:provider_name),
+        Sequel[:pvc][:sha].as(:pact_version_content_sha), Sequel[:pacts][:created_at], Sequel[:pacts][:updated_at]).
+        join(:versions, {:id => :version_id}, {:table_alias => :cv, implicit_qualifier: :pacts}).
+        join(:pacticipants, {:id => :pacticipant_id}, {:table_alias => :c, implicit_qualifier: :cv}).
+        join(:pacticipants, {:id => :provider_id}, {:table_alias => :p, implicit_qualifier: :pacts}).
+        join(:pact_version_contents, {:sha => :pact_version_content_sha}, {:table_alias => :pvc, implicit_qualifier: :pacts})
+      )
+
   end
 end

data/db/migrations/28_create_all_pact_publications.rb

@@ -2,11 +2,12 @@ Sequel.migration do
   up do
     # The denormalised pact publication details for each publication
     create_view(:all_pact_publications,
-      Sequel::Model.db[:pact_publications].select(:pact_publications__id,
-      :c__id___consumer_id, :c__name___consumer_name,
-      :cv__id___consumer_version_id, :cv__number___consumer_version_number, :cv__order___consumer_version_order,
-      :p__id___provider_id, :p__name___provider_name,
-      :pact_publications__revision_number, :pv__id___pact_version_id, :pv__sha___pact_version_sha, :pact_publications__created_at).
+      Sequel::Model.db[:pact_publications].select(
+      Sequel[:pact_publications][:id],
+      Sequel[:c][:id].as(:consumer_id), Sequel[:c][:name].as(:consumer_name),
+      Sequel[:cv][:id].as(:consumer_version_id), Sequel[:cv][:number].as(:consumer_version_number), Sequel[:cv][:order].as(:consumer_version_order),
+      Sequel[:p][:id].as(:provider_id), Sequel[:p][:name].as(:provider_name),
+      Sequel[:pact_publications][:revision_number], Sequel[:pv][:id].as(:pact_version_id), Sequel[:pv][:sha].as(:pact_version_sha), Sequel[:pact_publications][:created_at]).
       join(:versions, {:id => :consumer_version_id}, {:table_alias => :cv, implicit_qualifier: :pact_publications}).
       join(:pacticipants, {:id => :pacticipant_id}, {:table_alias => :c, implicit_qualifier: :cv}).
       join(:pacticipants, {:id => :provider_id}, {:table_alias => :p, implicit_qualifier: :pact_publications}).

data/lib/pact_broker/api/decorators/verification_decorator.rb

@@ -27,9 +27,6 @@ module PactBroker
           }
         end
 
-        def provider_name
-          represented.provider_name
-        end
       end
     end
   end

data/lib/pact_broker/api/decorators/verification_summary_decorator.rb

@@ -0,0 +1,36 @@
+require_relative 'base_decorator'
+require_relative 'version_decorator'
+require 'ostruct'
+
+module PactBroker
+  module Api
+    module Decorators
+
+      class VerificationSummaryDecorator < BaseDecorator
+
+        property :success
+        property :provider_summary, as: :providerSummary do
+          property :successful
+          property :failed
+          property :unknown
+        end
+
+        collection :verifications, as: :verificationResults, embedded: true, :extend => PactBroker::Api::Decorators::VerificationDecorator
+
+        link :self do | context |
+          {
+            href: context.fetch(:resource_url),
+            title: "Latest verification results for consumer #{context.fetch(:consumer_name)} version #{context.fetch(:consumer_version_number)}"
+          }
+        end
+
+        def provider_summary
+          OpenStruct.new(
+            successful: represented.select(&:success).collect(&:provider_name),
+            failed: represented.select{|verification| !verification.success }.collect(&:provider_name))
+        end
+
+      end
+    end
+  end
+end

data/lib/pact_broker/api/decorators/versions_decorator.rb

@@ -15,7 +15,7 @@ module PactBroker
         link :self do | context |
           {
             href: context[:resource_url],
-            title: "All versions of the pacticipant #{context[:pacticipant_name]}"
+            title: "All application versions of #{context[:pacticipant_name]}"
           }
         end
 

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

@@ -2,7 +2,7 @@ require 'pact_broker/api/resources/base_resource'
 require 'pact_broker/configuration'
 require 'pact_broker/domain/verification'
 require 'pact_broker/api/contracts/verification_contract'
-require 'pact_broker/api/decorators/verifications_decorator'
+require 'pact_broker/api/decorators/verification_summary_decorator'
 
 module PactBroker
   module Api
@@ -24,14 +24,14 @@ module PactBroker
         end
 
         def to_json
-          verifications = verification_service.find_latest_verifications_for_consumer_version(identifier_from_path)
-          decorator_for(verifications).to_json(user_options: decorator_context(identifier_from_path))
+          summary = verification_service.verification_summary_for_consumer_version(identifier_from_path)
+          decorator_for(summary).to_json(user_options: decorator_context(identifier_from_path))
         end
 
         private
 
-        def decorator_for verifications
-          PactBroker::Api::Decorators::VerificationsDecorator.new(verifications)
+        def decorator_for summary
+          PactBroker::Api::Decorators::VerificationSummaryDecorator.new(summary)
         end
       end
     end

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

@@ -25,7 +25,7 @@ module PactBroker
         end
 
         def versions
-          pacticipant_service.find_all_pacticipant_versions pacticipant_name
+          pacticipant_service.find_all_pacticipant_versions_in_reverse_order pacticipant_name
         end
 
       end

data/lib/pact_broker/app.rb

@@ -3,6 +3,7 @@ require 'pact_broker/db'
 require 'pact_broker/project_root'
 require 'rack/hal_browser'
 require 'rack/pact_broker/convert_file_extension_to_accept_header'
+require 'sucker_punch'
 
 module PactBroker
 
@@ -32,6 +33,7 @@ module PactBroker
       PactBroker::DB.connection = configuration.database_connection
       PactBroker::DB.connection.timezone = :utc
       PactBroker::DB.validate_connection_config if configuration.validate_database_connection_config
+      SuckerPunch.logger = configuration.logger
 
       if configuration.auto_migrate_db
         logger.info "Migrating database"

data/lib/pact_broker/configuration.rb

@@ -8,7 +8,8 @@ module PactBroker
 
     attr_accessor :log_dir, :database_connection, :auto_migrate_db, :use_hal_browser, :html_pact_renderer
     attr_accessor :validate_database_connection_config, :enable_diagnostic_endpoints, :version_parser
-    attr_accessor :use_case_sensitive_resource_names
+    attr_accessor :use_case_sensitive_resource_names, :order_versions_by_date
+    attr_accessor :semver_formats
     attr_writer :logger
 
     def logger
@@ -26,6 +27,10 @@ module PactBroker
       config.use_case_sensitive_resource_names = true
       config.html_pact_renderer = default_html_pact_render
       config.version_parser = PactBroker::Versions::ParseSemanticVersion
+      # Not recommended to set this to true unless there is no way to
+      # consistently extract an orderable object from the consumer application version number.
+      config.order_versions_by_date = false
+      config.semver_formats = ["%M.%m.%p%s%d","%M.%m", "%M"]
       config
     end
 

data/lib/pact_broker/domain/order_versions.rb

@@ -4,9 +4,17 @@ module PactBroker
   module Domain
     class OrderVersions
 
+      include PactBroker::Logging
+      # TODO select for update
       def self.call pacticipant_id
-        orderable_versions = PactBroker::Domain::Version.where(:pacticipant_id => pacticipant_id).all.collect{| version | OrderableVersion.new(version) }
-        orderable_versions.sort.each_with_index{ | version, i | version.update_model(i) }
+
+        orderable_versions = PactBroker::Domain::Version.for_update.where(:pacticipant_id => pacticipant_id).order(:created_at, :id).collect{| version | OrderableVersion.new(version) }
+        ordered_versions = if PactBroker.configuration.order_versions_by_date
+          orderable_versions # already ordered in SQL
+        else
+          orderable_versions.sort
+        end
+        ordered_versions.each_with_index{ | version, i | version.update_model(i) }
       end
 
       class OrderableVersion
@@ -23,9 +31,11 @@ module PactBroker
         end
 
         def update_model new_order
-          # Sequel will only run the update if the column value has changed, so in 99% of
-          # cases, only one update will occur.
-          version_model.update(:order => new_order)
+          if version_model.order != new_order
+            # Avoid modifying the updated_at flag by doing a manual update
+            # In 99% of cases, this will only set the order of the most recent version
+            PactBroker::Domain::Version.where(id: version_model.id).update(order: new_order)
+          end
         end
       end
     end

data/lib/pact_broker/domain/verification.rb

@@ -38,7 +38,7 @@ module PactBroker
         end
 
         def verification_number number
-          where("`verifications`.`number` = ? ", number)
+          where(Sequel.qualify("verifications", "number") => number)
         end
 
         def latest

data/lib/pact_broker/domain/webhook.rb

@@ -33,7 +33,6 @@ module PactBroker
         request && request.description
       end
 
-      #TODO retries
       def execute
         logger.info "Executing #{self}"
         request.execute

data/lib/pact_broker/domain/webhook_request.rb

@@ -22,7 +22,7 @@ module PactBroker
       include PactBroker::Logging
       include PactBroker::Messages
 
-      attr_accessor :method, :url, :headers, :body, :username, :password
+      attr_accessor :method, :url, :headers, :body, :username, :password, :uuid
 
       # Reform gets confused by the :method method, as :method is a standard
       # Ruby method.
@@ -35,6 +35,7 @@ module PactBroker
         @password = attributes[:password]
         @headers = attributes[:headers] || {}
         @body = attributes[:body]
+        @uuid = attributes[:uuid]
       end
 
       def description
@@ -64,17 +65,18 @@ module PactBroker
             end
           end
 
-          logger.info "Making webhook request #{to_s}"
+          logger.info "Making webhook #{uuid} request #{to_s}"
           response = Net::HTTP.start(uri.hostname, uri.port,
             :use_ssl => uri.scheme == 'https') do |http|
             http.request req
           end
 
-          logger.info "Received response status=#{response.code} body=#{response.body}"
+          logger.info "Received response for webhook #{uuid} status=#{response.code}"
+          logger.debug "body=#{response.body}"
           WebhookExecutionResult.new(response)
 
         rescue StandardError => e
-          logger.error "Error executing webhook #{e.class.name} - #{e.message}"
+          logger.error "Error executing webhook #{uuid} #{e.class.name} - #{e.message}"
           logger.error e.backtrace.join("\n")
           WebhookExecutionResult.new(nil, e)
         end

data/lib/pact_broker/logging.rb

@@ -17,6 +17,10 @@ module PactBroker
       @@logger = logger
     end
 
+    def log_error e
+      logger.error "#{e.class} #{e.message} #{e.backtrace.join("\n")}"
+    end
+
     def logger
       @@logger ||= begin
         FileUtils.mkdir_p(LOG_DIR)

data/lib/pact_broker/pacticipants/repository.rb

@@ -20,11 +20,12 @@ module PactBroker
         PactBroker::Domain::Pacticipant.order(:name).all
       end
 
-      def find_all_pacticipant_versions name
+      def find_all_pacticipant_versions_in_reverse_order name
         PactBroker::Domain::Version
-          .select(:versions__id, :versions__number, :versions__pacticipant_id, :versions__order, :versions__created_at, :versions__updated_at)
+          .select(Sequel[:versions][:id], Sequel[:versions][:number], Sequel[:versions][:pacticipant_id], Sequel[:versions][:order], Sequel[:versions][:created_at], Sequel[:versions][:updated_at])
           .join(:pacticipants, {id: :pacticipant_id})
           .where(name_like(:name, name))
+          .reverse_order(:order)
       end
 
       def find_by_name_or_create name