danger-yajp 0.0.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 66916c1769e7fecc6c05f5eb549e5775ae7bc8a98eac77bcf0efa727fdc5ec13
-  data.tar.gz: 24cca5f396eda224b143d326a152960a6dab5281634f02891d51ed0dfedfc97e
+  metadata.gz: 2b33191a7403d610580966c988913cd164f6eb2fba53b54aabb3bfc30c3d9796
+  data.tar.gz: b57ab4fe9c1a1b37b15647dd2358fd8640904f96801c91d680828a93d0b31779
 SHA512:
-  metadata.gz: 622b3ed1ff1e2168b1666861adfc20255a83573166a38ccf4edb352b9eaaf5e8d83e7eea900a3ce1d6e5954312ea3c6352ef89b23666b6b2f222c4079eaf5c9a
-  data.tar.gz: 5efc4d116c9c8e053c89086bde433515e09cfa11da105fe48f51ec76da0280281b90caefd8e7bb79a5c8da7e6590e47df3b66e5714dd2883af187f47fbc36740
+  metadata.gz: 0df505bcf5ff1c4eaa239607e4c759a09dc098e477e078b7a622bc58c97db93d7c8b9d61413b2da137cf858890ab450cebe26593bcaf249264d9adfc82c9e3a3
+  data.tar.gz: a2a95d33414764499d4879b52be808547343e3e6478c0f2493efa8d2db89ad91aac3a7f1d21555a88d386e5dfa9de2bc5249b511b37da1dfec05a7a5feeb22bb

data/.github/workflows/create-tag.yml

@@ -0,0 +1,39 @@
+name: Create release
+
+on:
+  push:
+    branches: main
+
+jobs:
+  build:
+    name: Retrieve version + create release
+    runs-on: ubuntu-latest
+    
+    steps:
+    - uses: actions/checkout@v2
+      with:
+        fetch-depth: 0
+      
+    - name: Set up Ruby 2.6
+      uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.6.x
+
+    - name: Retrieve version
+      id: version
+      run: |
+        version=$(ruby -e 'require "./lib/yajp/gem_version.rb"' -e 'puts Yajp::VERSION')
+        echo "TAG_NAME=$version" >> $GITHUB_ENV
+        latest_tag=$(git describe --tags --abbrev=0)
+        echo 'VERSION_DESCRIPTION<<EOF' >> $GITHUB_ENV
+        git log --pretty='format:%h %s' ${latest_tag}..HEAD >> $GITHUB_ENV
+        echo $'\nEOF' >> $GITHUB_ENV
+
+    - name: Create release
+      uses: actions/create-release@v1
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      with:
+        tag_name: ${{ env.TAG_NAME }}
+        release_name: v${{ env.TAG_NAME }}
+        body: ${{ env.VERSION_DESCRIPTION }}

data/.rubocop.yml

@@ -4,6 +4,7 @@
 
 AllCops:
   TargetRubyVersion: 2.6
+  NewCops: enable
 
 Style/StringLiterals:
   EnforcedStyle: single_quotes

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-yajp (0.0.1)
+    danger-yajp (0.1.3)
       danger-plugin-api
       jira-ruby
 
@@ -30,7 +30,7 @@ GEM
     cork (0.3.0)
       colored2 (~> 3.1)
     crack (0.4.4)
-    danger (8.2.0)
+    danger (8.2.1)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
@@ -82,7 +82,7 @@ GEM
       rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
-    listen (3.2.1)
+    listen (3.3.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
     lumberjack (1.2.8)
@@ -100,7 +100,7 @@ GEM
       faraday (>= 0.9)
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
-    parallel (1.19.2)
+    parallel (1.20.0)
     parser (2.7.2.0)
       ast (~> 2.4.1)
     pry (0.13.1)
@@ -115,29 +115,29 @@ GEM
     rchardet (1.8.0)
     regexp_parser (1.8.2)
     rexml (3.2.4)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.3)
-      rspec-support (~> 3.9.3)
-    rspec-expectations (3.9.3)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.0)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.1)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.4)
-    rubocop (1.0.0)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.0)
+    rubocop (1.3.0)
       parallel (~> 1.10)
       parser (>= 2.7.1.5)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8)
       rexml
-      rubocop-ast (>= 0.6.0)
+      rubocop-ast (>= 1.1.1)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 1.4.0, < 2.0)
-    rubocop-ast (1.1.0)
+    rubocop-ast (1.1.1)
       parser (>= 2.7.1.5)
     ruby-progressbar (1.10.1)
     ruby2_keywords (0.0.2)
@@ -149,10 +149,10 @@ GEM
       unicode-display_width (~> 1.1, >= 1.1.1)
     thor (1.0.1)
     thread_safe (0.3.6)
-    tzinfo (1.2.7)
+    tzinfo (1.2.8)
       thread_safe (~> 0.1)
     unicode-display_width (1.7.0)
-    webmock (3.9.3)
+    webmock (3.10.0)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
@@ -170,7 +170,7 @@ DEPENDENCIES
   pry
   rake (~> 13.0)
   rspec (~> 3.9)
-  rubocop (~> 1.0.0)
+  rubocop (>= 1.0.0)
   webmock (~> 3.9)
   yard (~> 0.9.11)
 

data/README.md

@@ -1,8 +1,10 @@
 # Yet Another Jira Plugin
 
-[![License](http://img.shields.io/badge/license-MIT-green.svg?style=flat)](LICENSE)
+[![License](https://img.shields.io/github/license/juliendms/danger-yajp)](LICENSE)
+[![Gem](https://img.shields.io/gem/v/danger-yajp)](https://rubygems.org/gems/danger-yajp)
+[![Dependencies](https://img.shields.io/librariesio/release/rubygems/danger-yajp)](https://libraries.io/rubygems/danger-yajp)
 
-Yet Another Jira Plugin (in short: yajp) is a [Danger](https://danger.systems/ruby/) plugin that provides methods to easily find and manipulate issues from within the Dangerfile. The major difference with the existing Jira plugins is the ability to transition and update issues with the same feeling as manipulating PR data from Danger. This plugin was build in the same mind as Danger, meaning that you will find methods to easily manipulate Jira data, but no predefined warning and/or message.
+Yet Another Jira Plugin (in short: yajp) is a [Danger](https://danger.systems/ruby/) plugin that provides methods to easily find and manipulate issues from within the Dangerfile. The major difference with the existing Jira plugins is the ability to transition and update issues with the same feeling as manipulating PR data from Danger. This plugin was build in the same mind as Danger, meaning that you will find methods to easily manipulate Jira data, but no predefined warning and/or message. It also does that by expanding the Issue class from `jira-ruby`.
 
 Inspired by [danger-jira](https://github.com/RestlessThinker/danger-jira), from which I borrowed the issue search, and by [danger-jira_sync](https://github.com/roverdotcom/danger-jira_sync) for their usage of the awesome [jira-ruby](https://github.com/sumoheavy/jira-ruby) gem.
 
@@ -16,12 +18,12 @@ gem 'danger-yajp'
 
 ## Usage
 
-You first need to define the environment variables `DANGER_JIRA_URL`, `DANGER_JIRA_USER` and `DANGER_JIRA_PASSWORD` in your CI environment, for example:
+You first need to define the environment variables `DANGER_JIRA_URL`, `DANGER_JIRA_USER` and `DANGER_JIRA_API_TOKEN` in your CI environment, for example:
 
 ```
 DANGER_JIRA_URL: https://jira.company.com/jira
 DANGER_JIRA_USER: username
-DANGER_JIRA_PASSWORD: abcd12345
+DANGER_JIRA_API_TOKEN: abcd12345
 ```
 
 ### Find issues
@@ -43,26 +45,40 @@ end
 
 ### Transition / update issues
 
-yajp allows to easily transition and update issues without the hassle of building custom json in the Dangerfile. The inputs are:
+yajp allows to easily transition and update issues without the hassle of building custom json in the Dangerfile. The methods are available in the issue object, or to handle multiple issues in the plugin object. The inputs are:
 
-* An issue (from `jira-ruby`) or an array of issues
 * For the transition action, the ID of the transition
-* Any number of fields to be updated in the form: `key: value`
+* When using the methods from the plugin object, the issues to handled, which is by default the issues found when the command `find_issues` was last run.
+* Any number of fields to be updated in a hash: `key: value`
 
+Example 1: transition all the issues found after running `find_issues`:
  ```rb
- jira.transition(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+ jira.transition_all(10, assignee: { name: 'username' }, customfield_11005: 'example')
  ```
 
-The `transition` method only takes fields available in the transition screen. Use the `split_transition_fields` method to separate the fields available in the transition screen, or use the `transition_and_update` method to transition and update issues (and automatically dispatch the fields to the correct action).
+Example 2: update a single issue:
+ ```rb
+ issue.update(assignee: { name: 'username' }, customfield_11005: 'example')
+ ```
+
+The `transition` and `transition_all` methods only take fields available in the transition screen. Use the `split_transition_fields` method to separate the fields available in the transition screen, or use the `transition_and_update_all` method to transition and update issues (and automatically dispatch the fields to the correct action).
 
 > Transition IDs can be found in Jira under Project Workflow > Edit Workflow in Text Mode.
 
+### Reference the PR as a remote link
+
+yajp can reference the PR as a remote link on Jira. It will use the icon of GitLab or Github depending on what you use. The remote link will use the URL of the PR as a `globalId` to not create duplicates. Optionnaly, you can specify the relationship with the issue (default is `relates to`), and the status, either as an object (eg. `{ "resolved": true, "icon": {...} }`) or as a boolean that will set the value of the property `resolved`. By default, no status is sent.
+
+```rb
+jira.pr_as_remotelink(issue, false)
+```
+
 ### Issue URL
 
-Use `issue_link` to retrieve the browse URL of the Jira issue.
+Use `link` to retrieve the browse URL of the Jira issue.
 
 ```rb
-message "<a href='#{jira.issue_link(issue)}'>#{issue.key} - #{issue.summary}</a>"
+message "<a href='#{issue.link}'>#{issue.key} - #{issue.summary}</a>"
 ```
 
 ### API
@@ -82,13 +98,13 @@ if issues.empty?
   warn 'This PR does not contain any Jira issue.'
 else
   issues.each do |issue|
-    message "<a href='#{jira.issue_link(issue)}'>#{issue.key} - #{issue.summary}</a>"
+    message "<a href='#{issue.link}'>#{issue.key} - #{issue.summary}</a>"
 
-    case issue.status
+    case issue.status.name
     when 'In Progress'
-      jira.transition_and_update(issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+      jira.transition_and_update_all(10, issue: issue, assignee: { name: 'username' }, customfield_11005: 'example')
     when 'To Do', 'Blocked'
-      warn "Issue <a href='#{jira.issue_link(issue)}'>#{issue.key}</a> is not in Dev status, please make sure the issue you're working on is in the correct status"
+      warn "Issue <a href='#{issue.link}'>#{issue.key}</a> is not in Dev status, please make sure the issue you're working on is in the correct status"
     end
   end
 end

data/danger-yajp.gemspec

@@ -33,7 +33,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'webmock', '~> 3.9'
 
   # Linting code and docs
-  spec.add_development_dependency 'rubocop', '~> 1.0.0'
+  spec.add_development_dependency 'rubocop', '>= 1.0.0'
   spec.add_development_dependency 'yard', '~> 0.9.11'
 
   # Makes testing easy via `bundle exec guard`

data/lib/yajp/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Yajp
-  VERSION = '0.0.1'
+  VERSION = '0.1.3'
 end

data/lib/yajp/issue.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require 'jira-ruby'
+
+module Danger
+  # This class extends (aka monkey patch) the `JIRA::Resource::Issue` class with straightforward methods to easily transition and update issues.
+  #
+  class JIRA::Resource::Issue
+    # Get the browse URL of the issue.
+    #
+    # @return [String] the URL of the issue
+    #
+    def link
+      "#{ENV['DANGER_JIRA_URL']}/browse/#{key}"
+    end
+
+    # Update the issue.
+    #
+    # @example Update the fields `assignee` and `customfield_11005`
+    #   issue.update(assignee: { name: 'username' }, customfield_11005: 'example')
+    #
+    # @param [Hash] fields Fields to update
+    #
+    # @return [Boolean] `true` if the issue was updated successfully, `false` otherwise.
+    #
+    def update(**fields)
+      return if fields.empty?
+
+      save({ fields: fields })
+    end
+
+    # Transition the issue using the ID of the transition. Transition IDs can be found in Jira under Project Workflow > Edit Workflow in Text Mode.
+    # The fields that can be updated with this method are only the fields available in the transition screen of the transition. Otherwise use `transition_and_update`.
+    #
+    # @example Transition the issue and set the fields `assignee` and `customfield_11005` available on the transition screens
+    #   jira.transition(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+    #
+    # @param [Integer] transition_id
+    # @param [Hash] fields Fields that can be updated on the transition screen
+    #
+    # @return [Boolean] `true` if the issue was transitioned successfully, `false` otherwise.
+    #
+    def transition(transition_id, **fields)
+      data = { transition: { id: transition_id.to_s } }
+      data[:fields] = fields unless fields.empty?
+
+      transitions.build.save(data)
+    end
+  end
+end

data/lib/yajp/plugin.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'jira-ruby'
+require_relative 'issue'
 
 module Danger
   # Yet Another Jira Plugin (in short: yajp) provides methods to easily find and manipulate issues from within the Dangerfile.
@@ -18,13 +18,13 @@ module Danger
   #     warn 'This PR does not contain any Jira issue.'
   #   else
   #     issues.each do |issue|
-  #       message "<a href='#{jira.issue_link(issue)}'>#{issue.key} - #{issue.summary}</a>"
+  #       message "<a href='#{issue.link}'>#{issue.key} - #{issue.summary}</a>"
   #
-  #       case issue.status
+  #       case issue.status.name
   #       when 'In Progress'
-  #         jira.transition_and_update(issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+  #         issue.transition(10, assignee: { name: 'username' }, customfield_11005: 'example')
   #       when 'To Do', 'Blocked'
-  #         warn "Issue <a href='#{jira.issue_link(issue)}'>#{issue.key}</a> is not in Dev status, please make sure the issue you're working on is in the correct status"
+  #         warn "Issue <a href='#{issue.link}'>#{issue.key}</a> is not in Dev status, please make sure the issue you're working on is in the correct status"
   #       end
   #     end
   #   end
@@ -69,11 +69,11 @@ module Danger
     #   jira.find_issues('KEY', search_title: false, search_branch: true)
     #
     # @param [Array<String>]  key An array of Jira project keys like `['KEY', 'JIRA']`, or a single `String` with a Jira project key
-    # @param [Boolean] search_title Option to search Jira issues from PR title, default `true`
-    # @param [Boolean] search_commits Option to search Jira issues from from commit messages, default `false`
-    # @param [Boolean] search_branch Option to search Jira issues from the name of the PR branch, default `false`
+    # @param [Boolean] search_title Option to search Jira issues from PR title
+    # @param [Boolean] search_commits Option to search Jira issues from from commit messages
+    # @param [Boolean] search_branch Option to search Jira issues from the name of the PR branch
     #
-    # @return [Array<JIRA::Issue>] An array containing all the unique issues found in the PR.
+    # @return [Array<JIRA::Resource::Issue>] An array containing all the unique issues found in the PR.
     #
     def find_issues(key, search_title: true, search_commits: false, search_branch: false)
       regexp = build_regexp_from_key(key)
@@ -84,29 +84,27 @@ module Danger
       jira_issues.concat(search_branch(regexp)) if search_branch
       jira_issues.concat(search_pr_body(regexp)) if jira_issues.empty?
 
-      jira_issues.uniq.map { |issue_key| @api.Issue.find(issue_key) }
+      @issues = jira_issues.uniq(&:downcase).map { |issue_key| @api.Issue.find(issue_key) }
     end
 
     # Transition the given Jira issue(s) using the ID of the transition. Transition IDs can be found in Jira under Project Workflow > Edit Workflow in Text Mode.
     # The fields that can be updated with this method are only the fields available in the transition screen of the transition. Otherwise use `transition_and_update`.
     #
     # @example Transition the issue `my_issue` and set the fields `assignee` and `customfield_11005` available on the transition screens
-    #   jira.transition(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+    #   jira.transition_all(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
     #
-    # @param [Array<JIRA::Issue>] issue An array of issues, or a single `JIRA::Issue`
     # @param [Integer] transition_id
+    # @param [Array<JIRA::Resource::Issue>, JIRA::Resource::Issue] issue An array of issues, or a single issue
     # @param [Hash] fields Fields that can be updated on the transition screen
     #
     # @return [Boolean] `true` if all the issues were transitioned successfully, `false` otherwise.
     #
-    def transition(issue, transition_id, **fields)
+    def transition_all(transition_id, issue: @issues, **fields)
       issues = issue.kind_of?(Array) ? issue : [] << issue
-      data = { transition: { id: transition_id.to_s } }
-      data[:fields] = fields unless fields.empty?
       result = true
 
       issues.each do |key|
-        result &= key.transitions.build.save(data)
+        result &= key.transition(transition_id, **fields)
       end
 
       return result
@@ -115,27 +113,29 @@ module Danger
     # Update the given Jira issue(s).
     #
     # @example Update the issue `my_issue` and set the fields `assignee` and `customfield_11005`
-    #   jira.update(my_issue, assignee: { name: 'username' }, customfield_11005: 'example')
+    #   jira.update_all(my_issue, assignee: { name: 'username' }, customfield_11005: 'example')
     #
-    # @param [Array<JIRA::Issue>] issue An array of issue, or a single `JIRA::Issue`
+    # @param [Array<JIRA::Resource::Issue>, JIRA::Resource::Issue] issue An array of issue, or a single issue
     # @param [Hash] fields Fields to update
     #
     # @return [Boolean] `true` if all the issues were updated successfully, `false` otherwise.
     #
-    def update(issue, **fields)
+    def update_all(issue: @issues, **fields)
       return if fields.empty?
 
       issues = issue.kind_of?(Array) ? issue : [] << issue
       result = true
 
       issues.each do |key|
-        result &= key.save(fields)
+        result &= key.update(**fields)
       end
+
+      return result
     end
 
     # Utility to split the given fields into fields that can be updated on the transition screen corresponding to the `transition_id` of the given `issue`.
     #
-    # @param [JIRA::Issue] issue
+    # @param [JIRA::Resource::Issue] issue
     # @param [Integer] transition_id
     # @param [Hash] fields Fields to split
     #
@@ -159,33 +159,77 @@ module Danger
     # and use the other fields with the update action.
     #
     # @example Transition the issue `my_issue` and set the fields `assignee` and `customfield_11005`
-    #   jira.transition_and_update(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
+    #   jira.transition_and_update_all(my_issue, 10, assignee: { name: 'username' }, customfield_11005: 'example')
     #
-    # @param [Array<JIRA::Issue>] issue An array of issues, or a single `JIRA::Issue`
     # @param [Integer] transition_id
+    # @param [Array<JIRA::Resource::Issue>, JIRA::Resource::Issue] issue An array of issues, or a single issue
     # @param [Hash] fields Fields to update
     #
     # @return [Boolean] `true` if all the issues were transitioned and updated successfully, `false` otherwise.
     #
-    def transition_and_update(issue, transition_id, **fields)
+    def transition_and_update_all(transition_id, issue: @issues, **fields)
       issues = issue.kind_of?(Array) ? issue : [] << issue
       result = issues.first.split_transition_fields(transition_id, fields)
       transition_fields = result[:transition_fields]
       fields = result[:other_fields]
 
-      result = transition(issues, transition_id, **transition_fields)
-      result & update(issues, **fields)
+      result = transition(transition_id, issue: issues, **transition_fields)
+      result & update(issue: issues, **fields)
     end
 
-    # Get the browse URL of a Jira issue.
-    #
-    # @param [JIRA::Issue] issue
-    #
-    # @return [String] the URL of the issue
+    # @deprecated Please use the new #{transition_and_update_all} method
+    def transition_and_update(issue, transition_id, **fields)
+      Warning.warn('Deprecated use of the transition_and_update method, please use the new method definition')
+      transition_and_update_all(transition_id, issue: issue, **fields)
+    end
+
+    # @deprecated Please use the new #{update_all} method
+    def update(issue, **fields)
+      Warning.warn('Deprecated use of the update method, please use the new method definition')
+      update_all(issue: issue, **fields)
+    end
+
+    # @deprecated Please use the new #{transition_all} method
+    def transition(issue, transition_id, **fields)
+      Warning.warn('Deprecated use of the transition method, please use the new method definition')
+      transition_all(transition_id, issue: issue, **fields)
+    end
+
+    # @deprecated Please use the method available on the issue directly [#JIRA::Resource::Issue.link]
     def issue_link(issue)
+      Warning.warn('Deprecated use of the issue_link method, please use the same method available in the Issue class')
       "#{ENV['DANGER_JIRA_URL']}/browse/#{issue.key}"
     end
 
+    # Add a remote link to the PR in the given Jira issues. It uses the link of the PR as the `globalId` of the remote link, thus avoiding to create duplicates each time the PR is updated.
+    #
+    # @param [Array<JIRA::Resource::Issue>, JIRA::Resource::Issue] issue An array of issues, or a single issue
+    # @param [<String>] relation Option to set the relationship of the remote link
+    # @param [<Hash>, <Boolean>] status Option to set the status property of the remote link, it can be <Hash> or a <Boolean> that will set the value of the property `resolved`
+    #
+    # @return [Boolean] `true` if all the remote links were added successfully, `false` otherwise.
+    #
+    def pr_as_remotelink(issue, relation: 'relates to', status: nil)
+      issues = issue.kind_of?(Array) ? issue : [] << issue
+      result = true
+
+      remote_link_prop = { object: { url: pr_link, title: vcs_host.pr_title, icon: link_icon } }
+      remote_link_prop[:globalId] = pr_link
+      remote_link_prop[:relationship] = relation
+
+      if status.kind_of?(Hash)
+        remote_link_prop[:object][:status] = status
+      elsif !status.nil?
+        remote_link_prop[:object][:status] = { resolved: status }
+      end
+
+      issues.each do |key|
+        result &= key.remotelink.build.save(remote_link_prop)
+      end
+
+      return result
+    end
+
     private
 
     def vcs_host
@@ -194,9 +238,27 @@ module Danger
       github
     end
 
+    def pr_link
+      return @pr_link unless @pr_link.nil?
+
+      if defined? @dangerfile.gitlab
+        @pr_link = vcs_host.pr_json['web_url']
+      else
+        @pr_link = vcs_host.pr_json['html_url']
+      end
+
+      return @pr_link
+    end
+
+    def link_icon
+      return { title: 'Gitlab', url16x16: 'https://gitlab.com/favicon.ico' } if defined? @dangerfile.gitlab
+
+      { title: 'Github', url16x16: 'https://github.com/favicon.ico' }
+    end
+
     def build_regexp_from_key(key)
       keys = key.kind_of?(Array) ? key.join('|') : key
-      return /((?:#{keys})-[0-9]+)/
+      return /((?:#{keys})-[0-9]+)/i
     end
 
     def search_title(regexp)

data/spec/support/remotelink.json

@@ -0,0 +1 @@
+{"object":{"url":"https://github.com/test/pull/1234","title":"PR Title","icon":{"title":"Github","url16x16":"https://github.com/favicon.ico"},"status":{"resolved":true}},"globalId":"https://github.com/test/pull/1234","relationship":"relates to"}

data/spec/yajp_spec.rb

@@ -1,9 +1,16 @@
 # frozen_string_literal: true
 
-require File.expand_path('spec_helper', __dir__)
+require_relative 'spec_helper'
 
+# rubocop:disable Metrics/ModuleLength
 module Danger
   describe Danger::DangerYajp do
+    before do
+      ENV['DANGER_JIRA_URL'] = 'https://jira.company.com/jira'
+      ENV['DANGER_JIRA_USER'] = 'username'
+      ENV['DANGER_JIRA_PASSWORD'] = 'password'
+    end
+
     it 'should be a plugin' do
       expect(Danger::DangerYajp.new(nil)).to be_a Danger::Plugin
     end
@@ -13,9 +20,6 @@ module Danger
       let(:plugin) { dangerfile.jira }
 
       before do
-        ENV['DANGER_JIRA_URL'] = 'https://jira.company.com/jira'
-        ENV['DANGER_JIRA_USER'] = 'username'
-        ENV['DANGER_JIRA_PASSWORD'] = 'password'
         DangerYajp.send(:public, *DangerYajp.private_instance_methods)
       end
 
@@ -43,9 +47,9 @@ module Danger
       end
 
       it 'can find jira issues via branch name' do
-        allow(plugin).to receive_message_chain('github.branch_for_head').and_return('bugfix/WEB-126')
+        allow(plugin).to receive_message_chain('github.branch_for_head').and_return('bugfix/web-126')
         issues = plugin.search_branch(plugin.build_regexp_from_key('WEB'))
-        expect(issues).to eq(['WEB-126'])
+        expect(issues).to eq(['web-126'])
       end
 
       it 'can find jira issues in pr body' do
@@ -58,17 +62,19 @@ module Danger
         issue = Object.new
 
         def issue.find(key)
-          return key
+          # The find method in jira-ruby plugin is not case sensitive, hence the upcase.
+          return key.upcase
         end
 
         allow_any_instance_of(JIRA::Client).to receive(:Issue).and_return(issue)
 
         allow(plugin).to receive_message_chain('github.pr_title').and_return('Fix for WEB-128 and WEB-129')
-        allow(plugin).to receive_message_chain('github.branch_for_head').and_return('bugfix/WEB-128')
+        allow(plugin).to receive_message_chain('github.branch_for_head').and_return('bugfix/web-128')
         issues = plugin.find_issues('WEB', search_branch: true)
         expect(issues).to eq(['WEB-128', 'WEB-129'])
       end
 
+      # rubocop:disable Naming/VariableNumber
       it 'can split transition field from other fields' do
         json = File.read("#{File.dirname(__FILE__)}/support/transitions.all.json")
         url = "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/WEB-130/transitions"
@@ -86,16 +92,52 @@ module Danger
 
       it 'can transition an issue' do
         expected_json = '{"transition":{"id":"2"},"fields":{"assignee":{"name":"username"},"customfield_11005":"example"}}'
-        url = "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/WEB-131/transitions"
-        issue = plugin.api.Issue.build
+        issue_id = Random.rand(1000)
+        url = "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/#{issue_id}/transitions"
+        issue = plugin.api.Issue.build({ 'id' => issue_id, 'key' => 'WEB-131' })
 
-        allow(issue).to receive(:key_value).and_return('WEB-131')
         stub = stub_request(:post, url).
           with(body: expected_json)
-        plugin.transition(issue, 2, assignee: { name: 'username' }, customfield_11005: 'example')
+        result = plugin.transition_all(2, issue: issue, assignee: { name: 'username' }, customfield_11005: 'example')
+
+        expect(stub).to have_been_requested.once
+        expect(result).to be true
+      end
+
+      it 'can update issues' do
+        expected_json = '{"fields":{"assignee":{"name":"username"},"customfield_11005":"example"}}'
+        uri_template = Addressable::Template.new "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/{issue}"
+        issue1 = plugin.api.Issue.build({ 'id' => Random.rand(1000), 'self' => "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/WEB-132", 'key' => 'WEB-132' })
+        issue2 = plugin.api.Issue.build({ 'id' => Random.rand(1000), 'self' => "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/WEB-133", 'key' => 'WEB-133' })
+
+        stub = stub_request(:put, uri_template).
+          with(body: expected_json)
+        result = plugin.update_all(issue: [issue1, issue2], assignee: { name: 'username' }, customfield_11005: 'example')
+
+        expect(stub).to have_been_requested.twice
+        expect(result).to be true
+      end
+      # rubocop:enable Naming/VariableNumber
+
+      it 'can add remote link' do
+        pr_title = 'PR Title'
+        pr_json = { 'html_url' => 'https://github.com/test/pull/1234' }
+        url = "#{ENV['DANGER_JIRA_URL']}/rest/api/2/issue/WEB-134/remotelink"
+        json = File.read("#{File.dirname(__FILE__)}/support/remotelink.json")
+        issue = plugin.api.Issue.build
+
+        allow(issue).to receive(:key_value).and_return('WEB-134')
+        allow(dangerfile.github).to receive(:pr_json).and_return(pr_json)
+        allow(dangerfile.github).to receive(:pr_title).and_return(pr_title)
+
+        stub = stub_request(:post, url).
+          with(body: json)
+        result = plugin.pr_as_remotelink(issue, status: true)
 
         expect(stub).to have_been_requested.once
+        expect(result).to be true
       end
     end
   end
 end
+# rubocop:enable Metrics/ModuleLength

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-yajp
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.3
 platform: ruby
 authors:
 - juliendms
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-11-02 00:00:00.000000000 Z
+date: 2020-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -98,14 +98,14 @@ dependencies:
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.0.0
 - !ruby/object:Gem::Dependency
@@ -171,6 +171,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/create-tag.yml"
 - ".github/workflows/gem-push.yml"
 - ".gitignore"
 - ".rubocop.yml"
@@ -185,8 +186,10 @@ files:
 - lib/danger_plugin.rb
 - lib/danger_yajp.rb
 - lib/yajp/gem_version.rb
+- lib/yajp/issue.rb
 - lib/yajp/plugin.rb
 - spec/spec_helper.rb
+- spec/support/remotelink.json
 - spec/support/transitions.all.json
 - spec/yajp_spec.rb
 homepage: https://github.com/juliendms/danger-yajp
@@ -215,5 +218,6 @@ summary: Yet Another Jira Plugin is a danger plugin to find issues, access their
   and perform operations on them.
 test_files:
 - spec/spec_helper.rb
+- spec/support/remotelink.json
 - spec/support/transitions.all.json
 - spec/yajp_spec.rb