jirametrics 3.3.1 → 3.4

data/lib/jirametrics/flow_efficiency_scatterplot.rb

@@ -12,10 +12,11 @@ class FlowEfficiencyScatterplot < ChartBase
     super()
 
     header_text 'Flow Efficiency'
+    no_data_text '<%= render_header %><div>No data matched the selected criteria. Nothing to show.</div>'
     description_text <<-HTML
       <div class="p">
         This chart shows the active time against the the total time spent on a ticket.
-        <a href="https://improvingflow.com/2024/07/06/flow-efficiency.html">Flow  efficiency</a> is the ratio
+        <a href="https://blog.mikebowler.ca/2024/07/06/flow-efficiency/">Flow  efficiency</a> is the ratio
         between these two numbers.
       </div>
       <div class="p">
@@ -63,10 +64,7 @@ class FlowEfficiencyScatterplot < ChartBase
       create_dataset(issues: issues, label: rules.label, color: rules.color)
     end
 
-    if data_sets.empty?
-      return "<h1 class='foldable'>#{@header_text}</h1>" \
-        '<div>No data matched the selected criteria. Nothing to show.</div>'
-    end
+    return render_no_data if data_sets.empty?
 
     wrap_and_render(binding, __FILE__)
   end

data/lib/jirametrics/html/aging_work_in_progress_chart.erb

@@ -42,21 +42,24 @@ new Chart(document.getElementById(<%= chart_id.inspect %>).getContext('2d'),
           z: 1 // draw the grid lines on top of the bars
         },
         stacked: false,
-        max: <%= (@max_age * 1.1).to_i %>
+        max: <%= (@max_age * 1.1).to_i %>,
+        ticks: {
+          // Uses labelExceptCrowdedCeiling() defined in index.js.
+          callback: labelExceptCrowdedCeiling
+        }
       }
     },
     plugins: {
       tooltip: {
         callbacks: {
           label: function(context) {
-            if( typeof(context.dataset.data[context.dataIndex]) == "number" ) {
-              let full_data = <%= @bar_data.inspect %>;
-              let columnIndex = context.dataIndex;
-              let rowIndex = context.datasetIndex - <%= @row_index_offset %>;
-              return context.dataset.label + " of completed work items left this column in " +full_data[rowIndex][columnIndex] + " days or less";
+            let point = context.dataset.data[context.dataIndex];
+            // A band is a floating bar, [bottom, top]. A dot is an object carrying its own title.
+            if( Array.isArray(point) ) {
+              return context.dataset.label + " of completed work items left this column in " + point[1] + " days or less";
             }
             else {
-              return context.dataset.data[context.dataIndex].title;
+              return point.title;
             }
           }
         }

data/lib/jirametrics/html/estimate_accuracy_chart.erb

@@ -6,7 +6,40 @@
 new Chart(document.getElementById('<%= chart_id %>').getContext('2d'), {
   type: 'bubble',
   data: {
-    datasets: <%= JSON.generate(data_sets) %>
+    datasets: (function() {
+      const dataSets = <%= JSON.generate(data_sets) %>;
+      // The still in progress series is drawn as right pointing arrows rather than discs. Those
+      // items have not finished, so their cycletime is a lower bound that keeps growing and the
+      // arrow says "this one is still moving that way". Same idea as the capped-value arrows on
+      // the scatterplot, turned to point along the axis that is actually still moving.
+      //
+      // Sized per point rather than at a fixed size, because on this chart the radius carries the
+      // number of issues stacked at that spot and we would be throwing that away.
+      function movementArrow(color, radius) {
+        const size = Math.max(12, Math.round(radius * 3));
+        const glyph = document.createElement('canvas');
+        glyph.width = size; glyph.height = size;
+        const pen = glyph.getContext('2d');
+        const middle = size / 2, head = size * 0.3, tip = size * 0.88;
+        pen.strokeStyle = color;
+        pen.lineWidth = Math.max(2, radius * 0.42);
+        pen.lineCap = 'round'; pen.lineJoin = 'round';
+        pen.beginPath();
+        pen.moveTo(size * 0.12, middle); pen.lineTo(tip, middle);
+        pen.moveTo(tip - head, middle - head);
+        pen.lineTo(tip, middle);
+        pen.lineTo(tip - head, middle + head);
+        pen.stroke();
+        return glyph;
+      }
+      dataSets.forEach(function(dataSet) {
+        if (!dataSet.still_in_progress) return;
+        dataSet.pointStyle = dataSet.data.map(function(point) {
+          return movementArrow(dataSet.backgroundColor, point.r);
+        });
+      });
+      return dataSets;
+    })()
   },
   options: {
     responsive: <%= canvas_responsive? %>, // If responsive is true then it fills the screen

data/lib/jirametrics/html/index.css

@@ -106,10 +106,16 @@
   --wip-by-column-chart-limit-line-color: #D55E00;        /* Okabe-Ito vermilion */
   --wip-by-column-chart-recommendation-color: #009E73;    /* Okabe-Ito bluish green */
 
-  --estimate-accuracy-chart-completed-fill-color: #92D9C0;    /* light bluish green */
-  --estimate-accuracy-chart-completed-border-color: #009E73;  /* Okabe-Ito bluish green */
-  --estimate-accuracy-chart-active-fill-color: #F4C6AD;       /* light vermilion */
-  --estimate-accuracy-chart-active-border-color: #D55E00;     /* Okabe-Ito vermilion */
+  /* Full strength, not lightened. The pastel versions of these two collapsed into the same beige
+     under protanopia (4.45 apart, where the palette's own floor is 11.08) and measured 1.6 against
+     white, half the contrast WCAG asks of a shape you need to see. At full strength they are 18.45
+     apart and clear the contrast floor. The border is the page colour rather than a second hue, so
+     it separates overlapping bubbles instead of competing with the fill, and needs no dark mode
+     override. See jirametrics-7ss. */
+  --estimate-accuracy-chart-completed-fill-color: #009E73;    /* Okabe-Ito bluish green */
+  --estimate-accuracy-chart-completed-border-color: var(--body-background);
+  --estimate-accuracy-chart-active-fill-color: #D55E00;       /* Okabe-Ito vermilion */
+  --estimate-accuracy-chart-active-border-color: var(--body-background);
 
   --expedited-chart-no-longer-expedited: gray;
   --expedited-chart-dot-issue-started-color: #E69F00;     /* Okabe-Ito orange */
@@ -315,8 +321,6 @@ html[data-theme="dark"] {
      band. Those are the two ends of the duration ramp, so confusing them is the worst case. This
      stays in the same warm family so the ramp still reads cool to warm as work ages. */
   --wip-chart-duration-more-than-four-weeks-color: #B16BAE;
-  --estimate-accuracy-chart-completed-border-color: #2DCB9A;
-  --estimate-accuracy-chart-active-border-color: #E69F00;
   --expedited-chart-dot-issue-stopped-color: #2DCB9A;
   --expedited-chart-dot-expedite-started-color: #E69F00;
   --expedited-chart-dot-expedite-stopped-color: #2DCB9A;
@@ -481,9 +485,6 @@ html[data-theme="light"] {
     --wip-chart-duration-four-weeks-or-less-color: #D55E00;
     --wip-chart-duration-more-than-four-weeks-color: #B16BAE; /* see jirametrics-3fg */
 
-    --estimate-accuracy-chart-completed-border-color: #2DCB9A;
-    --estimate-accuracy-chart-active-border-color: #E69F00;
-
     --expedited-chart-dot-issue-stopped-color: #2DCB9A;
     --expedited-chart-dot-expedite-started-color: #E69F00;
     --expedited-chart-dot-expedite-stopped-color: #2DCB9A;

data/lib/jirametrics/html/index.js

@@ -157,6 +157,19 @@ function withAlpha(color, alpha) {
   return resolved.replace(/^rgb\(/, 'rgba(').replace(/\)$/, `, ${alpha})`)
 }
 
+// A y-axis tick callback for charts whose ceiling is deliberately set above their tallest value, so
+// you can see that nothing is clipped. Chart.js labels that ceiling too, which gives you an arbitrary
+// number like 169 sitting just above 160, or 62 above 50. Drop that label, unless the ceiling happens
+// to land exactly on the regular grid, in which case it is a perfectly good label and worth keeping.
+// Pass it straight to ticks.callback.
+//   ticks 0 20 .. 160 169 -> 169 dropped, it breaks the spacing
+//   ticks 0 20 .. 100 120 -> 120 kept, it continues the spacing
+function labelExceptCrowdedCeiling(value, index, ticks) {
+  if (index !== ticks.length - 1 || index < 2) return value
+  let regularGap = ticks[index - 1].value - ticks[index - 2].value
+  return value - ticks[index - 1].value === regularGap ? value : null
+}
+
 function createDiagonalPattern(color = 'black') {
   // create a 5x5 px canvas for the pattern's base shape
   let shape = document.createElement('canvas')

data/lib/jirametrics/html/time_based_scatterplot.erb

@@ -82,9 +82,10 @@ new Chart(document.getElementById('<%= chart_id %>').getContext('2d'), {
           color: <%= CssVariable['--grid-line-color'].to_json %>
         },
         ticks: {
+          // Uses labelExceptCrowdedCeiling() defined in index.js.
           callback: function(value, index, ticks) {
             <% if @cap %>if (value > <%= @cap[:cutoff] %>) return null;<% end %>
-            return index === ticks.length - 1 ? null : value;
+            return labelExceptCrowdedCeiling(value, index, ticks);
           }
         }
       }

data/lib/jirametrics/issue.rb

@@ -63,7 +63,7 @@ class Issue
 
   def labels = raw_fields['labels'] || []
 
-  def author = raw_fields['creator']&.[]('displayName') || ''
+  def author = User.from_raw(raw_fields['creator'])&.display_name || ''
 
   def resolution = raw_fields['resolution']&.[]('name')
 
@@ -374,13 +374,11 @@ class Issue
     @changes.reverse.find(&:resolution?)
   end
 
-  def assigned_to
-    raw_fields['assignee']&.[]('displayName')
-  end
+  # nil rather than '' when there's no assignee, and daily_view relies on it to decide whether to
+  # render an assignee line at all.
+  def assigned_to = User.from_raw(raw_fields['assignee'])&.display_name
 
-  def assigned_to_icon_url
-    raw_fields['assignee']&.[]('avatarUrls')&.[]('16x16')
-  end
+  def assigned_to_icon_url = User.from_raw(raw_fields['assignee'])&.avatar_url
 
   # Many test failures are simply unreadable because the default inspect on this class goes
   # on for pages. Shorten it up.

data/lib/jirametrics/issue_printer.rb

@@ -19,10 +19,11 @@ class IssuePrinter
   end
 
   def assignee_line
-    assignee = @issue.raw['fields']['assignee']
-    return '' if assignee.nil?
+    user = User.from_raw @issue.raw['fields']['assignee']
+    return '' if user.nil?
 
-    "  [assignee] #{assignee['name'].inspect} <#{assignee['emailAddress']}>\n"
+    email = user.email_address
+    "  [assignee] #{user.display_name.inspect}#{" <#{email}>" if email}\n"
   end
 
   def links_section
@@ -78,6 +79,7 @@ class IssuePrinter
   def render_history history
     type_width = history.collect { |_time, type, _detail, _artificial| type&.length || 0 }.max
     sort_history!(history)
+    # Not all ruby versions return the same string for to_s so we force to a known format.
     history.map do |time, type, detail, _artificial|
       type = type.nil? ? '-' * type_width : type.rjust(type_width)
       "    #{time.strftime '%Y-%m-%d %H:%M:%S %z'} [#{type}] #{detail}\n"

data/lib/jirametrics/jira_gateway.rb

@@ -109,15 +109,13 @@ class JiraGateway
 
   VerifyResult = Data.define(:ok, :url, :message)
 
-  # Confirm that our credentials actually authenticate against Jira. We deliberately hit /myself,
-  # which requires authentication on every Jira edition (anonymous callers get a 401). An endpoint
-  # that also answers anonymously would happily "succeed" with bad credentials, which defeats the
-  # whole point. Cloud speaks v3, Server/Data Center speaks v2. Returns a VerifyResult rather than
-  # raising so the caller can report on every configured connection and set its own exit status.
   def verify_connection
+    # We use /myself because the only reason it can fail is if we aren't authenticated.
     json = call_url relative_url: "/rest/api/#{cloud? ? 3 : 2}/myself"
-    user = json['displayName'] || json['name'] || json['emailAddress'] || 'unknown user'
-    VerifyResult.new(ok: true, url: @jira_url, message: "Verified #{@jira_url} (authenticated as #{user})")
+
+    user = User.new raw: json
+    who = user.display_name || 'unknown user'
+    VerifyResult.new(ok: true, url: @jira_url, message: "Verified #{@jira_url} (authenticated as #{who})")
   rescue StandardError => e
     VerifyResult.new(ok: false, url: @jira_url, message: "Could not authenticate to #{@jira_url}: #{e.message}")
   end

data/lib/jirametrics/sprint_burndown.rb

@@ -37,13 +37,13 @@ class SprintBurndown < ChartBase
   end
 
   def run
-    return nil unless current_board.scrum?
+    return render_no_data unless current_board.scrum?
 
     sprints = sprints_in_time_range current_board
     change_data_by_sprint = gather_change_data_by_sprint sprints
 
     result = +''
-    result << render_top_text(binding)
+    result << render_top_text
 
     # HashEachMethods misreads this array of [method, title] pairs as a hash and thinks y_axis_title
     # is unused; in fact the ERB template reads it (and data_method) from the binding we pass to render.

data/lib/jirametrics/testing/mock_board.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+class JiraMetrics
+  module Testing
+    # Builds a Board from the two files jirametrics already downloads for you, so a test can use your
+    # own board rather than a fabricated one.
+    #
+    #   board = MockBoard.load statuses: 'target/mine_statuses.json',
+    #                          configuration: 'target/mine_board_1_configuration.json'
+    #
+    # Takes filenames rather than packaging a sample board, because a board that resembles yours is
+    # worth more than one of ours, and because sample data would have to be carried in the gem forever.
+    class MockBoard
+      class << self
+        def load statuses:, configuration:
+          board = Board.new(
+            raw: JSON.parse(File.read(configuration, encoding: 'UTF-8')),
+            possible_statuses: load_statuses(statuses)
+          )
+          board.project_config = ProjectConfig.new(
+            exporter: Exporter.new, target_path: File.dirname(configuration), jira_config: nil, block: nil
+          )
+          board
+        end
+
+        def load_statuses filename
+          JSON.parse(File.read(filename, encoding: 'UTF-8')).each_with_object(StatusCollection.new) do |raw, collection|
+            collection << Status.from_raw(raw)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/jirametrics/testing/mock_change_item.rb

@@ -0,0 +1,112 @@
+# frozen_string_literal: true
+
+class JiraMetrics
+  module Testing
+    # Builds a ChangeItem for tests without going through Jira's changelog format.
+    #
+    # Normalises the arguments a caller is likely to get slightly wrong (a status given as a name when
+    # an id is wanted, a time given as a string) and, for status changes with an issue, validates the
+    # status against that issue's board so a typo surfaces loudly rather than silently passing.
+    class MockChangeItem
+      def initialize(
+        field:, value:, time:, value_id: nil, old_value: nil, old_value_id: nil,
+        artificial: false, issue: nil, field_id: nil
+      )
+        @field = field
+        @value = value
+        # Callers write dates as strings far more often than as Times.
+        @time = JiraMetrics::Testing.to_time time
+        @value_id = value_id
+        @old_value = old_value
+        @old_value_id = old_value_id
+        @artificial = artificial
+        @issue = issue
+        @field_id = field_id
+      end
+
+      def to_change_item
+        normalize_status_arguments
+        validate_status_change if @field == 'status' && @issue
+
+        ChangeItem.new time: @time, artificial: @artificial, author_raw: nil, raw: {
+          'field' => @field,
+          'to' => @value_id,
+          'toString' => @value,
+          'from' => @old_value_id,
+          'fromString' => @old_value,
+          'fieldId' => @field_id
+        }
+      end
+
+      # If either value or old_value is a Status object then pull the name and id off it.
+      private
+
+      def normalize_status_arguments
+        if @value.is_a? Status
+          @value_id = @value.id
+          @value = @value.name
+        end
+        return unless @old_value.is_a? Status
+
+        @old_value_id = @old_value.id
+        @old_value = @old_value.name
+      end
+
+      # Status names aren't unique, so a status name always has to be paired with an explicit id.
+      def validate_status_change
+        require_value_id!
+        require_old_value_id!
+        verify_value_id!
+        verify_old_value_id!
+      end
+
+      def require_value_id!
+        return unless @value && !@value_id
+
+        guesses = possible_statuses.find_all_by_name(@value).collect(&:id)
+        message = "ID was not specified for new status #{@value.inspect}. "
+        if guesses.empty?
+          message << "No statuses with name #{@value.inspect} but did find these: #{possible_statuses.inspect}"
+        else
+          message << "Perhaps you meant one of #{guesses.inspect}"
+        end
+        raise message
+      end
+
+      def require_old_value_id!
+        return unless @old_value && !@old_value_id
+
+        guesses = possible_statuses.find_all_by_name(@old_value).collect(&:id)
+        raise "ID was not specified for old status #{@old_value.inspect}. Perhaps you meant one of #{guesses.inspect}"
+      end
+
+      def verify_value_id!
+        return unless @value_id
+
+        status = possible_statuses.find_by_id(@value_id)
+        raise "No status found for id: #{@value_id} (#{@value.inspect}) in #{possible_statuses.inspect}" unless status
+        return if status.name == @value
+
+        raise "Value passed to mock_change (#{@value.inspect}:#{@value_id.inspect}) " \
+          "doesn't match the status found in the board (#{status})"
+      end
+
+      def verify_old_value_id!
+        return unless @old_value_id
+
+        status = possible_statuses.find_by_id(@old_value_id)
+        unless status
+          raise "No status found for id: #{@old_value_id} (#{@old_value.inspect}) in #{possible_statuses.inspect}"
+        end
+        return if status.name == @old_value
+
+        raise "Old value passed to mock_change (#{@old_value.inspect}:#{@old_value_id.inspect}) " \
+          "doesn't match the status found in the board (#{status})"
+      end
+
+      def possible_statuses
+        @issue.board.possible_statuses
+      end
+    end
+  end
+end

data/lib/jirametrics/testing/mock_cycle_time_config.rb

@@ -0,0 +1,73 @@
+# frozen_string_literal: true
+
+class JiraMetrics
+  module Testing
+    # Stubs are matched by the issue's KEY, not by object identity, so every issue in a test needs its
+    # own key. Building two issues with the same key and stubbing both raises.
+    #
+    #   board.cycletime = MockCycleTimeConfig.new
+    #     .stub(issue1, started: '2021-01-02')
+    #     .stub(issue2, started: '2021-01-02', stopped: '2021-10-04')
+    #
+    # An issue with no stub at all reads as never started, which is a legitimate state and so cannot be
+    # distinguished from one you forgot.
+    class MockCycleTimeConfig < CycleTimeConfig
+      def initialize
+        super(possible_statuses: nil, label: nil, block: nil, settings: self.class.default_settings)
+        @stubs = []
+      end
+
+      # The same file ProjectConfig reads, resolved from this file's own location so that it works from
+      # wherever the caller happens to be rather than only from a checkout of this repo.
+      def self.default_settings
+        settings_file = File.expand_path '../settings.json', __dir__
+        JSON.parse(File.read(settings_file, encoding: 'UTF-8')).tap do |settings|
+          # A cached cycle time would outlive the stub that produced it, so a second stub for the same
+          # issue would appear to have no effect.
+          settings['cache_cycletime_calculations'] = false
+        end
+      end
+
+      # Returns self so that calls cascade.
+      def stub issue, started: nil, stopped: nil
+        key = issue.is_a?(Issue) ? issue.key : issue
+        assert_key_unused key
+        @stubs << [key, normalize_time(started), normalize_time(stopped)]
+        self
+      end
+
+      def started_stopped_changes(issue)
+        value = @stubs.find { |issue_key, _start, _stop| issue_key == issue.key }
+        return [nil, nil] unless value
+
+        [to_change(value[1]), to_change(value[2])]
+      end
+
+      private
+
+      def normalize_time value
+        value.is_a?(String) ? JiraMetrics::Testing.to_time(value) : value
+      end
+
+      # Only the first stub for a key is reachable, so a second one always means the test is asserting
+      # against something other than what it looks like.
+      def assert_key_unused key
+        return unless @stubs.any? { |existing, _start, _stop| existing == key }
+
+        raise "More than one stub for #{key}. Stubs are matched by issue key, so only the first would " \
+          'ever be used. Give each issue its own key.'
+      end
+
+      def to_change change
+        case change
+        when nil
+          nil
+        when ChangeItem
+          change
+        else
+          MockChangeItem.new(field: 'status', value: 'fake', value_id: 1_000_001, time: change&.to_time).to_change_item
+        end
+      end
+    end
+  end
+end

data/lib/jirametrics/testing/mock_issue.rb

@@ -0,0 +1,108 @@
+# frozen_string_literal: true
+
+class JiraMetrics
+  module Testing
+    # An Issue you can build without a fixture file and then add history to.
+    #
+    # MockIssue.new keeps Issue's own signature (raw:, board:) so that a MockIssue is substitutable for
+    # an Issue everywhere, including MockCycleTimeConfig's is_a?(Issue) check. The friendly constructor
+    # is MockIssue.empty, which builds the minimal raw hash for you.
+    class MockIssue < Issue
+      # Generated keys start well clear of the hand-written SP-1 and SP-2 that fixtures use.
+      FIRST_GENERATED_KEY_NUMBER = 1000
+
+      # A leap day on purpose. Anything that does its own date arithmetic, assumes 365-day years, or
+      # round-trips through a format that cannot represent Feb 29 will trip over this default rather
+      # than sailing past on a date that hides the bug.
+      DEFAULT_CREATED = '2024-02-29'
+
+      # The change is built and appended in one step, and returned for tests that need to hold on to it.
+      def add_change field:, value:, time:, value_id: nil, old_value: nil, old_value_id: nil,
+        artificial: false, field_id: nil
+        change = MockChangeItem.new(
+          issue: self, field: field, time: time, value: value, value_id: value_id,
+          old_value: old_value, old_value_id: old_value_id, artificial: artificial, field_id: field_id
+        ).to_change_item
+        changes << change
+        change
+      end
+
+      class << self
+        # board has no default. Supplying one would mean either packaging a sample board or reading
+        # from this repo's spec directory, and neither belongs in a shipped gem. MockBoard.load builds
+        # one from the files jirametrics already downloads.
+        def empty board:, created: DEFAULT_CREATED, key: nil, creation_status: nil,
+          current_sprint_ids: nil
+          new(
+            raw: raw_for(
+              created: created,
+              key: key || next_generated_key,
+              creation_status: resolve_creation_status(creation_status, board),
+              current_sprint_ids: current_sprint_ids,
+              board_id: board.id
+            ),
+            board: board
+          )
+        end
+
+        private
+
+        def next_generated_key
+          @next_key_number ||= FIRST_GENERATED_KEY_NUMBER
+          "SP-#{@next_key_number}".tap { @next_key_number += 1 }
+        end
+
+        # A Status carries its category, so the issue cannot end up claiming to be Done while sitting in
+        # To Do. Specs that need a status the board does not have can build one with Status.new, which
+        # is explicit about being artificial rather than being smuggled in as a name and id.
+        def resolve_creation_status creation_status, board
+          return creation_status if creation_status.is_a? Status
+
+          raise "creation_status must be a Status, got #{creation_status.class}" unless creation_status.nil?
+
+          backlog_statuses = board.possible_statuses.find_all_by_name('Backlog')
+          raise 'No Backlog status found' if backlog_statuses.empty?
+
+          backlog_statuses.first
+        end
+
+        # Mimics an issue created directly inside a sprint: that membership lives only in the current
+        # Sprint custom field and never appears as a changelog transition.
+        #
+        # Only two keys are ever read back. Issue#current_sprint_ids takes the ids, and
+        # Issue#sprint_field_id finds the sprint field by looking for an array of hashes carrying a
+        # boardId. Anything else written here would be decoration that could contradict the board.
+        def raw_for created:, key:, creation_status:, current_sprint_ids:, board_id:
+          sprint_field =
+            if current_sprint_ids
+              { 'customfield_10020' => current_sprint_ids.collect { |id| { 'id' => id, 'boardId' => board_id } } }
+            else
+              {}
+            end
+          created_time = JiraMetrics::Testing.to_time(created).to_s
+          {
+            'key' => key,
+            'changelog' => { 'histories' => [] },
+            'fields' => sprint_field.merge(
+              'created' => created_time,
+              'updated' => created_time,
+              'status' => {
+                'name' => creation_status.name,
+                'id' => creation_status.id.to_s,
+                'statusCategory' => {
+                  'name' => creation_status.category.name,
+                  'id' => creation_status.category.id,
+                  'key' => creation_status.category.key
+                }
+              },
+              'priority' => { 'name' => 'Medium', 'id' => '3' },
+              'issuetype' => { 'name' => 'Bug' },
+              'creator' => { 'displayName' => 'Tolkien' },
+              'summary' => 'Do the thing'
+            )
+          }
+        end
+      end
+    end
+  end
+end