ticket-replicator 1.0.0 → 1.2.0

data/lib/ticket/replicator/row_loader.rb

@@ -9,6 +9,13 @@ module Ticket
         new(jira_project, row).run
       end
 
+      def self.build_ticket_link_attributes(
+        url, title, application_name = Ticket::SOURCE_TICKET_REMOTE_LINK_APPLICATION
+      )
+        { "object" => { "url" => url, "title" => title } }
+          .merge(application_name ? { "application" => { "name" => application_name } } : {})
+      end
+
       private_class_method :new
 
       attr_reader :jira_project, :row
@@ -20,6 +27,7 @@ module Ticket
 
       def run
         save_ticket
+        update_source_ticket_remote_link
         transition_ticket_to_the_expected_status
       end
 
@@ -27,23 +35,35 @@ module Ticket
         jira_project.replicated_tickets.key?(id)
       end
 
-      # rubocop:disable Metrics/MethodLength
       def save_ticket
         return unless ticket_fields_need_to_be_updated?
 
-        ticket.jira_ticket.save!({
-                                   fields: {
-                                     project: { key: jira_project.project_key },
-                                     issuetype: { name: jira_project.ticket_type_name },
-                                     resolution: jira_resolution_value,
-                                     priority: { name: priority },
-                                     summary: summary
-                                   }
-                                 })
+        ticket.jira_ticket.save!(attributes_for_save)
 
         ticket.jira_ticket.fetch
       end
 
+      private
+
+      def attributes_for_save
+        {
+          fields: field_values
+        }
+      end
+
+      def field_values
+        {
+          project: { key: jira_project.project_key },
+          issuetype: { name: jira_project.ticket_type_name },
+          priority: { name: priority },
+          summary: summary
+        }.merge(exclude_resolution? ? {} : { resolution: jira_resolution_value })
+      end
+
+      def exclude_resolution?
+        ENV["TICKET_REPLICATOR_DISABLE_RESOLUTION_LOADING"] == "true"
+      end
+
       def jira_resolution_value
         return if resolution.blank?
         return { name: resolution } if jira_project.resolutions.key?(resolution)
@@ -56,8 +76,6 @@ module Ticket
         nil
       end
 
-      # rubocop:enable Metrics/MethodLength
-
       def ticket_fields_need_to_be_updated?
         !ticket_previously_replicated? || ticket_fields_changed?
       end
@@ -67,7 +85,47 @@ module Ticket
       end
 
       def fields_to_save
-        %i[summary priority]
+        %i[summary priority] + (exclude_resolution? ? [] : %i[resolution])
+      end
+
+      def update_source_ticket_remote_link
+        return unless source_ticket_link_needs_update?
+
+        ticket.source_ticket_link&.delete
+
+        ticket.jira_ticket.remotelink.build.save!(ticket_link_attributes)
+      end
+
+      # rubocop:disable Metrics/MethodLength
+      def source_ticket_link_needs_update?
+        existing_ticket_link_attributes = ticket.source_ticket_link&.attrs
+
+        object = existing_ticket_link_attributes&.[]("object")
+        existing_url = object&.[]("url")
+        existing_title = object&.[]("title")
+
+        filtered_existing_ticket_link_attributes = ticket_link_attributes(existing_url, existing_title)
+
+        log.debug do
+          <<~EOLOG
+
+            existing_ticket_link_attributes          = #{existing_ticket_link_attributes.inspect}
+            filtered_existing_ticket_link_attributes = #{filtered_existing_ticket_link_attributes}
+            ticket_link_attributes                   = #{ticket_link_attributes.inspect}
+          EOLOG
+        end
+
+        filtered_existing_ticket_link_attributes != ticket_link_attributes
+      end
+
+      # rubocop:enable Metrics/MethodLength
+
+      def source_ticket_link_title
+        "Source Ticket #{id}"
+      end
+
+      def ticket_link_attributes(url = source_ticket_url, title = source_ticket_link_title)
+        self.class.build_ticket_link_attributes(url, title)
       end
 
       def transition_ticket_to_the_expected_status
@@ -91,8 +149,6 @@ module Ticket
         Replicator::Ticket.new(jira_project.jira_auto_tool, jira_project.jira_client.Issue.build)
       end
 
-      private
-
       def method_missing(name, *args)
         if field_to_load?(name)
           row.fetch(name) { |key| raise "No value found for #{key.inspect} in #{row.inspect}" }
@@ -115,7 +171,8 @@ module Ticket
 
       class << self
         def fields_to_load
-          @fields_to_load ||= RowTransformer.fields_to_transform.collect { |field| field.to_s.downcase.to_sym }
+          @fields_to_load ||=
+            RowTransformer.fields_to_transform.collect { |field| field.to_s.parameterize.underscore.to_sym }
         end
       end
     end

data/lib/ticket/replicator/row_transformer.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 require "ticket/replicator/replicated_summary"
+require "ticket/replicator/file_transformer"
+require "erb"
 
 module Ticket
   class Replicator
@@ -18,11 +20,44 @@ module Ticket
       end
 
       def run
-        fields_to_transform.collect { |field| send("transformed_#{field.to_s.downcase}") }
+        fields_to_transform.collect { |field| send("transformed_#{field.to_s.parameterize.underscore}") }
       end
 
       def transformed_id
-        remapped_field_extracted_value_for :id
+        remapped_id = remapped_field_extracted_value_for :id
+
+        return remapped_id unless id_extraction_regex
+
+        match_data = id_extraction_regex.match(remapped_id)
+
+        return match_data[:id] if match_data
+
+        raise FileTransformer::TransformError,
+              "No match found for :id in #{remapped_id.inspect} using #{id_extraction_regex.inspect}"
+      end
+
+      private
+
+      def id_extraction_regex
+        @id_extraction_regex ||=
+          begin
+            regex_string = mappings["id_extraction_regex"]
+            Regexp.new(regex_string) if regex_string
+          end
+      end
+
+      public
+
+      def transformed_source_ticket_url
+        source_ticket_url_builder(transformed_id)
+      end
+
+      def source_ticket_url_builder(source_ticket_id)
+        ERB.new(ticket_replicator_source_ticket_url).result(binding)
+      end
+
+      def ticket_replicator_source_ticket_url
+        ENV.fetch("TICKET_REPLICATOR_SOURCE_TICKET_URL") { |name| raise "#{name}: not set in environment!" }
       end
 
       def transformed_status
@@ -42,7 +77,7 @@ module Ticket
       end
 
       def transformed_summary
-        ReplicatedSummary.build(remapped_field_extracted_value_for(:id),
+        ReplicatedSummary.build(transformed_id,
                                 remapped_field_extracted_value_for(:summary)).to_s
       end
 
@@ -50,7 +85,15 @@ module Ticket
         self.class.fields_to_transform
       end
 
+      def fields_to_remap
+        self.class.fields_to_remap
+      end
+
       def self.fields_to_transform
+        fields_to_remap + ["Source Ticket URL"]
+      end
+
+      def self.fields_to_remap
         %i[ID Status Resolution Priority Summary]
       end
 
@@ -77,7 +120,7 @@ module Ticket
       end
 
       def remapped_field_extracted_row
-        @remapped_field_extracted_row ||= fields_to_transform.to_h do |field|
+        @remapped_field_extracted_row ||= fields_to_remap.to_h do |field|
           mapped_field_key = remapped_field_key(field)
           [mapped_field_key, extracted_row_value_for(mapped_field_key)]
         end

data/lib/ticket/replicator/ticket.rb

@@ -36,8 +36,25 @@ module Ticket
         ReplicatedSummary.parse(summary).source_id
       end
 
+      SOURCE_TICKET_REMOTE_LINK_APPLICATION = "Ticket Source"
+
+      def source_ticket_link
+        @source_ticket_link ||=
+          jira_ticket.remotelink.all.find do |link|
+            source_ticket_link_attributes(link)&.[]("application")&.[]("name") == SOURCE_TICKET_REMOTE_LINK_APPLICATION
+          end
+      end
+
+      def source_ticket_url
+        source_ticket_link_attributes(source_ticket_link)&.[]("object")&.[]("url")
+      end
+
       private
 
+      def source_ticket_link_attributes(link)
+        link&.attrs
+      end
+
       def cmp_values(object)
         object&.source_id
       end

data/lib/ticket/replicator/version.rb

@@ -2,6 +2,6 @@
 
 module Ticket
   class Replicator
-    VERSION = "1.0.0"
+    VERSION = "1.2.0"
   end
 end

data/spec/ticket/replicator/file_transformer_spec.rb

@@ -43,7 +43,8 @@ module Ticket
 
         describe "#transformed_headers" do
           it "returns the headers" do
-            expect(transformer.transformed_headers).to eq(%w[ID Status Resolution Priority Summary])
+            expect(transformer.transformed_headers)
+              .to eq(%w[ID Status Resolution Priority Summary] + ["Source Ticket URL"])
           end
         end
 
@@ -69,11 +70,9 @@ module Ticket
             end
 
             it "raises an error message including the file name and line number of the row that was processed" do
-              expect { transformer.transformed_rows }.to raise_error(TransformError, <<~EOEXPECTEDERRORMSG)
-                source.csv:3: error while transforming row:
-                transformation error:
-                :another_row
-              EOEXPECTEDERRORMSG
+              expect { transformer.transformed_rows }
+                .to raise_error(TransformError,
+                                /source\.csv:3: error while transforming row:\ntransformation error:\n:another_row\n/)
             end
           end
         end