ticket-replicator 0.1.1 → 1.1.0

data/features/transform_and_load_extracted_ticket_queue.feature

@@ -8,6 +8,10 @@ Feature: Transform and load extracted ticket queue
       | name                                    |
       | TICKET_REPLICATOR_JIRA_PROJECT_KEY      |
       | TICKET_REPLICATOR_JIRA_TICKET_TYPE_NAME |
+    And the following environment variables have been set:
+      | name                                | value                                        |
+      | TICKET_REPLICATOR_SOURCE_TICKET_URL | http://url/to/source/ticket/<%= source_ticket_id %> |
+
     And a file named "config/ticket-replicator.mappings.yml" with:
       """
       field_mapping:
@@ -28,38 +32,11 @@ Feature: Transform and load extracted ticket queue
         defaults_to: keep_original_value
 
       resolution_mapping:
-        defaults_to: keep_original_value
-        "New":
-        "Open":
-        "In Process":
-        "In Review":
-        "On Hold":
-        Deferred:
-        Defect Correction in Process:
-        "Fixed": "Fixed"
+        defaults_to: blank_value
+        "Fixed": "Done"
         "Closed": "Done"
-        "Rejected": "Won't Do"
-        "Resolved": "Fixed"
-        "Confirmed":
-        "Forwarded":
-        "Information Required":
-        Wait for Defect Correction:
-        Solution Proposal:
-        Tester Action:
-        Wait on External:
-
-      team_mapping:
-        defaults_to: keep_original_value
-        "Frontend": "Web Team"
-        "Backend": "Server Team"
-        "Integration": "Integration Team"
-        "Mobile": "Mobile Team"
-        "Security": "Security Team"
-        "DevOps": "DevOps Team"
-        "QA": "Quality Assurance"
-        "Architecture": "Architecture Team"
-        "UX": "Design Team"
-        "Performance": "Performance Team"
+        "Withdrawn": "Won't Do"
+        "Resolved": "Done"
       """
     And the project has no tickets
 
@@ -74,7 +51,7 @@ Feature: Transform and load extracted ticket queue
       | 9400013816 | Summary    | 2: High         | Forwarded                    | A Team                  |
       | 9400011382 | Summary    | 3: Medium       | In Process                   | A Team                  |
       | 9400011393 | Summary    | 3: Medium       | Information Required         | A Team                  |
-      | 9400011381 | Summary    | 3: Medium       | New                          | A Team                  |
+      | 9400011381 | Summary    | 3: Medium       | Withdrawn                    | A Team                  |
       | 3000016617 | Summary    | 3: Medium       | No Error                     | A Team                  |
       | 9400011372 | Summary    | 1: Critical     | Open                         | A Team                  |
       | 9400011466 | Summary    | 3: Medium       | Withdrawn                    | A Team                  |
@@ -84,20 +61,20 @@ Feature: Transform and load extracted ticket queue
     And the current date time is "2025-05-10 07:52:00 UTC"
     When I successfully run `ticket-replicator --ticket-queue-transform-and-load`
     Then the Jira project should only have the following tickets:
-      | status                       | resolution | priority | team         | summary                    | source ticket url                     |
-      | Closed                       | Done       | Medium   | A Team       | SMAN-3000017049 \| Summary | https://example.com/defect/3000017049 |
-      | Confirmed                    |            | Low      | A Team       | SMAN-9400011377 \| Summary | https://example.com/defect/9400011377 |
-      | Defect Correction in Process |            | Medium   | A Team       | SMAN-3000016618 \| Summary | https://example.com/defect/3000016618 |
-      | Deferred                     |            | Medium   | A Team       | SMAN-9400013805 \| Summary | https://example.com/defect/9400013805 |
-      | Forwarded                    |            | High     | A Team       | SMAN-9400013816 \| Summary | https://example.com/defect/9400013816 |
-      | In Process                   |            | Medium   | A Team       | SMAN-9400011382 \| Summary | https://example.com/defect/9400011382 |
-      | Information Required         |            | Medium   | A Team       | SMAN-9400011393 \| Summary | https://example.com/defect/9400011393 |
-      | New                          |            | Medium   | A Team       | SMAN-9400011381 \| Summary | https://example.com/defect/9400011381 |
-      | No Error                     |            | Medium   | A Team       | SMAN-3000016617 \| Summary | https://example.com/defect/3000016617 |
-      | Open                         |            | Highest  | A Team       | SMAN-9400011372 \| Summary | https://example.com/defect/9400011372 |
-      | Withdrawn                    | Won't Do   | Medium   | A Team       | SMAN-9400011466 \| Summary | https://example.com/defect/9400011466 |
-      | Closed                       | Done       | Medium   | Another Team | SMAN-3000017667 \| Summary | https://example.com/defect/3000017667 |
-      | No Error                     |            | Medium   | A Team       | SMAN-3000018423 \| Summary | https://example.com/defect/3000018423 |
+      | status                       | resolution | priority | summary                    | source_ticket_url                      |
+      | Closed                       | Done       | Medium   | SMAN-3000017049 \| Summary | http://url/to/source/ticket/3000017049 |
+      | Confirmed                    |            | Low      | SMAN-9400011377 \| Summary | http://url/to/source/ticket/9400011377 |
+      | Defect Correction in Process |            | Medium   | SMAN-3000016618 \| Summary | http://url/to/source/ticket/3000016618 |
+      | Deferred                     |            | Medium   | SMAN-9400013805 \| Summary | http://url/to/source/ticket/9400013805 |
+      | Forwarded                    |            | High     | SMAN-9400013816 \| Summary | http://url/to/source/ticket/9400013816 |
+      | In Process                   |            | Medium   | SMAN-9400011382 \| Summary | http://url/to/source/ticket/9400011382 |
+      | Information Required         |            | Medium   | SMAN-9400011393 \| Summary | http://url/to/source/ticket/9400011393 |
+      | Withdrawn                    | Won't Do   | Medium   | SMAN-9400011381 \| Summary | http://url/to/source/ticket/9400011381 |
+      | No Error                     |            | Medium   | SMAN-3000016617 \| Summary | http://url/to/source/ticket/3000016617 |
+      | Open                         |            | Highest  | SMAN-9400011372 \| Summary | http://url/to/source/ticket/9400011372 |
+      | Withdrawn                    | Won't Do   | Medium   | SMAN-9400011466 \| Summary | http://url/to/source/ticket/9400011466 |
+      | Closed                       | Done       | Medium   | SMAN-3000017667 \| Summary | http://url/to/source/ticket/3000017667 |
+      | No Error                     |            | Medium   | SMAN-3000018423 \| Summary | http://url/to/source/ticket/3000018423 |
     # TODO: And the source ticket URL is found in the ticket descriptions of those tickets
     And a file named "queue/30.archived/2025-05-10.07h52m00.sap_solution_manager_defects.xlsx" should exist
     And the file named "queue/10.extracted/sap_solution_manager_defects.xlsx" should not exist anymore

data/lib/ticket/replicator/file_loader.rb

@@ -8,7 +8,7 @@ module Ticket
       class LoadError < StandardError; end
 
       def self.run_on(jira_project, file_path)
-        log.info { "Loading #{file_path} into #{jira_project.project_key}..." }
+        log.debug { "Loading #{file_path} into #{jira_project.project_key}..." }
 
         new(jira_project, file_path).run
       end

data/lib/ticket/replicator/jira_project.rb

@@ -30,6 +30,10 @@ module Ticket
         @ticket_type_name ||= ENV.fetch("TICKET_REPLICATOR_JIRA_TICKET_TYPE_NAME")
       end
 
+      def resolutions
+        @resolutions ||= jira_client.Resolution.all.to_h { |resolution| [resolution.name, resolution] }
+      end
+
       def replicated_tickets
         @replicated_tickets ||= all_replicated_ticket_pages.to_h { |ticket| [ticket.source_id, ticket] }
       end

data/lib/ticket/replicator/row_loader.rb

@@ -20,6 +20,7 @@ module Ticket
 
       def run
         save_ticket
+        update_source_ticket_remote_link
         transition_ticket_to_the_expected_status
       end
 
@@ -27,6 +28,7 @@ module Ticket
         jira_project.replicated_tickets.key?(id)
       end
 
+      # rubocop:disable Metrics/MethodLength
       def save_ticket
         return unless ticket_fields_need_to_be_updated?
 
@@ -34,9 +36,8 @@ module Ticket
                                    fields: {
                                      project: { key: jira_project.project_key },
                                      issuetype: { name: jira_project.ticket_type_name },
-                                     #                              resolution: { name: resolution },
+                                     resolution: jira_resolution_value,
                                      priority: { name: priority },
-                                     # implementation_team: { name: team },
                                      summary: summary
                                    }
                                  })
@@ -44,6 +45,20 @@ module Ticket
         ticket.jira_ticket.fetch
       end
 
+      def jira_resolution_value
+        return if resolution.blank?
+        return { name: resolution } if jira_project.resolutions.key?(resolution)
+
+        log.warn do
+          "Setting resolution to unset since " \
+            "resolution #{resolution.inspect} not found in #{jira_project.resolutions.inspect}!"
+        end
+
+        nil
+      end
+
+      # rubocop:enable Metrics/MethodLength
+
       def ticket_fields_need_to_be_updated?
         !ticket_previously_replicated? || ticket_fields_changed?
       end
@@ -56,6 +71,53 @@ module Ticket
         %i[summary priority]
       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 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
+
       def transition_ticket_to_the_expected_status
         ticket.transition_to(status)
       end
@@ -101,7 +163,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,7 @@
 # frozen_string_literal: true
 
 require "ticket/replicator/replicated_summary"
+require "erb"
 
 module Ticket
   class Replicator
@@ -18,13 +19,25 @@ 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
       end
 
+      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
         mapped_value_for :status
       end
@@ -50,17 +63,27 @@ module Ticket
         self.class.fields_to_transform
       end
 
+      def fields_to_remap
+        self.class.fields_to_remap
+      end
+
       def self.fields_to_transform
-        %i[ID Status Resolution Priority Team Summary]
+        fields_to_remap + ["Source Ticket URL"]
+      end
+
+      def self.fields_to_remap
+        %i[ID Status Resolution Priority Summary]
       end
 
       def mapped_value_for(field, extracted_value = remapped_field_extracted_value_for(field))
         mapping_for(field).fetch(extracted_value) do |key|
-          unless mapping_for(field)["defaults_to"] == "keep_original_value"
+          case mapping_for(field)["defaults_to"]
+          when "keep_original_value" then extracted_value
+          when "unset_value" then nil
+          when "blank_value" then ""
+          else
             raise "No mapping found for #{field.inspect} = #{key.inspect} in #{mapping_for(field).inspect}"
           end
-
-          extracted_value
         end
       end
 
@@ -75,7 +98,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
@@ -57,6 +74,8 @@ module Ticket
           value.name
         elsif value.respond_to?(:value)
           value.value
+        elsif value.is_a?(Hash)
+          value.values_at("name", "value").compact.first
         else
           value.nil? ? "" : value.to_s
         end

data/lib/ticket/replicator/version.rb

@@ -2,6 +2,6 @@
 
 module Ticket
   class Replicator
-    VERSION = "0.1.1"
+    VERSION = "1.1.0"
   end
 end

data/spec/ticket/replicator/file_loader_spec.rb

@@ -8,7 +8,7 @@ module Ticket
       RSpec.describe FileLoader do
         let(:loader) { described_class.send(:new, jira_project, "file.csv") }
         let(:jira_project) { instance_double(JiraProject) }
-        let(:logger) { instance_double(Logger, info: nil) }
+        let(:logger) { instance_double(Logger, debug: nil) }
 
         describe ".run_on" do
           it "loads the given file" do

data/spec/ticket/replicator/file_replicator_spec.rb

@@ -2,7 +2,6 @@
 
 module Ticket
   class Replicator
-    # rubocop:disable Metrics/ClassLength
     class FileReplicator
       RSpec.describe FileReplicator do
         let(:file_replicator) { described_class.send(:new, replicator, "extracted.csv") }
@@ -148,6 +147,5 @@ module Ticket
         end
       end
     end
-    # rubocop:enable Metrics/ClassLength
   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 Team Summary])
+            expect(transformer.transformed_headers)
+              .to eq(%w[ID Status Resolution Priority Summary] + ["Source Ticket URL"])
           end
         end
 

data/spec/ticket/replicator/jira_project_spec.rb

@@ -28,6 +28,24 @@ module Ticket
         end
       end
 
+      describe "#resolutions" do
+        def build_resolution(name, id)
+          double(JIRA::Resource::Resolution, name: name, id: id)
+        end
+
+        let(:a_resolution) { build_resolution("a resolution", 1) }
+        let(:another_resolution) { build_resolution("another resolution", 2) }
+
+        let(:resolution_query) { double("jira_client.Resolution", all: [a_resolution, another_resolution]) }
+
+        before { allow(jira_client).to receive_messages(Resolution: resolution_query) }
+
+        it do
+          expect(project.resolutions)
+            .to eq({ "a resolution" => a_resolution, "another resolution" => another_resolution })
+        end
+      end
+
       describe "#ticket_type_name" do
         it do
           expect(ENV).to receive(:fetch).with("TICKET_REPLICATOR_JIRA_TICKET_TYPE_NAME").and_return("Ticket Type Name")
@@ -83,6 +101,7 @@ module Ticket
             build_expected_ticket("5", "SMAN-5 | a summary")
           ]
         end
+
         let(:actual_tickets) do
           [
             build_non_replicated_ticket("a non replicated ticket"),