tefoji 1.0.8 → 1.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 888097f6ae0c3877e00fa2a2fc0318fb094f5e340621b445f510ffab51b01f85
-  data.tar.gz: 65af396135aa1f22b1385a3c2ed7738db817733429d2243651b3ced884a2a9fe
+  metadata.gz: f8859212ce735cdc0292cbbf58899931446e7a0b90e330899c15f277d071e723
+  data.tar.gz: 41baa8c0faa84d4c058adf946eb0a9944a8a997a464a9c83c172b7eb191e19d7
 SHA512:
-  metadata.gz: 9d5ce9983a3401f0fe6644fa35725a9c892733d7ad1bdb7ba6051e058fb6aa0a911a2a5cd30aba783c2260c19067fc1112c49c1bf4bf64501b7188096b5ae64e
-  data.tar.gz: 171807f4bf998fd42dd6296de339934c6a22dcd73722749886abf474a795a2e6dfdb2b6faea95a6a8bd5cf7956bd5f9c6d81d786b1c30c554756c7faba7d18db
+  metadata.gz: 8e4ca95fd24a97cbe1d24c25885abf7cad21a1ec135c2c346bd5e349b583403419c10c98d07ada2ffd5a44bcbc64aedfbce7ff97c754e9ce77c93b2a96133443
+  data.tar.gz: bd8c63324127e9d2a291af0d489fcb739d99d587c95360aacb1b5164003ec267c832bd30ad98dd57da399c071f17929dc6aa0605375f3b0554bfe9f6e165d44b

data/lib/mixins/logging.rb

@@ -3,7 +3,7 @@ module Logging
   # Another cheap hack to wrap 'logger' above. Called as 'fatal' rather than 'logger.fatal'
   # directly, does the exit for us so we can be lazy.
   def fatal(message)
-    logger.fatal(message)
+    @logger.fatal(message)
     exit 1
   end
 end

data/lib/mixins/user_functions.rb

@@ -34,7 +34,6 @@ module UserFunctions
   # Maps of Winston-style Jira names (projects, sprints, to the actual assets
   def jira_projects
     {
-      BOLT: 'BOLT',
       BUILD_TOOLS: 'BUILD',
       CLIENT_TOOLS: 'CT',
       CODE_MANAGEMENT: 'CODEMGMT',
@@ -48,16 +47,15 @@ module UserFunctions
       MODULES: 'MODULES',
       MODULES_INTERNAL: 'FM',
       OPERATIONS: 'OPS',
-      PDK: 'PDK',
+      PDK: 'CONT',
       PE_INTERNAL: 'PE',
+      POOLER: 'POOLER',
       PROJECT_CENTRAL: 'PC',
       PUPPETDB: 'PDB',
       PUPPETSERVER: 'SERVER',
       PUPPET_AGENT: 'PA',
       PUPPET: 'PUP',
       QUALITY_ENGINEERING: 'QENG',
-      DIO: 'DIO',
-      RAZOR: 'RAZOR',
       RELEASE_ENGINEERING: 'RE',
       SLV: 'SLV',
       SUPPORT: 'SUP',
@@ -89,12 +87,12 @@ module UserFunctions
       CD4PE: 'CD4PE',
       CODE_MANAGEMENT: 'Dumpling',
       DUMPLING: 'Dumpling',
-      FACTER: "Phoenix",
+      FACTER: 'Phoenix',
       INSTALLER: 'Installer and Management',
       NETWORKING: 'Network Automation',
       OPERATIONS: 'Operations',
       PE: 'Dumpling',
-      PLATFORM_OS: "Phoenix",
+      PLATFORM_OS: 'Phoenix',
       PUPPETDB: 'Dumpling',
       PUPPETSERVER: 'Dumpling',
       QE: 'Quality Engineering',

data/lib/tefoji/cli.rb

@@ -8,6 +8,9 @@ module Tefoji
     DEFAULT_JIRA_URL = 'https://tickets.puppetlabs.com'
     JIRA_TEST_URL = 'https://jira-app-dev-1.ops.puppetlabs.net'
     DEFAULT_JIRA_AUTH_FILE = "#{ENV['HOME']}/.tefoji-auth.yaml"
+    JIRA_CLOUD_URLS = [
+      'https://puppet.atlassian.net'
+    ]
 
     DOCUMENTATION = <<~DOCOPT
       Generate Jira issues from YAML files.
@@ -43,23 +46,28 @@ module Tefoji
       @user_options = parse_options(argv)
 
       if @user_options['--version']
-        warn "tefoji version #{Gem.loaded_specs['tefoji'].version.to_s}"
+        warn "tefoji version #{Gem.loaded_specs['tefoji'].version}"
         exit 0
       end
 
+      @user_options['--jira'] = JIRA_TEST_URL if @user_options['--jira-test']
+      @user_options['--jira'] = DEFAULT_JIRA_URL if @user_options['--jira'].nil?
+
       @user_options['jira-auth-string'] = nil
       @user_options['jira-auth-file'] = @user_options['--jira-auth'] || DEFAULT_JIRA_AUTH_FILE
 
+      @user_options['jira-cloud'] = false
+      @user_options['jira-cloud'] = true if JIRA_CLOUD_URLS.include?(@user_options['--jira'])
+
       jira_auth_file = @user_options['jira-auth-file']
-      if File.file?(jira_auth_file)
-        authentication = YAML.load_file(jira_auth_file)
-        if authentication.key?('jira-auth')
-          @user_options['jira-auth-string'] = authentication['jira-auth']
-        end
-      end
+      return unless File.file?(jira_auth_file)
 
-      @user_options['--jira'] = JIRA_TEST_URL if @user_options['--jira-test']
-      @user_options['--jira'] = DEFAULT_JIRA_URL if @user_options['--jira'].nil?
+      authentication = YAML.load_file(jira_auth_file)
+      if @user_options['jira-cloud'] && authentication.key?('jira-cloud-auth')
+        @user_options['jira-auth-string'] = authentication['jira-cloud-auth']
+      elsif authentication.key?('jira-auth')
+        @user_options['jira-auth-string'] = authentication['jira-auth']
+      end
     end
 
     # Iterate through issue templates, validating each. If that goes well, generate

data/lib/tefoji/jira_api.rb

@@ -2,6 +2,7 @@ require 'base64'
 require 'io/console'
 require 'json'
 require 'rest-client'
+RestClient.log = 'stdout'
 
 module Tefoji
   ## An interface to send API request to Jira using the rest-client gem.
@@ -14,6 +15,7 @@ module Tefoji
     include Logging
 
     ## Jira field keys
+    FIELD_ACCOUNT_ID = 'accountId'
     FIELD_ID = 'id'
     FIELD_KEY = 'key'
     FIELD_NAME = 'name'
@@ -74,19 +76,26 @@ module Tefoji
 
     # Do this so we can inform the user quickly that their credentials didn't work
     # There may be a better test, but this is the one the original Winston uses
-    def test_authentication
-      get_username
+    def test_authentication(jira_cloud)
+      get_username(@jira_username, jira_cloud)
     rescue RestClient::Forbidden
       fatal 'Forbidden: either the authentication is incorrect or ' \
             'Jira might be requiring a CAPTCHA response from the web interface.'
     end
 
-    def get_username
-      jira_get("user?username=#{@jira_username}")
+    # Get information about user in Jira
+    def get_username(username, jira_cloud, fail_if_not_found = true)
+      search_parameters = if jira_cloud
+                            "user/search?query=#{username}"
+                          else
+                            "user?username=#{username}"
+                          end
+
+      jira_get(search_parameters, fail_if_not_found)
     end
 
     # Save authentication YAML to the a file for reuse.
-    def save_authentication(save_file_name)
+    def save_authentication(save_file_name, jira_cloud)
       backup_file_name = "#{save_file_name}.bak"
 
       if File.readable?(save_file_name)
@@ -94,18 +103,22 @@ module Tefoji
         @logger.info "Saved #{save_file_name} to #{backup_file_name}"
       end
 
-      File.open(save_file_name, 'w') do |f|
-        f.puts "jira-auth: #{@jira_auth_string}"
-      end
+      jira_auth_string = if jira_cloud
+                           "jira-auth: #{@jira_auth_string}\n"
+                         else
+                           "jira-cloud-auth: #{@jira_auth_string}\n"
+                         end
+      File.write(save_file_name, jira_auth_string)
+
       File.chmod(0o600, save_file_name)
 
       @logger.info "Saved Jira authentication to #{save_file_name}"
     end
 
     # Create a Jira issue
-    def create_issue(issue_data)
+    def create_issue(issue_data, jira_cloud)
       request_path = 'issue'
-      jira_fields = issue_data_to_jira_fields(issue_data)
+      jira_fields = issue_data_to_jira_fields(issue_data, jira_cloud)
       jira_post(request_path, jira_fields)
     rescue RestClient::Forbidden
       fatal "Forbidden: could not send #{request_path} request to Jira. "\
@@ -159,15 +172,28 @@ module Tefoji
     end
 
     # https://www.youtube.com/watch?v=JsntlJZ9h1U
-    def add_watcher(issue_key, watcher)
+    def add_watcher(issue_key, watcher, jira_cloud)
       request_path = "issue/#{issue_key}/watchers"
+      watcher = useremail_to_id(watcher) if jira_cloud
       request_data = watcher
       jira_post(request_path, request_data)
     end
 
+    # jira cloud api calls now require an accountId rather than a username. This method will convert
+    # emails or display names to accountIds.
+    def useremail_to_id(email)
+      response = get_username(email, true, true)[0]
+      id = response['accountId']
+      if id.nil? || id.empty?
+        @logger.error "accountId not found for #{email}."
+        exit 1
+      end
+      return id
+    end
+
     private
 
-    def jira_get(jira_request_path)
+    def jira_get(jira_request_path, fail_if_not_found = true)
       # Jira likes to send complete URLs for responses. Handle
       # the case where we've received a 'self' query from Jira with
       # fully formed url
@@ -176,7 +202,6 @@ module Tefoji
         url = "#{@jira_base_rest_url}/#{jira_request_path}"
       end
       headers = @authentication_header
-
       begin
         response = RestClient.get(url, headers)
       rescue RestClient::MovedPermanently,
@@ -191,7 +216,9 @@ module Tefoji
       rescue RestClient::NotFound,
              SocketError,
              Errno::ECONNREFUSED => e
-        fatal "Cannot connect to #{@jira_base_rest_url}: #{e.message}"
+        # Return False if not found rather than fail to allow for checking the existence of users.
+        fatal "Cannot connect to #{@jira_base_rest_url}: #{e.message}" if fail_if_not_found
+        return false
       end
 
       return JSON.parse(response.body)
@@ -241,7 +268,7 @@ module Tefoji
     # Provide the needed translation of user-created data to required format
     # of Jira requests. This is mostly a list of fussing with the Jira field
     # names.
-    def issue_data_to_jira_fields(issue_data)
+    def issue_data_to_jira_fields(issue_data, jira_cloud)
       # Check to ensure we have what we need to create a issue
 
       unless issue_data['summary']
@@ -254,64 +281,130 @@ module Tefoji
       end
 
       # build the jira_fields hash describing the issue
-
       # These are required for all issues
       jira_fields = {
         'summary' => issue_data['summary'],
         'project' => { 'key' => issue_data['project'] }
       }
 
+      set_common_jira_fields(issue_data, jira_fields)
+
+      if jira_cloud
+        set_cloud_jira_fields(issue_data, jira_fields)
+      else
+        set_onprem_jira_fields(issue_data, jira_fields)
+      end
+
+      return { 'fields' => jira_fields }
+    end
+
+    def set_common_jira_fields(issue_data, jira_fields)
       # The following are optional
       if issue_data['description']
         jira_fields['description'] = issue_data['description']
       end
+
+      if issue_data['labels']
+        labels = issue_data['labels'].to_a.flatten.reject { |t| t =~ /^\s*$/ }
+        jira_fields['labels'] = labels unless labels.empty?
+      end
+      if issue_data['duedate']
+        jira_fields['duedate'] = issue_data['duedate']
+      end
+      if issue_data['fix_version']
+        jira_fields['fixVersions'] = [issue_data['fix_version']]
+      end
+
+      if issue_data['components']
+        components = issue_data[:components].to_a.flatten.reject { |t| t =~ /^\s*$/ }
+        unless components.empty?
+          jira_fields['components'] = components.map { |component| { FIELD_NAME => component } }
+        end
+      end
+
+      # Default issue type to ISSUE_TASK if it isn't already set
+      jira_fields['issuetype'] = { FIELD_NAME => ISSUE_TASK }
+
+      security = ENV['SECURITY'] || issue_data['security']
+      return unless security
+
+      case security.downcase
+      when 'confidential'
+        jira_fields['security'] = { FIELD_ID => '10002' }
+      when 'internal'
+        jira_fields['security'] = { FIELD_ID => '10001' }
+      when 'public'
+        # Nothing to do here - public is default
+      else
+        @logger.fatal "Unknown security type: #{security}"
+        exit 1
+      end
+    end
+
+    def set_cloud_jira_fields(issue_data, jira_fields)
       if issue_data['assignee']
-        jira_fields['assignee'] = { FIELD_NAME => issue_data['assignee'] }
+        assignee = issue_data['assignee']
+        assignee = useremail_to_id(assignee)
+        jira_fields['assignee'] = { FIELD_ACCOUNT_ID => assignee }
+      end
+
+      if issue_data['type']
+        jira_fields['issuetype'] = { FIELD_NAME => issue_data['type'] }
+        # If this is an epic, we need to add an epic name
+        if issue_data['type'].casecmp?(ISSUE_EPIC)
+          jira_fields['customfield_10011'] = issue_data['epic_name'] || issue_data['summary']
+        end
       end
+
       if issue_data['story_points']
-        jira_fields['customfield_10002'] = issue_data['story_points'].to_i
+        jira_fields['customfield_10038'] = issue_data['story_points'].to_i
       end
       if issue_data['team']
-        jira_fields['customfield_14200'] = { FIELD_VALUE => issue_data['team'] }
+        jira_fields['customfield_10052'] = { FIELD_VALUE => issue_data['team'] }
       end
       if issue_data['teams']
         teams = issue_data['teams'].to_a.flatten.reject { |t| t =~ /^\s*$/ }
         unless teams.empty?
-          jira_fields['customfield_14201'] = teams.map { |team| { FIELD_VALUE => team } }
+          jira_fields['customfield_10066'] = teams.map { |team| { FIELD_VALUE => team } }
         end
       end
       if issue_data['subteam']
-        jira_fields['customfield_11700'] = [issue_data['subteam']]
+        jira_fields['customfield_10045'] = [issue_data['subteam']]
       end
       if issue_data['sprint']
-        jira_fields['customfield_10005'] = issue_data['sprint'].to_i
+        jira_fields['customfield_10020'] = issue_data['sprint'].to_i
       end
       if issue_data['acceptance']
-        jira_fields['customfield_11501'] = issue_data['acceptance']
-      end
-      if issue_data['labels']
-        labels = issue_data['labels'].to_a.flatten.reject { |t| t =~ /^\s*$/ }
-        jira_fields['labels'] = labels unless labels.empty?
-      end
-      if issue_data['duedate']
-        jira_fields['duedate'] = issue_data['duedate']
-      end
-      if issue_data['fix_version']
-        jira_fields['fixVersions'] = [issue_data['fix_version']]
+        jira_fields['customfield_10062'] = issue_data['acceptance']
       end
       if issue_data['release_notes']
-        jira_fields['customfield_11100'] = { FIELD_VALUE => issue_data['release_notes'] }
+        jira_fields['customfield_10043'] = { FIELD_VALUE => issue_data['release_notes'] }
       end
 
-      if issue_data['components']
-        components = issue_data[:components].to_a.flatten.reject { |t| t =~ /^\s*$/ }
-        unless components.empty?
-          jira_fields['components'] = components.map { |component| { FIELD_NAME => component } }
+      # If a issue has a specified parent issue, prefer that. The parent issue *should* already
+      # be linked to the main epic. Otherwise, we need to set it to have an epic_parent. This can
+      # either be an epic linked to the main epic or the main epic itself.
+
+      if issue_data['parent']
+        unless issue_data['type'].casecmp?(ISSUE_SUB_TASK) || !issue_data['type']
+          @logger.fatal "A issue with a parent must be classified as a Sub-issue\n\n#{issue_data}"
+          exit 1
+        end
+        jira_fields['issuetype'] = { FIELD_NAME => ISSUE_SUB_TASK }
+        jira_fields['parent'] = { FIELD_KEY => issue_data['parent'] }
+      elsif issue_data['epic_parent']
+        if issue_data['type'].casecmp?(ISSUE_SUB_TASK)
+          @logger.fatal "This issue cannot be a subtask of an epic\n\n#{issue_data}"
+          exit 1
         end
+        jira_fields['customfield_10018'] = issue_data['epic_parent']
       end
+    end
 
-      # Default issue type to ISSUE_TASK if it isn't already set
-      jira_fields['issuetype'] = { FIELD_NAME => ISSUE_TASK }
+    def set_onprem_jira_fields(issue_data, jira_fields)
+      if issue_data['assignee']
+        jira_fields['assignee'] = { FIELD_NAME => issue_data['assignee'] }
+      end
 
       if issue_data['type']
         jira_fields['issuetype'] = { FIELD_NAME => issue_data['type'] }
@@ -321,6 +414,31 @@ module Tefoji
         end
       end
 
+      if issue_data['story_points']
+        jira_fields['customfield_10002'] = issue_data['story_points'].to_i
+      end
+      if issue_data['team']
+        jira_fields['customfield_14200'] = { FIELD_VALUE => issue_data['team'] }
+      end
+      if issue_data['teams']
+        teams = issue_data['teams'].to_a.flatten.reject { |t| t =~ /^\s*$/ }
+        unless teams.empty?
+          jira_fields['customfield_14201'] = teams.map { |team| { FIELD_VALUE => team } }
+        end
+      end
+      if issue_data['subteam']
+        jira_fields['customfield_11700'] = [issue_data['subteam']]
+      end
+      if issue_data['sprint']
+        jira_fields['customfield_10005'] = issue_data['sprint'].to_i
+      end
+      if issue_data['acceptance']
+        jira_fields['customfield_11501'] = issue_data['acceptance']
+      end
+      if issue_data['release_notes']
+        jira_fields['customfield_11100'] = { FIELD_VALUE => issue_data['release_notes'] }
+      end
+
       # If a issue has a specified parent issue, prefer that. The parent issue *should* already
       # be linked to the main epic. Otherwise, we need to set it to have an epic_parent. This can
       # either be an epic linked to the main epic or the main epic itself.
@@ -339,23 +457,6 @@ module Tefoji
         end
         jira_fields['customfield_10006'] = issue_data['epic_parent']
       end
-
-      security = ENV['SECURITY'] || issue_data['security']
-      if security
-        case security.downcase
-        when 'confidential'
-          jira_fields['security'] = { FIELD_ID => '10002' }
-        when 'internal'
-          jira_fields['security'] = { FIELD_ID => '10001' }
-        when 'public'
-          # Nothing to do here - public is default
-        else
-          @logger.fatal "Unknown security type: #{security}"
-          exit 1
-        end
-      end
-
-      return { 'fields' => jira_fields }
     end
   end
 

data/lib/tefoji.rb

@@ -28,6 +28,7 @@ module Tefoji
       @jira_auth_string = user_options['jira-auth-string']
       @jira_auth_file = user_options['jira-auth-file']
       @jira_mock = user_options['--jira-mock']
+      @jira_cloud = user_options['jira-cloud']
 
       @no_notes = user_options['--no-notes']
       @template_data = {}
@@ -74,6 +75,8 @@ module Tefoji
       resolve_variables(main_template_data['declare'])
       @logger.debug "Declarations: #{@declarations}"
 
+      check_assignees(main_template_data)
+
       # Process 'before' templates
       main_template.dig(:includes, :before)&.each do |before|
         default_epic_saved = @default_target_epic
@@ -270,13 +273,13 @@ module Tefoji
       @jira_api.logger = @logger
 
       @jira_api.authenticate(@jira_url, @jira_user, @jira_auth_string)
-      @jira_api.test_authentication unless @jira_mock
+      @jira_api.test_authentication(@jira_cloud) unless @jira_mock
     end
 
     # Save Jira auth data
     def save_authentication
       authenticate unless @jira_api
-      @jira_api.save_authentication(@jira_auth_file)
+      @jira_api.save_authentication(@jira_auth_file, @jira_cloud)
     end
 
     private
@@ -296,7 +299,7 @@ module Tefoji
 
       feature = variable_substitute(feature_to_do)
       feature['type'] = JiraApi::ISSUE_FEATURE
-      @feature_issue = @jira_api.create_issue(feature)
+      @feature_issue = @jira_api.create_issue(feature, @jira_cloud)
       @logger.info "Feature issue: #{@feature_issue['key']}"
       @feature_issue
     end
@@ -358,7 +361,7 @@ module Tefoji
           epic['security'] = 'internal'
         end
 
-        epic_issue = @jira_api.create_issue(epic)
+        epic_issue = @jira_api.create_issue(epic, @jira_cloud)
         epic_issue['short_name'] = short_name
         @logger.info 'Epic: %16s [%s]' % [epic_issue['key'], short_name]
         @jira_api.link_issues(@feature_issue['key'], epic_issue['key']) if @feature_issue
@@ -398,7 +401,7 @@ module Tefoji
         jira_ready_data, raw_issue_data = prepare_jira_ready_data(issue, issue_defaults)
         next if jira_ready_data.nil? || raw_issue_data.nil?
 
-        response_data = @jira_api.create_issue(jira_ready_data)
+        response_data = @jira_api.create_issue(jira_ready_data, @jira_cloud)
         jira_issue = @jira_api.retrieve_issue(response_data['self'])
         jira_issue['short_name'] = raw_issue_data['short_name']
 
@@ -608,7 +611,7 @@ module Tefoji
         issue_key = jira_issue_data['key']
         watchers = raw_issue_data[deferred_tag].value
         watchers.each do |watcher|
-          @jira_api.add_watcher(issue_key, watcher)
+          @jira_api.add_watcher(issue_key, watcher, @jira_cloud)
           @logger.info '%14s: watched by %s' % [issue_key, watcher]
         end
       end
@@ -641,12 +644,17 @@ module Tefoji
     def user_function_call(function_definition)
       @logger.debug "function_definition is: #{function_definition}"
       function_name = function_definition['name']
-      if function_definition.key?('arguments')
-        function_arguments = function_definition['arguments'].map do |a|
+      argument = function_definition['argument']
+      arguments = function_definition['arguments']
+
+      if argument.is_a?(Array) || arguments.is_a?(Array)
+        arguments = argument || function_definition['arguments']
+        function_arguments = arguments.map do |a|
           expand_right_value(a).value
         end
-      elsif function_definition.key?('argument')
-        function_arguments = [expand_right_value(function_definition['argument']).value]
+      elsif argument.is_a?(String) || arguments.is_a?(String)
+        argument = function_definition['argument'] || function_definition['arguments']
+        function_arguments = [expand_right_value(argument).value]
       else
         fatal("No arguments supplied to function \"#{function_name}\"")
       end
@@ -814,6 +822,124 @@ module Tefoji
       return true
     end
 
+    # Iterates through the assignees in a template and checks if they exist on jira. If any assignee in
+    # the template does not exist, it will fail, outputting a message saying which users failed and
+    # what epics/issues they are associated with.
+    # NOTE: This check does not guarantee a user is assignable, so there may be cases when a user passes
+    #       the check but still fails when being assigned a ticket/epic. If we find a reasonable way to
+    #       check assignability prior to the epic/issue being created, we should implement it in place of
+    #       using the get_username method.
+    def check_assignees(main_template_data)
+      valid_users = []
+
+      invalid_users_to_features = {}
+      main_template_data['features']&.each do |feature|
+        next unless feature['assignee']
+
+        assignee_username = get_username_from_assignee(feature['assignee'])
+        next if valid_users.include?(assignee_username)
+
+        update_user_validity(assignee_username, feature['summary'], valid_users, invalid_users_to_features)
+      end
+
+      if main_template_data.dig('feature', 'assignee')
+        assignee_username = get_username_from_assignee(main_template_data['feature']['assignee'])
+        unless valid_users.include?(assignee_username)
+          update_user_validity(assignee_username, main_template_data['feature']['summary'], valid_users,
+                               invalid_users_to_features)
+        end
+      end
+
+      invalid_users_to_epics = {}
+      main_template_data['epics']&.each do |epic|
+        next unless epic['assignee']
+
+        assignee_username = get_username_from_assignee(epic['assignee'])
+        next if valid_users.include?(assignee_username)
+
+        update_user_validity(assignee_username, epic['summary'], valid_users, invalid_users_to_epics)
+      end
+
+      if main_template_data.dig('epic', 'assignee')
+        assignee_username = get_username_from_assignee(main_template_data['epic']['assignee'])
+        unless valid_users.include?(assignee_username)
+          update_user_validity(assignee_username, main_template_data['epic']['summary'], valid_users,
+                               invalid_users_to_epics)
+        end
+      end
+
+      invalid_users_to_issue_defaults = {}
+      if main_template_data.dig('issue_defaults', 'assignee')
+        assignee_username = get_username_from_assignee(main_template_data['issue_defaults']['assignee'])
+        unless valid_users.include?(assignee_username)
+          update_user_validity(assignee_username, 'issue_defaults', valid_users, invalid_users_to_issue_defaults)
+        end
+      end
+
+      invalid_users_to_issues = {}
+      main_template_data['issues'].each do |issue|
+        next unless issue['assignee']
+
+        assignee_username = get_username_from_assignee(issue['assignee'])
+        next if valid_users.include?(assignee_username)
+
+        update_user_validity(assignee_username, issue['summary'], valid_users, invalid_users_to_issues)
+      end
+      unless invalid_users_to_features.empty? &&
+             invalid_users_to_epics.empty? &&
+             invalid_users_to_issue_defaults.empty? &&
+             invalid_users_to_issues.empty?
+        invalid_user_message = "Invalid assignees:\n"
+      end
+      unless invalid_users_to_features.empty?
+        invalid_users_to_features.each do |assignee, feature|
+          invalid_user_message += "Assignee #{assignee} associated with features #{feature}\n"
+        end
+      end
+      unless invalid_users_to_epics.empty?
+        invalid_users_to_epics.each do |assignee, epic|
+          invalid_user_message += "Assignee #{assignee} associated with epics #{epic}\n"
+        end
+      end
+      unless invalid_users_to_issue_defaults.empty?
+        invalid_users_to_issue_defaults.each do |assignee, _|
+          invalid_user_message += "Assignee #{assignee} associated with issue_defaults\n"
+        end
+      end
+      unless invalid_users_to_issues.empty?
+        invalid_users_to_issues.each do |assignee, issues|
+          invalid_user_message += "Assignee #{assignee} associated with issues #{issues}\n"
+        end
+      end
+      fatal invalid_user_message if invalid_user_message
+    end
+
+    def get_username_from_assignee(assignee)
+      # Assignee is a variable or direct input
+      username = expand_string(assignee) if assignee.is_a?(String)
+      # Assignee is a function
+      if assignee.is_a?(Hash) && assignee.key?('function')
+        username = user_function_call(assignee['function'])
+      end
+
+      unless username
+        fatal "invalid assignee format for issue: #{issue['short_name']} failed: must be a function or a string"
+      end
+
+      username = username.value unless username.is_a?(String)
+      username
+    end
+
+    def update_user_validity(username, issue_name, valid_users, invalid_users)
+      if invalid_users.key?(username)
+        invalid_users[username] += [issue_name]
+      elsif @jira_api.get_username(username, @jira_cloud, false)
+        valid_users << username
+      else
+        invalid_users[username] = invalid_users.fetch(username, []) + [issue_name]
+      end
+    end
+
     def logger_initialize(log_location)
       logger = Logger.new(log_location, progname: 'tefoji', level: @log_level)
       logger.formatter = proc do |severity, _, script_name, message|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tefoji
 version: !ruby/object:Gem::Version
-  version: 1.0.8
+  version: 1.0.10
 platform: ruby
 authors:
 - Puppet Labs
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-08-11 00:00:00.000000000 Z
+date: 2023-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry-byebug