pact_broker 1.0.0 → 1.1.0

data/lib/pact_broker/repositories/pacticipant_repository.rb

@@ -9,6 +9,10 @@ module PactBroker
         PactBroker::Models::Pacticipant.where(name: name).single_record
       end
 
+      def find_by_id id
+        PactBroker::Models::Pacticipant.where(id: id).single_record
+      end
+
       def find_all
         PactBroker::Models::Pacticipant.order(:name).all
       end

data/lib/pact_broker/repositories/webhook_repository.rb

@@ -0,0 +1,134 @@
+require 'sequel'
+require 'pact_broker/models/webhook'
+require 'pact_broker/models/pacticipant'
+require 'pact_broker/db'
+require 'base64'
+
+module PactBroker
+  module Repositories
+
+    class WebhookRepository
+
+      # Experimenting with decoupling the model from the database representation.
+      # Sure makes it messier for saving/retrieving.
+
+      include Repositories
+
+      def create webhook, consumer, provider
+        db_webhook = Webhook.from_model webhook, consumer, provider
+        db_webhook.uuid = SecureRandom.urlsafe_base64
+        db_webhook.save
+        webhook.request.headers.each_pair do | name, value |
+          db_webhook.add_header WebhookHeader.from_model(name, value, db_webhook.id)
+        end
+
+        find_by_uuid db_webhook.uuid
+      end
+
+      def find_by_uuid uuid
+        db_webhook = Webhook.where(uuid: uuid).single_record
+        return nil if db_webhook.nil?
+        db_webhook.to_model
+      end
+
+      def delete_by_uuid uuid
+        Webhook.where(uuid: uuid).destroy
+      end
+
+      def delete_by_pacticipant pacticipant
+        Webhook.where(consumer_id: pacticipant.id).destroy
+        Webhook.where(provider_id: pacticipant.id).destroy
+      end
+
+      def find_all
+        Webhook.all.collect { | db_webhook| db_webhook.to_model }
+      end
+
+      def find_by_consumer_and_provider consumer, provider
+        Webhook.where(consumer_id: consumer.id, provider_id: provider.id).collect { | db_webhook| db_webhook.to_model }
+      end
+
+    end
+
+    class Webhook < Sequel::Model
+
+      set_primary_key :id
+      associate(:many_to_one, :provider, :class => "PactBroker::Models::Pacticipant", :key => :provider_id, :primary_key => :id)
+      associate(:many_to_one, :consumer, :class => "PactBroker::Models::Pacticipant", :key => :consumer_id, :primary_key => :id)
+      one_to_many :headers, :class => "PactBroker::Repositories::WebhookHeader", :reciprocal => :webhook
+
+      def before_destroy
+        WebhookHeader.where(webhook_id: id).destroy
+      end
+
+      def self.from_model webhook, consumer, provider
+        is_json_request_body = !(String === webhook.request.body || webhook.request.body.nil?) # Can't rely on people to set content type
+        new(
+          uuid: webhook.uuid,
+          method: webhook.request.method,
+          url: webhook.request.url,
+          username: webhook.request.username,
+          password: not_plain_text_password(webhook.request.password),
+          body: (is_json_request_body ? webhook.request.body.to_json : webhook.request.body),
+          is_json_request_body: is_json_request_body
+        ).tap do | db_webhook |
+          db_webhook.consumer_id = consumer.id
+          db_webhook.provider_id = provider.id
+        end
+      end
+
+      def self.not_plain_text_password password
+        password.nil? ? nil : Base64.strict_encode64(password)
+      end
+
+      def to_model
+        Models::Webhook.new(
+          uuid: uuid,
+          consumer: consumer,
+          provider: provider,
+          request: Models::WebhookRequest.new(request_attributes),
+          created_at: created_at,
+          updated_at: updated_at)
+      end
+
+      def request_attributes
+        values.merge(headers: parsed_headers, body: parsed_body, password: plain_text_password)
+      end
+
+      def plain_text_password
+        password.nil? ? nil : Base64.strict_decode64(password)
+      end
+
+      def parsed_headers
+        WebhookHeader.where(webhook_id: id).all.each_with_object({}) do | header, hash |
+          hash[header[:name]] = header[:value]
+        end
+      end
+
+      def parsed_body
+        if body && is_json_request_body
+           JSON.parse(body)
+        else
+          body
+        end
+      end
+
+    end
+
+    Webhook.plugin :timestamps, :update_on_create=>true
+
+    class WebhookHeader < Sequel::Model
+
+      associate(:many_to_one, :webhook, :class => "PactBroker::Repositories::Webhook", :key => :webhook_id, :primary_key => :id)
+
+      def self.from_model name, value, webhook_id
+        db_header = new
+        db_header.name = name
+        db_header.value = value
+        db_header.webhook_id = webhook_id
+        db_header
+      end
+
+    end
+  end
+end

data/lib/pact_broker/services.rb

@@ -1,6 +1,8 @@
 require 'pact_broker/services/pact_service'
 require 'pact_broker/services/pacticipant_service'
 require 'pact_broker/services/tag_service'
+require 'pact_broker/services/group_service'
+require 'pact_broker/services/webhook_service'
 
 module PactBroker
   module Services
@@ -15,5 +17,13 @@ module PactBroker
     def tag_service
       TagService
     end
+
+    def group_service
+      GroupService
+    end
+
+    def webhook_service
+      WebhookService
+    end
   end
 end

data/lib/pact_broker/services/group_service.rb

@@ -0,0 +1,24 @@
+require 'pact_broker/repositories'
+require 'pact_broker/functions/groupify'
+
+module PactBroker
+
+  module Services
+    module GroupService
+
+      extend self
+
+      extend PactBroker::Repositories
+      extend PactBroker::Services
+
+      def find_group_containing pacticipant
+        groups.find { | group | group.include_pacticipant? pacticipant }
+      end
+
+      def groups
+        Functions::Groupify.call pacticipant_service.find_relationships
+      end
+
+    end
+  end
+end

data/lib/pact_broker/services/pact_service.rb

@@ -7,7 +7,8 @@ module PactBroker
 
       extend self
 
-      extend PactBroker::Repositories
+      extend Repositories
+      extend Services
 
       def find_latest_pact params
         pact_repository.find_latest_pact(params[:consumer_name], params[:provider_name], params[:tag])
@@ -24,18 +25,47 @@ module PactBroker
       def create_or_update_pact params
         provider = pacticipant_repository.find_by_name_or_create params[:provider_name]
         consumer = pacticipant_repository.find_by_name_or_create params[:consumer_name]
-        version = version_repository.find_by_pacticipant_id_and_number_or_create consumer.id, params[:consumer_version_number]
+        consumer_version = version_repository.find_by_pacticipant_id_and_number_or_create consumer.id, params[:consumer_version_number]
+        pact = pact_repository.find_by_version_and_provider(consumer_version.id, provider.id)
 
-        if pact = pact_repository.find_by_version_and_provider(version.id, provider.id)
-          pact.update(json_content: params[:json_content])
-          return pact, false
+        if pact
+          return update_pact params, pact
         else
-          pact = pact_repository.create json_content: params[:json_content], version_id: version.id, provider_id: provider.id
-          return pact, true
+          return create_pact params, consumer_version, provider
         end
 
       end
 
+      def pact_has_changed_since_previous_version? pact
+        previous_pact = pact_repository.find_previous_pact pact
+        previous_pact && pact.json_content != previous_pact.json_content
+      end
+
+      private
+
+      def update_pact params, pact
+        previous_json_content = pact.json_content
+        pact.update(json_content: params[:json_content])
+        if previous_json_content != params[:json_content]
+          webhook_service.execute_webhooks pact
+        end
+        return pact, false
+      end
+
+      def create_pact params, version, provider
+        pact = pact_repository.create json_content: params[:json_content], version_id: version.id, provider_id: provider.id
+        execute_webhooks pact
+        return pact, true
+      end
+
+
+
+      def execute_webhooks pact
+        if pact_has_changed_since_previous_version? pact
+          webhook_service.execute_webhooks pact
+        end
+      end
+
     end
   end
 end

data/lib/pact_broker/services/pacticipant_service.rb

@@ -7,7 +7,7 @@ module PactBroker
     class PacticipantService
 
       extend PactBroker::Repositories
-      extend PactBroker::Repositories
+      extend PactBroker::Services
 
       def self.find_all_pacticipants
         pacticipant_repository.find_all
@@ -40,6 +40,17 @@ module PactBroker
         pacticipant_repository.create(params)
       end
 
+      def self.delete name
+        pacticipant = find_pacticipant_by_name name
+        connection = PactBroker::Models::Pacticipant.new.db
+        connection.run("delete from tags where version_id IN (select id from versions where pacticipant_id IN (select id from pacticipants where name = '#{name}'))")
+        connection.run("delete from pacts where version_id IN (select id from versions where pacticipant_id IN (select id from pacticipants where name = '#{name}'))")
+        connection.run("delete from pacts where provider_id IN (select id from pacticipants where name = '#{name}')")
+        connection.run("delete from versions where pacticipant_id IN (select id from pacticipants where name = '#{name}')")
+        webhook_service.delete_by_pacticipant pacticipant
+        connection.run("delete from pacticipants where name = '#{name}'")
+      end
+
     end
   end
 end

data/lib/pact_broker/services/webhook_service.rb

@@ -0,0 +1,63 @@
+require 'pact_broker/repositories'
+require 'pact_broker/logging'
+
+module PactBroker
+
+  module Services
+    class WebhookService
+
+      extend Repositories
+      include Logging
+
+      def self.create webhook, consumer, provider
+        webhook_repository.create webhook, consumer, provider
+      end
+
+      def self.find_by_uuid uuid
+        webhook_repository.find_by_uuid uuid
+      end
+
+      def self.delete_by_uuid uuid
+        webhook_repository.delete_by_uuid uuid
+      end
+
+      def self.delete_by_pacticipant pacticipant
+        webhook_repository.delete_by_pacticipant pacticipant
+      end
+
+      def self.find_all
+        webhook_repository.find_all
+      end
+
+      def self.execute_webhook_now webhook
+        webhook.execute
+      end
+
+      def self.find_by_consumer_and_provider consumer, provider
+        webhook_repository.find_by_consumer_and_provider consumer, provider
+      end
+
+      def self.execute_webhooks pact
+        webhooks = webhook_repository.find_by_consumer_and_provider pact.consumer, pact.provider
+        if webhooks.any?
+          run_later(webhooks)
+        else
+          logger.debug "No webhook found for consumer \"#{pact.consumer.name}\" and provider \"#{pact.provider.name}\""
+        end
+      end
+
+      # TODO background job?
+      def self.run_later webhooks
+        Thread.new do
+          webhooks.each do | webhook |
+            begin
+              webhook.execute
+            rescue StandardError => e
+              # Exceptions are already logged, no need to log again.
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/pact_broker/tasks.rb

@@ -1,2 +1 @@
 require 'pact_broker/tasks/migration_task'
-load File.dirname(__FILE__) + '/tasks/delete.rake'

data/lib/pact_broker/ui/controllers/base_controller.rb

@@ -8,6 +8,7 @@ module PactBroker
       class Base < Padrino::Application
 
         set :root, File.join(File.dirname(__FILE__), '..')
+        set :show_exceptions, true
 
       end
     end

data/lib/pact_broker/ui/controllers/groups.rb

@@ -0,0 +1,25 @@
+require 'pact_broker/ui/controllers/base_controller'
+require 'pact_broker/ui/view_models/relationships'
+require 'haml'
+
+module PactBroker
+  module UI
+    module Controllers
+      class Groups < Base
+
+        include PactBroker::Services
+
+        get ":name" do
+          erb :'groups/show.html', {
+            locals: {
+              csv_path: "/groups/#{params[:name]}/csv",
+              pacticipant_name: params[:name]}
+            }, {
+              layout: 'layouts/main'
+            }
+        end
+
+      end
+    end
+  end
+end

data/lib/pact_broker/ui/controllers/relationships.rb

@@ -9,8 +9,6 @@ module PactBroker
 
         include PactBroker::Services
 
-        set :root, File.join(File.dirname(__FILE__), '..')
-
         get "/" do
           view_model = ViewModels::Relationships.new(pacticipant_service.find_relationships)
 

data/lib/pact_broker/ui/helpers/url_helper.rb

@@ -0,0 +1,17 @@
+require 'erb'
+
+module PactBroker
+  module UI
+    module Helpers
+      module URLHelper
+
+        extend self
+
+        def group_url pacticipant_name
+          "/groups/#{ERB::Util.url_encode(pacticipant_name)}"
+        end
+
+      end
+    end
+  end
+end

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

@@ -1,4 +1,5 @@
 require 'pact_broker/api/pact_broker_urls'
+require 'pact_broker/ui/helpers/url_helper'
 
 module PactBroker
   module UI
@@ -12,11 +13,19 @@ module PactBroker
         end
 
         def consumer_name
-          @relationship.consumer.name
+          @relationship.consumer_name
         end
 
         def provider_name
-          @relationship.provider.name
+          @relationship.provider_name
+        end
+
+        def consumer_group_url
+          Helpers::URLHelper.group_url consumer_name
+        end
+
+        def provider_group_url
+          Helpers::URLHelper.group_url provider_name
         end
 
         def latest_pact_url
@@ -24,9 +33,9 @@ module PactBroker
         end
 
         def <=> other
-          comp = consumer_name <=> other.consumer_name
+          comp = consumer_name.downcase <=> other.consumer_name.downcase
           return comp unless comp == 0
-          provider_name <=> other.provider_name
+          provider_name.downcase <=> other.provider_name.downcase
         end
 
       end