txgh-server 3.1.0 → 4.0.0.beta

data/lib/txgh-server/webhooks/git/request_handler.rb

@@ -0,0 +1,76 @@
+require 'json'
+
+module TxghServer
+  module Webhooks
+    module Git
+      class RequestHandler
+        include ResponseHelpers
+
+        class << self
+          def handle_request(request, logger)
+            new(request, logger).handle_request
+          end
+        end
+
+        attr_reader :request, :logger
+
+        def initialize(request, logger)
+          @request = request
+          @logger = logger
+        end
+
+        def handle_request
+          raise NotImplementedError
+        end
+
+        private
+
+        def attributes
+          raise NotImplementedError
+        end
+
+        def handle_safely
+          if authentic_request?
+            yield
+          else
+            respond_with_error(401, 'Unauthorized')
+          end
+        rescue => e
+          respond_with_error(500, "Internal server error: #{e.message}", e)
+        end
+
+        def payload
+          @payload ||= begin
+            if request.params[:payload]
+              JSON.parse(request.params[:payload])
+            else
+              JSON.parse(request.body.read)
+            end
+          rescue JSON::ParserError
+            {}
+          end
+        end
+
+        def git_repo_name
+          raise NotImplementedError
+        end
+
+        def config
+          @config ||= Txgh::Config::KeyManager.config_from_repo(git_repo_name)
+        end
+
+        def repo
+          config.git_repo
+        end
+
+        def project
+          config.transifex_project
+        end
+
+        def authentic_request?
+          raise NotImplementedError
+        end
+      end
+    end
+  end
+end

data/lib/txgh-server/webhooks/git/status_updater.rb

@@ -0,0 +1,15 @@
+module TxghServer
+  module Webhooks
+    module Git
+      class StatusUpdater
+        attr_reader :project, :repo, :branch
+
+        def initialize(project, repo, branch)
+          @project = project
+          @repo = repo
+          @branch = branch
+        end
+      end
+    end
+  end
+end

data/lib/txgh-server/webhooks/github/blank_attributes.rb

@@ -1,19 +1,7 @@
 module TxghServer
   module Webhooks
     module Github
-      class BlankAttributes
-        class << self
-          def from_webhook_payload(payload)
-            new
-          end
-        end
-
-        def initialize(options = {})
-        end
-
-        def to_h
-          {}
-        end
+      class BlankAttributes < TxghServer::Webhooks::Git::BlankAttributes
       end
     end
   end

data/lib/txgh-server/webhooks/github/delete_attributes.rb

@@ -1,53 +1,16 @@
 module TxghServer
   module Webhooks
     module Github
-      class DeleteAttributes
-        ATTRIBUTES = [
-          :event, :repo_name, :ref, :ref_type
-        ]
-
+      class DeleteAttributes < TxghServer::Webhooks::Git::DeleteAttributes
         class << self
-          def from_webhook_payload(payload)
-            new(
-              ATTRIBUTES.each_with_object({}) do |attr, ret|
-                ret[attr] = public_send(attr, payload)
-              end
-            )
-          end
-
-          def event(payload)
-            'delete'
-          end
-
           def repo_name(payload)
             payload.fetch('repository').fetch('full_name')
           end
 
-          def ref(payload)
-            payload.fetch('ref')
-          end
-
           def ref_type(payload)
             payload.fetch('ref_type')
           end
         end
-
-        attr_reader *ATTRIBUTES
-
-        def initialize(options = {})
-          ATTRIBUTES.each do |attr|
-            instance_variable_set(
-              "@#{attr}", options.fetch(attr) { options.fetch(attr.to_s) }
-            )
-          end
-        end
-
-        def to_h
-          ATTRIBUTES.each_with_object({}) do |attr, ret|
-            ret[attr] = public_send(attr)
-          end
-        end
-
       end
     end
   end

data/lib/txgh-server/webhooks/github/delete_handler.rb

@@ -1,44 +1,7 @@
 module TxghServer
   module Webhooks
     module Github
-      class DeleteHandler
-        include ResponseHelpers
-
-        attr_reader :project, :repo, :logger, :attributes
-
-        def initialize(project, repo, logger, attributes)
-          @project = project
-          @repo = repo
-          @logger = logger
-          @attributes = attributes
-        end
-
-        def execute
-          perform_delete if should_handle_request?
-          respond_with(200, true)
-        end
-
-        private
-
-        def perform_delete
-          deleter.delete_resources
-        end
-
-        def deleter
-          @deleter ||= Txgh::ResourceDeleter.new(project, repo, branch)
-        end
-
-        def should_handle_request?
-          # ref_type can be either 'branch' or 'tag' - we only care about branches
-          attributes.ref_type == 'branch' &&
-            repo.should_process_ref?(branch) &&
-            project.auto_delete_resources?
-        end
-
-        def branch
-          Txgh::Utils.absolute_branch(attributes.ref.sub(/^refs\//, ''))
-        end
-
+      class DeleteHandler < TxghServer::Webhooks::Git::DeleteHandler
       end
     end
   end

data/lib/txgh-server/webhooks/github/ping_handler.rb

@@ -3,18 +3,7 @@ module TxghServer
     module Github
       # Handles github's ping event, which is a test event fired whenever a new
       # webhook is set up.
-      class PingHandler
-        include ResponseHelpers
-
-        attr_reader :logger
-
-        def initialize(logger)
-          @logger = logger
-        end
-
-        def execute
-          respond_with(200, {})
-        end
+      class PingHandler < TxghServer::Webhooks::Git::PingHandler
       end
     end
   end

data/lib/txgh-server/webhooks/github/push_attributes.rb

@@ -1,41 +1,12 @@
 module TxghServer
   module Webhooks
     module Github
-      class PushAttributes
-        ATTRIBUTES = [
-          :event, :repo_name, :ref, :before, :after,
-          :added_files, :modified_files, :author
-        ]
-
+      class PushAttributes < TxghServer::Webhooks::Git::PushAttributes
         class << self
-          def from_webhook_payload(payload)
-            new(
-              ATTRIBUTES.each_with_object({}) do |attr, ret|
-                ret[attr] = public_send(attr, payload)
-              end
-            )
-          end
-
-          def event(payload)
-            'push'
-          end
-
           def repo_name(payload)
             payload.fetch('repository').fetch('full_name')
           end
 
-          def ref(payload)
-            payload.fetch('ref')
-          end
-
-          def before(payload)
-            payload.fetch('before')
-          end
-
-          def after(payload)
-            payload.fetch('after')
-          end
-
           def added_files(payload)
             extract_files(payload, 'added')
           end
@@ -57,27 +28,6 @@ module TxghServer
             payload.fetch('commits').flat_map { |c| c[state] }.uniq
           end
         end
-
-        def files
-          @files ||= added_files + modified_files
-        end
-
-        attr_reader *ATTRIBUTES
-
-        def initialize(options = {})
-          ATTRIBUTES.each do |attr|
-            instance_variable_set(
-              "@#{attr}", options.fetch(attr) { options.fetch(attr.to_s) }
-            )
-          end
-        end
-
-        def to_h
-          ATTRIBUTES.each_with_object({}) do |attr, ret|
-            ret[attr] = public_send(attr)
-          end
-        end
-
       end
     end
   end

data/lib/txgh-server/webhooks/github/push_handler.rb

@@ -1,98 +1,12 @@
-require 'set'
-require 'octokit'
-require 'txgh'
-
 module TxghServer
   module Webhooks
     module Github
-      class PushHandler
-        include ResponseHelpers
-
-        attr_reader :project, :repo, :logger, :attributes
-
-        def initialize(project, repo, logger, attributes)
-          @project = project
-          @repo = repo
-          @logger = logger
-          @attributes = attributes
-        end
-
-        def execute
-          # Check if the branch in the hook data is the configured branch we want
-          logger.info("request github branch: #{branch}")
-          logger.info("config github branch: #{repo.github_config_branch}")
-
-          if should_process?
-            logger.info('found branch in github request')
-
-            pusher.push_resources(added_and_modified_resources) do |tx_resource|
-              # block should return categories for the passed-in resource
-              { 'author' => attributes.author }
-            end
-
-            status_updater.update_status
-          end
-
-          respond_with(200, true)
-        rescue => e
-          status_updater.report_error_and_update_status(e)
-          respond_with_error(500, "Internal server error: #{e.message}", e)
-        end
-
+      class PushHandler < TxghServer::Webhooks::Git::PushHandler
         private
 
-        def pusher
-          @pusher ||= Txgh::Pusher.new(project, repo, branch)
-        end
-
         def status_updater
-          @status_updater = StatusUpdater.new(project, repo, branch)
+          @status_updater = TxghServer::Webhooks::Github::StatusUpdater.new(project, repo, branch)
         end
-
-        # finds the resources that were updated in each commit
-        def added_and_modified_resources
-          @amr ||= attributes.files.each_with_object(Set.new) do |file, ret|
-            logger.info("processing added/modified file: #{file}")
-
-            if tx_resources.include?(file)
-              ret << tx_resources[file]
-            end
-          end
-        end
-
-        # Build an index of known Tx resources, by source file
-        def tx_resources
-          @tx_resources ||= tx_config.resources.each_with_object({}) do |resource, ret|
-            ret[resource.source_file] = if repo.process_all_branches?
-              Txgh::TxBranchResource.new(resource, branch)  # maybe find instead?
-            else
-              resource
-            end
-          end
-        end
-
-        def tx_config
-          @tx_config ||= Txgh::Config::TxManager.tx_config(project, repo, branch)
-        end
-
-        def branch
-          @ref ||= attributes.ref.sub(/^refs\//, '')
-        end
-
-        def should_process?
-          should_process_branch? && should_process_commit?
-        end
-
-        def should_process_branch?
-          repo.should_process_ref?(branch)
-        end
-
-        def should_process_commit?
-          # return false if 'after' commit sha is all zeroes (indicates branch
-          # has been deleted)
-          !((attributes.after || '') =~ /\A0+\z/)
-        end
-
       end
     end
   end

data/lib/txgh-server/webhooks/github/request_handler.rb

@@ -3,22 +3,7 @@ require 'json'
 module TxghServer
   module Webhooks
     module Github
-      class RequestHandler
-        include ResponseHelpers
-
-        class << self
-          def handle_request(request, logger)
-            new(request, logger).handle_request
-          end
-        end
-
-        attr_reader :request, :logger
-
-        def initialize(request, logger)
-          @request = request
-          @logger = logger
-        end
-
+      class RequestHandler < TxghServer::Webhooks::Git::RequestHandler
         def handle_request
           handle_safely do
             case github_event
@@ -47,44 +32,14 @@ module TxghServer
           end
         end
 
-        def handle_safely
-          if authentic_request?
-            yield
-          else
-            respond_with_error(401, 'Unauthorized')
-          end
-        rescue => e
-          respond_with_error(500, "Internal server error: #{e.message}", e)
-        end
-
-        def payload
-          @payload ||= begin
-            if request.params[:payload]
-              JSON.parse(request.params[:payload])
-            else
-              JSON.parse(request.body.read)
-            end
-          rescue JSON::ParserError
-            {}
-          end
-        end
-
         def github_event
           request.env['HTTP_X_GITHUB_EVENT']
         end
 
-        def github_repo_name
+        def git_repo_name
           payload.fetch('repository', {})['full_name']
         end
 
-        def config
-          @config ||= Txgh::Config::KeyManager.config_from_repo(github_repo_name)
-        end
-
-        def repo
-          config.github_repo
-        end
-
         def project
           config.transifex_project
         end