pact_broker 1.1.0 → 1.2.0

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

@@ -8,57 +8,58 @@ module Webmachine
   end
 end
 
-module PactBroker::Api
+module PactBroker
+  module Api
+    module Resources
 
-  module Resources
+      class Pacticipant < BaseResource
 
-    class Pacticipant < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
 
-      def content_types_provided
-        [["application/hal+json", :to_json]]
-      end
+        def content_types_accepted
+          [["application/json", :from_json]]
+        end
 
-      def content_types_accepted
-        [["application/json", :from_json]]
-      end
+        def allowed_methods
+          ["GET", "PATCH", "DELETE"]
+        end
 
-      def allowed_methods
-        ["GET", "PATCH", "DELETE"]
-      end
+        def known_methods
+          super + ['PATCH']
+        end
 
-      def known_methods
-        super + ['PATCH']
-      end
+        def from_json
+          if @pacticipant
+            @pacticipant = pacticipant_service.update params.merge(name: pacticipant_name)
+          else
+            @pacticipant = pacticipant_service.create params.merge(name: pacticipant_name)
+            response.headers["Location"] = pacticipant_url(base_url, @pacticipant)
+          end
+          response.body = to_json
+        end
 
-      def from_json
-        if @pacticipant
-          @pacticipant = pacticipant_service.update params.merge(name: pacticipant_name)
-        else
-          @pacticipant = pacticipant_service.create params.merge(name: pacticipant_name)
-          response.headers["Location"] = pacticipant_url(base_url, @pacticipant)
+        def resource_exists?
+          @pacticipant = pacticipant_service.find_pacticipant_by_name(pacticipant_name)
+          @pacticipant != nil
         end
-        response.body = to_json
-      end
 
-      def resource_exists?
-        @pacticipant = pacticipant_service.find_pacticipant_by_name(pacticipant_name)
-        @pacticipant != nil
-      end
+        def delete_resource
+          pacticipant_service.delete pacticipant_name
+          true
+        end
 
-      def delete_resource
-        pacticipant_service.delete pacticipant_name
-        true
-      end
+        def to_json
+          PactBroker::Api::Decorators::PacticipantRepresenter.new(@pacticipant).to_json(base_url: base_url)
+        end
 
-      def to_json
-        PactBroker::Api::Decorators::PacticipantRepresenter.new(@pacticipant).to_json(base_url: base_url)
-      end
+        def pacticipant_name
+          identifier_from_path[:name]
+        end
 
-      def pacticipant_name
-        identifier_from_path[:name]
       end
-
     end
-  end
 
-end
+  end
+end

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

@@ -0,0 +1,19 @@
+module PactBroker
+  module Api
+    module Resources
+
+      module PacticipantResourceMethods
+
+        def potential_duplicate_pacticipants? pacticipant_names
+          messages = pacticipant_service.messages_for_potential_duplicate_pacticipants pacticipant_names, base_url
+          if messages.any?
+            response.body = messages.join("\n")
+            response.headers['Content-Type'] = 'text/plain'
+          end
+          messages.any?
+        end
+
+      end
+    end
+  end
+end

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

@@ -1,28 +1,63 @@
 require 'pact_broker/api/resources/base_resource'
+require 'pact_broker/api/decorators/pacticipant_decorator'
+require 'pact_broker/models/pacticipant'
 
-module PactBroker::Api
+module PactBroker
+  module Api
+    module Resources
 
-  module Resources
+      class Pacticipants < BaseResource
 
-    class Pacticipants < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
 
-      def content_types_provided
-        [["application/hal+json", :to_json]]
-      end
+        def content_types_accepted
+          [["application/json", :from_json]]
+        end
 
-      def allowed_methods
-        ["GET"]
-      end
+        def allowed_methods
+          ["GET", "POST"]
+        end
 
-      def to_json
-        generate_json(pacticipant_service.find_all_pacticipants)
-      end
+        def malformed_request?
+          if request.post?
+            return invalid_json? || validation_errors?(new_model)
+          end
+          false
+        end
 
-      def generate_json pacticipants
-        PactBroker::Api::Decorators::PacticipantCollectionRepresenter.new(pacticipants).to_json(base_url: base_url)
-      end
+        def post_is_create?
+          true
+        end
+
+        def from_json
+          created_model = pacticipant_service.create params
+          response.body = decorator_for(created_model).to_json(decorator_context)
+        end
+
+        def create_path
+          "/pacticpants/#{url_encode(params[:name])}"
+        end
+
+        def to_json
+          generate_json(pacticipant_service.find_all_pacticipants)
+        end
 
+        def generate_json pacticipants
+          PactBroker::Api::Decorators::PacticipantCollectionRepresenter.new(pacticipants).to_json(base_url: base_url)
+        end
+
+        def decorator_for model
+          PactBroker::Api::Decorators::PacticipantRepresenter.new(model)
+        end
+
+        def new_model
+          @new_model ||= decorator_for(PactBroker::Models::Pacticipant.new).from_json(request.body.to_s)
+        end
+
+      end
     end
-  end
 
-end
+  end
+end

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

@@ -1,29 +1,30 @@
 require 'pact_broker/api/resources/base_resource'
 require 'pact_broker/api/decorators/relationships_csv_decorator'
 
-module PactBroker::Api
+module PactBroker
+  module Api
+    module Resources
 
-  module Resources
+      class Relationships < BaseResource
 
-    class Relationships < BaseResource
+        def content_types_provided
+          [["text/csv", :to_csv]]
+        end
 
-      def content_types_provided
-        [["text/csv", :to_csv]]
-      end
+        def allowed_methods
+          ["GET"]
+        end
 
-      def allowed_methods
-        ["GET"]
-      end
+        def to_csv
+          generate_csv(pact_service.find_latest_pacts)
+        end
 
-      def to_csv
-        generate_csv(pact_service.find_latest_pacts)
-      end
+        def generate_csv pacts
+          PactBroker::Api::Decorators::RelationshipsCsvDecorator.new(pacts).to_csv
+        end
 
-      def generate_csv pacts
-        PactBroker::Api::Decorators::RelationshipsCsvDecorator.new(pacts).to_csv
       end
-
     end
-  end
 
-end
+  end
+end

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

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

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

@@ -1,44 +1,45 @@
 require 'pact_broker/services'
 require 'pact_broker/api/decorators/webhook_decorator'
 
-module PactBroker::Api
+module PactBroker
+  module Api
+    module Resources
 
-  module Resources
+      class Webhook < BaseResource
 
-    class Webhook < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
 
-      def content_types_provided
-        [["application/hal+json", :to_json]]
-      end
+        def allowed_methods
+          ["GET", "DELETE"]
+        end
 
-      def allowed_methods
-        ["GET", "DELETE"]
-      end
+        def resource_exists?
+          !webhook.nil?
+        end
 
-      def resource_exists?
-        !webhook.nil?
-      end
+        def to_json
+          Decorators::WebhookDecorator.new(webhook).to_json(base_url: base_url)
+        end
 
-      def to_json
-        Decorators::WebhookDecorator.new(webhook).to_json(base_url: base_url)
-      end
+        def delete_resource
+          webhook_service.delete_by_uuid uuid
+          true
+        end
 
-      def delete_resource
-        webhook_service.delete_by_uuid uuid
-        true
-      end
+        private
 
-      private
+        def webhook
+          @webhook ||= webhook_service.find_by_uuid uuid
+        end
 
-      def webhook
-        @webhook ||= webhook_service.find_by_uuid uuid
-      end
+        def uuid
+          identifier_from_path[:uuid]
+        end
 
-      def uuid
-        identifier_from_path[:uuid]
       end
-
     end
-  end
 
-end
+  end
+end

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

@@ -3,7 +3,6 @@ require 'pact_broker/api/decorators/webhook_execution_result_decorator'
 
 module PactBroker
   module Api
-
     module Resources
 
       class WebhookExecution < BaseResource

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

@@ -0,0 +1,24 @@
+module PactBroker
+  module Api
+
+    module Resources
+
+      module WebhookResourceMethods
+
+        def malformed_webhook_request? webhook
+          begin
+            if (errors = webhook.validate).any?
+              set_json_validation_error_messages errors
+              return true
+            end
+          rescue
+            set_json_error_message 'Invalid JSON'
+            return true
+          end
+          false
+        end
+      end
+
+    end
+  end
+end

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

@@ -1,29 +1,30 @@
 require 'pact_broker/services'
 require 'pact_broker/api/decorators/webhooks_decorator'
 
-module PactBroker::Api
+module PactBroker
+  module Api
+    module Resources
 
-  module Resources
+      class Webhooks < BaseResource
 
-    class Webhooks < BaseResource
+        def content_types_provided
+          [["application/hal+json", :to_json]]
+        end
 
-      def content_types_provided
-        [["application/hal+json", :to_json]]
-      end
+        def allowed_methods
+          ["GET"]
+        end
 
-      def allowed_methods
-        ["GET"]
-      end
+        def to_json
+          Decorators::WebhooksDecorator.new(webhooks).to_json(decorator_context(resource_title: "Webhooks"))
+        end
 
-      def to_json
-        Decorators::WebhooksDecorator.new(webhooks).to_json(decorator_context(resource_title: "Webhooks"))
-      end
+        def webhooks
+          webhook_service.find_all
+        end
 
-      def webhooks
-        webhook_service.find_all
       end
-
     end
-  end
 
-end
+  end
+end