jirametrics 2.20.1 → 2.20.2pre6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cbe1101b082615d38939850c0adc688aedefe02a74537503bcd390cdf11d0d4e
-  data.tar.gz: eeffbda7c7ba8280273e0d749ede1ed3c1caa33d06a1f50a2c47b2331035d5af
+  metadata.gz: 63b99b8edec3e017c9ef1be6ad6d7e9c4fa94038126a4fb84cf8d90face9ed9e
+  data.tar.gz: c2df48cad225aa8c270dfd3c8f04dc049d3c48642eaa971b885258d58f230a61
 SHA512:
-  metadata.gz: b73533c90e457c2c5f7a7f2d1759ccd44d218ebe984052fcb581d8ab88225abf43b5612923217b4fa7467ed11e82ba5b4fe2cc656a324f269b71c2497bf67659
-  data.tar.gz: 57cbc54fe6c739d0c85f68cc0efcfbc6005975af0b40174ed9ee35790a83dac9b5e524601b770dffc6a3fefbd10f0d577d19b840560b3acfb63b0b542728d5fc
+  metadata.gz: 50dcdb60c3626a3faa6260313fed8a015f24eb757b9f7f338218f7e789b5d0be242e390d9e56b51b2f2f7289eebe6d64df2e195af04a1ecec8acd2fff782772b
+  data.tar.gz: 6608ebbefbd28943fed9ab39fa8611a0948e523f76f9f3b37777fdef0a7f0d001df8db2ef4d7d936c588077ba45f27639e8273af279e74291fdbfa638941a9e9

data/lib/jirametrics/board.rb

@@ -116,4 +116,8 @@ class Board
   def estimation_configuration
     EstimationConfiguration.new raw: raw['estimation']
   end
+
+  def inspect
+    "Board(id:#{id}, name:#{name.inspect}, board_type:#{board_type.inspect})"
+  end
 end

data/lib/jirametrics/board_config.rb

@@ -24,7 +24,8 @@ class BoardConfig
     end
 
     @board.cycletime = CycleTimeConfig.new(
-      parent_config: self, label: label, block: block, file_system: project_config.file_system,
+      possible_statuses: project_config.possible_statuses,
+      label: label, block: block, file_system: project_config.file_system,
       settings: project_config.settings
     )
   end

data/lib/jirametrics/change_item.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class ChangeItem
-  attr_reader :field, :value_id, :old_value_id, :raw, :author_raw
+  attr_reader :field, :value_id, :old_value_id, :raw, :author_raw, :field_id
   attr_accessor :value, :old_value, :time
 
   def initialize raw:, author_raw:, time:, artificial: false
@@ -13,9 +13,15 @@ class ChangeItem
 
     @field = @raw['field']
     @value = @raw['toString']
-    @value_id = @raw['to'].to_i
     @old_value = @raw['fromString']
-    @old_value_id = @raw['from']&.to_i
+    if sprint?
+      @value_id = @raw['to'].split(', ').collect(&:to_i)
+      @old_value_id = (@raw['from'] || '').split(', ').collect(&:to_i)
+    else
+      @value_id = @raw['to'].to_i
+      @old_value_id = @raw['from']&.to_i
+    end
+    @field_id = @raw['fieldId']
     @artificial = artificial
   end
 

data/lib/jirametrics/chart_base.rb

@@ -22,6 +22,14 @@ class ChartBase
     @canvas_responsive = true
   end
 
+  def call_before_run &proc
+    (@call_before_run_procs ||= []) << proc
+  end
+
+  def before_run
+    @call_before_run_procs&.each { |proc| proc.call }
+  end
+
   def aggregated_project?
     @aggregated_project
   end
@@ -279,4 +287,19 @@ class ChartBase
       </div>
     TEXT
   end
+
+  # Set a cycletime for just this one chart, overriding the one for the report.
+  def cycletime &block
+    call_before_run do
+      @cycletime = CycleTimeConfig.new(
+        possible_statuses: possible_statuses, label: nil, block: block, file_system: file_system,
+        settings: settings
+      )
+    end
+  end
+
+  # Returns the cycletime in use right now, which may be specific to the chart or across the report.
+  def cycletime_for_issue issue
+    @cycletime || issue.board.cycletime
+  end
 end

data/lib/jirametrics/cycletime_config.rb

@@ -6,15 +6,14 @@ require 'date'
 class CycleTimeConfig
   include SelfOrIssueDispatcher
 
-  attr_reader :label, :parent_config, :settings, :file_system
+  attr_reader :label, :possible_statuses, :settings, :file_system
 
-  def initialize parent_config:, label:, block:, settings:, file_system: nil, today: Date.today
+  def initialize possible_statuses:, label:, block:, settings:, file_system: nil, today: Date.today
 
-    @parent_config = parent_config
+    @possible_statuses = possible_statuses
     @label = label
     @today = today
     @settings = settings
-    @cache_cycletime_calculations = settings['cache_cycletime_calculations']
 
     # If we hit something deprecated and this is nil then we'll blow up. Although it's ugly, this
     # may make it easier to find problems in the test code ;-)
@@ -68,7 +67,7 @@ class CycleTimeConfig
   def started_stopped_changes issue
     cache_key = "#{issue.key}:#{issue.board.id}"
     last_result = (@cache ||= {})[cache_key]
-    return *last_result if last_result && @cache_cycletime_calculations
+    return *last_result if last_result && settings['cache_cycletime_calculations']
 
     started = @start_at.call(issue)
     stopped = @stop_at.call(issue)

data/lib/jirametrics/daily_wip_chart.rb

@@ -66,7 +66,7 @@ class DailyWipChart < ChartBase
     hash = {}
 
     @issues.each do |issue|
-      start, stop = issue.board.cycletime.started_stopped_dates(issue)
+      start, stop = cycletime_for_issue(issue).started_stopped_dates(issue)
       next if start.nil? && stop.nil?
 
       # If it stopped but never started then assume it started at creation so the data points

data/lib/jirametrics/data_quality_report.rb

@@ -266,6 +266,8 @@ class DataQualityReport < ChartBase
 
   def scan_for_items_blocked_on_closed_tickets entry:
     entry.issue.issue_links.each do |link|
+      next unless settings['blocked_link_text'].include?(link.label)
+
       this_active = !entry.stopped
       other_active = !link.other_issue.board.cycletime.started_stopped_times(link.other_issue).last
       next unless this_active && !other_active

data/lib/jirametrics/fix_version.rb

@@ -11,11 +11,24 @@ class FixVersion
     @raw['name']
   end
 
+  def description
+    @raw['description']
+  end
+
   def id
     @raw['id'].to_i
   end
 
+  def release_date
+    text = @raw['releaseDate']
+    text.nil? ? nil : Date.parse(text)
+  end
+
   def released?
     @raw['released']
   end
+
+  def archived?
+    @raw['archived']
+  end
 end

data/lib/jirametrics/html_report_config.rb

@@ -52,7 +52,8 @@ class HtmlReportConfig
       raise 'Multiple cycletimes not supported' if board.cycletime
 
       board.cycletime = CycleTimeConfig.new(
-        parent_config: self, label: label, block: block, file_system: file_system, settings: settings
+        possible_statuses: file_config.project_config, label: label, block: block,
+        file_system: file_system, settings: settings
       )
     end
   end
@@ -175,6 +176,7 @@ class HtmlReportConfig
     after_init_block&.call chart
 
     @charts << chart
+    chart.before_run
     html chart.run
   end
 

data/lib/jirametrics/issue.rb

@@ -212,8 +212,79 @@ class Issue
     first_time_in_status(*board.visible_columns.collect(&:status_ids).flatten)
   end
 
+  # If this issue will ever be in an active sprint then return the time that it
+  # was first added to that sprint, whether or not the sprint was active at that
+  # time. Although it seems like an odd thing to calculate, it's a reasonable proxy
+  # for 'ready' in cases where the team doesn't have an explicit 'ready' status.
+  # You'd be better off with an explicit 'ready' but sometimes that's not an option.
+  def first_time_added_to_active_sprint
+    unless board.scrum?
+      raise 'first_time_added_to_active_sprint() can only be used with Scrum boards: ' \
+          "issue=#{key}, board=#{board.inspect}"
+    end
+    data_clazz = Struct.new(:sprint_id, :sprint_start, :sprint_stop, :change)
+
+    matching_changes = []
+    all_datas = []
+
+    @changes.each do |change|
+      next unless change.sprint?
+
+      added_sprint_ids = change.value_id - change.old_value_id
+      added_sprint_ids.each do |id|
+        data = data_clazz.new
+        data.sprint_id = id
+        data.change = change
+        data.sprint_start, data.sprint_stop = find_sprint_start_end(sprint_id: id, change: change)
+        all_datas << data
+      end
+
+      removed_sprint_ids = change.old_value_id - change.value_id
+      removed_sprint_ids.each do |id|
+        data = all_datas.find { |d| d.sprint_id == id }
+
+        all_datas.delete(data)
+
+        next if data.sprint_start.nil? || data.sprint_start >= change.time
+
+        matching_changes << data.change
+      end
+    end
+
+    # There can't be any more removes so whatever is left is a valid option
+    # Now all we care about is if the sprint has started.
+    all_datas.each do |data|
+      matching_changes << data.change if data.sprint_start
+    end
+
+    matching_changes.min_by(&:time)
+  end
+
+  def find_sprint_start_end sprint_id:, change:
+    start = nil
+    stop = nil
+
+    sprint = board.sprints.find { |s| s.id == sprint_id }
+    if sprint
+      start = sprint.start_time
+      stop = sprint.completed_time
+      puts "#{key}: Found sprint. values: #{start}/#{stop} from sprints file"
+    end
+
+    sprint_data = raw['fields'][change.field_id].find { |sd| sd['id'].to_i == sprint_id }
+    if sprint_data
+      start = parse_time(sprint_data['startDate'])
+      stop = parse_time(sprint_data['completeDate'])
+      puts "#{key}: Found sprint. values: #{start}/#{stop} from field: #{change.field_id}"
+    end
+
+    [start, stop]
+  end
+
   def parse_time text
-    if text.is_a? String
+    if text.nil?
+      nil
+    elsif text.is_a? String
       Time.parse(text).getlocal(@timezone_offset)
     else
       Time.at(text / 1000).getlocal(@timezone_offset)
@@ -225,6 +296,10 @@ class Issue
     parse_time @raw['fields']['created'] if @raw['fields']['created']
   end
 
+  def time_created
+    @changes.first
+  end
+
   def updated
     parse_time @raw['fields']['updated']
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jirametrics
 version: !ruby/object:Gem::Version
-  version: 2.20.1
+  version: 2.20.2pre6
 platform: ruby
 authors:
 - Mike Bowler
@@ -159,7 +159,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.3
 specification_version: 4
 summary: Extract Jira metrics
 test_files: []