lita-jira 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 48c00700fdf50c30b6ad579589c2fd0524c40910
-  data.tar.gz: 601010e495e0f28b68a3a7f854eb0af1c51bcb63
+  metadata.gz: 4bb3875a14b8411fc109e2f604e7620ba1b813a2
+  data.tar.gz: 945da5586a8a8bbd0c5b7437d52d30122cae27ee
 SHA512:
-  metadata.gz: 9b64f58dc863e610feb45e334859b7b21af89a0ff2dfcde1ee49b82a19702781eeb8bcddba9ea0ac7aa486deeb9a789c2260a814b96a7d1c0877b447bf4ff470
-  data.tar.gz: 873c4fb13c1f868e55f1edc193aa675178b50cd0dc93b51808ce02b5dfe484b7ad2153f6544328249eaf99dd7b2b1100f6b1b9c29b3bf0a910ae051ff5dfd424
+  metadata.gz: 3126e1e246e59c9ee7d72a1764bbb25cb9284e50ac6d946725533f47846c8de99b59368884c963673daa9307ca3e6d3eb2b56f49d61521e3a9974a78c5db2cc3
+  data.tar.gz: 2736cceeb65eab4331ececb8622081c906ae086c433e2f65d975fe5695a9bc8749cc598fdbac675794b438b0f832b292292dc80c4ecc12d9ab4df6ed9a711197

data/lib/jirahelper/regex.rb

@@ -9,5 +9,6 @@ module JiraHelper
     ISSUE_PATTERN   = /(?<issue>#{PROJECT_PATTERN}-[0-9]{1,5}+)/
     EMAIL_PATTERN   = /(?<email>[\w+\-.]+@[a-z\d\-]+(\.[a-z]+)*\.[a-z]+)/i
     AMBIENT_PATTERN = /(\s|^)#{ISSUE_PATTERN}/
+    POINTS_PATTERN  = /(?<points>\d{1,2})/
   end
 end

data/lib/lita/handlers/jira.rb

@@ -16,6 +16,7 @@ module Lita
       config :ignore, required: false, type: Array, default: []
       config :rooms, required: false, type: Array
       config :use_ssl, required: false, types: [TrueClass, FalseClass], default: true
+      config :points_field, required: false, type: String
 
       include ::JiraHelper::Issue
       include ::JiraHelper::Misc
@@ -67,6 +68,15 @@ module Lita
         }
       )
 
+      route(
+        /^jira\spoint\s#{ISSUE_PATTERN}\sas\s#{POINTS_PATTERN}$/,
+        :point,
+        command: true,
+        help: {
+          t('help.point.syntax') => t('help.point.desc')
+        }
+      )
+
       # Detect ambient JIRA issues in non-command messages
       route AMBIENT_PATTERN, :ambient, command: false
 
@@ -99,6 +109,13 @@ module Lita
       end
 
       # rubocop:disable Metrics/AbcSize
+      def point(response)
+        return response.reply(t('error.field_undefined')) if config.points_field.blank?
+        issue = fetch_issue(response.match_data['issue'])
+        return response.reply(t('error.request')) unless issue
+        set_points_on_issue(issue, response)
+      end
+
       def myissues(response)
         return response.reply(t('error.not_identified')) unless user_stored?(response.user)
 
@@ -130,6 +147,16 @@ module Lita
       def ignored?(user)
         config.ignore.include?(user.id) || config.ignore.include?(user.mention_name) || config.ignore.include?(user.name)
       end
+
+      def set_points_on_issue(issue, response)
+        points = response.match_data['points']
+        begin
+          issue.save!(fields: { config.points_field.to_sym => points.to_i })
+        rescue
+          return response.reply(t('error.unable_to_point'))
+        end
+        response.reply(t('point.added', issue: issue.key, points: points))
+      end
       # rubocop:enable Metrics/AbcSize
     end
     # rubocop:enable Metrics/ClassLength

data/lita-jira.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'lita-jira'
-  spec.version       = '0.8.0'
+  spec.version       = '0.8.1'
   spec.authors       = ['Eric Sigler', 'Matt Critchlow', 'Tristan Chong', 'Lee Briggs']
   spec.email         = ['me@esigler.com', 'matt.critchlow@gmail.com', 'ong@tristaneuan.ch', 'lee@brig.gs']
   spec.description   = 'A JIRA plugin for Lita.'

data/locales/en.yml

@@ -6,6 +6,8 @@ en:
           already_identified: "You are already identified as %{email}"
           not_identified: You do not have an email address on record
           request: Error fetching JIRA issue
+          field_undefined: You must define `points_field` in your lita_config
+          unable_to_point: Cannot set points on issue
         help:
           identify:
             syntax: jira identify <email address>
@@ -19,6 +21,9 @@ en:
           comment:
             syntax: jira comment on <issue> <comment text>
             desc: Adds <comment text> to <issue>
+          point:
+            syntax: jira point <issue> as <points>
+            desc: Adds <points> points to <issue>
           details:
             syntax: jira details <issue>
             desc: Shows detailed information for <issue>
@@ -42,6 +47,8 @@ en:
           summary: "%{key}: %{summary}"
         comment:
           added: "Comment added to %{issue}"
+        point:
+          added: "Added a point estimation of %{points} to %{issue}"
         myissues:
           empty: "You do not have any assigned issues. Great job!"
           info: "Here are issues currently assigned to you:"

data/spec/lita/handlers/jira_spec.rb

@@ -13,6 +13,7 @@ describe Lita::Handlers::Jira, lita_handler: true do
                     fixVersions: [{ 'name' => 'Sprint 2' }],
                     key: 'XYZ-987')
     allow(result).to receive('save') { true }
+    allow(result).to receive('save!') { true }
     allow(result).to receive('fetch') { true }
     result
   end
@@ -96,6 +97,7 @@ describe Lita::Handlers::Jira, lita_handler: true do
     is_expected.to route_command('jira ABC-123').to(:summary)
     is_expected.to route_command('jira details ABC-123').to(:details)
     is_expected.to route_command('jira comment on ABC-123 "You just need a cat"').to(:comment)
+    is_expected.to route_command('jira point ABC-123 as 2').to(:point)
     is_expected.to route_command('todo ABC "summary text"').to(:todo)
     is_expected.to route_command('todo ABC "summary text" "subject text"').to(:todo)
     is_expected.to route_command('jira myissues').to(:myissues)
@@ -165,6 +167,39 @@ describe Lita::Handlers::Jira, lita_handler: true do
     end
   end
 
+  describe '#point' do
+    before(:each) do
+      registry.config.handlers.jira.points_field = 'some_custom_points_field'
+    end
+
+    it 'updates the issue with a story point value' do
+      expect(saved_issue).to receive(:save!).with(fields: { some_custom_points_field: 5 })
+      grab_request(valid_client)
+      send_command('jira point XYZ-987 as 5')
+      expect(replies.last).to eq('Added a point estimation of 5 to XYZ-987')
+    end
+
+    it 'warns the user when the issue is not valid' do
+      grab_request(failed_find_issue)
+      send_command('jira point XYZ-987 as 5')
+      expect(replies.last).to eq('Error fetching JIRA issue')
+    end
+
+    it 'warns the user when the config points_field is not defined' do
+      registry.config.handlers.jira.points_field = ''
+      grab_request(valid_client)
+      send_command('jira point XYZ-987 as 5')
+      expect(replies.last).to eq('You must define `points_field` in your lita_config')
+    end
+
+    it 'warns the user when settings points fails' do
+      allow(saved_issue).to receive(:save!).and_throw(JIRA::HTTPError)
+      grab_request(valid_client)
+      send_command('jira point XYZ-987 as 5')
+      expect(replies.last).to eq('Cannot set points on issue')
+    end
+  end
+
   describe '#todo' do
     it 'creates a new issue if the project is valid and there is a summary' do
       grab_request(valid_client)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-jira
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Eric Sigler
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-26 00:00:00.000000000 Z
+date: 2016-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita