pact_broker 0.0.9 → 0.0.10

data/CHANGELOG.md

@@ -2,6 +2,20 @@ Do this to generate your change history
 
     $ git log --date=relative --pretty=format:'  * %h - %s (%an, %ad)' 'package/pact-broker-0.0.PRODVERSION'..'package/pact-broker-0.0.NEWVERSION'
 
+#### 0.0.10 (2014-03-24)
+
+* 7aee2ae - Implemented version tagging (Beth, 2 days ago)
+* cc78f92 - Added 'latest' pact url to pact representation in the 'latest pacts' response (Beth, 2 days ago)
+
+#### 0.0.9 (2014-02-27)
+
+* d07f4b7 - Using default gem publish tasks (Beth, 4 weeks ago)
+* d60b7ee - Comment (Beth, 7 weeks ago)
+* 836347c - Using local pacts (Beth, 7 weeks ago)
+* a2cb2bb - Fixed bug querying mysql DB, rather than sqlite (Beth, 7 weeks ago)
+* 9d5f83b - Using the to_json options to pass in the base_url instead of the nasty hack. (Beth, 4 months ago)
+* adb6148 - Changed 'last' to 'latest' (Beth, 4 months ago)
+
 #### 0.0.8 (2013-11-18)
 
 * 6022baa - Changed name to title in list pacticipants response (Beth, 7 hours ago)

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    pact_broker (0.0.9)
+    pact_broker (0.0.10)
       httparty
       json
       roar
@@ -20,6 +20,7 @@ GEM
     ci_reporter (1.9.0)
       builder (>= 2.1.2)
     coderay (1.0.9)
+    colored (1.2)
     columnize (0.3.6)
     debugger (1.6.2)
       columnize (>= 0.3.1)
@@ -30,7 +31,7 @@ GEM
     diff-lcs (1.2.5)
     fakefs (0.4.3)
     find_a_port (1.0.1)
-    httparty (0.12.0)
+    httparty (0.13.0)
       json (~> 1.8)
       multi_xml (>= 0.5.2)
     i18n (0.6.9)
@@ -41,8 +42,9 @@ GEM
     multi_xml (0.5.5)
     nokogiri (1.6.1)
       mini_portile (~> 0.5.0)
-    pact (1.0.26)
+    pact (1.1.0.rc1)
       awesome_print (~> 1.1)
+      colored
       find_a_port (~> 1.0.1)
       json
       rack-test (~> 0.6.2)
@@ -59,11 +61,11 @@ GEM
       rack (>= 1.0)
     rake (10.0.4)
     randexp (0.1.7)
-    representable (1.6.1)
+    representable (1.7.7)
       multi_json
       nokogiri
-    roar (0.12.0)
-      representable (~> 1.6.0)
+    roar (0.12.4)
+      representable (>= 1.6.0)
     rspec (2.14.1)
       rspec-core (~> 2.14.0)
       rspec-expectations (~> 2.14.0)
@@ -74,7 +76,7 @@ GEM
     rspec-fire (1.2.0)
       rspec (~> 2.11)
     rspec-mocks (2.14.4)
-    sequel (4.7.0)
+    sequel (4.8.0)
     shotgun (0.9)
       rack (>= 1.0)
     simplecov (0.7.1)
@@ -82,7 +84,7 @@ GEM
       simplecov-html (~> 0.7.1)
     simplecov-html (0.7.1)
     slop (3.4.6)
-    sqlite3 (1.3.8)
+    sqlite3 (1.3.9)
     thor (0.18.1)
     versionomy (0.4.4)
       blockenspiel (>= 0.4.5)
@@ -99,7 +101,7 @@ DEPENDENCIES
   ci_reporter
   debugger
   fakefs (~> 0.4)
-  pact
+  pact (= 1.1.0.rc1)
   pact_broker!
   pry
   rake (~> 10.0.3)

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

@@ -1,6 +1,6 @@
 require 'roar/decorator'
 require 'roar/representer/json/hal'
-require 'pact_broker/api/decorators/pact_broker_urls'
+require 'pact_broker/api/pact_broker_urls'
 
 module PactBroker
 

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

@@ -0,0 +1,24 @@
+require_relative 'pact_decorator'
+
+module PactBroker
+
+  module Api
+
+    module Decorators
+
+      class LatestPactDecorator < PactDecorator
+
+        links :self do | options |
+          [
+            {
+              href: latest_pact_url(options[:base_url], represented)
+            },{
+              href: pact_url(options[:base_url], represented)
+            }
+          ]
+        end
+
+      end
+    end
+  end
+end

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

@@ -1,6 +1,6 @@
 require_relative 'base_decorator'
 require_relative 'version_decorator'
-require_relative 'pact_decorator'
+require_relative 'latest_pact_decorator'
 require_relative 'representable_pact'
 
 module PactBroker
@@ -9,11 +9,11 @@ module PactBroker
 
     module Decorators
 
-      class PactCollectionRepresenter < BaseDecorator
+      class PactCollectionDecorator < BaseDecorator
         include Roar::Representer::JSON::HAL
         include PactBroker::Api::PactBrokerUrls
 
-        collection :pacts, decorator_scope: true, :class => PactBroker::Models::Pact, :extend => PactBroker::Api::Decorators::PactRepresenter
+        collection :pacts, decorator_scope: true, :class => PactBroker::Models::Pact, :extend => PactBroker::Api::Decorators::LatestPactDecorator
 
         def pacts
           represented.collect{ | pact | create_representable_pact(pact) }

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

@@ -7,7 +7,7 @@ module PactBroker
 
     module Decorators
 
-      class PactRepresenter < BaseDecorator
+      class PactDecorator < BaseDecorator
 
         property :consumer, :extend => PactBroker::Api::Decorators::PactPacticipantRepresenter, :embedded => true
         property :provider, :extend => PactBroker::Api::Decorators::PactPacticipantRepresenter, :embedded => true

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

@@ -1,4 +1,4 @@
-require_relative 'pact_broker_urls'
+require 'pact_broker/api/pact_broker_urls'
 require_relative 'version_decorator'
 require_relative 'base_decorator'
 

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

@@ -1,5 +1,5 @@
 require 'roar/representer/json/hal'
-require_relative 'pact_broker_urls'
+require 'pact_broker/api/pact_broker_urls'
 require_relative 'version_decorator'
 
 module Roar

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

@@ -0,0 +1,19 @@
+require_relative 'base_decorator'
+require_relative 'pact_pacticipant_decorator'
+
+module PactBroker
+
+  module Api
+
+    module Decorators
+
+      class TagDecorator < BaseDecorator
+
+        link :self do | options |
+          tag_url(options[:base_url], represented)
+        end
+
+      end
+    end
+  end
+end

data/lib/pact_broker/api/decorators.rb

@@ -3,4 +3,5 @@ require 'pact_broker/api/decorators/pacticipant_collection_decorator'
 require 'pact_broker/api/decorators/version_decorator'
 require 'pact_broker/api/decorators/pact_collection_decorator'
 require 'pact_broker/api/decorators/pact_pacticipant_decorator'
-require 'pact_broker/api/decorators/pact_decorator'
+require 'pact_broker/api/decorators/pact_decorator'
+require 'pact_broker/api/decorators/tag_decorator'

data/lib/pact_broker/api/{decorators/pact_broker_urls.rb → pact_broker_urls.rb}

@@ -23,7 +23,8 @@ module PactBroker
       end
 
       def pact_url base_url, pact
-        "#{pactigration_base_url(base_url, pact)}/version/#{pact.consumer.version.number}"
+        representable_pact = representable_pact(pact)
+        "#{pactigration_base_url(base_url, representable_pact)}/version/#{representable_pact.consumer.version.number}"
       end
 
       def latest_pact_url base_url, pact
@@ -34,8 +35,20 @@ module PactBroker
         "#{base_url}/pacts/latest"
       end
 
+      def tags_url base_url, version
+        "#{version_url(base_url, version)}/tags"
+      end
+
+      def tag_url base_url, tag
+        "#{tags_url(base_url, tag.version)}/#{tag.name}"
+      end
+
       private
 
+      def representable_pact pact
+        Decorators::RepresentablePact === pact ? pact : Decorators::RepresentablePact.new(pact)
+      end
+
       def pactigration_base_url base_url, pact
         "#{base_url}/pact/provider/#{url_encode(pact.provider.name)}/consumer/#{url_encode(pact.consumer.name)}"
       end

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

@@ -1,13 +1,21 @@
 require 'webmachine'
 require 'pact_broker/services'
 require 'pact_broker/api/decorators'
+require 'pact_broker/logging'
+require 'pact_broker/api/pact_broker_urls'
 
 module PactBroker::Api
 
   module Resources
 
+
     class ErrorHandler
+
+      include PactBroker::Logging
+
       def self.handle_exception e, response
+        logger.error e
+        logger.error e.backtrace
         response.body = {:message => e.message, :backtrace => e.backtrace }.to_json
         response.code = 500
       end
@@ -16,6 +24,7 @@ module PactBroker::Api
     class BaseResource < Webmachine::Resource
 
       include PactBroker::Services
+      include PactBroker::Api::PactBrokerUrls
 
       def identifier_from_path
         request.path_info.each_with_object({}) do | pair, hash|
@@ -23,13 +32,17 @@ module PactBroker::Api
         end
       end
 
-      def request_base_url
+      def resource_url
         request.uri.to_s.gsub(/#{request.uri.path}$/,'')
       end
 
       def handle_exception e
         PactBroker::Api::Resources::ErrorHandler.handle_exception(e, response)
       end
+
+      def params
+        JSON.parse(request.body.to_s, symbolize_names: true)
+      end
     end
   end
 end

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

@@ -19,7 +19,7 @@ module PactBroker::Api
       end
 
       def generate_json pacts
-        PactBroker::Api::Decorators::PactCollectionRepresenter.new(pacts).to_json(base_url: request_base_url)
+        PactBroker::Api::Decorators::PactCollectionDecorator.new(pacts).to_json(base_url: resource_url)
       end
 
     end

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

@@ -25,15 +25,9 @@ module PactBroker::Api
       end
 
       def from_json
-        pact, created = pact_service.create_or_update_pact(identifier_from_path.merge(:json_content => pact_content))
-        response.body = pact.json_content
-        @manual_response_code = 201 if created
-      end
-
-      def finish_request
-        if @manual_response_code
-          response.code = @manual_response_code
-        end
+        @pact, created = pact_service.create_or_update_pact(identifier_from_path.merge(:json_content => pact_content))
+        response.headers["Location"] = pact_url(resource_url, @pact) if created
+        response.body = to_json
       end
 
       def to_json

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

@@ -31,12 +31,13 @@ module PactBroker::Api
       end
 
       def from_json
-        pacticipant, created = pacticipant_service.create_or_update_pacticipant(
-          name: identifier_from_path[:name],
-          repository_url: params[:repository_url]
-        )
-        @manual_response_code = 201 if created
-        response.body = generate_json(pacticipant)
+        if @pacticipant
+          @pacticipant = pacticipant_service.update params.merge(name: identifier_from_path[:name])
+        else
+          @pacticipant = pacticipant_service.create params.merge(name: identifier_from_path[:name])
+          response.headers["Location"] = pacticipant_url(resource_url, @pacticipant)
+        end
+        response.body = to_json
       end
 
       def resource_exists?
@@ -44,22 +45,8 @@ module PactBroker::Api
         @pacticipant != nil
       end
 
-      def finish_request
-        if @manual_response_code
-          response.code = @manual_response_code
-        end
-      end
-
       def to_json
-        generate_json(@pacticipant)
-      end
-
-      def generate_json pacticipant
-        PactBroker::Api::Decorators::PacticipantRepresenter.new(pacticipant).to_json(base_url: request_base_url)
-      end
-
-      def params
-        JSON.parse(request.body.to_s)
+        PactBroker::Api::Decorators::PacticipantRepresenter.new(@pacticipant).to_json(base_url: resource_url)
       end
 
     end

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

@@ -19,7 +19,7 @@ module PactBroker::Api
       end
 
       def generate_json pacticipants
-        PactBroker::Api::Decorators::PacticipantCollectionRepresenter.new(pacticipants).to_json(base_url: request_base_url)
+        PactBroker::Api::Decorators::PacticipantCollectionRepresenter.new(pacticipants).to_json(base_url: resource_url)
       end
 
     end

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

@@ -0,0 +1,44 @@
+require 'pact_broker/api/resources/base_resource'
+
+module PactBroker::Api
+
+  module Resources
+
+    class Tag < BaseResource
+
+      def content_types_provided
+        [["application/hal+json", :to_json]]
+      end
+
+      def content_types_accepted
+        [["application/json", :from_json]]
+      end
+
+      def allowed_methods
+        ["GET","PUT"]
+      end
+
+      def from_json
+        unless @tag
+          @tag = tag_service.create identifier_from_path
+          response.headers["Location"] = tag_url(resource_url, @tag)
+        end
+        response.body = generate_json @tag
+      end
+
+      def resource_exists?
+        @tag = tag_service.find identifier_from_path
+      end
+
+      def to_json
+        generate_json(@tag)
+      end
+
+      def generate_json tag
+        PactBroker::Api::Decorators::TagDecorator.new(tag).to_json(base_url: resource_url)
+      end
+
+    end
+  end
+
+end

data/lib/pact_broker/api.rb

@@ -3,6 +3,7 @@ require 'pact_broker/api/resources/latest_pact'
 require 'pact_broker/api/resources/latest_pacts'
 require 'pact_broker/api/resources/pacticipant'
 require 'pact_broker/api/resources/pacticipants'
+require 'pact_broker/api/resources/tag'
 require 'pact_broker/api/resources/index'
 
 require 'webmachine/adapters/rack'
@@ -18,6 +19,7 @@ module PactBroker
         add ['pacts', 'latest'], Api::Resources::LatestPacts
         add ['pacticipants'], Api::Resources::Pacticipants
         add ['pacticipants', :name], Api::Resources::Pacticipant
+        add ['pacticipants', :pacticipant_name, 'versions', :pacticipant_version_number, 'tags', :tag_name], Api::Resources::Tag
         add [], Api::Resources::Index
       end
     end

data/lib/pact_broker/logging.rb

@@ -12,6 +12,10 @@ module PactBroker
       base.extend(self)
     end
 
+    def logger= logger
+      @@logger = logger
+    end
+
     def logger
       @@logger ||= begin
         FileUtils.mkdir_p(LOG_DIR)

data/lib/pact_broker/models/tag.rb

@@ -5,6 +5,11 @@ module PactBroker
   module Models
     class Tag < Sequel::Model
 
+      unrestrict_primary_key
+
+      associate(:many_to_one, :version, :class => "PactBroker::Models::Version", :key => :version_id, :primary_key => :id)
+
+
     end
   end
 end

data/lib/pact_broker/repositories/tag_repository.rb

@@ -0,0 +1,25 @@
+require 'pact_broker/models/tag'
+
+module PactBroker
+  module Repositories
+    class TagRepository
+
+
+      def create args
+        Models::Tag.new(name: args.fetch(:name), version: args.fetch(:version)).save
+      end
+
+      def find args
+        PactBroker::Models::Tag
+          .select(:tags__name, :tags__version_id)
+          .join(:versions, {id: :version_id})
+          .join(:pacticipants, {pacticipants__id: :versions__pacticipant_id})
+          .where(:tags__name => args.fetch(:tag_name))
+          .where(:versions__number => args.fetch(:pacticipant_version_number))
+          .where(:pacticipants__name => args.fetch(:pacticipant_name))
+          .single_record
+      end
+
+    end
+  end
+end

data/lib/pact_broker/repositories/version_repository.rb

@@ -9,6 +9,14 @@ module PactBroker
         PactBroker::Models::Version.where(number: number, pacticipant_id: pacticipant_id).single_record
       end
 
+      def find_by_pacticipant_name_and_number pacticipant_name, number
+        PactBroker::Models::Version
+          .where(number: number)
+          .join(:pacticipants, {id: :pacticipant_id})
+          .where(name: pacticipant_name)
+          .single_record
+      end
+
       def create args
         PactBroker.logger.info "Creating version #{args[:number]} for pacticipant_id=#{args[:pacticipant_id]}"
         PactBroker::Models::Version.new(number: args[:number], pacticipant_id: args[:pacticipant_id]).save

data/lib/pact_broker/repositories.rb

@@ -2,6 +2,7 @@ require 'pact_broker/models'
 require 'pact_broker/repositories/pacticipant_repository'
 require 'pact_broker/repositories/version_repository'
 require 'pact_broker/repositories/pact_repository'
+require 'pact_broker/repositories/tag_repository'
 
 module PactBroker
   module Repositories
@@ -17,6 +18,10 @@ module PactBroker
       PactRepository.new
     end
 
+    def tag_repository
+      TagRepository.new
+    end
+
     extend self
   end
 end

data/lib/pact_broker/services/pacticipant_service.rb

@@ -24,16 +24,16 @@ module PactBroker
         end
       end
 
-      def self.create_or_update_pacticipant params
-        pacticipant = pacticipant_repository.find_by_name(params[:name])
-        if pacticipant
-          pacticipant.update(repository_url: params[:repository_url])
-          return pacticipant, false
-        else
-          pacticipant = pacticipant_repository.create(name: params[:name], repository_url: params[:repository_url])
-          return pacticipant, true
-        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))
       end
+
+      def self.create params
+        pacticipant_repository.create(params)
+      end
+
     end
   end
 end

data/lib/pact_broker/services/tag_service.rb

@@ -0,0 +1,25 @@
+require 'pact_broker/repositories'
+
+module PactBroker
+
+  module Services
+    module TagService
+
+      extend self
+
+      extend PactBroker::Repositories
+
+      def create args
+        pacticipant = pacticipant_repository.find_by_name_or_create args.fetch(:pacticipant_name)
+        version = version_repository.find_by_pacticipant_id_and_number_or_create pacticipant.id, args.fetch(:pacticipant_version_number)
+        tag_repository.create version: version, name: args.fetch(:tag_name)
+      end
+
+      def find args
+        tag_repository.find args
+      end
+
+    end
+  end
+
+end

data/lib/pact_broker/services.rb

@@ -1,5 +1,6 @@
 require 'pact_broker/services/pact_service'
 require 'pact_broker/services/pacticipant_service'
+require 'pact_broker/services/tag_service'
 
 module PactBroker
   module Services
@@ -10,5 +11,9 @@ module PactBroker
     def pacticipant_service
       PacticipantService
     end
+
+    def tag_service
+      TagService
+    end
   end
 end

data/lib/pact_broker/version.rb

@@ -1,3 +1,3 @@
 module PactBroker
-  VERSION = '0.0.9'
+  VERSION = '0.0.10'
 end

data/pact_broker.gemspec

@@ -36,5 +36,5 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'ci_reporter'
   gem.add_development_dependency 'simplecov'
   gem.add_development_dependency 'shotgun'
-  gem.add_development_dependency 'pact'
+  gem.add_development_dependency 'pact', '1.1.0.rc1'
 end

data/spec/lib/pact_broker/api/decorators/pact_collection_decorator_spec.rb

@@ -5,7 +5,7 @@ require 'pact_broker/api/decorators'
 
 module PactBroker::Api::Decorators
 
-  describe PactCollectionRepresenter do
+  describe PactCollectionDecorator do
 
     let(:pact) do
       provider = PactBroker::Models::Pacticipant.create(:name => 'Pricing Service')
@@ -16,7 +16,7 @@ module PactBroker::Api::Decorators
     end
 
     xit "should description" do
-      puts PactCollectionRepresenter.new([pact]).to_json
+      puts PactCollectionDecorator.new([pact]).to_json
     end
   end
 

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

@@ -0,0 +1,118 @@
+require 'spec_helper'
+require 'pact_broker/api/resources/tag'
+
+module PactBroker
+  module Api
+
+    module Resources
+
+      describe Tag do
+
+        let(:tag) { double("PactBroker::Models::Tag") }
+        let(:tag_decorator) { instance_double("PactBroker::Api::Decorators::TagDecorator", :to_json => tag_json) }
+        let(:tag_json) { {"some" => "tag"}.to_json }
+        let(:tag_attributes) {
+          {
+            :pacticipant_name => "Condor",
+            :pacticipant_version_number => "1.3.0",
+            :tag_name => "prod"
+          }
+        }
+
+        describe "GET" do
+          before do
+            allow(Services::TagService).to receive(:find).and_return(tag)
+            allow(PactBroker::Api::Decorators::TagDecorator).to receive(:new).and_return(tag_decorator)
+          end
+
+          subject { get("/pacticipants/Condor/versions/1.3.0/tags/prod" ) }
+
+          context "when the tag exists" do
+
+            it "renders the tag" do
+              subject
+              expect(last_response.body).to eq tag_json
+            end
+
+            it "returns a success response" do
+              subject
+              expect(last_response).to be_successful
+            end
+
+          end
+
+          context "when the tag does not exist" do
+            let(:tag) { nil }
+
+            it "returns a 404" do
+              subject
+              expect(last_response.status).to eq 404
+            end
+          end
+        end
+
+        describe "PUT" do
+
+          let(:tag_url) { 'http://example.org/tag/url'}
+
+          before do
+            allow_any_instance_of(PactBroker::Api::Resources::Tag).to receive(:tag_url).and_return(tag_url)
+            allow(Services::TagService).to receive(:find).and_return(tag)
+            allow(PactBroker::Api::Decorators::TagDecorator).to receive(:new).and_return(tag_decorator)
+          end
+
+          subject { put("/pacticipants/Condor/versions/1.3.0/tags/prod", nil, "CONTENT_LENGTH" => "0", "CONTENT_TYPE" => "application/json") }
+
+          it "returns a success response" do
+            subject
+            expect(last_response).to be_successful
+          end
+
+          context "when the tag already exists" do
+
+            it "returns a 200" do
+              subject
+              expect(last_response.status).to be 200
+            end
+
+            it "renders the tag" do
+              expect(tag_decorator).to receive(:to_json).with(base_url: "http://example.org")
+              subject
+              expect(last_response.body).to eq tag_json
+            end
+
+          end
+
+          context "when the tag does not exist" do
+            before do
+              allow(Services::TagService).to receive(:find).and_return(nil)
+              allow(Services::TagService).to receive(:create).and_return(tag)
+            end
+
+
+            it "creates the tag" do
+              expect(Services::TagService).to receive(:create).with(tag_attributes)
+              subject
+            end
+
+            it "returns a 201" do
+              subject
+              expect(last_response.status).to be 201
+            end
+
+            it "renders the tag" do
+              expect(tag_decorator).to receive(:to_json).with(base_url: "http://example.org")
+              subject
+              expect(last_response.body).to eq tag_json
+            end
+
+          end
+
+
+        end
+
+      end
+
+    end
+  end
+end

data/spec/lib/pact_broker/repositories/tag_repository_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+require 'pact_broker/repositories/tag_repository'
+
+module PactBroker
+  module Repositories
+    describe TagRepository do
+
+      describe ".find" do
+
+        let(:pacticipant_name) { "test_pacticipant" }
+        let(:version_number) { "1.2.3" }
+        let(:tag_name) { "prod" }
+
+        subject { TagRepository.new }
+        let(:options) { {pacticipant_name: pacticipant_name, pacticipant_version_number: version_number, tag_name: tag_name} }
+        let(:find_tag) { subject.find options }
+
+        let!(:provider_state_builder) do
+          ProviderStateBuilder.new
+            .create_pacticipant("wrong_pacticipant")
+            .create_version(version_number)
+            .create_tag(tag_name) #Tag with wrong pacticipant
+            .create_pacticipant(pacticipant_name)
+            .create_version("2.0.0")
+            .create_tag(tag_name) # Tag with wrong version number
+            .create_version(version_number)
+            .create_tag("wrong tag") #Tag with wrong name
+        end
+
+        context "when the tag exists" do
+
+          before do
+            provider_state_builder.create_tag(tag_name) # Right tag!
+          end
+
+          it "returns the tag" do
+            expect(find_tag.name).to eq tag_name
+            expect(find_tag.version.number).to eq version_number
+            expect(find_tag.version.pacticipant.name).to eq pacticipant_name
+          end
+
+        end
+
+        context "when the tag does not exist" do
+          it "returns nil" do
+            expect(find_tag).to be_nil
+          end
+        end
+      end
+
+    end
+  end
+end

data/spec/lib/pact_broker/repositories/version_repository_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+require 'pact_broker/repositories/version_repository'
+
+module PactBroker
+  module Repositories
+    describe VersionRepository do
+
+      describe "#find_by_pacticipant_name_and_number" do
+
+        let(:pacticipant_name) { "test_pacticipant" }
+        let(:version_number) { "1.2.3" }
+
+        subject { described_class.new.find_by_pacticipant_name_and_number pacticipant_name, version_number }
+
+        context "when the version exists" do
+          before do
+            ProviderStateBuilder.new.create_version_with_hierarchy "other_pacticipant", version_number
+            ProviderStateBuilder.new.create_version_with_hierarchy pacticipant_name, version_number
+          end
+
+          it "returns the version" do
+            expect(subject.number).to eq version_number
+            expect(subject.pacticipant.name).to eq pacticipant_name
+          end
+        end
+
+        context "when the version doesn't exist" do
+          it "returns nil" do
+            expect(subject).to be_nil
+          end
+        end
+
+      end
+
+
+    end
+  end
+end

data/spec/lib/pact_broker/services/tag_service_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+require 'pact_broker/services/tag_service'
+
+module PactBroker
+  module Services
+    describe TagService do
+
+      describe ".create" do
+
+        let(:pacticipant_name) { "test_pacticipant" }
+        let(:version_number) { "1.2.3" }
+        let(:tag_name) { "prod" }
+
+        let(:options) { {pacticipant_name: pacticipant_name, pacticipant_version_number: version_number, tag_name: tag_name}}
+        let(:provider_state_builder) { ProviderStateBuilder.new }
+
+        subject { TagService.create(options) }
+
+        # Naughty integration test... didn't seem much point unit testing this
+
+        it "creates the new tag" do
+          expect(subject.name).to eq tag_name
+          expect(subject.version.number).to eq version_number
+          expect(subject.version.pacticipant.name).to eq pacticipant_name
+        end
+
+      end
+
+    end
+  end
+end

data/spec/service_consumers/pact_helper.rb

@@ -10,6 +10,7 @@ require 'pact_broker/api/resources/pact'
 
 Pact.configure do | config |
   config.logger.level = Logger::DEBUG
+  #config.diff_format = :plus_and_minus
 end
 
 Pact.service_provider "Pact Broker" do

data/spec/service_consumers/provider_states_for_pact_broker_client.rb

@@ -54,6 +54,15 @@ Pact.provider_states_for "Pact Broker Client" do
     end
   end
 
+  provider_state "'Condor' exists in the pact-broker with version 1.3.0, tagged with 'prod'" do
+    set_up do
+      ProviderStateBuilder.new
+        .create_pacticipant("Condor")
+        .create_version('1.3.0')
+        .create_tag('prod')
+    end
+  end
+
   provider_state "'Condor' does not exist in the pact-broker" do
      no_op
    end

data/spec/spec_helper.rb

@@ -4,7 +4,11 @@ RACK_ENV = 'test'
 
 $: << File.expand_path("../../", __FILE__)
 
+require 'rack/test'
 require 'pact_broker/db'
+require './spec/support/provider_state_builder'
+require 'pact_broker/api'
+require 'rspec/fire'
 
 RSpec.configure do | config |
   config.before :suite do
@@ -19,4 +23,10 @@ RSpec.configure do | config |
     DB::PACT_BROKER_DB[:pacticipants].truncate
   end
 
+  config.include Rack::Test::Methods
+  config.include RSpec::Fire
+
+  def app
+    PactBroker::API
+  end
 end

data/spec/support/provider_state_builder.rb

@@ -62,6 +62,31 @@ class ProviderStateBuilder
     PactBroker::Models::Pact.create(:consumer_version => version, :provider => provider)
   end
 
+  def create_version_with_hierarchy pacticipant_name, pacticipant_version
+    pacticipant = PactBroker::Models::Pacticipant.create(:name => pacticipant_name)
+    PactBroker::Models::Version.create(:number => pacticipant_version, :pacticipant => pacticipant)
+  end
+
+  def create_tag_with_hierarchy pacticipant_name, pacticipant_version, tag_name
+    version = create_version_with_hierarchy pacticipant_name, pacticipant_version
+    PactBroker::Models::Tag.create(name: tag_name, version: version)
+  end
+
+  def create_pacticipant pacticipant_name
+    @pacticipant = PactBroker::Models::Pacticipant.create(:name => pacticipant_name)
+    self
+  end
+
+  def create_version version_number
+    @version = PactBroker::Models::Version.create(:number => version_number, :pacticipant => @pacticipant)
+    self
+  end
+
+  def create_tag tag_name
+    @tag = PactBroker::Models::Tag.create(name: tag_name, version: @version)
+    self
+  end
+
   private
 
   # def create_pacticipant name

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pact_broker
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
   prerelease: 
 platform: ruby
 authors:
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-27 00:00:00.000000000 Z
+date: 2014-03-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -258,17 +258,17 @@ dependencies:
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.1.0.rc1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.1.0.rc1
 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:
@@ -299,14 +299,16 @@ files:
 - lib/pact_broker/api.rb
 - lib/pact_broker/api/decorators.rb
 - lib/pact_broker/api/decorators/base_decorator.rb
-- lib/pact_broker/api/decorators/pact_broker_urls.rb
+- lib/pact_broker/api/decorators/latest_pact_decorator.rb
 - lib/pact_broker/api/decorators/pact_collection_decorator.rb
 - lib/pact_broker/api/decorators/pact_decorator.rb
 - lib/pact_broker/api/decorators/pact_pacticipant_decorator.rb
 - lib/pact_broker/api/decorators/pacticipant_collection_decorator.rb
 - lib/pact_broker/api/decorators/pacticipant_decorator.rb
 - lib/pact_broker/api/decorators/representable_pact.rb
+- lib/pact_broker/api/decorators/tag_decorator.rb
 - lib/pact_broker/api/decorators/version_decorator.rb
+- lib/pact_broker/api/pact_broker_urls.rb
 - lib/pact_broker/api/resources/base_resource.rb
 - lib/pact_broker/api/resources/index.rb
 - lib/pact_broker/api/resources/latest_pact.rb
@@ -314,6 +316,7 @@ files:
 - lib/pact_broker/api/resources/pact.rb
 - lib/pact_broker/api/resources/pacticipant.rb
 - lib/pact_broker/api/resources/pacticipants.rb
+- lib/pact_broker/api/resources/tag.rb
 - lib/pact_broker/db.rb
 - lib/pact_broker/logging.rb
 - lib/pact_broker/models.rb
@@ -325,10 +328,12 @@ files:
 - lib/pact_broker/repositories.rb
 - lib/pact_broker/repositories/pact_repository.rb
 - lib/pact_broker/repositories/pacticipant_repository.rb
+- lib/pact_broker/repositories/tag_repository.rb
 - lib/pact_broker/repositories/version_repository.rb
 - lib/pact_broker/services.rb
 - lib/pact_broker/services/pact_service.rb
 - lib/pact_broker/services/pacticipant_service.rb
+- lib/pact_broker/services/tag_service.rb
 - lib/pact_broker/services/version_service.rb
 - lib/pact_broker/tasks.rb
 - lib/pact_broker/tasks/migration_task.rb
@@ -337,8 +342,12 @@ files:
 - pact_broker_client-pact_broker.json
 - spec/lib/pact_broker/api/decorators/pact_collection_decorator_spec.rb
 - spec/lib/pact_broker/api/decorators/representable_pact_spec.rb
+- spec/lib/pact_broker/api/resources/tag_spec.rb
 - spec/lib/pact_broker/models/order_versions_spec.rb
 - spec/lib/pact_broker/repositories/pact_repository_spec.rb
+- spec/lib/pact_broker/repositories/tag_repository_spec.rb
+- spec/lib/pact_broker/repositories/version_repository_spec.rb
+- spec/lib/pact_broker/services/tag_service_spec.rb
 - spec/service_consumers/pact_helper.rb
 - spec/service_consumers/provider_states_for_pact_broker_client.rb
 - spec/spec_helper.rb
@@ -361,7 +370,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 615091496951534936
+      hash: -449205238157124147
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -370,7 +379,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 615091496951534936
+      hash: -449205238157124147
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -380,8 +389,12 @@ summary: See description
 test_files:
 - spec/lib/pact_broker/api/decorators/pact_collection_decorator_spec.rb
 - spec/lib/pact_broker/api/decorators/representable_pact_spec.rb
+- spec/lib/pact_broker/api/resources/tag_spec.rb
 - spec/lib/pact_broker/models/order_versions_spec.rb
 - spec/lib/pact_broker/repositories/pact_repository_spec.rb
+- spec/lib/pact_broker/repositories/tag_repository_spec.rb
+- spec/lib/pact_broker/repositories/version_repository_spec.rb
+- spec/lib/pact_broker/services/tag_service_spec.rb
 - spec/service_consumers/pact_helper.rb
 - spec/service_consumers/provider_states_for_pact_broker_client.rb
 - spec/spec_helper.rb