pact_broker 2.7.0.beta.2 → 2.7.0.beta.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 55ed087a4bac3721c9d21d00522d4aa6af0d804b
-  data.tar.gz: 1b6058269a9c42452b3fc0ac7321f7467ee97f62
+  metadata.gz: '09bb5e6f6b13f93ded89b80289de3954432e5fbe'
+  data.tar.gz: 4091e9ee2e13413a98dd9179bd00118b92dfa85c
 SHA512:
-  metadata.gz: 86f0f33980f4d8be7f9d19a391f6fc5329dd2c40d99a25ad08580181c30459b5dad4c80b4fd45c5e923e6edc0b54f5d507a950185a0c20c2ec7077a20eeba1b3
-  data.tar.gz: d29c13d0da4027ee2876759e7fcd31bc46f4819a65f935da844808bc3b06315ccbbd5dcdf33ad7c43ec28c6bd2becd32ce21148aa7408258a3d10d208ef2e877
+  metadata.gz: 4e12f025d4183d03cad7a7d45ebfd203357ddf63779f8cc39b359e83f30adec8eb3cc018b9d024d4e79ae435b1b847847a87f13dd5c8df5d789b15dac2c08c17
+  data.tar.gz: 64153bb8d9aef540cabd32fb6bdaeb1270f2d24ebdfcb17e6b558bf5b0bcb30236e8655c072396d8ef0b33aedfb87bfaee26398d6ad8365d28d8ad398050fe4f

data/CHANGELOG.md

@@ -1,3 +1,23 @@
+<a name="v2.7.0.beta.3"></a>
+### v2.7.0.beta.3 (2017-11-01)
+
+#### Features
+
+* **matrix**
+  * implement querying by latest without a tag	 ([3d78f79](/../../commit/3d78f79))
+  * update query to handle tag and latest flag	 ([c5c800f](/../../commit/c5c800f))
+  * update validation of selectors to allow tag and latest flag to be specified	 ([0fa33f1](/../../commit/0fa33f1))
+  * parse latest=true and tag=TAG in matrix query	 ([abcab9e](/../../commit/abcab9e))
+  * update validation to allow latest tag to be specified	 ([6da6e02](/../../commit/6da6e02))
+  * allow version to be specified by latest_tag	 ([fe498a7](/../../commit/fe498a7))
+
+* **pacticipant**
+  * expose repositoryUrl in resource	 ([8f0f16a](/../../commit/8f0f16a))
+
+#### Bug Fixes
+
+* correct captialization in require	 ([db2e676](/../../commit/db2e676))
+
 <a name="v2.7.0.beta.2"></a>
 ### v2.7.0.beta.2 (2017-10-31)
 

data/RELEASING.md

@@ -1,15 +1,5 @@
 # Releasing
 
-1. Increment the version in `./lib/pact_broker/version.rb`
-2. Update the `CHANGELOG.md` using:
+Run
 
-      $ bundle exec rake generate_changelog
-
-3. Add files to git
-
-      $ git add CHANGELOG.md lib/pact_broker/version.rb
-      $ git commit -m "chore(release): version $(ruby -r ./lib/pact_broker/version.rb -e "puts PactBroker::VERSION")" && git push
-
-3. Release:
-
-      $ bundle exec rake release
+    script/release.sh [major|minor|patch] # default is minor

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

@@ -32,9 +32,9 @@ module PactBroker
 
         def from_json
           if pacticipant
-            @pacticipant = pacticipant_service.update params.merge(name: pacticipant_name)
+            @pacticipant = pacticipant_service.update params_with_string_keys.merge('name' => pacticipant_name)
           else
-            @pacticipant = pacticipant_service.create params.merge(name: pacticipant_name)
+            @pacticipant = pacticipant_service.create params_with_string_keys.merge('name' => pacticipant_name)
             response.headers["Location"] = pacticipant_url(base_url, pacticipant)
           end
           response.body = to_json
@@ -62,9 +62,7 @@ module PactBroker
         def pacticipant_name
           identifier_from_path[:name]
         end
-
       end
     end
-
   end
 end

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

@@ -32,7 +32,7 @@ module PactBroker
         end
 
         def from_json
-          created_model = pacticipant_service.create params
+          created_model = pacticipant_service.create params_with_string_keys
           response.body = decorator_for(created_model).to_json(user_options: decorator_context)
         end
 
@@ -55,9 +55,7 @@ module PactBroker
         def new_model
           @new_model ||= decorator_for(PactBroker::Domain::Pacticipant.new).from_json(request.body.to_s)
         end
-
       end
     end
-
   end
 end

data/lib/pact_broker/domain/tag.rb

@@ -1,10 +1,15 @@
 require 'pact_broker/db'
+require 'pact_broker/repositories/helpers'
 
 module PactBroker
 
   module Domain
     class Tag < Sequel::Model
 
+      dataset_module do
+        include PactBroker::Repositories::Helpers
+      end
+
       unrestrict_primary_key
 
       associate(:many_to_one, :version, :class => "PactBroker::Domain::Version", :key => :version_id, :primary_key => :id)

data/lib/pact_broker/matrix/parse_query.rb

@@ -5,7 +5,14 @@ module PactBroker
     class ParseQuery
       def self.call query
         params = Rack::Utils.parse_nested_query(query)
-        selectors = (params['q'] || []).collect{ |i| { pacticipant_name: i['pacticipant'], pacticipant_version_number: i['version'] } }
+        selectors = (params['q'] || []).collect do |i|
+          p = {}
+          p[:pacticipant_name] = i['pacticipant'] if i['pacticipant']
+          p[:pacticipant_version_number] = i['version'] if i['version']
+          p[:latest] = true if i['latest'] == 'true'
+          p[:tag] = i['tag'] if i['tag']
+          p
+        end
         options = {}
         if params.key?('success') && params['success'].is_a?(Array)
           options[:success] = params['success'].collect do | value |

data/lib/pact_broker/matrix/repository.rb

@@ -37,6 +37,8 @@ module PactBroker
       # If the version is nil, it means all versions for that pacticipant are to be included
       #
       def find_all selectors
+        selectors = look_up_versions_for_tags(selectors)
+
         query = PactBroker::Pacts::LatestPactPublicationsByConsumerVersion
           .select_append(:consumer_version_number, :provider_name, :consumer_name, :provider_version_id, :provider_version_number, :success)
           .select_append(Sequel[:latest_pact_publications_by_consumer_versions][:created_at].as(:pact_created_at))
@@ -55,6 +57,29 @@ module PactBroker
           .collect(&:values)
       end
 
+      def look_up_versions_for_tags(selectors)
+        selectors.collect do | selector |
+          # resource validation currently stops tag being specified without latest=true
+
+          if selector[:tag] && selector[:latest]
+            version = version_repository.find_by_pacticpant_name_and_latest_tag(selector[:pacticipant_name], selector[:tag])
+            # validation in resource should ensure we always have a version
+            {
+              pacticipant_name: selector[:pacticipant_name],
+              pacticipant_version_number: version.number
+            }
+          elsif selector[:latest]
+            version = version_repository.find_latest_by_pacticpant_name(selector[:pacticipant_name])
+            {
+              pacticipant_name: selector[:pacticipant_name],
+              pacticipant_version_number: version.number
+            }
+          else
+            selector
+          end
+        end
+      end
+
       def where_consumer_and_provider_within selectors, query
           query.where{
             Sequel.&(

data/lib/pact_broker/matrix/service.rb

@@ -23,25 +23,36 @@ module PactBroker
       def validate_selectors selectors
         error_messages = []
 
-        selectors.each do | selector |
-          if selector[:pacticipant_name].nil? && selector[:pacticipant_version_number].nil?
+        selectors.each do | s |
+          if s[:pacticipant_name].nil? && s[:pacticipant_version_number].nil?
             error_messages << "Please specify the pacticipant name and version"
-          elsif selector[:pacticipant_name].nil?
+          elsif s[:pacticipant_name].nil?
             error_messages << "Please specify the pacticipant name"
+          else
+            if s.key?(:pacticipant_version_number) && s.key?(:latest)
+              error_messages << "A version and latest flag cannot both be specified for #{s[:pacticipant_name]}"
+            end
+
+            if s.key?(:tag) && !s.key?(:latest)
+              error_messages << "Querying for all versions with a tag is not currently supported. The latest=true flag must be specified when a tag is given."
+            end
           end
         end
 
         selectors.collect{ |selector| selector[:pacticipant_name] }.compact.each do | pacticipant_name |
           unless pacticipant_service.find_pacticipant_by_name(pacticipant_name)
-            error_messages << "Pacticipant '#{pacticipant_name}' not found"
+            error_messages << "Pacticipant #{pacticipant_name} not found"
           end
         end
 
         if error_messages.empty?
-          selectors.each do | selector |
-            if selector[:pacticipant_version_number]
-              version = version_service.find_by_pacticipant_name_and_number(pacticipant_name: selector[:pacticipant_name], pacticipant_version_number: selector[:pacticipant_version_number])
-              error_messages << "No pact or verification found for #{selector[:pacticipant_name]} version #{selector[:pacticipant_version_number]}" if version.nil?
+          selectors.each do | s |
+            if s[:pacticipant_version_number]
+              version = version_service.find_by_pacticipant_name_and_number(pacticipant_name: s[:pacticipant_name], pacticipant_version_number: s[:pacticipant_version_number])
+              error_messages << "No pact or verification found for #{s[:pacticipant_name]} version #{s[:pacticipant_version_number]}" if version.nil?
+            elsif s[:tag]
+              version = version_service.find_by_pacticpant_name_and_latest_tag(s[:pacticipant_name], s[:tag])
+              error_messages << "No version of #{s[:pacticipant_name]} found with tag #{s[:tag]}" if version.nil?
             end
           end
         end

data/lib/pact_broker/pacticipants/service.rb

@@ -70,13 +70,17 @@ module PactBroker
       end
 
       def self.update params
-        pacticipant = pacticipant_repository.find_by_name(params.fetch(:name))
-        pacticipant.update(params)
-        pacticipant_repository.find_by_name(params.fetch(:name))
+        pacticipant = pacticipant_repository.find_by_name(params.fetch('name'))
+        PactBroker::Api::Decorators::PacticipantDecorator.new(pacticipant).from_hash(params)
+        pacticipant.save
+        pacticipant_repository.find_by_name(params.fetch('name'))
       end
 
       def self.create params
-        pacticipant_repository.create(params)
+        pacticipant = PactBroker::Domain::Pacticipant.new
+        PactBroker::Api::Decorators::PacticipantDecorator.new(pacticipant).from_hash(params)
+        pacticipant.save
+        pacticipant
       end
 
       def self.delete name

data/lib/pact_broker/tags/repository.rb

@@ -14,7 +14,7 @@ module PactBroker
 
       def find args
         PactBroker::Domain::Tag
-          .select(Sequel.qualify("tags", "name"), Sequel.qualify("tags", "version_id"), Sequel.qualify("tags", "created_at"), Sequel.qualify("tags", "updated_at"))
+          .select_all_qualified
           .join(:versions, {id: :version_id})
           .join(:pacticipants, {Sequel.qualify("pacticipants", "id") => Sequel.qualify("versions", "pacticipant_id")})
           .where(name_like(Sequel.qualify("tags", "name"), args.fetch(:tag_name)))

data/lib/pact_broker/version.rb

@@ -1,3 +1,3 @@
 module PactBroker
-  VERSION = '2.7.0.beta.2'
+  VERSION = '2.7.0.beta.3'
 end

data/lib/pact_broker/versions/repository.rb

@@ -12,6 +12,26 @@ module PactBroker
         PactBroker::Domain::Version.where(number: number, pacticipant_id: pacticipant_id).single_record
       end
 
+      def find_by_pacticpant_name_and_latest_tag pacticipant_name, tag
+        PactBroker::Domain::Version
+          .select_all_qualified
+          .join(:pacticipants, {id: :pacticipant_id}, {implicit_qualifier: :versions})
+          .join(:tags, {version_id: :id}, {implicit_qualifier: :versions})
+          .where(name_like(Sequel[:tags][:name], tag))
+          .where(name_like(Sequel[:pacticipants][:name], pacticipant_name))
+          .reverse_order(:order)
+          .first
+      end
+
+      def find_latest_by_pacticpant_name pacticipant_name
+        PactBroker::Domain::Version
+          .select_all_qualified
+          .join(:pacticipants, {id: :pacticipant_id}, {implicit_qualifier: :versions})
+          .where(name_like(Sequel[:pacticipants][:name], pacticipant_name))
+          .reverse_order(:order)
+          .first
+      end
+
       def find_by_pacticipant_name_and_number pacticipant_name, number
         PactBroker::Domain::Version
           .select(Sequel[:versions][:id], Sequel[:versions][:number], Sequel[:versions][:pacticipant_id], Sequel[:versions][:order], Sequel[:versions][:created_at], Sequel[:versions][:updated_at])

data/lib/pact_broker/versions/service.rb

@@ -11,18 +11,15 @@ module PactBroker
         version_repository.find_by_pacticipant_name_and_number params.fetch(:pacticipant_name), params.fetch(:pacticipant_version_number)
       end
 
+      def self.find_by_pacticpant_name_and_latest_tag(pacticipant_name, tag)
+        version_repository.find_by_pacticpant_name_and_latest_tag(pacticipant_name, tag)
+      end
+
       def self.delete version
         tag_repository.delete_by_version_id version.id
         pact_repository.delete_by_version_id version.id
         version_repository.delete_by_id version.id
       end
-
-      def self.find_versions_by_selector selectors
-        selectors.collect do | selector |
-          pacticipant_name, type, number = selector.split("/")
-          version_repository.find_by_pacticipant_name_and_number pacticipant_name, number
-        end
-      end
     end
   end
 end

data/pact_broker.gemspec

@@ -51,5 +51,5 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'database_cleaner', '~>1.6'
   gem.add_development_dependency 'pg', '~>0.21'
   gem.add_development_dependency 'conventional-changelog', '~>1.3'
-
+  gem.add_development_dependency 'bump', '~> 0.5'
 end

data/script/release.sh

@@ -0,0 +1,7 @@
+#!/bin/sh
+set -e
+bundle exec bump ${1:-minor} --no-commit
+bundle exec rake generate_changelog
+git add CHANGELOG.md lib/pact_broker/version.rb
+git commit -m "chore(release): version $(ruby -r ./lib/pact_broker/version.rb -e "puts PactBroker::VERSION")" && git push
+bundle exec rake release

data/spec/features/create_pacticipant_spec.rb

@@ -4,7 +4,7 @@ describe "Creating a pacticipant" do
   let(:path) { "/pacticipants" }
   let(:headers) { {'CONTENT_TYPE' => 'application/json'} }
   let(:response_body) { JSON.parse(last_response.body, symbolize_names: true)}
-  let(:pacticipant_hash) { {name: 'Foo Thing'}}
+  let(:pacticipant_hash) { { name: 'Foo Thing' } }
 
   subject { post path, pacticipant_hash.to_json, headers; last_response }
 

data/spec/features/update_pacticipant_spec.rb

@@ -1,25 +1,23 @@
-xdescribe "Publishing a pact" do
+describe "Publishing a pact" do
 
-  let(:request_body) { load_fixture('update_pacticipant.json') }
+  let(:request_body) { {'repositoryUrl' => 'http://foo'} }
   let(:path) { "/pacticipants/Some%20Consumer" }
   let(:response_body_json) { JSON.parse(subject.body) }
 
-  subject { patch path, request_body, {'CONTENT_TYPE' => 'application/json-patch+json' }; last_response  }
+  subject { patch path, request_body.to_json, {'CONTENT_TYPE' => 'application/json' }; last_response  }
 
   context "when the pacticipant exists" do
+
+    before do
+      TestDataBuilder.new.create_pacticipant("Some Consumer")
+    end
     it "returns a 200 OK" do
-      expect(subject.status).to be 201
+      puts subject.body unless subject.status == 200
+      expect(subject.status).to be 200
     end
 
     it "returns a json body with the updated pacticipant" do
-      expect(subject.headers['Content-Type']).to eq "application/json"
-    end
-
-  end
-
-  context "when the pacticipant does not exist" do
-    it "returns a 404" do
-      expect(subject.status).to be 404
+      expect(subject.headers['Content-Type']).to eq "application/hal+json;charset=utf-8"
     end
   end
 end

data/spec/lib/pact_broker/api/resources/pacticipant_spec.rb

@@ -64,8 +64,6 @@ module PactBroker::Api
           end
         end
       end
-
     end
   end
-
 end

data/spec/lib/pact_broker/api/resources/pacticipants_spec.rb

@@ -47,7 +47,7 @@ module PactBroker
 
           context "with valid JSON" do
             it "creates the pacticipant" do
-              expect(PactBroker::Pacticipants::Service).to receive(:create).with(params)
+              expect(PactBroker::Pacticipants::Service).to receive(:create).with('name' => 'New Consumer')
               subject
             end
 

data/spec/lib/pact_broker/matrix/parse_query_spec.rb

@@ -32,7 +32,7 @@ module PactBroker
           let(:query) { "q[][wrong]=Foo&q[][blah]=1.2.3" }
 
           it "returns nil keys or values" do
-            expect(subject.first).to eq [{ pacticipant_name: nil, pacticipant_version_number: nil }]
+            expect(subject.first).to eq [{}]
           end
         end
 
@@ -73,6 +73,30 @@ module PactBroker
             expect(subject.last).to eq(success: [nil])
           end
         end
+
+        context "when latest is true" do
+          let(:query) { "q[][pacticipant]=Foo&q[][latest]=true" }
+
+          it "returns a selector with latest true" do
+            expect(subject.first).to eq [{ pacticipant_name: 'Foo', latest: true }]
+          end
+        end
+
+        context "when latest is not true" do
+          let(:query) { "q[][pacticipant]=Foo&q[][latest]=false" }
+
+          it "returns a selector with no latest key" do
+            expect(subject.first).to eq [{ pacticipant_name: 'Foo' }]
+          end
+        end
+
+        context "when there is a tag" do
+          let(:query) { "q[][pacticipant]=Foo&q[][tag]=prod" }
+
+          it "returns a selector with a tag" do
+            expect(subject.first).to eq [{ pacticipant_name: 'Foo', tag: 'prod' }]
+          end
+        end
       end
     end
   end

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

@@ -32,15 +32,15 @@ module PactBroker
           end
 
           it "returns an row with a blank provider_version_number" do
-            expect(subject.first).to include consumer_name: "A",
+            expect(subject).to include_hash_matching consumer_name: "A",
               provider_name: "B",
               consumer_version_number: "1.2.3",
               provider_version_number: nil
 
-            expect(subject.last).to include consumer_name: "A",
+            expect(subject).to include_hash_matching({consumer_name: "A",
               provider_name: "C",
               consumer_version_number: "1.2.3",
-              provider_version_number: nil
+              provider_version_number: nil})
           end
 
           context "when only 2 version selectors are specified" do
@@ -219,6 +219,79 @@ module PactBroker
             end
           end
         end
+
+        context "when the latest tag is specified for a provider instead of a version" do
+          before do
+            td.create_pact_with_hierarchy("A", "1.2.3", "B")
+              .create_verification(provider_version: "1.0.0")
+              .use_provider_version("1.0.0")
+              .create_provider_version_tag("prod")
+              .create_verification(provider_version: "2.0.0", number: 2)
+              .use_provider_version("2.0.0")
+              .create_provider_version_tag("prod")
+              .create_verification(provider_version: "3.0.0", number: 3)
+          end
+
+          let(:selectors) do
+            [
+              { pacticipant_name: "A", pacticipant_version_number: "1.2.3" },
+              { pacticipant_name: "B", latest: true, tag: "prod" }
+            ]
+          end
+
+          subject { Repository.new.find(selectors) }
+
+          it "returns the row for the version " do
+            expect(subject.first).to include provider_version_number: "2.0.0"
+            expect(subject.size).to eq 1
+          end
+        end
+
+        context "when the latest version is specified for a provider without a tag" do
+          before do
+            td.create_pact_with_hierarchy("A", "1.2.3", "B")
+              .create_verification(provider_version: "1.0.0")
+              .use_provider_version("1.0.0")
+              .create_verification(provider_version: "2.0.0", number: 2)
+              .use_provider_version("2.0.0")
+              .create_verification(provider_version: "3.0.0", number: 3)
+          end
+
+          let(:selectors) do
+            [
+              { pacticipant_name: "A", pacticipant_version_number: "1.2.3" },
+              { pacticipant_name: "B", latest: true }
+            ]
+          end
+
+          subject { Repository.new.find(selectors) }
+
+          it "returns the row for the version " do
+            expect(subject.first).to include provider_version_number: "3.0.0"
+            expect(subject.size).to eq 1
+          end
+        end
+
+        context "when the latest version is specified for a provider without a tag but the latest known version for a provider does not have a verification" do
+          before do
+            td.create_pact_with_hierarchy("A", "1.2.3", "B")
+              .create_verification(provider_version: "1.0.0")
+              .create_provider_version("5.0.0")
+          end
+
+          let(:selectors) do
+            [
+              { pacticipant_name: "A", pacticipant_version_number: "1.2.3" },
+              { pacticipant_name: "B", latest: true }
+            ]
+          end
+
+          subject { Repository.new.find(selectors) }
+
+          it "returns no data - this may be confusing. Might need to re-think this logic." do
+            expect(subject.size).to eq 0
+          end
+        end
       end
 
       describe "#find_for_consumer_and_provider" do

data/spec/lib/pact_broker/matrix/service_spec.rb

@@ -36,7 +36,7 @@ module PactBroker
           let(:selectors) { [{ pacticipant_name: "Foo", pacticipant_version_number: "1" }] }
 
           it "returns error messages" do
-            expect(subject.first).to eq "Pacticipant 'Foo' not found"
+            expect(subject.first).to eq "Pacticipant Foo not found"
           end
         end
 
@@ -70,6 +70,65 @@ module PactBroker
             expect(subject.first).to eq "Please specify the pacticipant name and version"
           end
         end
+
+        context "when the latest_tag is used instead of a version" do
+          before do
+            td.create_pacticipant("Foo")
+              .create_version("1")
+              .create_tag("prod")
+              .create_pacticipant("Bar")
+              .create_version("2")
+          end
+
+          let(:selectors) { [{ pacticipant_name: "Foo", latest_tag: "prod" }, { pacticipant_name: "Bar", pacticipant_version_number: "2" }] }
+
+          context "when there is a version for the tag" do
+            it "returns no error messages" do
+              expect(subject).to eq []
+            end
+          end
+
+          context "when there is not a version for the tag" do
+
+            let(:selectors) { [{ pacticipant_name: "Foo", latest: true, tag: "wiffle" }, { pacticipant_name: "Bar", pacticipant_version_number: "2" }] }
+
+            it "returns an error message" do
+              expect(subject).to eq ["No version of Foo found with tag wiffle"]
+            end
+          end
+        end
+
+        context "when the latest is used as well as a version" do
+          before do
+            td.create_pacticipant("Foo")
+              .create_version("1")
+              .create_tag("prod")
+              .create_pacticipant("Bar")
+              .create_version("2")
+          end
+
+          let(:selectors) { [{ pacticipant_name: "Foo", pacticipant_version_number: "1", latest: true }, { pacticipant_name: "Bar", pacticipant_version_number: "2" }] }
+
+          it "returns an error message" do
+            expect(subject).to eq ["A version and latest flag cannot both be specified for Foo"]
+          end
+        end
+
+        context "when a tag is specified without latest=true" do
+          before do
+            td.create_pacticipant("Foo")
+              .create_version("1")
+              .create_tag("prod")
+              .create_pacticipant("Bar")
+              .create_version("2")
+          end
+
+          let(:selectors) { [{ pacticipant_name: "Foo", tag: "1"}] }
+
+          it "returns an error message" do
+            expect(subject).to eq ["Querying for all versions with a tag is not currently supported. The latest=true flag must be specified when a tag is given."]
+          end
+        end
       end
     end
   end

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

@@ -10,6 +10,22 @@ module PactBroker
 
       subject{ Service }
 
+      let(:td) { TestDataBuilder.new }
+
+      describe ".update" do
+        before do
+          td.create_pacticipant("Foo")
+        end
+
+        let(:params) { { 'name' => 'Foo', 'repositoryUrl' => 'http://foo' } }
+
+        subject { Service.update(params) }
+
+        it "updates the repositoryUrl" do
+          expect(subject.repository_url).to eq 'http://foo'
+        end
+      end
+
       describe ".messages_for_potential_duplicate_pacticipants" do
 
         let(:base_url) { 'http://example.org' }
@@ -126,7 +142,6 @@ module PactBroker
         it "returns a list of relationships" do
           expect(subject.find_relationships).to eq([PactBroker::Domain::Relationship.create(consumer, provider, pact, verification, webhooks)])
         end
-
       end
 
       describe "delete" do

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

@@ -5,9 +5,33 @@ module PactBroker
   module Versions
     describe Repository do
 
+      let(:td) { TestDataBuilder.new }
       let(:pacticipant_name) { "test_pacticipant" }
       let(:version_number) { "1.2.3" }
 
+
+      describe "#find_by_pacticpant_name_and_latest_tag" do
+        before do
+          td.create_consumer("Bar")
+            .create_consumer_version("2.3.4")
+            .create_consumer_version_tag("prod")
+            .create_consumer("Foo")
+            .create_consumer_version("1.2.3")
+            .create_consumer_version_tag("prod")
+            .create_consumer_version("2.3.4")
+            .create_consumer_version_tag("prod")
+            .create_consumer_version("5.6.7")
+        end
+
+        subject { Repository.new.find_by_pacticpant_name_and_latest_tag("Foo", "prod") }
+
+        it "returns the most recent version that has the specified tag" do
+
+          expect(subject.number).to eq "2.3.4"
+          expect(subject.pacticipant.name).to eq "Foo"
+        end
+      end
+
       describe "#create" do
         context "when a previous version exists" do
 

data/spec/lib/pact_broker/versions/service_spec.rb

@@ -4,44 +4,6 @@ module PactBroker
 
   module Versions
     describe Service do
-
-      describe ".find_versions_by_selector" do
-
-        context "without a version selector" do
-          let(:selector) { "Foo" }
-        end
-
-        context "with a version selector" do
-          before do
-            TestDataBuilder.new
-              .create_pacticipant("Foo")
-              .create_version("1.2.3")
-              .create_version("4.5.6")
-              .create_pacticipant("Bar")
-              .create_version("1.2.3")
-          end
-
-          let(:selector) { "Foo/version/1.2.3" }
-
-          subject { Service.find_versions_by_selector [selector] }
-
-          context "when the version exists" do
-            it "returns the specfied version" do
-              expect(subject.first.number).to eq "1.2.3"
-              expect(subject.first.pacticipant.name).to eq "Foo"
-            end
-          end
-
-          context "when the version does not exist" do
-            let(:selector) { "Wiffle/version/0.0.0" }
-
-            it "returns nil" do
-              expect(subject.first).to eq nil
-            end
-          end
-        end
-      end
-
       describe ".delete" do
         let!(:version) do
           TestDataBuilder.new

data/spec/service_consumers/provider_states_for_pact_broker_client.rb

@@ -48,6 +48,20 @@ Pact.provider_states_for "Pact Broker Client" do
     end
   end
 
+  provider_state "the pact for Foo version 1.2.3 has been successfully verified by Bar version 4.5.6 with tag prod, and 1.2.4 unsuccessfully by 9.9.9" do
+      set_up do
+        TestDataBuilder.new
+          .create_pact_with_hierarchy("Foo", "1.2.3", "Bar")
+          .create_verification(provider_version: "4.5.6")
+          .use_provider("Bar")
+          .use_provider_version("4.5.6")
+          .create_provider_version_tag("prod")
+          .create_consumer_version("1.2.4")
+          .create_pact
+          .create_verification(provider_version: "9.9.9", success: false)
+      end
+    end
+
   provider_state "the 'Pricing Service' does not exist in the pact-broker" do
     no_op
   end

data/spec/support/shared_examples_for_responses.rb

@@ -41,3 +41,13 @@ RSpec::Matchers.define :be_a_404_response do
     expect(actual.status).to be 404
   end
 end
+
+RSpec::Matchers.define :include_hash_matching do |expected|
+  match do |array_of_hashes|
+    array_of_hashes.any? { |actual| slice(actual, expected.keys) == expected }
+  end
+
+  def slice actual, keys
+    keys.each_with_object({}) { |k, hash| hash[k] = actual[k] if actual.has_key?(k) }
+  end
+end

data/spec/support/test_data_builder.rb

@@ -154,6 +154,11 @@ class TestDataBuilder
     self
   end
 
+  def use_provider_version version_number
+    @provider_version = PactBroker::Domain::Version.where(pacticipant_id: @provider.id, number: version_number).single_record
+    self
+  end
+
   def create_tag tag_name
     @tag = PactBroker::Domain::Tag.create(name: tag_name, version: @version)
     self
@@ -164,6 +169,11 @@ class TestDataBuilder
     self
   end
 
+  def create_provider_version_tag tag_name
+    @tag = PactBroker::Domain::Tag.create(name: tag_name, version: @provider_version)
+    self
+  end
+
   def create_label label_name
     @label = PactBroker::Domain::Label.create(name: label_name, pacticipant: @pacticipant)
     self

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker
 version: !ruby/object:Gem::Version
-  version: 2.7.0.beta.2
+  version: 2.7.0.beta.3
 platform: ruby
 authors:
 - Bethany Skurrie
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-10-30 00:00:00.000000000 Z
+date: 2017-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -430,6 +430,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: bump
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
 description: A server that stores and returns pact files generated by the pact gem.
   It enables head/prod cross testing of the consumer and provider projects.
 email:
@@ -796,6 +810,7 @@ files:
 - script/publish-not-a-pact.sh
 - script/publish.sh
 - script/recreate-pg-db.sh
+- script/release.sh
 - script/seed-matrix.rb
 - script/seed.rb
 - script/update-hal-browser